Missing/incorrect regex for authority in Vandelay.pm
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Undecided
|
Unassigned | ||
2.4 |
Fix Released
|
Undecided
|
Unassigned | ||
2.5 |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
EG Master (01/07/2014)
File: Vandelay.pm, in function import_
---
1.
---
On line 934, there is the following code: $auto_overlay_func = s/bib/auth/o
This is missing a "~" in order for 'auth' to replace 'bib'.
So, the corrected code is: $auto_overlay_func =~ s/bib/auth/o
Restoring the missing "~" correctly modified the value of the $auto_overlay_func variable from vandelay.
However, there is no such method in the database in the vandelay schema named vandelay.
So, the corrected code is now: $auto_overlay_func =~ s/bib/authority/o;
---
2
---
In the same way, on line 933, there is the following code: $overlay_func =~ s/bib/auth/o;
This modified the value of the variable $overlay_func from "vandelay.
However, there is no such method in the database in the vandelay schema named vandelay.
---
The fact that auth overlaying is not available via the Batch Marc Importer / Exporter is perhaps one of the reasons why the missing "~" went unnoticed.
Going to use this as an opportunity to learn how to push to the working repo.
Branch to come with the changes detailed above.
tags: | added: authority |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
Working Branch: git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ sreyseng/ lp1266937- fix
http://