Bugzilla – Attachment 48155 Details for
Bug 15358
merge.pl does not populate values to merge
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[3.22.x] Bug 15358: Fix authorities merge
Bug-15358-Fix-authorities-merge.patch (text/plain), 7.87 KB, created by
Jonathan Druart
on 2016-02-17 10:30:20 UTC
(
hide
)
Description:
[3.22.x] Bug 15358: Fix authorities merge
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2016-02-17 10:30:20 UTC
Size:
7.87 KB
patch
obsolete
>From c96fdf93d2f101d9f5844a4457a8c07eb57a4664 Mon Sep 17 00:00:00 2001 >From: Julian Maurice <julian.maurice@biblibre.com> >Date: Wed, 6 Jan 2016 19:11:27 +0100 >Subject: [PATCH] Bug 15358: Fix authorities merge > >Bug 8064 (Merge several biblio records) change some code used in both >biblios and authorities merge tool without updating the authorities >merge tool. >This patch fixes that. > >Signed-off-by: Josef Moravec <josef.moravec@gmail.com> > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >--- > authorities/merge.pl | 20 ++++++++++++++++---- > cataloguing/merge.pl | 2 +- > .../intranet-tmpl/prog/en/includes/merge-record.inc | 20 ++++++++++---------- > .../prog/en/modules/authorities/merge.tt | 8 ++++---- > 4 files changed, 31 insertions(+), 19 deletions(-) > >diff --git a/authorities/merge.pl b/authorities/merge.pl >index 45785ee..4956335 100755 >--- a/authorities/merge.pl >+++ b/authorities/merge.pl >@@ -133,15 +133,27 @@ else { > > # Creating a loop for display > >- my @record1 = $recordObj1->createMergeHash($tagslib); >- my @record2 = $recordObj2->createMergeHash($tagslib); >+ my @records = ( >+ { >+ recordid => $mergereference, >+ record => $recordObj1->record, >+ frameworkcode => $recordObj1->authtype, >+ display => $recordObj1->createMergeHash($tagslib), >+ reference => 1, >+ }, >+ { >+ recordid => $notreference, >+ record => $recordObj2->record, >+ frameworkcode => $recordObj2->authtype, >+ display => $recordObj2->createMergeHash($tagslib), >+ }, >+ ); > > # Parameters > $template->param( > recordid1 => $mergereference, > recordid2 => $notreference, >- record1 => @record1, >- record2 => @record2, >+ records => \@records, > framework => $framework, > ); > } >diff --git a/cataloguing/merge.pl b/cataloguing/merge.pl >index a882daa..fc96d3c 100755 >--- a/cataloguing/merge.pl >+++ b/cataloguing/merge.pl >@@ -203,7 +203,7 @@ if ($merge) { > my $frameworkcode = GetFrameworkCode($biblionumber); > my $recordObj = new Koha::MetadataRecord({'record' => $marcrecord, schema => $marcflavour}); > my $record = { >- biblionumber => $biblionumber, >+ recordid => $biblionumber, > record => $marcrecord, > frameworkcode => $frameworkcode, > display => $recordObj->createMergeHash($tagslib), >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc >index 111252a..df79d8e 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/merge-record.inc >@@ -1,16 +1,16 @@ > [% BLOCK sourcetab %] >- <div id="tabrecord[% record.biblionumber %]"> >+ <div id="tabrecord[% record.recordid %]"> > <div class="record"> >- <ul id="ulrecord[% record.biblionumber %]"> >+ <ul id="ulrecord[% record.recordid %]"> > [% FOREACH field IN record.display %] > [% IF field.tag != biblionumbertag %] > <li id="k[% field.key %]"> > [% IF (tabrecord.reference) %] >- <input type="checkbox" checked="checked" class="fieldpick" id="rec_[% record.biblionumber %]_[% field.key %]" /> >+ <input type="checkbox" checked="checked" class="fieldpick" id="rec_[% record.recordid %]_[% field.key %]" /> > [% ELSE %] >- <input type="checkbox" class="fieldpick" id="rec_[% record.biblionumber %]_[% field.key %]" /> >+ <input type="checkbox" class="fieldpick" id="rec_[% record.recordid %]_[% field.key %]" /> > [% END %] >- <label for="rec_[% record.biblionumber %]_[% field.key %]"><span class="field">[% field.tag %]</span></label> >+ <label for="rec_[% record.recordid %]_[% field.key %]"><span class="field">[% field.tag %]</span></label> > > <input type="hidden" name="tag_[% field.tag %]_indicator1_[% field.key %]" value="[% field.indicator1 %]" /> > <input type="hidden" name="tag_[% field.tag %]_indicator2_[% field.key %]" value="[% field.indicator2 %]" /> >@@ -25,11 +25,11 @@ > [% FOREACH subfield IN field.subfield %] > <li id="k[% subfield.subkey %]"> > [% IF (tabrecord.reference) %] >- <input type="checkbox" checked="checked" class="subfieldpick" id="rec_[% record.biblionumber %]_[% subfield.subkey %]" /> >+ <input type="checkbox" checked="checked" class="subfieldpick" id="rec_[% record.recordid %]_[% subfield.subkey %]" /> > [% ELSE %] >- <input type="checkbox" class="subfieldpick" id="rec_[% record.biblionumber %]_[% subfield.subkey %]" /> >+ <input type="checkbox" class="subfieldpick" id="rec_[% record.recordid %]_[% subfield.subkey %]" /> > [% END %] >- <label for="rec_[% record.biblionumber %]_[% subfield.subkey %]"><span class="subfield">[% subfield.subtag %]</span> / [% subfield.value %]</label> >+ <label for="rec_[% record.recordid %]_[% subfield.subkey %]"><span class="subfield">[% subfield.subtag %]</span> / [% subfield.value %]</label> > <input type="hidden" name="tag_[% field.tag %]_code_[% subfield.subtag %]_[% field.key %]_[% subfield.subkey %]" value="[% subfield.subtag %]" /> > <input type="hidden" name="tag_[% field.tag %]_subfield_[% subfield.subtag %]_[% subfield.key %]_[% subfield.subkey %]" value="[% subfield.value |html%]" /> > </li> >@@ -50,8 +50,8 @@ > <ul> > [% FOREACH record IN sourcerecords %] > <li> >- <a href="#tabrecord[% record.biblionumber %]"> >- [% record.biblionumber %] >+ <a href="#tabrecord[% record.recordid %]"> >+ [% record.recordid %] > [% IF record.reference %](ref)[% END %] > </a> > </li> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt >index 520b9cd..3203c26 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/merge.tt >@@ -15,15 +15,15 @@ div#result { margin-top: 1em; } > > // When submiting the form > function mergeformsubmit() { >- $("ul#ulrecord1").remove(); >- $("ul#ulrecord2").remove(); >-} >+ $('#tabs').remove(); >+ } > > $(document).ready(function(){ > // Getting marc structure via ajax > tagslib = []; > $.getJSON("/cgi-bin/koha/authorities/merge_ajax.pl", {frameworkcode : "[% framework %]" }, function(json) { > tagslib = json; >+ rebuild_target($("#tabs"), $("#resultul")); > }); > > $('.preview-merge-reference').click(function (ev) { >@@ -129,7 +129,7 @@ function changeFramework(fw) { > > <div class="yui-g"> > <div class="yui-u first"> >-[% PROCESS mergesource %] >+[% PROCESS mergesource sourcerecords=records %] > </div> > <div class="yui-u"> > [% PROCESS mergetarget %] >-- >2.7.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 15358
:
45666
|
45767
|
46343
|
46393
|
46531
|
46651
|
46652
|
46653
|
47508
|
47509
|
47510
|
47537
|
47538
|
47539
|
47543
|
47640
|
47641
| 48155