[Koha-bugs] [Bug 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 Jonathan Druart changed: What|Removed |Added Version|unspecified |master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@biblibre.co ||m, ||katrin.fisc...@bsz-bw.de, ||neng...@gmail.com Status|NEW |In Discussion See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=11343, ||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=8246 --- Comment #1 from Jonathan Druart --- Nicole, Katrin, Could I get your opinion on this one? Bug 8246 has been pushed a couple of weeks ago and there is inconsistency between these 2 bug reports. According to the syspref description it should be considered as a bug. "MaxItemsForBatch: Process up to XXX items in a single modification or deletion batch." -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 --- Comment #2 from Katrin Fischer --- Hi Jonathan, I'd agree that the behaviour described doesn't match the syspref description. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 --- Comment #3 from Jonathan Druart --- Not sure to know what you should do here. MaxItemsForBatch has been added because "1000" was hardcoded in the code. In the batch item deletion tool it does not block the deletion, but does not display the item info. In the batch item mod tool it completely blocks the modification. This behavior seems to exist previous to bug 11343. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 --- Comment #4 from Katrin Fischer --- I feel like we should honor the system preference. Wondering: Does it display a warning, that not all information is displayed? (batch item deletion) If we don't honor the pref to its current description, we should rephrase it and also make it clear to the user that not all information is displayed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 Jonathan Druart changed: What|Removed |Added CC||gaetan.bois...@biblibre.com --- Comment #5 from Jonathan Druart --- (In reply to Katrin Fischer from comment #4) > I feel like we should honor the system preference. > > Wondering: Does it display a warning, that not all information is displayed? > (batch item deletion) > > If we don't honor the pref to its current description, we should rephrase it > and also make it clear to the user that not all information is displayed. I am not sure. In this case, people won't be able to modify more than "MaxItemsForBatch" items. I think the limit has been introduced in order not to freeze the browser if there is too much info to display. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 --- Comment #6 from Jonathan Druart --- Created attachment 51424 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51424&action=edit Bug 11490: Add prefs MaxItemsToProcessForBatchMod and MaxItemsToDisplayForBatchDel -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 Jonathan Druart changed: What|Removed |Added Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 --- Comment #7 from Jonathan Druart --- Created attachment 51425 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=51425&action=edit Bug 11490: Split MaxItemsForBatch into 2 prefs to clarify things Prior to this patch, a single pref (MaxItemsForBatch) was used to restrict the use of the batch item modification and deletion. On the batch modification tool, the pref is used to limit the number of items a librarian is allowed to modify in a batch, but on the batch deletion tool the pref is used to limit the number of items to display. To clarify things, this patch split the pref into 2 new prefs: MaxItemsToDisplayForBatchDel and MaxItemsToProcessForBatchMod. Later we could add a MaxItemsToProcessForBatchDel if needed. Note that this patch should not introduce any changes in the current behaviors. Test plan: 0/ Do not execute the update DB entry 1/ Set a value != 1000 for MaxItemsForBatch 2/ Execute the update DB entry and confirm that the 2 new prefs are initially set with the value of MaxItemsForBatch and that MaxItemsForBatch has been deleted 3/ Set different values for these 2 prefs (2 and 3 for instance to ease the next steps) 4/ Try to delete less than MaxItemsToDisplayForBatchDel items => You must see the items details 5/ Try to delete more than MaxItemsToDisplayForBatchDel items => You must not see the items details but are allowed to delete them 6/ Try to modify less than MaxItemsToProcessForBatchMod items => You must see the items details 7/ Try to modify more than MaxItemsToProcessForBatchMod items => You must be restricted and not allowed to process -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 11490] Deleting Batch Items
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11490 Jonathan Druart changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=10163 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/