Attachment 124220 Details for Bug 28843 – Bug 28843: Add view and edit buttons to result of MARC record import

[patch] Bug 28843: Add view and edit buttons to result of MARC record import

Bug-28843-Add-view-and-edit-buttons-to-result-of-M.patch (text/plain), 3.43 KB, created by Katrin Fischer on 2021-08-29 11:06:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2021-08-29 11:06:14 UTC

Size: 3.43 KB

patch

obsolete

>From 2700424a6af8a42cfb11efda1a49f039dee0bc33 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 11 Aug 2021 12:21:19 +0000
>Subject: [PATCH] Bug 28843: Add view and edit buttons to result of MARC record
> import
>
>This patch adds "View" and "Edit" buttons to the output of the result of
>a MARC record import.
>
>To test, apply the patch and go to Tools -> State MARC for import.
>
> - Import a file of MARC records.
> - Click the "Manage staged records" button.
> - In the table of staged records the last column labeled "Records"
>   should be empty.
> - Click "Import this batch into the catalog."
> - The table of records will be shown again, and this time the last
>   column should contain "View" and "Edit" buttons for each row.
> - Confirm that the buttons work correctly, opening the correct record
>   for viewing and editing.
>
>Test with both bibliographic and authority records to confirm that the
>correct view and edit pages open for each.
>
>EDIT: Add permission check for "Edit" link
>Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../prog/en/modules/tools/manage-marc-import.tt       | 19 +++++++++++++++----
> 1 file changed, 15 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>index 601d0a3dbd..75777fc3a3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt
>@@ -508,6 +508,11 @@
>                         [% ELSE %]
>                             var record_details_url = "/cgi-bin/koha/catalogue/detail.pl?biblionumber=";
>                         [% END %]
>+                        [% IF(record_type == 'auth') %]
>+                            var record_edit_url = "/cgi-bin/koha/authorities/authorities.pl?authid=";
>+                        [% ELSE %]
>+                            var record_edit_url = "/cgi-bin/koha/cataloguing/addbiblio.pl?biblionumber=";
>+                        [% END %]
> 
>                         var additional_details = "";
>                         if( aData['author'] ){
>@@ -559,10 +564,16 @@
>                                 '<a href="' + aData['diff_url'] + '">' + _("View") + '</a>'
>                             );
>                         }
>-                        $('td:eq(6)', nRow).html(
>-                            '<a target="_blank" href="' + record_details_url
>-                            + aData['matched'] + '">' + aData['matched'] + '</a>'
>-                        );
>+                        if (aData['matched']) {
>+                            $('td:eq(6)', nRow).html(
>+                                '<a class="btn btn-default btn-xs" target="_blank" href="' + record_details_url
>+                                + aData['matched'] + '"><i class="fa fa-eye"></i> ' + _("View") + '</a> '
>+                                [% IF ( CAN_user_editcatalogue_edit_catalogue ) %]
>+                                    + '<a class="btn btn-default btn-xs" target="_blank" href="' + record_edit_url
>+                                    + aData['matched'] + '"><i class="fa fa-pencil"></i> ' + _("Edit") + '</a>'
>+                                [% END %]
>+                            );
>+                        }
>                     },
>                 }));
>                 $("#import_batch_form").on("submit", function() {
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 28843: 123791 | 123804 | 123938 | 123986 | 124220