Authorities not validating in 2.2 + patch

Bug #1013783 reported by Steve Callender
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.2
Fix Released
Medium
Unassigned
2.3
Fix Released
Medium
Unassigned

Bug Description

Tested in 2.2 rc1. Authorities were not validating.

First there was an error with the actual validation portion. When clicking the validate button, Evergreen would stop and error on the first checked field causing the validation button to append the tag to the end of it in a never ending cycle.

For example, clicking Validate enough times would cause the button to look like, "Validate - 100 - 100 - 100 - 100 - 100 - 100".

This problem was due to the way the authority.normalize_heading was processing marc records.

I've included a patch that I worked on with Mike Rylander to fix this problem.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=4910db432470c1922da516c029c37d523dac8451

Secondary issue, Evergreen is not building the test code correctly when validating marc.

CALL: open-ils.cstore open-ils.cstore.json_query.atomic {"from":"are","select":{"are":["id"]},"where":{"marc":{"startwith":{"transform":"authority.simple_normalize_heading","value":["authority.simple_normalize_heading","<record xmlns=\"http://www.loc.gov/MARC21/slim\"><leader>00000czm a2200205Ka 4500</leader><datafield tag=\"100\" ind1=\" \" ind2=\" \"><subfield code=\"a\">d</subfield></datafield></record>"]}},"control_set":"1","deleted":"f"}}

In the above, there should be a subfield a + the field contents, and a subfield d + the field contents. It's being translated to subfield a with a contents of d instead, so whatever code is building this is off. This part still needs to be investigated and fixed.

Steve

Changed in evergreen:
assignee: nobody → Mike Rylander (mrylander)
Revision history for this message
Mike Rylander (mrylander) wrote :
Changed in evergreen:
status: New → Confirmed
milestone: none → 2.4.0-alpha
assignee: Mike Rylander (mrylander) → nobody
Ben Shum (bshum)
tags: added: signedoff
no longer affects: evergreen/2.1
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-alpha1 → 2.4.0-beta
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-beta → 2.4.0-rc
importance: Undecided → Medium
Revision history for this message
Mike Rylander (mrylander) wrote :

This has been heavily tested in production at several sites. It's time to merge.

Changed in evergreen:
status: Confirmed → Fix Committed
Ben Shum (bshum)
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.