[Bug 28805] New: Batch checkout shoudl have on-site checkout option
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Bug ID: 28805 Summary: Batch checkout shoudl have on-site checkout option Change sponsored?: --- Product: Koha Version: 20.05 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: katrin.fischer@bsz-bw.de QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com The batch checkout is very useful, but currently doesn't support all advanced options of the normal search page. It's missing the "on-site checkout" functionality. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Batch checkout shoudl have |Batch checkout should have |on-site checkout option |on-site checkout option -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|20.05 |master -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fabricio Molins <fabriciomolina@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fabriciomolina@theke.io Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |fabriciomolina@theke.io |ity.org | Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Fabricio, any progress on this? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #2 from Fabricio Molina <fabriciomolina@theke.io> --- Hi Katrin I'm pleased to inform you that the development is almost complete. I just need to finish linking the OnSiteCheckoutAutoCheck preference to make the development more comprehensive. It will be finished this week. Best regards. Fabricio. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thanks for the update! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fabricio Molina <fabriciomolina@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #4 from Fabricio Molina <fabriciomolina@theke.io> --- Created attachment 152992 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152992&action=edit Bug 28805: Add on-site checkout checkbox in batch checkout. Add the 'on-site checkout' checkbox to the 'Batch checkout'. To verify the functionality, follow these steps: 1) Before the modification is added, enable the 'OnSiteCheckouts' and 'BatchCheckout' system preferences. 2) In the 'BatchCheckoutsValidCategories' preference, select all categories. 3) Find a user in the system. For example, Henry Acevedo. 4) In the 'Check out' section, you can see that the 'Checkout settings' option displays the 'On-site Checkout' checkbox when clicked. 5) Check the 'On-site checkout' option. This will display the current date with a runtime deadline of 23:59. 6) Enter a barcode (for example 39999000011418). 7) Click the 'Check out' button. This will show that the loan was made on-site. 8) Check the 'Batch checkout' section. This section does not hace the 'On-site checkout' checkbox. When loading the patch with the addition of the 'on-site checkout' checkbox in 'Batch checkout' starting from step 3), you can already find the checkbox in the 'Batch checkout' section. To use it, you can try the barcodes 39999000006179 and 39999000008494 together. Then continue from step 5). When viewing the loans, you will see that both items (39999000006179 and 39999000008494) will appear with the label 'On-site checkout'. If this patch is deactivated, the functionality will remain the same as it was before this change. Sponsored-by: Banco Central de la República Argentina. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Sam Lau <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |samalau@gmail.com Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #152992|0 |1 is obsolete| | --- Comment #5 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 153105 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153105&action=edit Bug 28805: Add on-site checkout checkbox in batch checkout Add the 'on-site checkout' checkbox to the 'Batch checkout'. To verify the functionality, follow these steps: 1) Before the modification is added, enable the 'OnSiteCheckouts' and 'BatchCheckout' system preferences. 2) In the 'BatchCheckoutsValidCategories' preference, select all categories. 3) Find a user in the system. For example, Henry Acevedo. 4) In the 'Check out' section, you can see that the 'Checkout settings' option displays the 'On-site Checkout' checkbox when clicked. 5) Check the 'On-site checkout' option. This will display the current date with a runtime deadline of 23:59. 6) Enter a barcode (for example 39999000011418). 7) Click the 'Check out' button. This will show that the loan was made on-site. 8) Check the 'Batch checkout' section. This section does not hace the 'On-site checkout' checkbox. When loading the patch with the addition of the 'on-site checkout' checkbox in 'Batch checkout' starting from step 3), you can already find the checkbox in the 'Batch checkout' section. To use it, you can try the barcodes 39999000006179 and 39999000008494 together. Then continue from step 5). When viewing the loans, you will see that both items (39999000006179 and 39999000008494) will appear with the label 'On-site checkout'. If this patch is deactivated, the functionality will remain the same as it was before this change. Sponsored-by: Banco Central de la República Argentina -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #6 from Fabricio Molina <fabriciomolina@theke.io> --- Hello Sam. Thank you very much for taking the time to make the correction. I apologize for the mistake made in the previously uploaded patch. Tomás helped me fix it. Could you please review it again?. Thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Sam Lau <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Sam Lau <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #153105|0 |1 is obsolete| | --- Comment #7 from Sam Lau <samalau@gmail.com> --- Created attachment 153185 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153185&action=edit Bug 28805: Add on-site checkout checkbox in batch checkout Add the 'on-site checkout' checkbox to the 'Batch checkout'. To verify the functionality, follow these steps: 1) Before the modification is added, enable the 'OnSiteCheckouts' and 'BatchCheckout' system preferences. 2) In the 'BatchCheckoutsValidCategories' preference, select all categories. 3) Find a user in the system. For example, Henry Acevedo. 4) In the 'Check out' section, you can see that the 'Checkout settings' option displays the 'On-site Checkout' checkbox when clicked. 5) Check the 'On-site checkout' option. This will display the current date with a runtime deadline of 23:59. 6) Enter a barcode (for example 39999000011418). 7) Click the 'Check out' button. This will show that the loan was made on-site. 8) Check the 'Batch checkout' section. This section does not hace the 'On-site checkout' checkbox. When loading the patch with the addition of the 'on-site checkout' checkbox in 'Batch checkout' starting from step 3), you can already find the checkbox in the 'Batch checkout' section. To use it, you can try the barcodes 39999000006179 and 39999000008494 together. Then continue from step 5). When viewing the loans, you will see that both items (39999000006179 and 39999000008494) will appear with the label 'On-site checkout'. If this patch is deactivated, the functionality will remain the same as it was before this change. Sponsored-by: Banco Central de la República Argentina Signed-off-by: Sam Lau <samalau@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #8 from Sam Lau <samalau@gmail.com> --- (In reply to Fabricio Molina from comment #6)
Hello Sam. Thank you very much for taking the time to make the correction. I apologize for the mistake made in the previously uploaded patch. Tomás helped me fix it. Could you please review it again?. Thanks!
Fabricio, no need to apologize! The bug is now signed off :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com Status|Signed Off |Failed QA --- Comment #9 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- 1. If you turn off SpecifyDueDate, the checkbox is no longer displayed 2. Do you really need that? + [% Asset.js("js/members-menu.js") | $raw %] + [% Asset.js("js/pages/circulation.js") | $raw %] -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Comment on attachment 153185 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153185 Bug 28805: Add on-site checkout checkbox in batch checkout Review of attachment 153185: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=28805&attachment=153185) ----------------------------------------------------------------- Hi Fabricio, thanks for working on this! Joubu already made 2 good points, I have a bit of an explanation and one more: ::: koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt @@ +99,4 @@
</li> </ol> </fieldset> + [% IF Koha.Preference('SpecifyDueDate') && Koha.Preference('OnSiteCheckouts') %]
To explain a bit more what Jonathan said: the specify due date feature and the onsite-checkout only partially depend on each other. So this should probably be split into separate conditions or be ||. * If you have both, you are able to change the pre-selected due date for the on-site checkout. * If you have only OnSiteCheckouts, the date is automatically set to today 23:59:00 in the background. * Only SpecifyDueDate works like now. You can compare how it should work with the same options on the checkouts tab. @@ +104,5 @@
+ <legend>Due date:</legend> + <label for="duedatespec">Hard due date: [% INCLUDE 'date-format.inc' %]:</label> + <input type="text" size="20" id="duedatespec" name="duedatespec" value="[% duedatespec | html %]" class="flatpickr" data-flatpickr-enable-time="true"/> + <div id="onsite_checkout-select" class="circ-setting"> + <div class="circ-setting">
You double-up the circ-setting classes here in 2 different div - this makes the font size smaller than for the other options. Maybe one or both should be removed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fabricio Molina <fabriciomolina@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #153185|0 |1 is obsolete| | --- Comment #11 from Fabricio Molina <fabriciomolina@theke.io> --- Created attachment 154346 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154346&action=edit Bug 28805: Add on-site checkout checkbox in batch checkout Add the 'on-site checkout' checkbox to the 'Batch checkout'. To verify the functionality, follow these steps: 1) Before the modification is added, enable the 'OnSiteCheckouts' and 'BatchCheckout' system preferences. 2) In the 'BatchCheckoutsValidCategories' preference, select all categories. 3) Find a user in the system. For example, Henry Acevedo. 4) In the 'Check out' section, you can see that the 'Checkout settings' option displays the 'On-site Checkout' checkbox when clicked. 5) Check the 'On-site checkout' option. This will display the current date with a runtime deadline of 23:59. 6) Enter a barcode (for example 39999000011418). 7) Click the 'Check out' button. This will show that the loan was made on-site. 8) Check the 'Batch checkout' section. This section does not hace the 'On-site checkout' checkbox. When loading the patch with the addition of the 'on-site checkout' checkbox in 'Batch checkout' starting from step 3), you can already find the checkbox in the 'Batch checkout' section. To use it, you can try the barcodes 39999000006179 and 39999000008494 together. Then continue from step 5). When viewing the loans, you will see that both items (39999000006179 and 39999000008494) will appear with the label 'On-site checkout'. If this patch is deactivated, the functionality will remain the same as it was before this change. Sponsored-by: Banco Central de la República Argentina Signed-off-by: Sam Lau <samalau@gmail.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fabricio Molina <fabriciomolina@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #12 from Fabricio Molina <fabriciomolina@theke.io> --- Hi Thanks for your feedback and suggestions. In this fix, the "OnSiteCheckouts" preference can be used without depending on the "SpecifyDueData" preference. Regarding the functionality of being able to modify the preselected due date, I used the same behavior that on-site checkout has for individual loans (the preselected date can also be modified there). I look forward to any feedback. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #154346|0 |1 is obsolete| | --- Comment #13 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 156182 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156182&action=edit Bug 28805: Add on-site checkout checkbox in batch checkout Add the 'on-site checkout' checkbox to the 'Batch checkout'. To verify the functionality, follow these steps: 1) Before the modification is added, enable the 'OnSiteCheckouts' and 'BatchCheckout' system preferences. 2) In the 'BatchCheckoutsValidCategories' preference, select all categories. 3) Find a user in the system. For example, Henry Acevedo. 4) In the 'Check out' section, you can see that the 'Checkout settings' option displays the 'On-site Checkout' checkbox when clicked. 5) Check the 'On-site checkout' option. This will display the current date with a runtime deadline of 23:59. 6) Enter a barcode (for example 39999000011418). 7) Click the 'Check out' button. This will show that the loan was made on-site. 8) Check the 'Batch checkout' section. This section does not hace the 'On-site checkout' checkbox. When loading the patch with the addition of the 'on-site checkout' checkbox in 'Batch checkout' starting from step 3), you can already find the checkbox in the 'Batch checkout' section. To use it, you can try the barcodes 39999000006179 and 39999000008494 together. Then continue from step 5). When viewing the loans, you will see that both items (39999000006179 and 39999000008494) will appear with the label 'On-site checkout'. If this patch is deactivated, the functionality will remain the same as it was before this change. Sponsored-by: Banco Central de la República Argentina Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #14 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 156183 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156183&action=edit Bug 28805: (QA follow-up) Restore removed asset Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #15 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 156184 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156184&action=edit Bug 28805: (QA follow-up) Pass onsite variable to confirmation screen Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com --- Comment #16 from Nick Clemens <nick@bywatersolutions.com> --- I still got a warning about tidiness, however, the lines I added are tidied and nothing around them is not -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #17 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- If SpecifyDueDate and OnSiteCheckouts are both disabled, you see "Due date" at the bottom of the screen. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fabricio Molina <fabriciomolina@theke.io> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #18 from Fabricio Molina <fabriciomolina@theke.io> --- Created attachment 158108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=158108&action=edit Bug 28805: (follow-up) Fix template logic error This patch fixes a problem in the template logic that made the 'Due date' legend display when `OnSiteCheckouts` and `SpecifyDueDate` are not set, as highlighted by QA. Sponsored-by: Banco Central de la República Argentina -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #19 from Fabricio Molina <fabriciomolina@theke.io> --- (In reply to Jonathan Druart from comment #17)
If SpecifyDueDate and OnSiteCheckouts are both disabled, you see "Due date" at the bottom of the screen.
Sorry for the delay. The last patch fixes the issue. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #158108|0 |1 is obsolete| | --- Comment #20 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 158279 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=158279&action=edit Bug 28805: (follow-up) Fix template logic error This patch fixes a problem in the template logic that made the 'Due date' legend display when `OnSiteCheckouts` and `SpecifyDueDate` are not set, as highlighted by QA. Sponsored-by: Banco Central de la República Argentina Signed-off-by: Nick Clemens <nick@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #21 from Nick Clemens <nick@bywatersolutions.com> --- QA tool is not happy, but the template pattern works and is used elsewhere - I had to tidy the entire issuing block to remove the tidy error, did not seem worth it -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #22 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |additional_work_needed --- Comment #23 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- It looks like this makes the tests failing: prove -v xt/author/valid-templates.t # Failed test '/kohadevbox/koha/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt' # at xt/author/valid-templates.t line 141. # file error - parse error - /kohadevbox/koha/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt line 103: unexpected token ( Koha.Preference('OnSiteCheckouts')) # [% IF Koha.Preference('SpecifyDueDate') || Koha.Preference('OnSiteCheckouts') %] # Looks like you failed 1 test of 822. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #24 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 158386 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=158386&action=edit Bug 28805: (follow-up) Fix non-printable char that breaks tests -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #25 from David Cook <dcook@prosentient.com.au> --- (In reply to Katrin Fischer from comment #24)
Created attachment 158386 [details] [review] Bug 28805: (follow-up) Fix non-printable char that breaks tests
Nice job! I don't know how you spotted that C2A0 no-break space, but that was masterful 👍 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #26 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to David Cook from comment #25)
(In reply to Katrin Fischer from comment #24)
Created attachment 158386 [details] [review] [review] Bug 28805: (follow-up) Fix non-printable char that breaks tests
Nice job! I don't know how you spotted that C2A0 no-break space, but that was masterful 👍
Thanks David :) It took a bit to figure that one out. As there was no obvious issue with the template logic, I looked at the patches here in bugzilla again and it showed in the browser. Guess more lucky than masterful ;) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |nick@bywatersolutions.com |y.org | Keywords|additional_work_needed | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 --- Comment #27 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- I have the following in my .vimrc set list listchars=tab:>-,trail:.,extends:>,nbsp:· And (9e675d2ac43 is right before the last patch has been pushed): % git show 9e675d2ac43:koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation_batch_checkouts.tt|vim - Shows me the non-printable char: 103 [% IF Koha.Preference('SpecifyDueDate') ||·Koha.Preference('OnSiteCheckouts') %] -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Needs documenting CC| |fridolin.somers@biblibre.co | |m --- Comment #28 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 23.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Summary|Batch checkout should have |Add on-site option to batch |on-site checkout option |checkout functionality -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|martin.renvoize@ptfs-europe | |.com | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28805 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |Caroline Cyr La Rose contact| | CC| |caroline.cyr-la-rose@inlibr | |o.com Status|Needs documenting |RESOLVED Documentation| |https://gitlab.com/koha-com submission| |munity/koha-manual/-/merge_ | |requests/983 Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org