Basically the same bug as 28103 but newer Koha version, and this time we get "undefined" instead of a console error. -- 0. Set "autoBarcode" to "generated in the form <branchcode>yymm0001" 1. Create a vendor 2. Create a backet with "Create items when" set to "receiving an order" 3. Add an order (any order) 4. Close the basket 5. Receive the shipment 6. Click in the barcode field 7. Note that you get a barcode like undefined24090001
This time it's because the Javascript code written by C4/Barcodes/ValueBuilder.pm only works for the MARC basic editor.
Fortunately, there's a very simple "fix" for this. Yikes...
(In reply to David Cook from comment #2) > Fortunately, there's a very simple "fix" for this. Yikes... Oh I take that back. Fixes the Javascript but wrecks the styling :D
Created attachment 171115 [details] [review] Bug 37854: Add fieldset.rows so that Javascript works This change adds a fieldset.rows that the Javascript produced by C4/Barcodes/ValueBuilder.pm will work here too like it does for /cgi-bin/koha/cataloguing/additem.pl and /cgi-bin/koha/acqui/neworderempty.pl The fieldset.rows element ruins the styling, so we add some context specific styling to the styling is preserved. Test plan: 0. Apply the patch 1. Set "autoBarcode" to "generated in the form <branchcode>yymm0001" 2. Create a vendor 3. Create a backet with "Create items when" set to "receiving an order" 4. Add an order (any order) 5. Close the basket 6. Receive the shipment 7. Click in the barcode field 8. Note that you get a barcode like CPL24090001 and not undefined24090001
Created attachment 171116 [details] [review] Bug 37854: Re-indent HTML (whitespace-only) This whitespace only change re-indents the HTML
Looking like bug 8179 is what broke this one
Created attachment 171234 [details] [review] Bug 37854: Add fieldset.rows so that Javascript works This change adds a fieldset.rows that the Javascript produced by C4/Barcodes/ValueBuilder.pm will work here too like it does for /cgi-bin/koha/cataloguing/additem.pl and /cgi-bin/koha/acqui/neworderempty.pl The fieldset.rows element ruins the styling, so we add some context specific styling to the styling is preserved. Test plan: 0. Apply the patch 1. Set "autoBarcode" to "generated in the form <branchcode>yymm0001" 2. Create a vendor 3. Create a backet with "Create items when" set to "receiving an order" 4. Add an order (any order) 5. Close the basket 6. Receive the shipment 7. Click in the barcode field 8. Note that you get a barcode like CPL24090001 and not undefined24090001 Signed-off-by: Olivier V <olivier.vezina@inLibro.com>
Created attachment 171235 [details] [review] Bug 37854: Re-indent HTML (whitespace-only) This whitespace only change re-indents the HTML Signed-off-by: Olivier V <olivier.vezina@inLibro.com>
Created attachment 174546 [details] [review] Bug 37854: Add fieldset.rows so that Javascript works This change adds a fieldset.rows that the Javascript produced by C4/Barcodes/ValueBuilder.pm will work here too like it does for /cgi-bin/koha/cataloguing/additem.pl and /cgi-bin/koha/acqui/neworderempty.pl The fieldset.rows element ruins the styling, so we add some context specific styling to the styling is preserved. Test plan: 0. Apply the patch 1. Set "autoBarcode" to "generated in the form <branchcode>yymm0001" 2. Create a vendor 3. Create a backet with "Create items when" set to "receiving an order" 4. Add an order (any order) 5. Close the basket 6. Receive the shipment 7. Click in the barcode field 8. Note that you get a barcode like CPL24090001 and not undefined24090001 Signed-off-by: Olivier V <olivier.vezina@inLibro.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 174547 [details] [review] Bug 37854: Re-indent HTML (whitespace-only) This whitespace only change re-indents the HTML Signed-off-by: Olivier V <olivier.vezina@inLibro.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed for 24.11! Well done everyone, thank you!