[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #34 from Emmi Takkinen --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #33 from Emmi Takkinen --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen changed: What|Removed |Added Status|Failed QA |ASSIGNED --- Comment #32 from Emmi Takkinen --- (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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens (kidclamp) changed: What|Removed |Added Status|Signed Off |Failed QA CC||n...@bywatersolutions.com --- Comment #31 from Nick Clemens (kidclamp) --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Cécile Garcia-Belmonte changed: What|Removed |Added CC||cecilegarciabelmonte@loiref ||orez.fr -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura O'Neil changed: What|Removed |Added CC||la...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #164051|0 |1 is obsolete|| --- Comment #30 from Lucas Gass --- Created attachment 164063 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164063=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163796|0 |1 is obsolete|| --- Comment #29 from Lucas Gass --- Created attachment 164062 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164062=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163544|0 |1 is obsolete|| --- Comment #28 from Lucas Gass --- Created attachment 164061 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164061=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163543|0 |1 is obsolete|| --- Comment #27 from Lucas Gass --- Created attachment 164060 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164060=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 Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163542|0 |1 is obsolete|| --- Comment #26 from Lucas Gass --- Created attachment 164059 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164059=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 Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163541|0 |1 is obsolete|| --- Comment #25 from Lucas Gass --- Created attachment 164058 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164058=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 Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Attachment #163540|0 |1 is obsolete|| --- Comment #24 from Lucas Gass --- Created attachment 164057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164057=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 Signed-off-by: Lucas Gass -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #23 from Lucas Gass --- Thanks Emmi! This looks great. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen changed: What|Removed |Added Attachment #164037|0 |1 is obsolete|| --- Comment #22 from Emmi Takkinen --- Created attachment 164051 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164051=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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #21 from Emmi Takkinen --- Fixed QA issues which Lucas pointed out. For this >Expiration date: >Suspended until: >These should probably be not . 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #20 from Emmi Takkinen --- Created attachment 164037 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=164037=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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Michelle Spinney changed: What|Removed |Added CC||mspin...@clamsnet.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added Status|Needs Signoff |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #19 from Lucas Gass --- This looks good but there are some small fixes to be made: 1. WARNinstaller/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.FAILtt_valid lines 134 This probably needs to be something like: [% IF hold.found %] [% ELSE %] [% END %] 4. Line 123: Title 5. Line 111: Missing a closing 6. Overall there needs to be some improvement to the HTML: Expiration date: Suspended until: These should probably be not . elements inside of ? 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 57 To 58 59 91 To 92 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen changed: What|Removed |Added Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #18 from Emmi Takkinen --- Created attachment 163796 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163796=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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #17 from Emmi Takkinen --- (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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Katie Bliss changed: What|Removed |Added CC||kebl...@dmpl.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #16 from Laura Escamilla --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Attachment #163530|0 |1 is obsolete|| --- Comment #15 from Laura Escamilla --- Created attachment 163544 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163544=edit Bug 36135: (follow-up) Adjust for CSRF Signed-off-by: Laura Escamilla -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Attachment #163024|0 |1 is obsolete|| --- Comment #14 from Laura Escamilla --- Created attachment 163543 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163543=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Attachment #163023|0 |1 is obsolete|| --- Comment #13 from Laura Escamilla --- Created attachment 163542 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163542=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Attachment #163022|0 |1 is obsolete|| --- Comment #12 from Laura Escamilla --- Created attachment 163541 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163541=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Attachment #163021|0 |1 is obsolete|| --- Comment #11 from Laura Escamilla --- Created attachment 163540 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163540=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 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens changed: What|Removed |Added Attachment #163529|0 |1 is obsolete|| --- Comment #10 from Nick Clemens --- Created attachment 163530 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163530=edit Bug 36135: (follow-up) Adjust for CSRF -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 --- Comment #9 from Nick Clemens --- Created attachment 163529 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=163529=edit Bug 36135: (follow-up) Adjust for CSRF -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Nick Clemens changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Lisette Scheer changed: What|Removed |Added CC||lisette.scheer@bywatersolut ||ions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Laura Escamilla changed: What|Removed |Added CC||Laura.escamilla@bywatersolu ||tions.com Status|Needs Signoff |Failed QA --- Comment #8 from Laura Escamilla --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Brenda changed: What|Removed |Added CC||bow...@washoecounty.us -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 36135] Add tool to batch modify holds
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36135 Emmi Takkinen 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 --- 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/