Missing/incorrect regex for authority in Vandelay.pm

[Srey Seng]

EG Master (01/07/2014)

File: Vandelay.pm, in function import_record_list_impl

---
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.auto_overlay_bib_record to vandelay.auto_overlay_auth_record.

However, there is no such method in the database in the vandelay schema named vandelay.auto_overlay_auth_record, only (the next closest one) vandelay.auto_overlay_authority_record

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.overlay_bib_record" to "vandelay.overlay_auth_record".

However, there is no such method in the database in the vandelay schema named vandelay.overlay_auth_record, only (the next closest one) vandelay.overlay_authority_record

---

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.

[Srey Seng (sreyseng)]

Working Branch:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sreyseng/lp1266937-fix

[Bill Erickson (berick)]

The fix looks good. I've merged the change to 2.4 and up.

In the future, when a patch is ready, please add a "pullrequest" tag to the ticket.

Thanks, Srey!

[Srey Seng (sreyseng)]

Great, I will do so in the future, thanks!