Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 30579: (follow-up) Fix rebase of multi holds form and add CSRF

Browse source 
This patch restores missing elemtns to multi-hold form and adds CSRF variables

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Nick Clemens 2024-05-07 12:01:33 +00:00 committed by Katrin Fischer
parent 2f70239201
commit fe4be8be6e
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834
1 changed files with 42 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

40
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
View file

@ -1049,6 +1049,8 @@
<fieldset class="rows">
<legend>Hold details</legend>
<form action="placerequest.pl" method="post" name="form" id="hold-request-form">
[% INCLUDE 'csrf-token.inc' %]
<input type="hidden" name="op" value="cud-placerequest" />
<input type="hidden" name="multi_holds" id="multi_holds" value="1" />
<input type="hidden" name="request" value="any"/>
<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
@ -1061,6 +1063,44 @@
[% END %]
[% END %]
<ol>
<li>
<span class="label">Patron:</span>
[% IF ( patron.borrowernumber ) %]
[% INCLUDE 'patron-title.inc' patron => patron no_title => 1 hide_patron_infos_if_needed => 1 %]
[% ELSE %]
Not defined yet
[% END %]
</li>
<li>
<label for="holdnotes">Notes:</label>
<textarea id="holdnotes" name="notes" cols="30" rows="1"></textarea>
</li>
<li>
<label for="pickup">Pickup at:</label>
<select name="pickup" id="pickup_multi" data-patron-id="[% patron.borrowernumber | html %]">
<option value="" selected="selected"></option>
[% FOREACH pickup_location IN multi_pickup_locations %]
<option value="[% pickup_location.branchcode | html %]">[% pickup_location.branchname | html %]</option>
[% END %]
</select>
</li>
[% IF ( Koha.Preference('AllowHoldDateInFuture') ) %]
<li>
<label for="from">Hold starts on date:</label>
<input id="reserve_date" name="reserve_date" id="from" size="10" type="text" data-date_to="expiration_date" class="flatpickr" data-flatpickr-futureinclusive="true" />
</li>
[% END %]
<li>
<label for="to">Hold expires on date:</label>
<input id="expiration_date" name="expiration_date" id="to" size="10" type="text" class="flatpickr" data-flatpickr-futuredate="true" />
</li>
</ol>
<table id="requesttitles">
<tr>
<th>&nbsp;</th>