Bug 14387: Merge reference selection has no effect when merging authorities.
authormxbeaulieu <maxime.beaulieu@inlibro.com>
Mon, 15 Jun 2015 15:05:51 +0000 (11:05 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 24 Jul 2015 16:44:58 +0000 (13:44 -0300)
This patch swaps the authority records according to the refenrece record selection.

To TEST:
Merge two authority records, select the second as merge reference.
The reference authority is always the first.

Apply the patch.
Repeat previous steps, the authority is now merged using the selected reference record.

modified:   authorities/merge.pl

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
authorities/merge.pl

index 6f2b154..476b774 100755 (executable)
@@ -123,10 +123,13 @@ else {
                 }
             }
 
-            my $notreference =
-              ( $authid[0] == $mergereference )
-              ? $authid[1]
-              : $authid[0];
+            #Setting $notreference
+            my $notreference = $authid[1];
+            if($mergereference == $notreference){
+                $notreference = $authid[0];
+                #Swap so $recordObj1 is always the correct merge reference
+                ($recordObj1, $recordObj2) = ($recordObj2, $recordObj1);
+            }
 
             # Creating a loop for display