More right-click authority cleanup

Bug #1287791 reported by Mike Rylander on 2014-03-04
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Medium
Unassigned
2.6
Undecided
Unassigned
2.7
Undecided
Unassigned

Bug Description

Back in bug 1175308 I added some logic that improves the behavior of the MARC editor's right-click authority popup. Now we need to go a little farther and adjust that logic to be more careful about which subfields it uses to browse for authority records. Specifically, it should only use controlled subfields.

Code to do that coming as soon as I have an LP number with which to stamp it... :)

Revision history for this message
Mike Rylander (mrylander) wrote :
tags: added: authority pullrequest
Changed in evergreen:
milestone: 2.6.0-rc1 → 2.next
Yamil (ysuarez) on 2014-08-26
Changed in evergreen:
assignee: nobody → Yamil (ysuarez)
Kathy Lussier (klussier) on 2014-11-10
Changed in evergreen:
assignee: Yamil (ysuarez) → nobody
Revision history for this message
Kathy Lussier (klussier) wrote :

Hi Mike,

I might take a look at this patch. Can you give me an example of the problem that occurred pre-patch?

Thanks!
Kathy

Revision history for this message
Mike Rylander (mrylander) wrote :

FTR, in IRC I said:

[10:54:16] <eeevil> kmlussier: the pre-patch bug is that we were searching on all subfields, not just controlled ones, with the right-click. so, extraneous (from the authority POV) breaks the search

Thanks for poking at this, Kathy!

Revision history for this message
Gislaine Hamelin (gislaine-hamelin) wrote :

Just so I understand this bug (I believe we may have reported it years ago), what is the behaviour? You right click and what is the error?

Revision history for this message
Dan Wells (dbw2) wrote :

Remington loaded this on our test server, and it works fine in his and my testing. Pushed to master through rel_2_6. Thanks, Mike!

Changed in evergreen:
status: Confirmed → Fix Committed
milestone: 2.next → 2.8-beta
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers