Bug 35489: Add a hidden input for itemnumber even if item has no barcode
This page is a single form, all fields must have inputs to avoid data loss/mismatch
To test:
1 - Find a record in the catalog and add or edit items to have one item with a barcode and one without
2 - Place an item level hold on the item with no barcode
3 - Place another hold for a different patron on the item with a barcode
4 - From the holds table add an expiration date to the hold with a barcode and save
5 - Note the hold on the item with no barcode is now on the item with a barcode and the other hold is now a next available hold
6 - Delete the holds
7 - Apply patch
8 - Repeat 2-4
9 - Confirm the itemnumbers for the holds have not changed
Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit b78be9bddb
)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
parent
e5c5424023
commit
9fe3d5bc48
1 changed files with 2 additions and 2 deletions
|
@ -167,10 +167,10 @@
|
|||
<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% hold.biblionumber | uri %]&itemnumber=[% hold.itemnumber | uri %]#item[% hold.itemnumber | uri %]">
|
||||
[%- IF ( hold.barcodenumber ) -%]
|
||||
[%- hold.barcodenumber | html -%]
|
||||
<input type="hidden" name="itemnumber" value="[% hold.itemnumber | html %]" />
|
||||
[%- ELSE -%]
|
||||
<span>No barcode</span>
|
||||
[%- END -%]
|
||||
<input type="hidden" name="itemnumber" value="[% hold.itemnumber | html %]" />
|
||||
</a>
|
||||
[%- ELSE -%]
|
||||
[%- IF ( hold.item_level_hold ) -%]
|
||||
|
@ -179,10 +179,10 @@
|
|||
<a href="/cgi-bin/koha/catalogue/moredetail.pl?biblionumber=[% hold.biblionumber | uri %]&itemnumber=[% hold.itemnumber | uri %]#item[% hold.itemnumber | uri %]">
|
||||
[%- IF ( hold.barcodenumber ) -%]
|
||||
[%- hold.barcodenumber | html -%]
|
||||
<input type="hidden" name="itemnumber" value="[% hold.itemnumber | html %]" />
|
||||
[%- ELSE -%]
|
||||
<span>No barcode</span>
|
||||
[%- END -%]
|
||||
<input type="hidden" name="itemnumber" value="[% hold.itemnumber | html %]" />
|
||||
</a>
|
||||
</em>
|
||||
[%- ELSE -%]
|
||||
|
|
Loading…
Reference in a new issue