Attachment 149591 Details for Bug 32418 – Bug 32418: (follow-up) Remove jQuery, Fix typos in url, add biblio as template param

[patch] Bug 32418: (follow-up) Remove jQuery, Fix typos in url, add biblio as template param

Bug-32418-follow-up-Remove-jQuery-Fix-typos-in-url.patch (text/plain), 2.96 KB, created by Paul Derscheid on 2023-04-13 09:30:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Paul Derscheid

Created: 2023-04-13 09:30:15 UTC

Size: 2.96 KB

patch

obsolete

>From 3d5b847688ca104d63a636412cb1993ec1281138 Mon Sep 17 00:00:00 2001
>From: Paul Derscheid <paul.derscheid@lmscloud.de>
>Date: Thu, 13 Apr 2023 09:27:31 +0000
>Subject: [PATCH] Bug 32418: (follow-up) Remove jQuery, Fix typos in url, add
> biblio as template param
>
>To test:
>1. Go to any record
>2. Hit the edit button on an item in the holdings table
>3. Modify the URL so that the query param for the itemnumber is either
>    3.1 empty: /cgi-bin/koha/cataloguing/additem.pl?op=edititem&biblionumber=384&itemnumber=
>    3.2 an itemnumber that doesn't exist:
>      /cgi-bin/koha/cataloguing/additem.pl?op=edititem&biblionumber=384&itemnumber=9999999999
>    3.3 whatever else you come up with..
>4. Check the same thing for the dupe option (op=dupeitem)
>5. Sign off
>---
> cataloguing/additem.pl                                   | 2 +-
> .../intranet-tmpl/prog/en/modules/cataloguing/additem.tt | 9 +--------
> 2 files changed, 2 insertions(+), 9 deletions(-)
>
>diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl
>index e1f4650e09..44cd0bc19c 100755
>--- a/cataloguing/additem.pl
>+++ b/cataloguing/additem.pl
>@@ -160,7 +160,7 @@ if ( $op eq 'edititem' || $op eq 'dupeitem' ) {
>     my $item = Koha::Items->find($itemnumber);
>     if ( !$item ) {
>         $itemnumber = undef;
>-        $template->param( item_doesnt_exist => 1 );
>+        $template->param( biblio => $biblio, item_doesnt_exist => 1 );
>         output_and_exit( $input, $cookie, $template, 'unknown_item' );
>     }
> }
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
>index 0844f68e5b..57f53b4c52 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
>@@ -27,13 +27,6 @@
> [% Asset.js("js/cataloging_additem.js") | $raw %]
>     <script>
>         var has_item_groups = "[% item_groups.size | html %]";
>-        $(document).ready(function() {
>-            [% IF item_doesnt_exist %]
>-                $("#itemst_wrapper").hide();
>-                $("#menu").hide();
>-                $("#cataloguing_additem_newitem").hide();
>-            [% END %]
>-        });
>     </script>
> </head>
> 
>@@ -68,7 +61,7 @@
>             <main>
>                 [% IF item_doesnt_exist %]
>                     <div class="dialog alert">
>-                        <a href="/cgi/bin/koha/catloguing/additem.pl?biblionumber=[% biblio.biblionumber | uri %]">Add a new item</a> or <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% biblio.biblionumber | uri %]#holdings">go to the records holdings</a>.
>+                        <a href="/cgi-bin/koha/cataloguing/additem.pl?biblionumber=[% biblio.biblionumber | uri %]">Add a new item</a> or <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% biblio.biblionumber | uri %]#holdings">go to the records holdings</a>.
>                     </div>
>                 [% END %]
>                 [% INCLUDE 'blocking_errors.inc' %]
>-- 
>2.39.2 (Apple Git-143)

Actions: View | Diff | Splinter Review

Attachments on bug 32418: 144571 | 144639 | 144640 | 144641 | 144643 | 144794 | 144812 | 145306 | 145307 | 145505 | 146944 | 149092 | 149591 | 149626 | 149627 | 149635 | 149636 | 149637