Add summary creation to embedded alt dist editor

Bug #1164725 reported by Dan Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
High
Unassigned

Bug Description

In the Serial Control View, summaries are created automatically with a new distribution. To avoid unhappy surprises, the embedded version of the alt distribution editor should do the same.

To test:
Create a distribution in the integrated editor, then switch to the Distribution tab and refresh the tree. Before the fix, there will be no "summary" leaves ("Basic Summary", etc.) hanging off the distribution. After the fix, you should see the summaries exist by doing the same.

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

working/user/dbwells/integrated_alt_dist_must_create_summaries

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

At first I didn't get that this doesn't change the behavior in alt serials control view, nor did I realize that summary creation is one of the side effects of the open-ils.serial.distribution.fleshed.batch.update method, but now I get it.

This works in the serial control view as Dan says, and doesn't hurt the other one at all, so merged to master with thanks.

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.