Attachment 165464 Details for Bug 36221 – Bug 36221: Move patron section to top of create form

[patch] Bug 36221: Move patron section to top of create form

Bug-36221-Move-patron-section-to-top-of-create-for.patch (text/plain), 2.69 KB, created by Pedro Amorim on 2024-04-24 14:22:51 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2024-04-24 14:22:51 UTC

Size: 2.69 KB

patch

obsolete

>From f65b2795d54818661db6693a00d116ea10401572 Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Fri, 8 Mar 2024 15:21:16 +0000
>Subject: [PATCH] Bug 36221: Move patron section to top of create form
>
>---
> Koha/ILL/Backend/opac-includes/create.inc | 43 +++++++++++------------
> 1 file changed, 21 insertions(+), 22 deletions(-)
>
>diff --git a/Koha/ILL/Backend/opac-includes/create.inc b/Koha/ILL/Backend/opac-includes/create.inc
>index 6f9ca8a06fe..906f1cc8214 100644
>--- a/Koha/ILL/Backend/opac-includes/create.inc
>+++ b/Koha/ILL/Backend/opac-includes/create.inc
>@@ -22,6 +22,27 @@
> <h2>Create a manual ILL request</h2>
> <form id="create_form" method="POST" action="">
>   [% INCLUDE 'csrf-token.inc' %]
>+    <fieldset class="rows">
>+    <legend>Patron options</legend>
>+    <ol>
>+      <li>
>+        [% WRAPPER ill_select_field required = 1 id = 'branchcode' label = t('Destination library') %]
>+          <option value="" />
>+          [% FOREACH branch IN branches %]
>+            [% IF whole.value.other.branchcode && branch.branchcode == whole.value.other.branchcode %]
>+            <option value="[% branch.branchcode | html %]" selected>
>+              [% branch.branchname | html %]
>+            </option>
>+            [% ELSE %]
>+            <option value="[% branch.branchcode | html %]">
>+              [% branch.branchname | html %]
>+            </option>
>+            [% END %]
>+          [% END %]
>+        [% END #ill_select_field %]
>+      </li>
>+    </ol>
>+  </fieldset>
>   <fieldset class="rows">
>     <legend>General details</legend>
>     <ol id="general-standard-fields">
>@@ -82,28 +103,6 @@
>       [% INCLUDE "${cwd}/shared-includes/forms/${type}.inc" %]
>   [% END %]
>   [% INCLUDE "${cwd}/shared-includes/custom_fields.inc" %]
>-  <fieldset class="rows">
>-    <legend>Patron options</legend>
>-    <ol>
>-      <li>
>-        [% WRAPPER ill_select_field required = 1 id = 'branchcode' label = t('Destination library') %]
>-          <option value="" />
>-          [% FOREACH branch IN branches %]
>-            [% IF whole.value.other.branchcode && branch.branchcode == whole.value.other.branchcode %]
>-            <option value="[% branch.branchcode | html %]" selected>
>-              [% branch.branchname | html %]
>-            </option>
>-            [% ELSE %]
>-            <option value="[% branch.branchcode | html %]">
>-              [% branch.branchname | html %]
>-            </option>
>-            [% END %]
>-          [% END %]
>-        [% END #ill_select_field %]
>-      </li>
>-    </ol>
>-  </fieldset>
>-
>   <fieldset class="action">
>     <input id="ill-submit" class="btn btn-default" type="submit" value="Create"/>
>     <a class="cancel" href="/cgi-bin/koha/opac-illrequests.pl">Cancel</a>
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36221: 162715 | 162716 | 162933 | 162934 | 162948 | 162949 | 162970 | 162971 | 165462 | 165463 | 165464 | 165465 | 173706 | 173752 | 173753 | 173754 | 173755 | 173756 | 173939 | 173940 | 173941 | 173965 | 173969 | 174017 | 174018 | 174019 | 174020 | 174021 | 174022 | 174023 | 174024 | 174025 | 174026 | 174027 | 174385 | 174398 | 174454