[Koha-bugs] [Bug 21549] Lock expired patron accounts after x days
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #40 from Fridolin Somers --- Enhancement not pushed to 20.11.x -- 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 21549] Lock expired patron accounts after x days
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Katrin Fischer changed: What|Removed |Added Keywords|release-notes-needed|Manual -- 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 21549] Lock expired patron accounts after x days
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Katrin Fischer changed: What|Removed |Added Summary|Lock expired patron |Lock expired patron |accounts|accounts after x days -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Katrin Fischer changed: What|Removed |Added Text to go in the|This report adds pref |This report adds pref release notes|LockExpiredDelay. When you |LockExpiredDelay. When you |enter a value there, the|enter a value there, the |cleanupdatabase cron job|cleanup_database.pl cron |will lock the accounts of |job will lock the accounts |patrons that expired at |of patrons that expired at |least the specified number |least the specified number |of days. |of days. |This follows the|This follows the |same pattern as existing|same pattern as existing |code that already allows|code that already allows |you to anonymize or delete |you to anonymize or delete |locked patrons. |locked patrons. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #39 from Jonathan Druart --- Pushed to master for 21.05, thanks to everybody involved! -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Jonathan Druart changed: What|Removed |Added Version(s)||21.05.00 released in|| Status|Passed QA |Pushed to master -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Text to go in the||This report adds pref release notes||LockExpiredDelay. When you ||enter a value there, the ||cleanupdatabase cron job ||will lock the accounts of ||patrons that expired at ||least the specified number ||of days. ||This follows the ||same pattern as existing ||code that already allows ||you to anonymize or delete ||locked patrons. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #38 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #34) > Do you agree with those 3 patches, Marcel? Sure, thanks. Found a solution in the middle ;) -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #119464|0 |1 is obsolete|| --- Comment #37 from Marcel de Rooy --- Created attachment 119527 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119527=edit Bug 21549: Make filter_by_last_update returns a Koha::Objects-based object Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #119463|0 |1 is obsolete|| --- Comment #36 from Marcel de Rooy --- Created attachment 119526 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119526=edit Bug 21549: Rename filter_by_dateexpiry with filter_by_expiration_date Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #119462|0 |1 is obsolete|| --- Comment #35 from Marcel de Rooy --- Created attachment 119525 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119525=edit Bug 21549: Use filter_by_last_update from filter_by_dateexpiry To not reengineer the patch this patch does not remove the new subroutine but make it use the logic from Koha::Objects->filter_by_last_update It also adds a new parameter "days_inclusive" Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Signed Off Keywords||release-notes-needed --- Comment #34 from Jonathan Druart --- Do you agree with those 3 patches, Marcel? -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #32 from Jonathan Druart --- Created attachment 119463 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119463=edit Bug 21549: Rename filter_by_dateexpiry with filter_by_expiration_date -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #31 from Jonathan Druart --- Created attachment 119462 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119462=edit Bug 21549: Use filter_by_last_update from filter_by_dateexpiry To not reengineer the patch this patch does not remove the new subroutine but make it use the logic from Koha::Objects->filter_by_last_update It also adds a new parameter "days_inclusive" -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #33 from Jonathan Druart --- Created attachment 119464 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119464=edit Bug 21549: Make filter_by_last_update returns a Koha::Objects-based object -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #30 from Jonathan Druart --- Created attachment 119461 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119461=edit Bug 21549: (QA follow-up) Get pref at start of run and rename variable This patch ensures the script will lock patrons if the pref is set but no other actions specified It also changes the variable name to be a bit more explicit Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #29 from Jonathan Druart --- Created attachment 119460 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119460=edit Bug 21549: (follow-up) Adjust preference description See QA remarks. Test plan: Check the preference in staff client. Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #28 from Jonathan Druart --- Created attachment 119459 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119459=edit Bug 21549: (follow-up) Switch to filter_by See QA remarks. Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #27 from Jonathan Druart --- Created attachment 119458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119458=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Jonathan Druart changed: What|Removed |Added Attachment #118517|0 |1 is obsolete|| Attachment #118518|0 |1 is obsolete|| Attachment #118519|0 |1 is obsolete|| Attachment #118520|0 |1 is obsolete|| Attachment #118521|0 |1 is obsolete|| --- Comment #26 from Jonathan Druart --- Created attachment 119457 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119457=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #25 from Jonathan Druart --- Well, I do consider it's a valid reason. Adding a method that is already inherited is not acceptable (duplication of code). If you are not willing to do it I will have a look and investigate when possible. Note that there is a difference between the 2 methods, < vs <=. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Status|Failed QA |Passed QA --- Comment #24 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #23) > It seems that we should use filter_by_last_update (inherited from > Koha::Objects) here. I do not consider this as a valid reason for failing this patch. Note that a filter on dateexpiry is not a filter on "last update". Perhaps this routine should be made more generic first? -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #23 from Jonathan Druart --- It seems that we should use filter_by_last_update (inherited from Koha::Objects) here. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Nick Clemens changed: What|Removed |Added Blocks||28000 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28000 [Bug 28000] Locking expired patrons via cronjob should have a switch to remove holds/article requests -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Nick Clemens changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |n...@bywatersolutions.com |y.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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #22 from Nick Clemens --- Created attachment 118521 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118521=edit Bug 21549: (QA follow-up) Get pref at start of run and rename variable This patch ensures the script will lock patrons if the pref is set but no other actions specified It also changes the variable name to be a bit more explicit Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #21 from Nick Clemens --- Created attachment 118520 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118520=edit Bug 21549: (follow-up) Adjust preference description See QA remarks. Test plan: Check the preference in staff client. Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #20 from Nick Clemens --- Created attachment 118519 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118519=edit Bug 21549: (follow-up) Switch to filter_by See QA remarks. Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #19 from Nick Clemens --- Created attachment 118518 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118518=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Nick Clemens changed: What|Removed |Added Attachment #118493|0 |1 is obsolete|| Attachment #118494|0 |1 is obsolete|| Attachment #118495|0 |1 is obsolete|| Attachment #118496|0 |1 is obsolete|| --- Comment #18 from Nick Clemens --- Created attachment 118517 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118517=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar Signed-off-by: Nick Clemens -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Nick Clemens 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #17 from Marcel de Rooy --- (In reply to Nick Clemens from comment #12) > -I think the method needs to be 'filter_by' -see bug 11983 > > -The pref description should include reliance on the cleanup database cron: > NOTE: This system preference requires the misc/cronjobs/cleanup_database.pl > cronjob. Ask your system administrator to schedule it. Thx for QAing, Nick. The two follow-ups should do the job. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #16 from Marcel de Rooy --- Created attachment 118496 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118496=edit Bug 21549: (follow-up) Adjust preference description See QA remarks. Test plan: Check the preference in staff client. Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #15 from Marcel de Rooy --- Created attachment 118495 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118495=edit Bug 21549: (follow-up) Switch to filter_by See QA remarks. Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #113915|0 |1 is obsolete|| --- Comment #14 from Marcel de Rooy --- Created attachment 118494 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118494=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #113914|0 |1 is obsolete|| --- Comment #13 from Marcel de Rooy --- Created attachment 118493 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118493=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Status|Failed QA |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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com Status|Signed Off |Failed QA --- Comment #12 from Nick Clemens --- -I think the method needs to be 'filter_by' -see bug 11983 -The pref description should include reliance on the cleanup database cron: NOTE: This system preference requires the misc/cronjobs/cleanup_database.pl cronjob. Ask your system administrator to schedule it. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #11 from Marcel de Rooy --- (In reply to Michal Denar from comment #10) > Created attachment 113915 [details] [review] > Bug 21549: Database revision for new preference > > Introducing LockExpiredDelay. > > Test plan: > Run upgrade. Check syspref. > > Signed-off-by: Marcel de Rooy > > Signed-off-by: Michal Denar Thanks Michal -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar changed: What|Removed |Added Attachment #111481|0 |1 is obsolete|| --- Comment #10 from Michal Denar --- Created attachment 113915 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113915=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar changed: What|Removed |Added Attachment #111480|0 |1 is obsolete|| --- Comment #9 from Michal Denar --- Created attachment 113914 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113914=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy Signed-off-by: Michal Denar -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #8 from Michal Denar --- Sorry, Patch wors for me now. My fault. Back to NSO. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #7 from Michal Denar --- Hi, plaease rebase on master. Thank you. -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Michal Denar changed: What|Removed |Added CC||blac...@gmail.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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 David Cook changed: What|Removed |Added CC||dc...@prosentient.com.au -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #6 from Marcel de Rooy --- Rebased. Please test ! -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #108000|0 |1 is obsolete|| --- Comment #4 from Marcel de Rooy --- Created attachment 111480 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111480=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Attachment #108001|0 |1 is obsolete|| --- Comment #5 from Marcel de Rooy --- Created attachment 111481 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=111481=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Status|Patch doesn't apply |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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Amit Gupta changed: What|Removed |Added CC||amit.gupta@informaticsgloba ||l.com, ||amitddng...@gmail.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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch Status|ASSIGNED|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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #3 from Marcel de Rooy --- Created attachment 108001 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108001=edit Bug 21549: Database revision for new preference Introducing LockExpiredDelay. Test plan: Run upgrade. Check syspref. Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #2 from Marcel de Rooy --- Created attachment 108000 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=108000=edit Bug 21549: Lock expired patron accounts Adding a search on locked patrons to the search_expired in cron script. This prevents relocking. Test plan: Prove Koha/Patrons.t Signed-off-by: Marcel de Rooy -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |m.de.r...@rijksmuseum.nl |ity.org | -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 Marcel de Rooy changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 21549] Lock expired patron accounts
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21549 --- Comment #1 from Marcel de Rooy --- Essentially, we need something like: Koha::Patrons->search_expired({ days => $days })->search({ login_attempts => { '!=' => -1 } })->lock({ remove => 1, verbose => $verbose }); -- 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 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/