From 1446b9cb5045c08f77285eba1eac65cb585bcbd7 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

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
---
 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 b5adfd2d10..a00ab33185 100644
--- a/Koha/ILL/Backend/opac-includes/create.inc
+++ b/Koha/ILL/Backend/opac-includes/create.inc
@@ -21,6 +21,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">
@@ -81,28 +102,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.47.0