Adding/deleting MFHD records is not synced throughout interface

Bug #798923 reported by Dan Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Jason Etheridge

Bug Description

Adding or deleting MFHD records from the XUL menus requires coordinating the OPAC display, the XUL menus, and the Serial Control distribution editors. The current code frequently requires some non-obvious manual refreshing to make it work.

This branch keeps them in sync using custom events instead.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbwells/mfhd_serctrl_sync

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

This looks fine to me, but I'd like someone with more staff-client-attuned eyes to look at it and try it out before I push it (if I indeed end up being the pusher).

Dan Wells (dbw2)
Changed in evergreen:
assignee: Dan Wells (dbw2) → nobody
Changed in evergreen:
assignee: nobody → Jason Etheridge (phasefx)
Revision history for this message
Dan Wells (dbw2) wrote :

Here is a basic run through which can be used to test this branch. I have also pushed a number of fixes, so please pull again.

From a record with no MFHD or Serials data, do the following:
1. From the "Actions for this Record" menu, choose "Serial Control View"
2. Go to the Subscriptions tab
3. On the left sidebar, find your desired library location and right-click it; choose "Add Subscription"
4. Enter a valid date (YYYY-MM-DD) in the "Start Date" field, click "Apply", then click "Create Subscription(s)"
5. On the left sidebar under your subscription, right-click "Distributions" and choose "Add Distribution"
6. Click "Create Distribution(s)" (all fields are optional)
7. The Distributions Editor will disappear. Go back to your distribution so that it shows in the editor again. Note that the "Legacy Record Entry" dropdown has no MFHD Records.
8. On the "Actions for this Record" menu, choose the "MFHD Holdings" submenu, then "Add MFHD Record"; on the popup, choose the same Org Unit as your Distribution's Holding Lib, then click "Select"
9. On the Distributions Editor, click on "Legacy Record Entry" and notice that your new MFHD Record appears on the list.
10. On the "Actions for this Record" menu, choose the "MFHD Holdings" submenu, then choose the "Delete MFHD Record" submenu, then choose the MFHD Record; check the box to confirm and click "Delete"
11. On the Distributions Editor, click on "Legacy Record Entry" and notice that your deleted MFHD Record is gone from the list.

This doesn't actually result in a "working" setup, but that's beyond the scope of this bugfix.

Thanks Jason!

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

Jason, I just realized at least one portion of this is not going to backport cleanly. I'll get a rel_2_0 branch pushed as soon as I can.

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

First branch should pick cleanly into 2_1, new branch for 2_0 at user/dbwells/mfhd_serctrl_sync_rel_2_0

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbwells/mfhd_serctrl_sync_rel_2_0

Thanks again.

Revision history for this message
Jason Etheridge (phasefx) wrote :
Changed in evergreen:
status: New → 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.