Bugzilla – Attachment 114924 Details for
Bug 11299
Add a button to automatically link authority records in cataloguing (AJAX)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 11299: Add a button to the biblio edition page to automatically add authority links in the current biblio record via AJAX. Also adds a button to easily create missing authority records.
Bug-11299-Add-a-button-to-the-biblio-edition-page-.patch (text/plain), 34.74 KB, created by
Nick Clemens (kidclamp)
on 2021-01-07 16:05:30 UTC
(
hide
)
Description:
Bug 11299: Add a button to the biblio edition page to automatically add authority links in the current biblio record via AJAX. Also adds a button to easily create missing authority records.
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2021-01-07 16:05:30 UTC
Size:
34.74 KB
patch
obsolete
>From bc6690e8865cd3c8bd213897bc8befca5a9b99c6 Mon Sep 17 00:00:00 2001 >From: Bouzid Fergani <bouzid.fergani@inlibro.com> >Date: Wed, 6 Nov 2013 10:49:09 -0500 >Subject: [PATCH] Bug 11299: Add a button to the biblio edition page to > automatically add authority links in the current biblio record via AJAX. Also > adds a button to easily create missing authority records. > > add authority type in the form to create the missing authority. > when authority was found, the 600$9 field have the authid. > >Testing scenario (Creating an authority record for a failed automatic link) : > > 1 - In your system preferences set: > AutoCreateAuthorities: Don't generate > BiblioAddsAuthorities: Allow > 2 - Go to the Cataloging -> New record (koha/cataloguing/addbiblio.pl) > Ensure you are using the basic editor > 3 - Click the "Link authorities automatically" button. > A message should appear, telling the user "No authority link was changed." > 4 - Add random informations in field 600$a of the biblio record. > 5 - Click the "Link authorities automatically" button. > the message box should now show "600 - No matching authority found.". > the 9 subfield is red > Above the 9 subfield is a red X with a blue plus next to it > Hover on the plus, see it is titled 'Create authority' > 6 - Click the 'Create authority' link > 7 - A new authroity form pops up, the info from the cataloging editor is prefilled > Click the 100 field heading to expand and confirm info is transferred > 8 - Fill in necessary fields and save the new authority > 9 - The cataloging screen now has the 9 subfield populated and is green >10 - Click "Link authorities automatically" again > Dialog says "No authority link was changed" >11 - In another tab go to System preferences and set AutoCreateAuthorities to 'Generate' >12 - Add random information to the 650 field >13 - Click 'Link authorities' button >14 - Dialog says:650 - No matching authority found. A new authority was created automatically. >15 - The subfield 9 is green and has the id of the new authority record >16 - In another tab search authorities and find an existing subject heading >17 - Add a new 650 with the info from the existing record >18 - Click 'Link authorities' >19 - The new field is correctly linked to existing authority > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> > >https://bugs.koha-community.org/show_bug.cgi?id=12299 >Signed-off-by: Michal Denar <black23@gmail.com> > >Signed-off-by: Nick Clemens <nick@bywatersolutions.com> > >Signed-off-by: Michal Denar <black23@gmail.com> >--- > C4/Biblio.pm | 25 ++- > authorities/authorities.pl | 5 +- > cataloguing/z3950_auth_search.pl | 3 + > koha-tmpl/intranet-tmpl/prog/css/addbiblio.css | 16 ++ > .../prog/en/modules/authorities/authorities.tt | 7 +- > .../authorities/blinddetail-biblio-search.tt | 19 +- > .../prog/en/modules/cataloguing/addbiblio.tt | 241 ++++++++++++++++++++- > .../en/modules/cataloguing/z3950_auth_search.tt | 3 +- > koha-tmpl/intranet-tmpl/prog/js/cataloging.js | 5 + > koha-tmpl/intranet-tmpl/prog/js/z3950_search.js | 7 +- > svc/cataloguing/automatic_linker.pl | 61 ++++++ > t/Biblio.t | 2 +- > t/db_dependent/Biblio.t | 28 ++- > 13 files changed, 404 insertions(+), 18 deletions(-) > create mode 100755 svc/cataloguing/automatic_linker.pl > >diff --git a/C4/Biblio.pm b/C4/Biblio.pm >index 6e53842ee1..88a69153e3 100644 >--- a/C4/Biblio.pm >+++ b/C4/Biblio.pm >@@ -499,6 +499,7 @@ Returns the number of headings changed > sub BiblioAutoLink { > my $record = shift; > my $frameworkcode = shift; >+ my $verbose = shift; > if (!$record) { > carp('Undefined record passed to BiblioAutoLink'); > return 0; >@@ -516,15 +517,15 @@ sub BiblioAutoLink { > > my $linker = $linker_module->new( > { 'options' => C4::Context->preference("LinkerOptions") } ); >- my ( $headings_changed, undef ) = >- LinkBibHeadingsToAuthorities( $linker, $record, $frameworkcode, C4::Context->preference("CatalogModuleRelink") || '' ); >+ my ( $headings_changed, $results ) = >+ LinkBibHeadingsToAuthorities( $linker, $record, $frameworkcode, C4::Context->preference("CatalogModuleRelink") || '', $verbose ); > # By default we probably don't want to relink things when cataloging >- return $headings_changed; >+ return $headings_changed, $results; > } > > =head2 LinkBibHeadingsToAuthorities > >- my $num_headings_changed, %results = LinkBibHeadingsToAuthorities($linker, $marc, $frameworkcode, [$allowrelink]); >+ my $num_headings_changed, %results = LinkBibHeadingsToAuthorities($linker, $marc, $frameworkcode, [$allowrelink, $verbose]); > > Links bib headings to authority records by checking > each authority-controlled field in the C<MARC::Record> >@@ -547,6 +548,7 @@ sub LinkBibHeadingsToAuthorities { > my $frameworkcode = shift; > my $allowrelink = shift; > my $tagtolink = shift; >+ my $verbose = shift; > my %results; > if (!$bib) { > carp 'LinkBibHeadingsToAuthorities called on undefined bib record'; >@@ -554,7 +556,6 @@ sub LinkBibHeadingsToAuthorities { > } > require C4::Heading; > require C4::AuthoritiesMarc; >- > $allowrelink = 1 unless defined $allowrelink; > my $num_headings_changed = 0; > foreach my $field ( $bib->fields() ) { >@@ -570,6 +571,7 @@ sub LinkBibHeadingsToAuthorities { > if ( defined $current_link && (!$allowrelink || !C4::Context->preference('LinkerRelink')) ) > { > $results{'linked'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $current_link, status => 'UNCHANGED'}) if $verbose; > next; > } > >@@ -577,17 +579,23 @@ sub LinkBibHeadingsToAuthorities { > if ($authid) { > $results{ $fuzzy ? 'fuzzy' : 'linked' } > ->{ $heading->display_form() }++; >- next if defined $current_link and $current_link == $authid; >+ if(defined $current_link and $current_link == $authid) { >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $current_link, status => 'UNCHANGED'}) if $verbose; >+ next; >+ } > > $field->delete_subfield( code => '9' ) if defined $current_link; > $field->add_subfields( '9', $authid ); > $num_headings_changed++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $authid, status => 'LOCAL_FOUND'}) if $verbose; > } > else { >+ my $authority_type = Koha::Authority::Types->find( $heading->auth_type() ); > if ( defined $current_link > && (!$allowrelink || C4::Context->preference('LinkerKeepStale')) ) > { > $results{'fuzzy'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $current_link, status => 'UNCHANGED'}) if $verbose; > } > elsif ( C4::Context->preference('AutoCreateAuthorities') ) { > if ( _check_valid_auth_link( $current_link, $field ) ) { >@@ -663,24 +671,29 @@ sub LinkBibHeadingsToAuthorities { > $num_headings_changed++; > $linker->update_cache($heading, $authid); > $results{'added'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $authid, status => 'CREATED'}) if $verbose; > } > } > elsif ( defined $current_link ) { > if ( _check_valid_auth_link( $current_link, $field ) ) { > $results{'linked'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => $authid, status => 'UNCHANGED'}) if $verbose; > } > else { > $field->delete_subfield( code => '9' ); > $num_headings_changed++; > $results{'unlinked'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => undef, status => 'NONE_FOUND', auth_type => $heading->auth_type(), tag_to_report => $authority_type->auth_tag_to_report}) if $verbose; > } > } > else { > $results{'unlinked'}->{ $heading->display_form() }++; >+ push(@{$results{'details'}}, { tag => $field->tag(), authid => undef, status => 'NONE_FOUND', auth_type => $heading->auth_type(), tag_to_report => $authority_type->auth_tag_to_report}) if $verbose; > } > } > > } >+ push(@{$results{'details'}}, { tag => '', authid => undef, status => 'UNCHANGED'}) unless %results; > return $num_headings_changed, \%results; > } > >diff --git a/authorities/authorities.pl b/authorities/authorities.pl >index c8de9de788..bf555aa5c7 100755 >--- a/authorities/authorities.pl >+++ b/authorities/authorities.pl >@@ -289,6 +289,9 @@ sub create_input { > > } > } >+ if ($cgi->param('tagreport') && $subfield_data{tag} == $cgi->param('tagreport')) { >+ $subfield_data{marc_value}{value} = $cgi->param('tag'. $cgi->param('tagbiblio') . 'subfield' . $subfield_data{subfield}); >+ } > $subfield_data{'index_subfield'} = $index_subfield; > return \%subfield_data; > } >@@ -576,7 +579,7 @@ my ($template, $loggedinuser, $cookie) > flagsrequired => {editauthorities => 1}, > debug => 1, > }); >-$template->param(nonav => $nonav,index=>$myindex,authtypecode=>$authtypecode,breedingid=>$breedingid,); >+$template->param(nonav => $nonav,index=>$myindex,authtypecode=>$authtypecode,breedingid=>$breedingid); > > $tagslib = GetTagsLabels(1,$authtypecode); > $mandatory_z3950 = GetMandatoryFieldZ3950($authtypecode); >diff --git a/cataloguing/z3950_auth_search.pl b/cataloguing/z3950_auth_search.pl >index 615d7e7ce3..e97c47d91f 100755 >--- a/cataloguing/z3950_auth_search.pl >+++ b/cataloguing/z3950_auth_search.pl >@@ -42,6 +42,7 @@ my $subjectsubdiv = $input->param('subjectsubdiv'); > my $srchany = $input->param('srchany'); > my $op = $input->param('op')||''; > my $page = $input->param('current_page') || 1; >+my $index =$input->param('index'); > $page = $input->param('goto_page') if $input->param('changepage_goto'); > my $controlnumber = $input->param('controlnumber'); > >@@ -65,6 +66,7 @@ $template->param( > srchany => $srchany, > authid => $authid, > controlnumber => $controlnumber, >+ index => $index, > ); > > if ( $op ne "do_search" ) { >@@ -74,6 +76,7 @@ if ( $op ne "do_search" ) { > $template->param( > serverloop => $serverloop, > opsearch => "search", >+ index => $index, > ); > output_html_with_http_headers $input, $cookie, $template->output; > exit; >diff --git a/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css b/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css >index febbd2096d..4592e25a20 100644 >--- a/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css >+++ b/koha-tmpl/intranet-tmpl/prog/css/addbiblio.css >@@ -352,6 +352,22 @@ tbody tr.active td { > display: inline-block; > } > >+.matching_authority{ >+ color: #99FF99; >+} >+ >+.no_matching_authority{ >+ color: #FFAAAA; >+} >+ >+.matching_authority_field{ >+ background-color: #99FF99 !important; >+ } >+ >+ .no_matching_authority_field{ >+ background-color: #FFAAAA !important; >+ } >+ > @media (min-width: 768px) { > li.subfield_line label { > width: 20em; >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >index 601726a25f..ed34c91b91 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt >@@ -32,8 +32,9 @@ > }); > $("#z3950submit").click(function(){ > var strQuery = GetZ3950Terms(); >+ var index = "&index=[% index | html %]"; > if(strQuery){ >- window.open("/cgi-bin/koha/cataloguing/z3950_auth_search.pl?authid=[% authid | html %]"+strQuery,"z3950search",'width=740,height=450,location=yes,toolbar=no,scrollbars=yes,resize=yes'); >+ window.open("/cgi-bin/koha/cataloguing/z3950_auth_search.pl?authid=[% authid | html %]"+strQuery+index,"z3950search",'width=740,height=450,location=yes,toolbar=no,scrollbars=yes,resize=yes'); > } > return false; > }); >@@ -211,9 +212,9 @@ function confirmnotdup(redirect){ > <input type="hidden" name="op" value="add" /> > <input type="hidden" name="addfield_field" value="" /> > <input type="hidden" name="repeat_field" value="" /> >- <input type="hidden" name="authtypecode" value="[% authtypecode | html %]" /> >+ <input type="hidden" name="authtypecode" value="[% authtypecode | html%]" /> > <input type="hidden" name="authid" value="[% authid | html %]" /> >- <input type="hidden" name="index" value="[% index | html %]" /> >+ <input type="hidden" name="index" value="[% index | html %]" /> > <input type="hidden" value="0" id="confirm_not_duplicate" name="confirm_not_duplicate" /> > > <div id="toolbar" class="btn-toolbar"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-search.tt >index 16360262f5..e86f46db77 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-search.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/blinddetail-biblio-search.tt >@@ -44,8 +44,10 @@ > var whichfield; > try { > whichfield = opener.opener.document.getElementById(index_start); >+ autolinker_dialog = opener.opener.document.getElementById("autolinker_dialog"); > } catch(e) { >- return; >+ whichfield = opener.document.getElementById(index_start); >+ autolinker_dialog = opener.document.getElementById("autolinker_dialog"); > } > var field_start = whichfield.parentNode.parentNode.parentNode.parentNode; > >@@ -138,6 +140,7 @@ > [% ELSE %] > if(code.value=='9'){ > subfield.value = "[% To.json(authid) | $raw %]"; >+ subfield.className = subfield.className.replace("no_matching_authority_field", "matching_authority_field"); > break; > } > [% END %] >@@ -145,6 +148,20 @@ > } > [% END %] > >+ //Delete create authority button and status >+ var linkinfos = field_start.getElementsByTagName('i'); >+ for (index = linkinfos.length - 1; index >= 0; index--) { >+ linkinfos[index].parentNode.removeChild(linkinfos[index]); >+ } >+ //Change message autolinker_dialog >+ var subfield_dialog = $(autolinker_dialog).children('ul').children('li'); >+ for (index = subfield_dialog.length - 1; index >= 0; index--){ >+ var tag_field = field_start.id.split("_")[1]; >+ if (subfield_dialog[index].innerHTML.indexOf(tag_field) !== -1 ){ >+ subfield_dialog[index].innerHTML = subfield_dialog[index].innerHTML.replace(_("No matching authority found."), _("No matching authority found. A new authority was created or imported")); >+ break; >+ } >+ } > opener.close(); > window.close(); > return false; >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >index 32aeac779d..be5bc2b0df 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt >@@ -91,6 +91,10 @@ > PopupZ3950(); > }); > >+ $("#linkerbutton").click(function(){ >+ AutomaticLinker(); >+ }); >+ > $("#saverecord").click(function(){ > $(".btn-group").removeClass("open"); > onOption(); >@@ -177,8 +181,240 @@ > var onOption = function () { > return Check(); > } >- [% END %] >+[% END %] >+ >+function confirmnotdup(redirect){ >+ $("#confirm_not_duplicate").attr("value","1"); >+ $("#redirect").attr("value",redirect); >+ Check(); >+} >+ >+function Dopop(link,i) { >+ defaultvalue = document.getElementById(i).value; >+ window.open(link+"&result="+defaultvalue,"valuebuilder",'width=700,height=550,toolbar=false,scrollbars=yes'); >+} >+ >+/** >+ * this function open a popup to search on z3950 server. >+ */ >+function PopupZ3950() { >+ var strQuery = GetZ3950Terms(); >+ if(strQuery){ >+ window.open("/cgi-bin/koha/cataloguing/z3950_search.pl?biblionumber=[% biblionumber | html %]"+strQuery,"z3950search",'width=740,height=450,location=yes,toolbar=no,scrollbars=yes,resize=yes'); >+ } >+} >+ >+/** >+* this function append button for create new authority if not found >+*/ >+ >+function addCreateAuthorityButton(tag_subfield_line, heading, tag_index) { >+ var title = _("Create authority"); >+ var elem = $('<a href="#" title="' + title + '"><i class="fa fa-plus-circle"></i></a>'); >+ tag_subfield_line.append(elem); >+ var tag_subfield_line_a = $('.subfield_line[id^=subfield' + heading.tag + 'a]').eq(tag_index); >+ var subfield_a = tag_subfield_line_a.children('.input_marceditor').eq(0); >+ var index = subfield_a.attr('id'); >+ elem.click(function() { >+ var popup = window.open("", "new_auth_popup",'fullscreen,toolbar=false,scrollbars=yes'); >+ if(popup !== null) { >+ // Create a new form that will be POSTed in the new window >+ var form = $('<form>').attr({ >+ method: 'post', >+ action: "../authorities/authorities.pl", >+ target: "new_auth_popup" >+ }); >+ //add the authtypecode >+ form.append($('<input>').attr({ >+ type: 'hidden', >+ name: 'authtypecode', >+ value: heading.auth_type >+ })); >+ form.append($('<input>').attr({ >+ type: 'hidden', >+ name: 'tagreport', >+ value: heading.tag_to_report >+ })); >+ form.append($('<input>').attr({ >+ type: 'hidden', >+ name: 'tagbiblio', >+ value: heading.tag >+ })); >+ form.append($('<input>').attr({ >+ type: 'hidden', >+ name: 'index', >+ value: index >+ })); >+ $('.tag[id^=tag_' + heading.tag + '_]').eq(tag_index).find(':input').each(function(){ >+ form.append($('<input>').attr({ >+ type: 'hidden', >+ name: this.name.split('_',4).join(''), >+ value: $(this).val() >+ })); >+ }); >+ >+ $('body').append(form); >+ form.submit(); >+ form.remove(); >+ } >+ return false; >+ }); >+ >+} >+ >+/** >+ * Updates the authid for every heading field >+ * Adds visual feedback for the changes made on the form. >+ */ >+function updateHeadingLinks(links) { >+ var current_tag = ''; >+ var tag_index = 0; >+ >+ // Delete the old message dialog and create a new one >+ $('#autolinker_dialog').remove(); >+ var message_dialog = $('<div id="autolinker_dialog" class="dialog"><strong>' + _("Automatic authority link results:") + '</strong><ul></ul></div>'); >+ var message_dialog_ul = message_dialog.find('ul'); >+ >+ $.each(links, function(index, heading) { >+ if(current_tag == heading.tag) { >+ tag_index++; >+ } >+ else { >+ current_tag = heading.tag; >+ tag_index = 0; >+ } >+ >+ // Find the $9 field to update >+ var tag_subfield_line = $('.subfield_line[id^=subfield' + heading.tag + '9]').eq(tag_index); >+ var subfield = tag_subfield_line.children('.input_marceditor').eq(0); >+ >+ // Delete the old status if one exists >+ tag_subfield_line.children('.subfield_status').remove(); >+ >+ // If the field wasn't modified. Skip it. >+ if(heading.status == 'UNCHANGED') { >+ return; >+ } >+ >+ >+ // Make the subfield line visible and update its value >+ tag_subfield_line.show(); >+ subfield.val(heading.authid); >+ >+ // Add the new status >+ var image = '<i class="fa fa-close no_matching_authority" </i> '; >+ var message = ''; >+ var field_class = 'no_matching_authority_field'; >+ switch(heading.status) { >+ case 'LOCAL_FOUND': >+ image = '<i class="fa fa-check matching_authority"</i> '; >+ message = _("A matching authority was found in the local database."); >+ field_class = 'matching_authority_field'; >+ break; >+ case 'CREATED': >+ image = '<i class="fa fa-check matching_authority"></i> '; >+ message = _("No matching authority found. A new authority was created automatically."); >+ field_class = 'matching_authority_field'; >+ break; >+ case 'MULTIPLE_MATCH': >+ message = _("More than one local match found. Possibly a duplicate authority!"); >+ break; >+ case 'NONE_FOUND': >+ message = _("No matching authority found."); >+ break; >+ default: >+ message = heading.status; >+ break; >+ } > >+ subfield[0].classList.add(field_class); >+ tag_subfield_line.find('i').each(function() { >+ this.remove(); >+ }); >+ tag_subfield_line.append(image); >+ >+ // Add the message to the dialog >+ message_dialog_ul.append('<li><strong>' + heading.tag + '</strong> - ' + message + '</li>'); >+ >+ // Add a link to create a new authority if none was found >+ if(heading.status == 'NONE_FOUND' && tag_subfield_line.find('i').length == 1) { >+ addCreateAuthorityButton(tag_subfield_line, heading , tag_index); >+ } >+ }); >+ >+ if(message_dialog.find('li').length == 0) { >+ message_dialog_ul.append("<li>" + _("No authority link was changed.") + "</li>"); >+ } >+ $('#addbibliotabs').before(message_dialog); >+} >+ >+/** >+ * Use an ajax request to automatically find authority links for the current record >+ */ >+function AutomaticLinker() { >+ // Show the Loading overlay >+ $("#loading").show(); >+ >+ // Remove fields that are completely empty >+ $('#f').find('.tag').each(function() { >+ var empty = true; >+ $(this).find('.input_marceditor').each(function() { >+ if($(this).val() != '') { >+ empty = false; >+ return false; >+ } >+ }); >+ if(empty) { >+ UnCloneField($(this).attr('id')); >+ } >+ }); >+ >+ // Get all the form values to post via AJAX >+ var form_data = {}; >+ $('#f').find(':input').each(function(){ >+ form_data[this.name] = $(this).val(); >+ }); >+ delete form_data['']; >+ >+ // Send the data to automatic_linker.pl >+ $.ajax({ >+ url:'/cgi-bin/koha/svc/cataloguing/automatic_linker.pl', >+ type:'post', >+ data: form_data, >+ dataType: 'json', >+ error: function(xhr) { >+ alert("Error : \n" + xhr.responseText); >+ }, >+ success: function(json) { >+ switch(json.status) { >+ case 'UNAUTHORIZED': >+ alert(_("Error : You do not have the permissions necessary to use this functionality.")); >+ break; >+ case 'OK': >+ updateHeadingLinks(json.links); >+ break; >+ } >+ }, >+ complete: function() { >+ $("#loading").hide(); >+ } >+ }); >+} >+ >+ >+function PopupMARCFieldDoc(field) { >+ [% IF Koha.Preference('marcfielddocurl') %] >+ var docurl = "[% Koha.Preference('marcfielddocurl').replace('"','"') | html %]"; >+ docurl = docurl.replace("{MARC}", "[% marcflavour | html %]"); >+ docurl = docurl.replace("{FIELD}", ""+field); >+ docurl = docurl.replace("{LANG}", "[% lang | html %]"); >+ window.open(docurl); >+ [% ELSIF ( marcflavour == 'MARC21' ) %] >+ _MARC21FieldDoc(field); >+ [% ELSIF ( marcflavour == 'UNIMARC' ) %] >+ _UNIMARCFieldDoc(field); >+ [% END %] >+} > function confirmnotdup(redirect){ > $("#confirm_not_duplicate").attr("value","1"); > $("#redirect").attr("value",redirect); >@@ -587,6 +823,9 @@ > [% ELSE %] > <div class="btn-group"><a class="btn btn-default" href="#" id="z3950search"><i class="fa fa-search"></i> Z39.50/SRU search</a></div> > [% END %] >+ [% IF ( CAN_user_editauthorities ) %] >+ <div class="btn-group"><a class="btn btn-default" href="#" id="linkerbutton"><i class="fa fa-refresh"></i> Link authorities automatically</a></div> >+ [% END %] > [% END %] > > <div class="btn-group"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_auth_search.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_auth_search.tt >index 51aa454619..ee07f953d7 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_auth_search.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_auth_search.tt >@@ -20,6 +20,7 @@ > <form method="post" action="z3950_auth_search.pl" name="f" class="checkboxed"> > <input type="hidden" name="op" id="op" value="do_search" /> > <input type="hidden" name="authid" value="[% authid | html %]" /> >+ <input type="hidden" name="index" value="[% index | html %]" /> > <div class="row"> > <div class="col-xs-6"> > <fieldset class="rows"> >@@ -94,7 +95,7 @@ > </a> > <ul class="dropdown-menu pull-right" role="menu" aria-labelledby="cataloguesearchactions[% breeding_loo.breedingid | html %]"> > <li><a href="/cgi-bin/koha/catalogue/showmarc.pl?importid=[% breeding_loo.breedingid | uri %]" class="previewMARC"><i class="fa fa-eye"></i> MARC preview</a></li> >- <li><a href="#" class="import_record" data-breedingid="[% breeding_loo.breedingid | html %]" data-heading_code="[% breeding_loo.heading_code | html %]" data-authid="[% breeding_loo.authid | html %]"><i class="fa fa-download"></i> Import</a></li> >+ <li><a href="#" class="import_record" data-breedingid="[% breeding_loo.breedingid | html %]" data-heading_code="[% breeding_loo.heading_code | html %]" data-authid="[% breeding_loo.authid | html %]" data-index="[% index | html %]" ><i class="fa fa-download"></i> Import</a></li> > </ul> > </div> > </td> >diff --git a/koha-tmpl/intranet-tmpl/prog/js/cataloging.js b/koha-tmpl/intranet-tmpl/prog/js/cataloging.js >index a66edff559..b7b9ab3d5c 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/cataloging.js >+++ b/koha-tmpl/intranet-tmpl/prog/js/cataloging.js >@@ -187,6 +187,9 @@ function CloneField(index, hideMarc, advancedMARCEditor) { > for( j = 0 ; j < inputs.length ; j++ ) { > if(inputs[j].getAttribute("id") && inputs[j].getAttribute("id").match(/^tag_/) ){ > inputs[j].value = ""; >+ >+ //Remove the color added by the automatic linker >+ $(inputs[j]).css({backgroundColor:""}); > } > } > var textareas = divs[i].getElementsByTagName('textarea'); >@@ -195,6 +198,8 @@ function CloneField(index, hideMarc, advancedMARCEditor) { > textareas[j].value = ""; > } > } >+ // Remove the status icons added by the automatic linker >+ $(divs[i]).find('.subfield_status').remove(); > if( inputs.length > 0 ){ > inputs[0].setAttribute('id',inputs[0].getAttribute('id')+new_key); > inputs[0].setAttribute('name',inputs[0].getAttribute('name')+new_key); >diff --git a/koha-tmpl/intranet-tmpl/prog/js/z3950_search.js b/koha-tmpl/intranet-tmpl/prog/js/z3950_search.js >index 39a7af0c0d..cafa6ad016 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/z3950_search.js >+++ b/koha-tmpl/intranet-tmpl/prog/js/z3950_search.js >@@ -1,11 +1,11 @@ > /* global __ total_pages */ > //z3950_search.js for Authorities, Bib records and Acquisitions module >-function Import(Breeding, recordid, AuthType, FrameworkCode) { >+function Import(Breeding, recordid, AuthType, FrameworkCode, index) { > > if ( AuthType == false ) { > opener.document.location="../cataloguing/addbiblio.pl?biblionumber="+recordid+"&z3950=1&frameworkcode="+FrameworkCode+"&breedingid="+Breeding; > } else { >- opener.document.location="../authorities/authorities.pl?breedingid="+Breeding+"&authtypecode="+AuthType+"&authid="+recordid; >+ opener.document.location="../authorities/authorities.pl?breedingid="+Breeding+"&authtypecode="+AuthType+"&authid="+recordid+"&index="+index; > } > window.close(); > return false; >@@ -141,10 +141,11 @@ $( document ).ready( function() { > var data_authid = $( this ).data( "authid" ); > var data_biblionumber = $( this ).data( "biblionumber" ); > var data_frameworkcode = $( this ).data( "frameworkcode" ); >+ var data_index = $( this ).data( "index" ); > if ( data_headingcode == undefined ) { > Import( data_breedingid, data_biblionumber, false , data_frameworkcode ); > } else { >- Import( data_breedingid, data_authid, data_headingcode ); >+ Import( data_breedingid, data_authid, data_headingcode, "", data_index ); > } > return false; > }); >diff --git a/svc/cataloguing/automatic_linker.pl b/svc/cataloguing/automatic_linker.pl >new file mode 100755 >index 0000000000..711debf92d >--- /dev/null >+++ b/svc/cataloguing/automatic_linker.pl >@@ -0,0 +1,61 @@ >+#!/usr/bin/perl >+ >+# Bouzid Fergani, 2020 - inLibro >+# >+# This file is part of Koha. >+# >+# Koha is free software; you can redistribute it and/or modify it under the >+# terms of the GNU General Public License as published by the Free Software >+# Foundation; either version 3 of the License, or (at your option) any later >+# version. >+# >+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY >+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR >+# A PARTICULAR PURPOSE. See the GNU General Public License for more details. >+# >+# You should have received a copy of the GNU General Public License along >+# with Koha; if not, write to the Free Software Foundation, Inc., >+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. >+ >+use Modern::Perl; >+use CGI; >+use CGI::Cookie; >+use JSON; >+use C4::Auth; >+use C4::Biblio; >+use C4::Context; >+ >+my $input = new CGI; >+print $input->header('application/json'); >+ >+# Check the user's permissions >+my %cookies = CGI::Cookie->fetch; >+my $sessid = $cookies{'CGISESSID'}->value || $input->param('CGISESSID'); >+my ( $auth_status, $auth_sessid ) = >+ C4::Auth::check_cookie_auth( $sessid, { editauthorities => 1, editcatalogue => 1 } ); >+if ( $auth_status ne "ok" ) { >+ print to_json( { status => 'UNAUTHORIZED' } ); >+ exit 0; >+} >+ >+# Link the biblio headings to authorities and return a json containing the status of all the links. >+# Example : {"status":"OK","links":[{"authid":"123","status":"LINK_CHANGED","tag":"650"}]} >+# >+# tag = the tag number of the field >+# authid = the value of the $9 subfield for this tag >+# status = The status of the link (LOCAL_FOUND, NONE_FOUND, MULTIPLE_MATCH, UNCHANGED, CREATED) >+ >+my $json; >+ >+my $record = TransformHtmlToMarc($input,1); >+ >+my ( $headings_changed, $results ) = BiblioAutoLink ( >+ $record, >+ $input->param('frameworkcode'), >+ 1 >+); >+ >+$json->{status} = 'OK'; >+$json->{links} = $results->{details} || ''; >+ >+print to_json($json); >diff --git a/t/Biblio.t b/t/Biblio.t >index 5596b4bf19..a847edaa97 100755 >--- a/t/Biblio.t >+++ b/t/Biblio.t >@@ -157,4 +157,4 @@ warnings_like { $ret = UpdateTotalIssues() } > "UpdateTotalIssues returns carped warnings if biblio record does not exist"; > > ok( !defined $ret, 'UpdateTotalIssues returns carped warning if biblio record does not exist'); >- >+1; >diff --git a/t/db_dependent/Biblio.t b/t/db_dependent/Biblio.t >index ac6a6a1255..4286565018 100755 >--- a/t/db_dependent/Biblio.t >+++ b/t/db_dependent/Biblio.t >@@ -732,7 +732,7 @@ subtest 'ModBiblio called from linker test' => sub { > }; > > subtest "LinkBibHeadingsToAuthorities record generation tests" => sub { >- plan tests => 6; >+ plan tests => 12; > > # Set up mocks to ensure authorities are generated > my $biblio_mod = Test::MockModule->new( 'C4::Linker::Default' ); >@@ -766,6 +766,20 @@ subtest "LinkBibHeadingsToAuthorities record generation tests" => sub { > "The generated record contains the correct subfields" > ); > >+ #Add test for this case using verbose >+ my ( $num_headings_changed, $results ) = LinkBibHeadingsToAuthorities($linker, $record, "",undef, 1); >+ my $details = $results->{details}; >+ is( $num_headings_changed, 1, 'We changed the one we passed' ); >+ is( $results->{details}->[0]->{status}, 'CREATED', "We added an authority record for the heading using verbose"); >+ >+ # Now we check the authority record itself >+ my $authority = GetAuthority($results->{details}->[0]->{authid}); >+ >+ is( $authority->field('150')->as_string(), >+ "Beach city Weirdness Fiction Books 21st Century Fish Stew Pizza", >+ "The generated record contains the correct subfields when using verbose" >+ ); >+ > # Example series link with volume and punctuation > $record = MARC::Record->new(); > $field = MARC::Field->new('800','','','a' => 'Tolkien, J. R. R.', 'q' => '(John Ronald Reuel),', 'd' => '1892-1973.', 't' => 'Lord of the rings ;', 'v' => '1'); >@@ -785,6 +799,18 @@ subtest "LinkBibHeadingsToAuthorities record generation tests" => sub { > "Tolkien, J. R. R. (John Ronald Reuel), 1892-1973. Lord of the rings", > "The generated record contains the correct subfields" > ); >+ >+ # The same exemple With verbose >+ ( $num_headings_changed, $results ) = LinkBibHeadingsToAuthorities($linker, $record, "",undef, 1); >+ is( $num_headings_changed, 1, 'We changed the one we passed' ); >+ is( $results->{details}->[0]->{status}, 'CREATED', "We added an authority record for the heading using verbose"); >+ >+ # Now we check the authority record itself >+ my $authority = GetAuthority($results->{details}->[0]->{authid}); >+ is( $authority->field('100')->as_string(), >+ "Tolkien, J. R. R. (John Ronald Reuel), 1892-1973. Lord of the rings", >+ "The generated record contains the correct subfields" >+ ); > }; > > # Cleanup >-- >2.11.0
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=114924">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11299
:
23146
|
26715
|
33074
|
37912
|
37913
|
37972
|
38737
|
39477
|
39478
|
39479
|
40831
|
41229
|
45218
|
56440
|
56508
|
56509
|
58307
|
60998
|
63311
|
63312
|
63313
|
64075
|
67162
|
67708
|
67709
|
67710
|
67712
|
67713
|
67714
|
67824
|
67825
|
67826
|
67827
|
67828
|
67829
|
79385
|
79386
|
79387
|
79388
|
79389
|
79390
|
79391
|
79392
|
79393
|
79394
|
79395
|
79396
|
79397
|
79398
|
79399
|
79447
|
79448
|
79449
|
79450
|
79451
|
79452
|
79545
|
94818
|
95126
|
95152
|
98571
|
98572
|
99249
|
108091
|
108309
|
108310
|
108396
|
108552
|
108834
|
108835
|
109062
|
109063
|
109069
|
109181
|
110553
|
110554
|
110555
|
110556
|
110557
|
110568
|
110569
|
110570
|
110571
|
110572
|
111706
|
111707
|
111708
|
111709
|
111710
|
112125
|
112126
|
112127
|
112128
|
112129
|
112130
|
112131
|
112132
|
112133
|
112134
|
112135
|
113562
|
113563
|
113564
|
113565
|
113566
|
113567
|
113568
|
114803
| 114924 |
114925
|
114926
|
114927
|
114928
|
114929
|
114930
|
114931
|
114932