Attachment 40144 Details for Bug 14387 – Bug 14387 - Merge reference selection has no effect when merging authorities.

[patch] Bug 14387 - Merge reference selection has no effect when merging authorities.

Bug-14387---Merge-reference-selection-has-no-effec.patch (text/plain), 1.41 KB, created by Maxime Beaulieu on 2015-06-15 15:12:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Maxime Beaulieu

Created: 2015-06-15 15:12:04 UTC

Size: 1.41 KB

patch

obsolete

>From b3d3fcce1fd4f190c86e3459b5590f456a79cd0e Mon Sep 17 00:00:00 2001
>From: mxbeaulieu <maxime.beaulieu@inlibro.com>
>Date: Mon, 15 Jun 2015 11:05:51 -0400
>Subject: [PATCH] Bug 14387 -  Merge reference selection has no effect when
> merging authorities.
>
>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
>---
> authorities/merge.pl |   11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
>diff --git a/authorities/merge.pl b/authorities/merge.pl
>index 6f2b154..476b774 100755
>--- a/authorities/merge.pl
>+++ b/authorities/merge.pl
>@@ -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
> 
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 14387: 40144 | 40695 | 40875