LP#1406786: fix remapping copies during parts merge
authorGalen Charlton <gmc@esilibrary.com>
Thu, 29 Oct 2015 19:30:31 +0000 (19:30 +0000)
committerDan Wells <dbw2@calvin.edu>
Wed, 4 Nov 2015 17:05:57 +0000 (12:05 -0500)
This patch fixes an issue with how copies are mapped
on the target part when using "Merge Selected" in the
Monograph Parts interface.

To test:

[1] Create some monograph parts on a record and assign
    some copies to use them.
[2] Use the merge parts function to collapse all of the
    parts created in step 1 onto a single part.
[3] Verify that the copies whose original parts were removed
    are now mapped onto the part that was the target of the
    merge.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/web/js/dojo/openils/biblio/monographPartMerge.js

index 59b557e..31644fd 100755 (executable)
@@ -71,10 +71,11 @@ if(!dojo._hasResource["openils.biblio.monographPartMerge"]) {
            dojo.forEach(cpmList,
                    function (g) {
                        g.part(parseInt(obj.itemID))            /* Assign "winner" DB id of mono_part. */
+                       g.ischanged(true);
                    });
 
            if (cpmList.length > 0) {
-               pcrud.update( cpmList, {});
+               pcrud.apply( cpmList);
            }
  
            /*