[Bug 36135] New: Add tool to batch edit pick up dates of holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Bug ID: 36135 Summary: Add tool to batch edit pick up dates of holds Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: emmi.takkinen@koha-suomi.fi QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com There are sometimes situations where libraries have to extend their holds pick up dates. We should at same kind of tool for this like "Batch extend due dates". However criteria for dates to extend can be complicated (we've excluded e.g. items with specific state, holds with found status, items on transfer to specific library). So rather than using predefined criteria for holds to edit (at least for the start), tool should have option to upload file of SQL query results. -- 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=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Add tool to batch edit pick |Add tool to batch edit |up dates of holds |expiration dates of holds -- 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=36135 --- Comment #1 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Corrected -- 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=36135 --- Comment #2 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Emmi Takkinen from comment #1)
Corrected
... the title, we should extend expiration dates, not pick up dates. -- 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=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |emmi.takkinen@koha-suomi.fi |ity.org | -- 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=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #3 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 163021 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163021&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #4 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 163022 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163022&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #5 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 163023 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163023&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #6 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 163024 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163024&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Add tool to batch edit |Add tool to batch modify |expiration dates of holds |holds Status|ASSIGNED |Needs Signoff --- Comment #7 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- We did some briefing on this and ended up adding ability to modify other fields as well, not just expiration date. I'll adjust bugs title accordingly. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Brenda <bowens@washoecounty.us> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bowens@washoecounty.us -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |Laura.escamilla@bywatersolu | |tions.com Status|Needs Signoff |Failed QA --- Comment #8 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Hi Emmi, I am running into the following error when attempting to use the Batch Modify Holds tool: Programming error - op 'list' must start with 'cud-' for POST http://localhost:8081/intranet/tools/batch_modify_holds.pl (referer: http://localhost:8081/cgi-bin/koha/tools/batch_modify_holds.pl) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lisette Scheer <lisette.scheer@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisette.scheer@bywatersolut | |ions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #9 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 163529 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163529&action=edit Bug 36135: (follow-up) Adjust for CSRF -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163529|0 |1 is obsolete| | --- Comment #10 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 163530 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163530&action=edit Bug 36135: (follow-up) Adjust for CSRF -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.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=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163021|0 |1 is obsolete| | --- Comment #11 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 163540 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163540&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163022|0 |1 is obsolete| | --- Comment #12 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 163541 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163541&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163023|0 |1 is obsolete| | --- Comment #13 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 163542 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163542&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163024|0 |1 is obsolete| | --- Comment #14 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 163543 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163543&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163530|0 |1 is obsolete| | --- Comment #15 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Created attachment 163544 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163544&action=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #16 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- Hi! Thank you again for working on this patch! It works as described. :) These could be follow-up bugs, but I think it would be helpful to have a column in the batch modify holds tool that either has the item barcode, or item number / bib record hyperlinked for easy access. It would also be helpful to display which items have a holds status that can not be suspended. It is difficult to determine at a glance which item cannot be suspended and it might be frustrating to libraries who are trying to suspend hundreds of holds at a time. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Katie Bliss <kebliss@dmpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kebliss@dmpl.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #17 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Laura Escamilla from comment #16)
Hi!
Thank you again for working on this patch! It works as described. :)
These could be follow-up bugs, but I think it would be helpful to have a column in the batch modify holds tool that either has the item barcode, or item number / bib record hyperlinked for easy access.
It would also be helpful to display which items have a holds status that can not be suspended. It is difficult to determine at a glance which item cannot be suspended and it might be frustrating to libraries who are trying to suspend hundreds of holds at a time.
Hi! Thank you for feedback :) I'll take a look what I can do to these and other feedback I received from our superlibrarians. I'll probably provide separate feedback patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #18 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 163796 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163796&action=edit Bug 36135: Add several new features This patch adds following new features to batch modify holds tool. - Do not automatically select login library as search param - Add biblios title and items barcode to found holds table - Add count of found holds at top of the table - Add ability to do not change pick up library - Display those hold rows with some found status in different color Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #19 from Lucas Gass <lucas@bywatersolutions.com> --- This looks good but there are some small fixes to be made: 1. WARN installer/data/mysql/atomicupdate/bug_36135.pl WARN tidiness The file is less tidy than before (bad/messy lines before: 0, now: 4) Perltidy your atomicupdate 2. forbidden pattern: simple-quote string (line 312) Line 312: tools/batch_modify_holds.tt, needs double quotes, not single. 3. FAIL tt_valid lines 134 This probably needs to be something like: [% IF hold.found %] <tr class="highlighted-row"> [% ELSE %] <tr> [% END %] 4. Line 123: <th>Title</title> 5. Line 111: Missing a closing </div> 6. Overall there needs to be some improvement to the HTML: <label for="suspend_until">Expiration date:</label> <label for="suspend_until">Suspended until:</label> These should probably be <legend> not <label>. <li> elements inside of <li>? Not a blocker but should these be separate fieldsets? 7. These both use the ID "to" and they should use #expirationdate_to and #suspend_until_to, respectively. 56 <li> 57 <label for="expirationdate_to">To</label> 58 <input type="text" size="10" id="to" name="expirationdate_to" class="flatpickr"/> 59 </li> 91 <label for="suspend_until_to">To</label> 92 <input type="text" size="10" id="to" name="suspend_until_to" class="flatpickr"/> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Michelle Spinney <mspinney@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mspinney@clamsnet.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #20 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 164037 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164037&action=edit Bug 36135: Fix QA issues and add improvemnents This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #21 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Fixed QA issues which Lucas pointed out. For this
<label for="suspend_until">Expiration date:</label> <label for="suspend_until">Suspended until:</label> These should probably be <legend> not <label>.
I removed them and added labels "Expiration date from:" and "Expiration date to:" labels for corresponding li elements. Same for suspend until. Legend is used for as caption of fieldset so it wasn't suitable to be used in this case. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164037|0 |1 is obsolete| | --- Comment #22 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 164051 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164051&action=edit Bug 36135: Fix QA issues and add improvements This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #23 from Lucas Gass <lucas@bywatersolutions.com> --- Thanks Emmi! This looks great. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.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=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163540|0 |1 is obsolete| | --- Comment #24 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164057&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163541|0 |1 is obsolete| | --- Comment #25 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164058 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164058&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163542|0 |1 is obsolete| | --- Comment #26 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164059 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164059&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163543|0 |1 is obsolete| | --- Comment #27 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164060 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164060&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163544|0 |1 is obsolete| | --- Comment #28 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164061 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164061&action=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163796|0 |1 is obsolete| | --- Comment #29 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164062 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164062&action=edit Bug 36135: Add several new features This patch adds following new features to batch modify holds tool. - Do not automatically select login library as search param - Add biblios title and items barcode to found holds table - Add count of found holds at top of the table - Add ability to do not change pick up library - Display those hold rows with some found status in different color Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164051|0 |1 is obsolete| | --- Comment #30 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 164063 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164063&action=edit Bug 36135: Fix QA issues and add improvements This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura O'Neil <laura@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |laura@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Cécile Garcia-Belmonte <cecilegarciabelmonte@loireforez.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cecilegarciabelmonte@loiref | |orez.fr -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |nick@bywatersolutions.com --- Comment #31 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- I like this a lot - self contained and makes minimal changes, the patches are clean and easy to read, but I think we need to consider some things here before this moves ahead Must fix: The JS alerts to prevent submission fire an alert per warning, so you need to click through several if you make a mistake - these should be combined into a single alert There is no way to see the limits you have placed on your search - I think these need to be shown to the user, ideally they would be editable, but showing them would be a start Should fix: While I like the idea of minimalizing data for privacy, excluding the borrower info on the holds feels a bit too far. Ideally this would be under columns configuration, and we have patron-title.inc for showing borrower info only to staff who should see it. The current column selection feels The display should be paginated, ideally it would use the holds API to load the holds to ensure performance, as scrolling can get really long really fast. It would be nice too to be able to select all in a filtered search - right now 'Select all' selects even those not visible in the table Could fix: If implemented with the datatables API this could then work like the patron search, with filters present on the page. The batch edit implies more batch actions - cancellation, reverting, etc. but these can all be follow-ups I think -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED --- Comment #32 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Nick Clemens (kidclamp) from comment #31)
I like this a lot - self contained and makes minimal changes, the patches are clean and easy to read, but I think we need to consider some things here before this moves ahead
Must fix: The JS alerts to prevent submission fire an alert per warning, so you need to click through several if you make a mistake - these should be combined into a single alert
There is no way to see the limits you have placed on your search - I think these need to be shown to the user, ideally they would be editable, but showing them would be a start
Should fix: While I like the idea of minimalizing data for privacy, excluding the borrower info on the holds feels a bit too far. Ideally this would be under columns configuration, and we have patron-title.inc for showing borrower info only to staff who should see it. The current column selection feels
The display should be paginated, ideally it would use the holds API to load the holds to ensure performance, as scrolling can get really long really fast. It would be nice too to be able to select all in a filtered search - right now 'Select all' selects even those not visible in the table
Could fix: If implemented with the datatables API this could then work like the patron search, with filters present on the page.
The batch edit implies more batch actions - cancellation, reverting, etc. but these can all be follow-ups I think
Thank you Nick for feedback. First two points are indeed something that should be fixed. I'm just bit unsure how to display those search terms in search result page. I also agree that using API does have benefits and table should be paginated (even my test data might produce result of over 2000 holds and that is cumbersome for users go through). I'll take a look what I can do to these. Setting this as assigned for now. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #33 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Moving on with this really depends on whether API is used or not in this page. Otherwise adding it later would probably mean making same work again with some of the features Nick suggested. As for the filters displayed on page, I think it would be better if they are displayed like in item search page behind "Edit search" button. Putting them on margin just doesn't seem (visually) good idea since there are other links there already. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #34 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Couple problems, holds endpoint doesn't show patron_expiration_date column (see bug 29975). It also doesn't allow fetching items barcode, adding items to datatables embed property leads to error. So we need to make changes to holds endpoint to make this function properly. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |37032 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37032 [Bug 37032] REST API: Unable to call item info via holds endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #35 from Laura O'Neil <laura@bywatersolutions.com> --- I would love to see this one pick up again. It's a popular request from our partner libraries. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #36 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Looks like the templates now need to be adjusted for Bootstrap 5 changes. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #37 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177518 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177518&action=edit Bug 36135: Report errors to a modal -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #38 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177519 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177519&action=edit Bug 36135: Fix breadcrumbs/Bootstrap issues -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 George Williams (NEKLS) <george@nekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |george@nekls.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Kristi <kkrueger@cuyahogalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kkrueger@cuyahogalibrary.or | |g -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Rebecca Coert <rcoert@arlingtonva.us> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rcoert@arlingtonva.us -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #39 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177959 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177959&action=edit Bug 36135: Fix some markup errors -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164057|0 |1 is obsolete| | --- Comment #40 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177960 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177960&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164058|0 |1 is obsolete| | --- Comment #41 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177961 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177961&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164059|0 |1 is obsolete| | --- Comment #42 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177962 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177962&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164060|0 |1 is obsolete| | --- Comment #43 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177963 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177963&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164061|0 |1 is obsolete| | --- Comment #44 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177964 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177964&action=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164062|0 |1 is obsolete| | --- Comment #45 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177965 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177965&action=edit Bug 36135: Add several new features This patch adds following new features to batch modify holds tool. - Do not automatically select login library as search param - Add biblios title and items barcode to found holds table - Add count of found holds at top of the table - Add ability to do not change pick up library - Display those hold rows with some found status in different color Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #164063|0 |1 is obsolete| | --- Comment #46 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177966 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177966&action=edit Bug 36135: Fix QA issues and add improvements This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177518|0 |1 is obsolete| | --- Comment #47 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177967 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177967&action=edit Bug 36135: Report errors to a modal -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177519|0 |1 is obsolete| | --- Comment #48 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177968 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177968&action=edit Bug 36135: Fix breadcrumbs/Bootstrap issues -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177959|0 |1 is obsolete| | --- Comment #49 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177969 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177969&action=edit Bug 36135: Fix some markup errors -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #50 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Rebased using auto_rebase.pl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163021|1 |0 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177960|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #51 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177975 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177975&action=edit Bug 36135: API the found_holds table -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #52 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 177976 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177976&action=edit Bug 36135: Embed items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #53 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- This is still a work in progress but want to mention a few things: 1. Handling state across paginated AJAX table is hard, for me at least. So, I don't think we actually need checkboxes. Instead, you do the search and can bulk update everything that is returned. There should be an option to refine the search. 2. Instead of handling errors with JS before you submit I think can simply report back on holds that were not updated. Example, trying to suspend a waiting hold. 3. The report table in the final step should report back successes and errors, so I think it cannot be an API/AJAX table since you can't get that data from the API. But maybe a standard DataTable? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #54 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 177981 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177981&action=edit Bug 36135: Add ability to fetch patron info via API holds endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #55 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 177982 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=177982&action=edit Bug 36135: Add more API functionality to display holds to modify -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #56 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Seems that Lucas and I both have been working with this one :D Sorry for not updating status on this, we've been testing this on our testing environment and I've been holding up to provide patches here until there are no more major bugs in this. I added my patches top of Lucas' patches, they use API to fetch holds and display them in datatable. However modifying them is still done in back-end where they are then send back to front-end as JSON and to another datatable. There is also a feature to modify search before submiting the changes. There is still a problem with handling the suspend status which I'm currently working with (changing it doesn't work in other languages than english).
1. Handling state across paginated AJAX table is hard, for me at least. So, I don't think we actually need checkboxes. Instead, you do the search and can bulk update everything that is returned. There should be an option to refine the search.
I do agree, handling checkboxes is a mess with pagination. But I'm not yet ready to give up on that so I'll see what I can do.
2. Instead of handling errors with JS before you submit I think can simply report back on holds that were not updated. Example, trying to suspend a waiting hold.
I'm not sure if this is a good idea. It feels more user-friendly to provide feedback for users before anything is changed.
3. The report table in the final step should report back successes and errors, so I think it cannot be an API/AJAX table since you can't get that data from the API. But maybe a standard DataTable?
Again if we handle errors before making modifications there is no need for this. I think we need more feedback from users on how to handle these. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #57 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- There seems to be a way to save the state of checkboxes with pagination, at least I was able to do this via patron search. So we can probably use same method with this also. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177982|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177981|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177976|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177975|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177969|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177968|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177967|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #58 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- (In reply to Emmi Takkinen from comment #57)
There seems to be a way to save the state of checkboxes with pagination, at least I was able to do this via patron search. So we can probably use same method with this also.
Emmi, my apologies here. Sorry for the confusion, I meant to help here but looks like I only got in the way! Maybe using something like this to manage state? https://datatables.net/extensions/select/index Lucas -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #59 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Lucas Gass (lukeg) from comment #58)
(In reply to Emmi Takkinen from comment #57)
There seems to be a way to save the state of checkboxes with pagination, at least I was able to do this via patron search. So we can probably use same method with this also.
Emmi, my apologies here. Sorry for the confusion, I meant to help here but looks like I only got in the way!
Maybe using something like this to manage state?
https://datatables.net/extensions/select/index
Lucas
Hmm, for some reason this isn't working. Which is a bummer, this seems like easy way achieve saving the state. Maybe I'm missing something here even thought at least in paper this should be pretty simple to take in use :D -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #163021|0 |1 is obsolete| | --- Comment #60 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178154 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178154&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177961|0 |1 is obsolete| | --- Comment #61 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178155 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178155&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177962|0 |1 is obsolete| | --- Comment #62 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178156 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178156&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177963|0 |1 is obsolete| | --- Comment #63 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178157 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178157&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177964|0 |1 is obsolete| | --- Comment #64 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178158 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178158&action=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177965|0 |1 is obsolete| | --- Comment #65 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178159 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178159&action=edit Bug 36135: Add several new features This patch adds following new features to batch modify holds tool. - Do not automatically select login library as search param - Add biblios title and items barcode to found holds table - Add count of found holds at top of the table - Add ability to do not change pick up library - Display those hold rows with some found status in different color Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #177966|0 |1 is obsolete| | --- Comment #66 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178160 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178160&action=edit Bug 36135: Fix QA issues and add improvements This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #67 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178161 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178161&action=edit Bug 36135: Fix breadcrumbs/Bootstrap issues -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #68 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178162 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178162&action=edit Bug 36135: Fix some markup errors -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #69 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178163 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178163&action=edit Bug 36135: Embed items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #70 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178164 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178164&action=edit Bug 36135: Add ability to fetch patron info via API holds endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #71 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178165 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178165&action=edit Bug 36135: Use API and datatables to display holds to modify -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #72 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178166 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178166&action=edit Bug 36135: Save state of selected checkboxes and fix modifying suspend -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff --- Comment #73 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- All patches are now rebased and ready for testing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #74 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- There is still some issues with modifying suspend status. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178166|0 |1 is obsolete| | --- Comment #75 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178167 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178167&action=edit Bug 36135: Save state of selected checkboxes and fix modifying suspend -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Avery Campbell (Butte County Library (CA)) <acampbell@buttecounty.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |acampbell@buttecounty.net --- Comment #76 from Avery Campbell (Butte County Library (CA)) <acampbell@buttecounty.net> --- Batch editing holds would be great to have. I don't know if batch changing priorities is planned as part of this, but we have a large number of tech devices (hotpots) which are being deactivated. We have placed them on hold so we can get them before another patron checks out an unusable device. I'm working on moving them to the top of the records' hold queue. Batch changing the priorities would be much better than manually clicking the double-up for all ~180 items. This would be a huge time saver if another library were to need to do something this in the future. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #77 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 178954 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178954&action=edit Bug 36135: Batch clear hold notes and use select2 in multiple filters This patch adds ability to clear hold note from all selected holds. It also adds select2 to libraries and holds status filters. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #78 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Sorry Emmi, patches don't apply to current main. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #79 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Lucas Gass (lukeg) from comment #78)
Sorry Emmi, patches don't apply to current main.
That's odd, my main is up to date and patches apply cleanly to it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #80 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- I've just tried to apply these and was also unable. This is on my ktd system, on main, immediately after a git pull. Specifically: Applying: Bug 36135: Add links to the batch modify holds tool Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc error: Failed to merge in the changes. Patch failed at 0001 Bug 36135: Add links to the batch modify holds tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #81 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Andrew Fuerste-Henry from comment #80)
I've just tried to apply these and was also unable. This is on my ktd system, on main, immediately after a git pull.
Specifically: Applying: Bug 36135: Add links to the batch modify holds tool Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc error: Failed to merge in the changes. Patch failed at 0001 Bug 36135: Add links to the batch modify holds tool
Ah, I see. I'm using git-installation for testing, so I guess it and ktd differ from each other somehow. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #82 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Emmi Takkinen from comment #81)
(In reply to Andrew Fuerste-Henry from comment #80)
I've just tried to apply these and was also unable. This is on my ktd system, on main, immediately after a git pull.
Specifically: Applying: Bug 36135: Add links to the batch modify holds tool Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/tools-menu.inc error: Failed to merge in the changes. Patch failed at 0001 Bug 36135: Add links to the batch modify holds tool
Ah, I see. I'm using git-installation for testing, so I guess it and ktd differ from each other somehow.
And thinking this a little more, it's not possible that they differ. I'll try rebase again. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178154|0 |1 is obsolete| | --- Comment #83 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179028 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179028&action=edit Bug 36135: Add new permission batch_modify_holds This patch adds new permission "batch_modify_holds". To test: 1. Apply this patch. 2. Run updatedatabase.pl. 3. Check that new permission was added to Patrons->Set permissions. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178155|0 |1 is obsolete| | --- Comment #84 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179029 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179029&action=edit Bug 36135: Add links to the batch modify holds tool This patch adds links to new tool batch modify holds tool. To test: 1. Apply this patch. 2. Add patron permission "batch_modify_holds". 3. Login as chosen patron. 4. Navigate to Tools page. => Note there's a link "Batch modify holds". 5. Remove permission from user. => Link to new tool is not displayed. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178156|0 |1 is obsolete| | --- Comment #85 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179030 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179030&action=edit Bug 36135: Add new tool to batch modify holds We sometimes need to modify holds due library closing down etc. For this we need tool to batch modify holds which allows us to filter holds and modify them. Tool allows users to select holds with following parameters: - expiration date -> if patron_expiration_date is set, it is used as main search filter -> otherwise expirationdate is used - library - found status - suspended status - suspend until date - hold note Users are able to modify following with tool: - expiration date (not expiration date set by patron) - pick up library - suspend status (only if hold has no found status) - suspend until date (only if suspend status is set) - hold note To test: 1. Apply this patch. 2. Navigate to new tool Tools->Batch modify holds. 3. Fill in some criteria to search holds with. => Confirm that holds found match your search criteria. 4. Set e.g. new expiration date for all or just selected holds. => Confirm that holds were modified as they should have. => Confirm this also from database. To test suspending holds with found status: 1. Find holds with found status. 2. Attempt to modify their suspend status. => Alert box with text "One or more holds have found status and can't be suspended." should be displayed. Play around, test different search compinations and modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178157|0 |1 is obsolete| | --- Comment #86 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179031 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179031&action=edit Bug 36135: Allow using SQL results in batch modify holds tool This patch adds ability to use SQL results for batch holds tool if report contains column reserve_id. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Sponsored-by: Koha-Suomi Oy Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178158|0 |1 is obsolete| | --- Comment #87 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179032 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179032&action=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178159|0 |1 is obsolete| | --- Comment #88 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179033 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179033&action=edit Bug 36135: Add several new features This patch adds following new features to batch modify holds tool. - Do not automatically select login library as search param - Add biblios title and items barcode to found holds table - Add count of found holds at top of the table - Add ability to do not change pick up library - Display those hold rows with some found status in different color Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178160|0 |1 is obsolete| | --- Comment #89 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179034 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179034&action=edit Bug 36135: Fix QA issues and add improvements This patch: - Fixes issues reported by qa-tool - Adds info about how many holds were modified - Add title and barcode columns to table of modified holds - Adds div with class "page-section" to "Modify holds" table Sponsored-by: Koha-Suomi Oy Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #90 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179035 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179035&action=edit Bug 36135: Report errors to a modal -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178161|0 |1 is obsolete| | --- Comment #91 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179036 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179036&action=edit Bug 36135: Fix breadcrumbs/Bootstrap issues -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178162|0 |1 is obsolete| | --- Comment #92 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179037 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179037&action=edit Bug 36135: Fix some markup errors -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178163|0 |1 is obsolete| | --- Comment #93 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179038 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179038&action=edit Bug 36135: Embed items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178164|0 |1 is obsolete| | --- Comment #94 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179039 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179039&action=edit Bug 36135: Add ability to fetch patron info via API holds endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178165|0 |1 is obsolete| | --- Comment #95 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179040 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179040&action=edit Bug 36135: Use API and datatables to display holds to modify -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178167|0 |1 is obsolete| | --- Comment #96 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179041 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179041&action=edit Bug 36135: Save state of selected checkboxes and fix modifying suspend -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #178954|0 |1 is obsolete| | --- Comment #97 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 179042 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179042&action=edit Bug 36135: Batch clear hold notes and use select2 in multiple filters -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff --- Comment #98 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- There was something wrong with patches I had in my development branch, they applied cleanly against current main. But after creating a new clean branch and applying patches from Bugzilla I got same merge conflict as Andrew. I had also dropped one of the patches at some point. These should now apply, but I was wondering do they still need some clean up to match changes made for tidying up the whole code base (bug 38664). -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #99 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 179081 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179081&action=edit Bug 36135: (QA follow-up): fix typo in tool permissions The flagsrequired for the new page was incorrectly set to 'batch_modify_holds.tt' -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #100 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 179082 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=179082&action=edit Bug 36135: (follow-up) Fix link style on guided report page -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=23258 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmanuel Bétemps <e.betemps@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply CC| |e.betemps@gmail.com --- Comment #101 from Emmanuel Bétemps <e.betemps@gmail.com> --- Hi, I created 3 holds pour the same patron Steps 1 to 4 are ok but when I click on the "Batch modify holds" option, the Tolls page doesn't list any hold I just see the staff interface, the title of this tool (Batch modify holds) but no hold is displayed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #102 from Emmanuel Bétemps <e.betemps@gmail.com> --- oups not 'pour' but 'for' the same patron -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #103 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Emmanuel Bétemps from comment #101)
Hi,
I created 3 holds pour the same patron Steps 1 to 4 are ok but when I click on the "Batch modify holds" option, the Tolls page doesn't list any hold I just see the staff interface, the title of this tool (Batch modify holds) but no hold is displayed
I'm guessing you tested importing SQL results to tool? We noticed the same problem a while ago and I'm think it has something to do with API. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #104 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 181049 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=181049&action=edit Bug 36135: Fix ability to send SQL results in batch modify holds tool Ability to send SQL results to batch holds tool was broken due use of API to fetch holds. This patch fixes this. To test: 1. Create SQL report which contains column for reserve_id. => e.g "SELECT reserve_id FROM reserves limit 10". 2. Run report. 3. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 4. Click "Batch modify holds" option. => Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. Note that when SQL results are send to the tool, search form is displayed for a second before API call is launched. This is expected behaviour since we need to first navigate to the search form so we can then trigger the forms submit event automatically. Also tidy some code to make QA tool happy. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #105 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- On searching, it appears this only uses dates (expiration or suspension) if both ends of the range are supplied. If only one end of the date range is supplied, we should interpret that as an open-ended search -- entering 2025-05-01 fr "Expiration date from" and leaving "Expiration date to" empty should only return holds that have an expiration date greater than 2025-05-01. We're missing the word "and" when repeating back the user's suspend dates on search results: "suspend date between 2025-04-17-2025-04-18" You give the message "You have to suspend holds if new suspend until date is set." if one tries to set Suspend Until date without also clicking Suspend Holds. Could you automatically select Suspend Holds for the user if they set a suspend until? Error when trying to send holds from a report: Programming error - op 'form' must start with 'cud-' for POST http://localhost:8081/intranet/tools/batch_modify_holds.pl (referer: http://localhost:8081/cgi-bin/koha/reports/guided_reports.pl?id=1&op=run) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emily Lamancusa (emlam) <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |emily.lamancusa@montgomeryc | |ountymd.gov -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #106 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- I can select checkboxes from multiple pages now, but when click 'Modify holds' only the ones on my current page are handled. Also, we have Bug 23258 which creates/cancels holds. It would be great if we could bring these 2 bugs together, instead of having 2 seperate but similar features? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #107 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Lucas Gass (lukeg) from comment #106)
I can select checkboxes from multiple pages now, but when click 'Modify holds' only the ones on my current page are handled. Oof, so it seems. I'll fix it with other problems Andrew pointed out.
Also, we have Bug 23258 which creates/cancels holds. It would be great if we could bring these 2 bugs together, instead of having 2 seperate but similar features? I see your point and agree, these should probably be one and same tool. I'm just wondering what's the best plan here, push tools as they are to main (when they are ready to be pushed to main) and then "merge" them. Or push one to main and then add other to it? Or merge them before the push?
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #108 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Emmi Takkinen from comment #107)
(In reply to Lucas Gass (lukeg) from comment #106)
I can select checkboxes from multiple pages now, but when click 'Modify holds' only the ones on my current page are handled. Oof, so it seems. I'll fix it with other problems Andrew pointed out.
Also, we have Bug 23258 which creates/cancels holds. It would be great if we could bring these 2 bugs together, instead of having 2 seperate but similar features? I see your point and agree, these should probably be one and same tool. I'm just wondering what's the best plan here, push tools as they are to main (when they are ready to be pushed to main) and then "merge" them. Or push one to main and then add other to it? Or merge them before the push?
Although... Now when I think about this more, our tools to modify and delete patrons, items and records are all separated. So from that point of view these hold tools could also be different tools. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #109 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Lucas Gass (lukeg) from comment #106)
I can select checkboxes from multiple pages now, but when click 'Modify holds' only the ones on my current page are handled.
Looked into this and it seems to be the default for datatables to handle data like this. I tried turning option "serverSide" as false as suggested in the DataTables forum to fetch all data from table, but no luck, it still changes only the holds in current page. We could fetch hold_ids from localStorage, but then we lose ability to check in front-end if modification is possible (e.g. if user tries to suspend hold with found status) and we can't display error message pop-up before attempting to modify holds. I came up with following ways to handle this: 1) Fetch found and suspended status for selected holds with API and display error message as pop-up like it is currently displayed. 2) Make check from back-end and send error message from there back to front. 3) Do not display error message, but do not modify holds which can't be modified either. Show "Following holds couldn't be modified. Reason:..." in "Holds modified" page. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #110 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- We have encountered some other tricky problems with this (modified holds table isn't rendered in some cases), so getting this to move might take a while. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #111 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 183151 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183151&action=edit Bug 36135: Add several fixes This patch adds several fixes and improvements: -Allow open-ended search To test: 1. Find holds to modify and fill either "Expiration date from" field or "Expiration date to" field. 2. Click "Search" button. => Confirm that your search contains holds that match your search. -Suspend hold if new suspend until date is set To test: 1. Find some holds to modify which can be suspended. 2. Select holds to modify from your search and set new suspended date for them. Do not select new suspended status from "Suspend holds" dropdown. 3. Click "Modify holds" button. => Confirm that pop-up preventing modificating holds suspend date without setting new suspend status is not displayed and holds are now suspended with new suspended until date. -Add word "and" to suspend date filter message -Change "form" to "cud-form" in guided_reports_start.tt to prevent programming error message Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|emmi.takkinen@koha-suomi.fi |lucas@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179028|0 |1 is obsolete| | --- Comment #112 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183294 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183294&action=edit Bug 36135: Add new permission batch_modify_holds Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #113 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183295 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183295&action=edit Bug 36135: Allow using SQL results in batch modify holds tool Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179038|0 |1 is obsolete| | --- Comment #114 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183296 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183296&action=edit Bug 36135: Embed items -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179039|0 |1 is obsolete| | --- Comment #115 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183297 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183297&action=edit Bug 36135: Add ability to fetch patron info via API holds endpoint -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179082|0 |1 is obsolete| | --- Comment #116 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183298 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183298&action=edit Bug 36135: (follow-up) Fix link style on guided report page Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #117 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183299 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183299&action=edit Bug 36135: Fix syntax error in permissions.inc -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #118 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183300 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183300&action=edit Bug 36135: Fix forbidden pattern -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183151|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #181049|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179081|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179042|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179041|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179040|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179037|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179036|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179035|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179034|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179033|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179032|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179031|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179030|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #179029|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183294|Bug 36135: Add new |Bug 36135: Add description|permission |batch_modify_holds |batch_modify_holds | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #119 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- I squashed some of these patches and am resetting to NSO. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #120 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183305 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183305&action=edit Bug 36135: Add batch_modify_holds permissions Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #121 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183306 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183306&action=edit Bug 36135: Add ability to batch modify holds TO TEST: 0. APPLY PATCHES, updatedatabase, restart_all 1. Create a user with limited permissions, making sure you give them the "batch_modify_holds" permission 2. Login as chosen patron. 3. Navigate to Tools page. 4. Note there's a link "Batch modify holds". 5. Remove permission from user. 6. Link to new tool is not displayed. 7. Navigate to new tool Tools->Batch modify holds. 8. Fill in some criteria to search holds with. 9. Confirm that holds found match your search criteria. 10. Set e.g. new expiration date for all or just selected holds. 11. Confirm that holds were modified as they should have. 12. Confirm this also from database. 13. Find holds with a found/waiting status. 14. Attempt to modify their suspend status. 15. Alert box with text "One or more holds have found status and can't be suspended." should be displayed. 16. Play around, test different modifications. 17. Create SQL report which contains column for reserve_id. 18. e.g "SELECT reserve_id FROM reserves limit 10". 19.. Run report. 20. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 21. Click "Batch modify holds" option. 22. Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. 23. Do a holds search that will bring up multiple pages of results 24. Try selecting some holds from different pages to modify 25. Ensure all selected holds are being modified 26. Play around, test different modifications. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183299|0 |1 is obsolete| | --- Comment #122 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183307 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183307&action=edit Bug 36135: Fix syntax error in permissions.inc Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183300|0 |1 is obsolete| | --- Comment #123 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183308 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183308&action=edit Bug 36135: Fix forbidden pattern Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183298|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183297|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183296|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183295|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183294|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #124 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- squashed patches and added complete test plan, reading for testing! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Medium patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #125 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- (In reply to Lucas Gass (lukeg) from comment #124)
squashed patches and added complete test plan, reading for testing!
Noting that this all works as intended by chokes if you try to modify a lot of holds at once due to a URI length error. Confirmed it works with about holds but dies with about 1000, so the issue is somewhere in there :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #126 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- (In reply to Andrew Fuerste-Henry from comment #125)
(In reply to Lucas Gass (lukeg) from comment #124)
squashed patches and added complete test plan, reading for testing!
Noting that this all works as intended by chokes if you try to modify a lot of holds at once due to a URI length error. Confirmed it works with about holds but dies with about 1000, so the issue is somewhere in there :)
This seems to be a dataTable issue. All of the modified hold ID's are send back to be filtered and the number of IDs seems to be too much for dataTables when we approach 1000 modified holds. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #127 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- (In reply to Lucas Gass (lukeg) from comment #126)
(In reply to Andrew Fuerste-Henry from comment #125)
(In reply to Lucas Gass (lukeg) from comment #124)
squashed patches and added complete test plan, reading for testing!
Noting that this all works as intended by chokes if you try to modify a lot of holds at once due to a URI length error. Confirmed it works with about holds but dies with about 1000, so the issue is somewhere in there :)
This seems to be a dataTable issue. All of the modified hold ID's are send back to be filtered and the number of IDs seems to be too much for dataTables when we approach 1000 modified holds.
I think we could make the modification results table not be generated from the API. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #128 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 183771 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183771&action=edit Bug 36135: Make modified_holds table not use the API -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|37032 |40332 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37032 [Bug 37032] REST API: Unable to call item info via holds endpoint https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40332 [Bug 40332] Tools menu sidebar category not shown for users with batch_extend_due_dates only -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #129 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 183893 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183893&action=edit Bug 36135: Correct tools-menu category logic -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #130 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Small searching issue here. When searching by expiration date, one can provide both To and From dates to do a Between search, or one can provide just one date to perform a Before or After search. But when searching by suspension date, one *must* provide two dates and do a Between search. If only one date is given, the search is not limited on suspension date at all. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all task changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #131 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 183924 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183924&action=edit Bug 36135: (follow-up) Fix suspension search and other improvements This patch tries to fix the hold suspension search by copying the code from the expiration date search function. The patch also makes these changes: - Change page title to use i18n functions - Add specific breadcrumb for results page - Update headings between search and results page - Correct table settings (definition was missing from columns_settings.yml - Correct capitalization (e.g. "Pickup location" instead of "Pickup Location" - Use include files for displaying title and patron in update results - Add subtitle and medium to title in search results - Remove quotes from DataTables option names - Add hold date to the search results view. - Without a specified default sort, the table will sort the first column. Since the first column is checkboxes, we ought to explicitly set a default sorting column. I thought hold date seemed like a better default than title. -- You are receiving this mail because: You are watching all task changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> 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=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183305|0 |1 is obsolete| | --- Comment #132 from Jason Robb <jrobb@sekls.org> --- Created attachment 183959 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183959&action=edit Bug 36135: Add batch_modify_holds permissions Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183306|0 |1 is obsolete| | --- Comment #133 from Jason Robb <jrobb@sekls.org> --- Created attachment 183960 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183960&action=edit Bug 36135: Add ability to batch modify holds TO TEST: 0. APPLY PATCHES, updatedatabase, restart_all 1. Create a user with limited permissions, making sure you give them the "batch_modify_holds" permission 2. Login as chosen patron. 3. Navigate to Tools page. 4. Note there's a link "Batch modify holds". 5. Remove permission from user. 6. Link to new tool is not displayed. 7. Navigate to new tool Tools->Batch modify holds. 8. Fill in some criteria to search holds with. 9. Confirm that holds found match your search criteria. 10. Set e.g. new expiration date for all or just selected holds. 11. Confirm that holds were modified as they should have. 12. Confirm this also from database. 13. Find holds with a found/waiting status. 14. Attempt to modify their suspend status. 15. Alert box with text "One or more holds have found status and can't be suspended." should be displayed. 16. Play around, test different modifications. 17. Create SQL report which contains column for reserve_id. 18. e.g "SELECT reserve_id FROM reserves limit 10". 19.. Run report. 20. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 21. Click "Batch modify holds" option. 22. Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. 23. Do a holds search that will bring up multiple pages of results 24. Try selecting some holds from different pages to modify 25. Ensure all selected holds are being modified 26. Play around, test different modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183307|0 |1 is obsolete| | --- Comment #134 from Jason Robb <jrobb@sekls.org> --- Created attachment 183961 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183961&action=edit Bug 36135: Fix syntax error in permissions.inc Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183308|0 |1 is obsolete| | --- Comment #135 from Jason Robb <jrobb@sekls.org> --- Created attachment 183962 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183962&action=edit Bug 36135: Fix forbidden pattern Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183771|0 |1 is obsolete| | --- Comment #136 from Jason Robb <jrobb@sekls.org> --- Created attachment 183963 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183963&action=edit Bug 36135: Make modified_holds table not use the API Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183893|0 |1 is obsolete| | --- Comment #137 from Jason Robb <jrobb@sekls.org> --- Created attachment 183964 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183964&action=edit Bug 36135: Correct tools-menu category logic Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183924|0 |1 is obsolete| | --- Comment #138 from Jason Robb <jrobb@sekls.org> --- Created attachment 183965 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183965&action=edit Bug 36135: (follow-up) Fix suspension search and other improvements This patch tries to fix the hold suspension search by copying the code from the expiration date search function. The patch also makes these changes: - Change page title to use i18n functions - Add specific breadcrumb for results page - Update headings between search and results page - Correct table settings (definition was missing from columns_settings.yml - Correct capitalization (e.g. "Pickup location" instead of "Pickup Location" - Use include files for displaying title and patron in update results - Add subtitle and medium to title in search results - Remove quotes from DataTables option names - Add hold date to the search results view. - Without a specified default sort, the table will sort the first column. Since the first column is checkboxes, we ought to explicitly set a default sorting column. I thought hold date seemed like a better default than title. Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |new feature -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jrobb@sekls.org --- Comment #139 from Jason Robb <jrobb@sekls.org> --- Worked great in testing, excited for this! I did get confused when testing batch modification from a report. If you try to start a new search from that same page of results, the report-supplied filter remains in addition to any newly applied filters. If you want to lift that filter, you have to fully exit the tool and start over. It's not a major thing but maybe in the future there could be an indication that the results are filtered with an option to clear that filter, similar to how you can see when a filter is present when browsing saved reports. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #140 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Thank you Lucas for taking over this! Our libraries have used this tool for a while now and they have pointed out that it should be possible to also modify holds with different status. We thought this should be prevented when we were planning what users should be able to do with the tool but now this has quickly become an issue when holds need to be modified etc. due library being closed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 carthur@slolibrary.org <carthur@slolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |carthur@slolibrary.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pedro.amorim@openfifth.co.u | |k QA Contact|testopia@bugs.koha-communit |pedro.amorim@openfifth.co.u |y.org |k Status|Signed Off |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=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183959|0 |1 is obsolete| | --- Comment #141 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185051 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185051&action=edit Bug 36135: Add batch_modify_holds permissions Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183960|0 |1 is obsolete| | --- Comment #142 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185052 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185052&action=edit Bug 36135: Add ability to batch modify holds TO TEST: 0. APPLY PATCHES, updatedatabase, restart_all 1. Create a user with limited permissions, making sure you give them the "batch_modify_holds" permission 2. Login as chosen patron. 3. Navigate to Tools page. 4. Note there's a link "Batch modify holds". 5. Remove permission from user. 6. Link to new tool is not displayed. 7. Navigate to new tool Tools->Batch modify holds. 8. Fill in some criteria to search holds with. 9. Confirm that holds found match your search criteria. 10. Set e.g. new expiration date for all or just selected holds. 11. Confirm that holds were modified as they should have. 12. Confirm this also from database. 13. Find holds with a found/waiting status. 14. Attempt to modify their suspend status. 15. Alert box with text "One or more holds have found status and can't be suspended." should be displayed. 16. Play around, test different modifications. 17. Create SQL report which contains column for reserve_id. 18. e.g "SELECT reserve_id FROM reserves limit 10". 19.. Run report. 20. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 21. Click "Batch modify holds" option. 22. Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. 23. Do a holds search that will bring up multiple pages of results 24. Try selecting some holds from different pages to modify 25. Ensure all selected holds are being modified 26. Play around, test different modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183961|0 |1 is obsolete| | --- Comment #143 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185053 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185053&action=edit Bug 36135: Fix syntax error in permissions.inc Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183962|0 |1 is obsolete| | --- Comment #144 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185054 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185054&action=edit Bug 36135: Fix forbidden pattern Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183963|0 |1 is obsolete| | --- Comment #145 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185055 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185055&action=edit Bug 36135: Make modified_holds table not use the API Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183964|0 |1 is obsolete| | --- Comment #146 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185056 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185056&action=edit Bug 36135: Correct tools-menu category logic Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183965|0 |1 is obsolete| | --- Comment #147 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185057&action=edit Bug 36135: (follow-up) Fix suspension search and other improvements This patch tries to fix the hold suspension search by copying the code from the expiration date search function. The patch also makes these changes: - Change page title to use i18n functions - Add specific breadcrumb for results page - Update headings between search and results page - Correct table settings (definition was missing from columns_settings.yml - Correct capitalization (e.g. "Pickup location" instead of "Pickup Location" - Use include files for displaying title and patron in update results - Add subtitle and medium to title in search results - Remove quotes from DataTables option names - Add hold date to the search results view. - Without a specified default sort, the table will sort the first column. Since the first column is checkboxes, we ought to explicitly set a default sorting column. I thought hold date seemed like a better default than title. Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #148 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 185073 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185073&action=edit Funky filters 1) The new permission batch_modify_holds is not enough to use this tool. If the staff member has that but does not have reserveforothers->place_holds, they can still access the tool but get a 403 error when performing a search at /cgi-bin/koha/tools/batch_modify_holds.pl 2) Something funky happening with select2 filters, check attached picture. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #149 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Thanks for looking, Pedro:
1) The new permission batch_modify_holds is not enough to use this tool. If the staff member has that but does not have reserveforothers->place_holds, they can still access the tool but get a 403 error when performing a search at /cgi-bin/koha/tools/batch_modify_holds.pl Good catch, I'll have a closer look.
2) Something funky happening with select2 filters, check attached picture. I've noticed this too with other select2 dropdowns. I filed a bug to hopefully fix that everywhere:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40583 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Brendan Lawlor <blawlor@clamsnet.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |blawlor@clamsnet.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #150 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> ---
1) The new permission batch_modify_holds is not enough to use this tool. If the staff member has that but does not have reserveforothers->place_holds, they can still access the tool but get a 403 error when performing a search at /cgi-bin/koha/tools/batch_modify_holds.pl Good catch, I'll have a closer look.
There are several solutions to this but I don't know which is right. 1. Check for both place_holds and batch_modify_holds before allowing to use tool. 2. Update holds API endpoint to allow for those who have batch_modify_holds 3. Move the permission from tools to reserveforothers. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #151 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 185829 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185829&action=edit Bug 36135: Require users to have batch_modify_holds and place_holds permssion to use the tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #152 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Changing to 'Signed Off' (by Jason Robb <jrobb@sekls.org>) as it was previous to the FQA. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> 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=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185051|0 |1 is obsolete| | --- Comment #153 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186072 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186072&action=edit Bug 36135: Add batch_modify_holds permissions Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185052|0 |1 is obsolete| | --- Comment #154 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186073 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186073&action=edit Bug 36135: Add ability to batch modify holds TO TEST: 0. APPLY PATCHES, updatedatabase, restart_all 1. Create a user with limited permissions, making sure you give them the "batch_modify_holds" permission 2. Login as chosen patron. 3. Navigate to Tools page. 4. Note there's a link "Batch modify holds". 5. Remove permission from user. 6. Link to new tool is not displayed. 7. Navigate to new tool Tools->Batch modify holds. 8. Fill in some criteria to search holds with. 9. Confirm that holds found match your search criteria. 10. Set e.g. new expiration date for all or just selected holds. 11. Confirm that holds were modified as they should have. 12. Confirm this also from database. 13. Find holds with a found/waiting status. 14. Attempt to modify their suspend status. 15. Alert box with text "One or more holds have found status and can't be suspended." should be displayed. 16. Play around, test different modifications. 17. Create SQL report which contains column for reserve_id. 18. e.g "SELECT reserve_id FROM reserves limit 10". 19.. Run report. 20. Over report select "Batch operations" should be displayed and under that there should be option "Batch modify holds". 21. Click "Batch modify holds" option. 22. Confirm that holds displayed in "Batch mofidy holds" tool are same as found with your report. 23. Do a holds search that will bring up multiple pages of results 24. Try selecting some holds from different pages to modify 25. Ensure all selected holds are being modified 26. Play around, test different modifications. Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185053|0 |1 is obsolete| | --- Comment #155 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186074 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186074&action=edit Bug 36135: Fix syntax error in permissions.inc Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185054|0 |1 is obsolete| | --- Comment #156 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186075 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186075&action=edit Bug 36135: Fix forbidden pattern Sponsored-by: Koha-Suomi Oy Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185055|0 |1 is obsolete| | --- Comment #157 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186076 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186076&action=edit Bug 36135: Make modified_holds table not use the API Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185056|0 |1 is obsolete| | --- Comment #158 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186077 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186077&action=edit Bug 36135: Correct tools-menu category logic Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185057|0 |1 is obsolete| | --- Comment #159 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186078 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186078&action=edit Bug 36135: (follow-up) Fix suspension search and other improvements This patch tries to fix the hold suspension search by copying the code from the expiration date search function. The patch also makes these changes: - Change page title to use i18n functions - Add specific breadcrumb for results page - Update headings between search and results page - Correct table settings (definition was missing from columns_settings.yml - Correct capitalization (e.g. "Pickup location" instead of "Pickup Location" - Use include files for displaying title and patron in update results - Add subtitle and medium to title in search results - Remove quotes from DataTables option names - Add hold date to the search results view. - Without a specified default sort, the table will sort the first column. Since the first column is checkboxes, we ought to explicitly set a default sorting column. I thought hold date seemed like a better default than title. Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Pedro Amorim <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #185829|0 |1 is obsolete| | --- Comment #160 from Pedro Amorim <pedro.amorim@openfifth.co.uk> --- Created attachment 186079 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186079&action=edit Bug 36135: Require users to have batch_modify_holds and place_holds permssion to use the tool Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |25.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #161 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Nice work everyone! Pushed to main for 25.11 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patchset adds a new release notes| |feature that allows for the | |batch modification of holds | |via a new tool. It also | |adds a new permission to | |allow use of the batch | |modify holds tool. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #162 from Laura O'Neil <laura@bywatersolutions.com> --- Would it be possible to backport this to 25.05.xx as well? It would be so so helpful to many of our partners when they face unexpected closures. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #163 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 186141 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186141&action=edit Bug 36135: Fix t/Koha/Auth/Permissions.t -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |additional_work_needed CC| |jonathan.druart@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #164 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Follow-up pushed to main, thanks Jonathan. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=36135 --- Comment #165 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- I believe the addition of patron to the REST API definition in api/v1/swagger/definitions/hold.yaml is contributing to a failing cypress test: t/cypress/integration/KohaTable/PendingHolds_spec.ts -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #166 from David Cook <dcook@prosentient.com.au> --- (In reply to Lucas Gass (lukeg) from comment #165)
I believe the addition of patron to the REST API definition in api/v1/swagger/definitions/hold.yaml is contributing to a failing cypress test:
t/cypress/integration/KohaTable/PendingHolds_spec.ts
Yep, looks that way. I may was well fix this while I'm fixing bug 15516. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #167 from David Cook <dcook@prosentient.com.au> --- Created attachment 186186 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186186&action=edit Bug 36135: Fix Cypress test -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #168 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 186187 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=186187&action=edit Bug 36135: Fix KohaTable/PendingHolds_spec.ts -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #169 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Jonathan Druart from comment #168)
Created attachment 186187 [details] [review] Bug 36135: Fix KohaTable/PendingHolds_spec.ts
Oops, I didn't see David's patch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #170 from David Cook <dcook@prosentient.com.au> --- (In reply to Jonathan Druart from comment #169)
(In reply to Jonathan Druart from comment #168)
Created attachment 186187 [details] [review] [review] Bug 36135: Fix KohaTable/PendingHolds_spec.ts
Oops, I didn't see David's patch.
And here I was trying to help so you didn't have to do it all yourself hehe. There's another one on bug 15516 for the hold_group_id one -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #186187|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|additional_work_needed | --- Comment #171 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Follow-ups pushed to main. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Pushed to stable Version(s)|25.11.00 |25.11.00,25.05.04 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #172 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Nice work everyone! Pushed to 25.05.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Paul Derscheid <paul.derscheid@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)|25.11.00,25.05.04 |25.11.00 released in| | CC| |paul.derscheid@lmscloud.de --- Comment #173 from Paul Derscheid <paul.derscheid@lmscloud.de> --- Reverted for 25.05.04, removed from Version(s) released in. Sorry to disappoint as this was specifically requested but you'll need to wait for 25.11 @Laura O'Neil. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.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=36135 Lucas Gass (lukeg) <lucas@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=36135 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #174 from Laura O'Neil <laura@bywatersolutions.com> --- That's okay, Paul. Something to look forward to! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=38739 --- Comment #175 from Jonathan Druart <jonathan.druart@gmail.com> --- Please use main-container.inc, see bug 38739. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to main |Needs documenting --- Comment #176 from Laura Escamilla <Laura.escamilla@bywatersolutions.com> --- This is a new feature and will not be backported to the 25.05.x branch. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=41334 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Sara Brown <sbrown@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=41882 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Sara Brown <sbrown@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=41883 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emily Lamancusa (emlam) <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |41882 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41882 [Bug 41882] Batch hold modification tool updates pickup locations to disallowed libraries -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emily Lamancusa (emlam) <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42249 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42249 [Bug 42249] Wrong body id on hold batch modification tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emily Lamancusa (emlam) <emily.lamancusa@montgomerycountymd.gov> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42250 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42250 [Bug 42250] Add link from patron page to hold batch modification tool -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42583 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42583 [Bug 42583] Batch modify holds job should be able to revert hold statuses -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #177 from Aude Charillon <aude.charillon@openfifth.co.uk> --- For documentation: see also bug 42822. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Sara Brown <sbrown@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |Sara Brown contact| | CC| |sbrown@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Aude Charillon <aude.charillon@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |https://gitlab.com/koha-com submission| |munity/koha-manual/-/merge_ | |requests/1239 CC| |aude.charillon@openfifth.co | |.uk, philip.orr@lmscloud.de Documentation|Sara Brown |Philip Orr, Sara Brown contact| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Philip Orr <philip.orr@lmscloud.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Needs documenting |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org