Vandelay batch import of authorities does not work, gives JS method error

Bug #989391 reported by Yamil
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned

Bug Description

* Evergreen version
EG 2.2 beta2

  * OpenSRF version
2.1.0 RC1

  * PostgreSQL version
9.1

  * Linux distribution (type and version) on which Evergreen is running
Debian 6.0.4 "Multi-architecture"distro - running as a VM inside XenServer

I can't seem to batch import authority records with Vandelay. For my tests I am not using "match sets" or "overlay profiles". I am just clicking on "import non-matching records". This works fine for bib record imports, but nothing happens when I click "upload" when using auth records.

BTW, I do get this error on the JS console when I click upload....

---------------- Error: Method error: undefined : *** Call to [open-ils.vandelay.authority_queue.create] failed for session [0.9289250462796181335209562004], thread trace [2]: Can't use an undefined value as an ARRAY reference at /usr/local/share/perl/5.10.1/OpenILS/Application/Vandelay.pm line 113.

Source File: http://10.4.45.5/js/dojo/dojo/dojo.js Line: 188

-----------------

Revision history for this message
Kathy Lussier (klussier) wrote :

FWIW, I just tested this on 2.2beta2 as well and had the same problem, same error .

Revision history for this message
Bill Erickson (berick) wrote :

Looks like a small thinko in the code. It's searching for existing authority queues among the bib queues.

Fix pushed to working => user/berick/vandelay-authority-queue-create-fix

Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
status: New → In Progress
milestone: none → 2.2.0
tags: added: pullrequest
Changed in evergreen:
status: In Progress → Confirmed
assignee: Bill Erickson (erickson-esilibrary) → nobody
Revision history for this message
Mike Rylander (mrylander) wrote :

Since this fixes a currently broken (as in, won't work) UI, I've marked it High.

Changed in evergreen:
importance: Undecided → High
Revision history for this message
Yamil (ysuarez) wrote :

I was just able to import about 325 authority records with this fix works on my EG 2.2 beta2 test server. Thanks Bill.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Yep, this works. In master and rel_2_2.

Changed in evergreen:
status: Confirmed → Fix Committed
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.