[Koha-bugs] [Bug 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Version(s)|22.11.00,21.11, 22.05.06|22.11.00,,22.05.06,21.11.12 released in|| -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #28 from Arthur Suzuki --- Thanks! Pushed to 21.11 for 21.11.12 -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Arthur Suzuki changed: What|Removed |Added Version(s)|22.11.00, 22.05.06 |22.11.00,21.11, 22.05.06 released in|| -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Victor Grousset/tuxayo changed: What|Removed |Added Status|Pushed to stable|RESOLVED Resolution|--- |FIXED --- Comment #27 from Victor Grousset/tuxayo --- Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Nothing to document it seems, marking resolved. -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Lucas Gass changed: What|Removed |Added Version(s)|22.11.00|22.11.00, 22.05.06 released in|| Status|Pushed to master|Pushed to stable --- Comment #26 from Lucas Gass --- Backported to 22.05.x for 22.05.06 -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #25 from Tomás Cohen Arazi --- Pushed to master for 22.11. Nice work everyone, thanks! -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||22.11.00 released in|| -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Attachment #138379|0 |1 is obsolete|| --- Comment #24 from Katrin Fischer --- Created attachment 138388 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138388=edit Bug 28864: Have patron card creator use PatronsPerPage Add pageLength to members/member.tt Add aLengthMenu and pageLength to patroncards/add_user_search (common/patron_search.tt). Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to Tools -> Patron card creator. 3. Start a new Card Batch. 4. Click the 'Add patron(s)' button to open the patron search window. 5. Do a search. 6. Add more than 20 patrons to the batch. 7. Close patron search window. 8. Click the 'Add patron(s)' button to add the selected patrons to the batch. 9. Observe that the table of patrons is has 20 as the default per page setting. 10. Apply patch. 11. Reload the batch. 12. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. Signed-off-by: Sally Signed-off-by: Katrin Fischer -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #23 from Katrin Fischer --- Thanks for clearing that up! -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Victor Grousset/tuxayo changed: What|Removed |Added Attachment #138197|0 |1 is obsolete|| -- 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #21 from Victor Grousset/tuxayo --- Created attachment 138379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138379=edit Bug 28864: Have patron card creator use PatronsPerPage Add pageLength to members/member.tt Add aLengthMenu and pageLength to patroncards/add_user_search (common/patron_search.tt). Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to Tools -> Patron card creator. 3. Start a new Card Batch. 4. Click the 'Add patron(s)' button to open the patron search window. 5. Do a search. 6. Add more than 20 patrons to the batch. 7. Close patron search window. 8. Click the 'Add patron(s)' button to add the selected patrons to the batch. 9. Observe that the table of patrons is has 20 as the default per page setting. 10. Apply patch. 11. Reload the batch. 12. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. Signed-off-by: Sally -- 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Sally changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #12 from Sally --- (In reply to Michael Hafen from comment #7) > My test plan missed hitting the 'add patrons' button the second time. Great, thank you. Patch works as described with the updated test plan - so I've signed off. Unfortunately, the sandbox is playing havoc at the moment and won't sign off both of the patches. After signing one off twice, it has now added a patch from a different bug - rather than signing off the two patches Michael added. Nightmare. :( But the two patches from Michael both work here. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Michael Hafen changed: What|Removed |Added Resolution|WORKSFORME |--- Status|RESOLVED|REOPENED -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #19 from Sally --- (In reply to Victor Grousset/tuxayo from comment #17) > Email sent to Sally to ask if their testing matches the new test plan. To > know if the signoff is still valid. Yes, it's valid. I used the new test plan and checked the bug was valid on master. Then I used the new test plan against the patch, and it worked perfectly - so that's why I added the sign off. I'll change the status back. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Sally changed: What|Removed |Added Status|REOPENED|Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #10 from ByWater Sandboxes --- Created attachment 138196 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138196=edit Bug 31120: Items will renew for zero ( 0 ) days if renewalperiod is blank/empty value If the rule renewalperiod is the blank empty string instead of being null/undefined or non-existant, Koha will interpret the renewal period as being zero days instead of falling back to the issuelength rule. It makes sense to me that a literal 0 here should make it renew for zero days even though that is nonsensical. Test Plan: 1) Delete all your rules 2) Create an all/all/all rules with an empty string for renewal base period 3) Note that renewing an item does nothing 4) Apply this patch 5) Restart all the things! 6) Renew again 7) Note the renewal uses the issuelength rule as intended Signed-off-by: Sally Signed-off-by: Martin Renvoize Signed-off-by: Tomas Cohen Arazi Signed-off-by: Sally -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #22 from Victor Grousset/tuxayo --- (In reply to Michael Hafen from comment #18) > though I like the commit message of the other commit better, but > if you can't change the commit message, then that's ok. Ok I changed to "Bug 28864: Have patron card creator use PatronsPerPage" and added test plan from Comment 15 -- 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Michael Hafen changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|WORKSFORME |--- --- Comment #7 from Michael Hafen --- My test plan missed hitting the 'add patrons' button the second time. The patron table in the search window that open the first time you hit the 'add patrons' button is fine, it honors the PatronsPerPage setting. After adding the patrons to the batch though, the patron table that appears on the main windows does not honor the setting. I just loaded the patron card creator on my workstation, which is running 22.05.3-1, and checked. My PatronsPerPage setting is 50. I created a batch with more than 50 patrons. In the patrons table in the card batch after adding patrons to the batch, the default per page selected is 20, not 50 as it should be. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |katrin.fisc...@bsz-bw.de |y.org | Summary|The patron search results |The patron search results |in the Patron card creator |in the patron card creator |doesn't seem to use |doesn't seem to use |PatronsPerPage syspref |PatronsPerPage syspref -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Sally changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |michael.ha...@washk12.org |ity.org | --- Comment #20 from Katrin Fischer --- Michael, please don't forget to assign yourself! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Attachment #134004|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 ByWater Sandboxes changed: What|Removed |Added Attachment #138195|0 |1 is obsolete|| --- Comment #11 from ByWater Sandboxes --- Created attachment 138197 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138197=edit Bug 28864: Have patron search use PatronsPerPage better. Add pageLength to members/member.tt Add aLengthMenu and pageLength to patroncards/add_user_search (common/patron_search.tt). Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to the Patrons page and search for patrons. 3. Observe that the per page drop down defaults to 20. 4. Go to Tools -> Patron card creator. 5. Start a new Card Batch. 6. Click the 'Add patron(s)' button to open the patron search window. 7. Do a search. 8. Observe that the per page drop down defaults to 20. 8b. If the PatronsPerPage was set to a value other than 10, 20, 50, or 100 observe that that value is not in the per page drop down. 9. Close patron search window. 10. Apply patch. 11. Click the 'Add patron(s)' button again to open the patron search window. 12. Do a search. 13. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. 14. Go back to the Patrons page and search for patrons. 15. Observe that the per page drop down defaults to the value of the PatronsPerPage system preference. Signed-off-by: Sally Signed-off-by: Sally Signed-off-by: Sally -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #18 from Michael Hafen --- (In reply to Victor Grousset/tuxayo from comment #16) > The commits > Bug 28864: Have patron card creator use PatronsPerPage > Bug 28864: Have patron search use PatronsPerPage better. > are duplicate, which the is the right one? > > Thanks Michael for the correct test plan. Which patch name? With this, we > will be able to cleanup this. > > The name and test plan was changed on the 1st signoff via a sandbox. I don't > know how is that possible to change a commit message via a sandbox. The line numbers are the only difference. It seems that the commit "Bug 28864: Have patron search use PatronsPerPage better." is the more correct, looking at the line numbers (this is the only attachment that isn't marked obsolete); though I like the commit message of the other commit better, but if you can't change the commit message, then that's ok. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Victor Grousset/tuxayo changed: What|Removed |Added Attachment #138196|0 |1 is obsolete|| --- Comment #13 from Victor Grousset/tuxayo --- Comment on attachment 138196 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138196 Bug 31120: Items will renew for zero ( 0 ) days if renewalperiod is blank/empty value A patch for Bug 31120 was attached by mistake here. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Victor Grousset/tuxayo changed: What|Removed |Added CC||vic...@tuxayo.net Patch complexity|--- |Small patch --- Comment #16 from Victor Grousset/tuxayo --- The commits Bug 28864: Have patron card creator use PatronsPerPage Bug 28864: Have patron search use PatronsPerPage better. are duplicate, which the is the right one? Thanks Michael for the correct test plan. Which patch name? With this, we will be able to cleanup this. The name and test plan was changed on the 1st signoff via a sandbox. I don't know how is that possible to change a commit message via a sandbox. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #8 from ByWater Sandboxes --- Created attachment 138194 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138194=edit Bug 28864: Have patron search use PatronsPerPage better. Add pageLength to members/member.tt Add aLengthMenu and pageLength to patroncards/add_user_search (common/patron_search.tt). Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to the Patrons page and search for patrons. 3. Observe that the per page drop down defaults to 20. 4. Go to Tools -> Patron card creator. 5. Start a new Card Batch. 6. Click the 'Add patron(s)' button to open the patron search window. 7. Do a search. 8. Observe that the per page drop down defaults to 20. 8b. If the PatronsPerPage was set to a value other than 10, 20, 50, or 100 observe that that value is not in the per page drop down. 9. Close patron search window. 10. Apply patch. 11. Click the 'Add patron(s)' button again to open the patron search window. 12. Do a search. 13. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. 14. Go back to the Patrons page and search for patrons. 15. Observe that the per page drop down defaults to the value of the PatronsPerPage system preference. Signed-off-by: Sally -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Katrin Fischer changed: What|Removed |Added Status|Signed Off |RESOLVED Resolution|--- |WORKSFORME --- Comment #14 from Katrin Fischer --- I have tested without applying the patch and this works nicely on master. I believe this might be because of the standardizing of patron search that has taken place since this patch was written. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #17 from Victor Grousset/tuxayo --- Email sent to Sally to ask if their testing matches the new test plan. To know if the signoff is still valid. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #15 from Michael Hafen --- (In reply to Katrin Fischer from comment #14) > I have tested without applying the patch and this works nicely on master. I > believe this might be because of the standardizing of patron search that has > taken place since this patch was written. The test plan, since the standardizing of the patron search, is wrong. This is still an issue on the edit-batch page. The contents of the patch reflect this though I neglected to update the test plan. Here is the new test plan: Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to Tools -> Patron card creator. 3. Start a new Card Batch. 4. Click the 'Add patron(s)' button to open the patron search window. 5. Do a search. 6. Add more than 20 patrons to the batch. 7. Close patron search window. 8. Click the 'Add patron(s)' button to add the selected patrons to the batch. 9. Observe that the table of patrons is has 20 as the default per page setting. 10. Apply patch. 11. Reload the batch. 12. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 ByWater Sandboxes changed: What|Removed |Added Attachment #138194|0 |1 is obsolete|| --- Comment #9 from ByWater Sandboxes --- Created attachment 138195 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=138195=edit Bug 28864: Have patron search use PatronsPerPage better. Add pageLength to members/member.tt Add aLengthMenu and pageLength to patroncards/add_user_search (common/patron_search.tt). Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to the Patrons page and search for patrons. 3. Observe that the per page drop down defaults to 20. 4. Go to Tools -> Patron card creator. 5. Start a new Card Batch. 6. Click the 'Add patron(s)' button to open the patron search window. 7. Do a search. 8. Observe that the per page drop down defaults to 20. 8b. If the PatronsPerPage was set to a value other than 10, 20, 50, or 100 observe that that value is not in the per page drop down. 9. Close patron search window. 10. Apply patch. 11. Click the 'Add patron(s)' button again to open the patron search window. 12. Do a search. 13. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. 14. Go back to the Patrons page and search for patrons. 15. Observe that the per page drop down defaults to the value of the PatronsPerPage system preference. Signed-off-by: Sally Signed-off-by: Sally -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Sally changed: What|Removed |Added Status|REOPENED|Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Sally changed: What|Removed |Added Status|Needs Signoff |RESOLVED CC||sally.healey@cheshireshared ||services.gov.uk Resolution|--- |WORKSFORME --- Comment #6 from Sally --- I think this has since been fixed; I can't replicate the reported issue on the current branch or on 20.11. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Michael Hafen changed: What|Removed |Added Attachment #124787|0 |1 is obsolete|| --- Comment #5 from Michael Hafen --- Created attachment 134004 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=134004=edit Bug 28864: Have patron card creator use PatronsPerPage Rebase after update to how patron search works elsewhere. Add aLengthMenu, pageLength, and iDisplayLength to patroncards/edit-batch Test plan: 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to Tools -> Patron card creator. 3. Start a new Card Batch. 4. Click the 'Add patron(s)' button to open the patron search window. 5. Do a search. 6. Add more than 20 patrons to the batch and close the search window. 7. Observe that the per page drop down defaults to 20. 8. If the PatronsPerPage was set to a value other than 10, 20, 50, or 100 observe that that value is not in the per page drop down. 9. Apply patch. 10. Reload the Patron Card Batch 11. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #4 from Michael Hafen --- The test plan is in the patch, but it isn't complete. Here it is for you. 1. Set the PatronsPerPage system preference to any integer other than 20. 2. Go to the Patrons page and search for patrons. 3. Observe that the per page drop down defaults to 20. 4. Go to Tools -> Patron card creator. 5. Start a new Card Batch. 6. Click the 'Add patron(s)' button to open the patron search window. 7. Do a search with more then 20 results. 8. Observe that the per page drop down defaults to 20. 8b. If the PatronsPerPage was set to a value other than 10, 20, 50, or 100 observe that that value is not in the per page drop down. 9. Add patrons to the batch. 10. Close patron search window. 11. Click the 'Add patron(s)' button again to add the patrons to the batch. Observe that the per page defaults to 20. 12. Delete the card batch. 13. Apply patch. 14. Start a new card batch. Click the 'Add patron(s)' button again to open the patron search window. 15. Do a search. 16. Observe that the per page drop down includes the value of PatronsPerPage and that value is selected by default. 17. Add patrons to batch and close the search window. 18. Click the 'Add patron(s)' button again and observe that the per page drop down includes and defaults to the PatronsPerPage value. 19. Delete the card batch. 17. Go back to the Patrons page and search for patrons. 18. Observe that the per page drop down defaults to the value of the PatronsPerPage system preference. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #3 from Lucas Gass --- Micheal, I'm happy to test this if you could add a test plan. -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #2 from Michael Hafen --- Created attachment 124787 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124787=edit Proposed patch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Michael Hafen changed: What|Removed |Added Attachment #123848|0 |1 is obsolete|| -- You are receiving this mail because: You are the assignee for the bug. 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 Michael Hafen changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 28864] The patron search results in the Patron card creator doesn't seem to use PatronsPerPage syspref
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28864 --- Comment #1 from Michael Hafen --- Created attachment 123848 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=123848=edit Proposed patch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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/