[Koha-bugs] [Bug 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Caroline Cyr La Rose changed: What|Removed |Added CC||caroline.cyr-la-rose@inlibr ||o.com Documentation||https://gitlab.com/koha-com submission||munity/koha-manual/-/merge_ ||requests/1006 Resolution|--- |FIXED Status|Needs documenting |RESOLVED Documentation||Caroline Cyr La Rose contact|| -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass (lukeg) changed: What|Removed |Added Status|Pushed to main |Needs documenting --- Comment #90 from Lucas Gass (lukeg) --- Enhancement, no 24.05 backport. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Katrin Fischer changed: What|Removed |Added Keywords|release-notes-needed| -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Katrin Fischer changed: What|Removed |Added Text to go in the||This adds a new option to release notes||the patron category ||administration pages, that ||allows to enforce a ||password reset on first ||login for any patron ||accounts created manually ||in the staff interface. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #89 from Jonathan Druart --- DB revs: say_success statement must be inside the if ! column_exists block. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Tomás Cohen Arazi (tcohen) changed: What|Removed |Added CC||tomasco...@gmail.com --- Comment #85 from Tomás Cohen Arazi (tcohen) --- This patchset added a column to the categories table, but not to the spec. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #88 from Tomás Cohen Arazi (tcohen) --- Created attachment 173865 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173865&action=edit Bug 33462: Add new system preference to sysprefs.sql Signed-off-by: Tomas Cohen Arazi -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #87 from Tomás Cohen Arazi (tcohen) --- Follow-up pushed to main. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #86 from Tomás Cohen Arazi (tcohen) --- Created attachment 173864 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=173864&action=edit Bug 33462: (QA follow-up) Adapt API spec Signed-off-by: Tomas Cohen Arazi -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Katrin Fischer changed: What|Removed |Added Status|Passed QA |Pushed to main Version(s)||24.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #83 from Katrin Fischer --- 1) Database update (fixed in follow-up) We have a mismatch in the sequence of columns: ALTER TABLE categories ADD COLUMN `force_password_reset_when_set_by_staff` TINYINT(1) NULL DEFAULT NULL AFTER `require_strong_password But kohastructure.sql: `noissueschargeguarantorswithguarantees` int(11) DEFAULT NULL COMMENT 'define maximum amount that the guarantors with guarantees of a patron in this category can have outstanding before checkouts are blocked', + `force_password_reset_when_set_by_staff` tinyint(1) DEFAULT NULL COMMENT 'if patrons of this category are required to reset password after being created by a staff member', -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #84 from Katrin Fischer --- Pushed for 24.11! Well done everyone, 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Katrin Fischer changed: What|Removed |Added Keywords|rel_24_11_candidate |release-notes-needed -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #82 from Katrin Fischer --- Skipped the last patch Bug 33462: (QA follow-up) Tidy code because it gave me an sha1 error. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Katrin Fischer changed: What|Removed |Added Keywords|Sandbox |rel_24_11_candidate -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Kyle M Hall (khall) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171640|0 |1 is obsolete|| --- Comment #73 from Olivier Vezina --- Created attachment 172036 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172036&action=edit Bug 33462: Adjust tests Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #78 from Kyle M Hall (khall) --- Created attachment 172068 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172068&action=edit Bug 33462: (Follow up) Fix Perl Tidy Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #81 from Kyle M Hall (khall) --- Created attachment 172071 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172071&action=edit Bug 33462: (QA follow-up) Tidy code Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #80 from Kyle M Hall (khall) --- Created attachment 172070 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172070&action=edit Bug 33462: Adjust tests Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #79 from Kyle M Hall (khall) --- Created attachment 172069 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172069&action=edit Bug 33462: Update Schema Files (Don't push) Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #77 from Kyle M Hall (khall) --- Created attachment 172067 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172067&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Kyle M Hall (khall) changed: What|Removed |Added Attachment #172030|0 |1 is obsolete|| Attachment #172031|0 |1 is obsolete|| Attachment #172032|0 |1 is obsolete|| Attachment #172033|0 |1 is obsolete|| Attachment #172034|0 |1 is obsolete|| Attachment #172035|0 |1 is obsolete|| Attachment #172036|0 |1 is obsolete|| --- Comment #74 from Kyle M Hall (khall) --- Created attachment 172064 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172064&action=edit Bug 33462: DB Updates Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #76 from Kyle M Hall (khall) --- Created attachment 172066 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172066&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-off :) Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #75 from Kyle M Hall (khall) --- Created attachment 172065 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172065&action=edit Bug 33462: Manual Schema Changes (define as boolean) Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V Signed-off-by: Kyle M Hall -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171634|0 |1 is obsolete|| --- Comment #67 from Olivier Vezina --- Created attachment 172030 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172030&action=edit Bug 33462: DB Updates Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171638|0 |1 is obsolete|| --- Comment #71 from Olivier Vezina --- Created attachment 172034 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172034&action=edit Bug 33462: (Follow up) Fix Perl Tidy Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Kyle M Hall (khall) changed: What|Removed |Added CC||k...@bywatersolutions.com QA Contact|testo...@bugs.koha-communit |k...@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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||ovez...@inlibro.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171639|0 |1 is obsolete|| --- Comment #72 from Olivier Vezina --- Created attachment 172035 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172035&action=edit Bug 33462: Update Schema Files (Don't push) Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171637|0 |1 is obsolete|| --- Comment #70 from Olivier Vezina --- Created attachment 172033 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172033&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171636|0 |1 is obsolete|| --- Comment #69 from Olivier Vezina --- Created attachment 172032 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172032&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-off :) Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Olivier Vezina changed: What|Removed |Added Attachment #171635|0 |1 is obsolete|| --- Comment #68 from Olivier Vezina --- Created attachment 172031 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=172031&action=edit Bug 33462: Manual Schema Changes (define as boolean) Signed-off-by: Laura_Escamilla Signed-off-by: Olivier V -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #66 from Lucas Gass --- rebased for current main. resetting to "Needs signoff" as there were a lot of merge conflicts and I am not the original author. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #65 from Lucas Gass --- Created attachment 171640 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171640&action=edit Bug 33462: Adjust tests -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168453|0 |1 is obsolete|| --- Comment #64 from Lucas Gass --- Created attachment 171639 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171639&action=edit Bug 33462: Update Schema Files (Don't push) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168457|0 |1 is obsolete|| --- Comment #63 from Lucas Gass --- Created attachment 171638 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171638&action=edit Bug 33462: (Follow up) Fix Perl Tidy 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168456|0 |1 is obsolete|| --- Comment #62 from Lucas Gass --- Created attachment 171637 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171637&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168455|0 |1 is obsolete|| --- Comment #61 from Lucas Gass --- Created attachment 171636 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171636&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-off :) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168454|0 |1 is obsolete|| --- Comment #60 from Lucas Gass --- Created attachment 171635 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171635&action=edit Bug 33462: Manual Schema Changes (define as boolean) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Attachment #168452|0 |1 is obsolete|| --- Comment #59 from Lucas Gass --- Created attachment 171634 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=171634&action=edit Bug 33462: DB Updates 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas Gass changed: What|Removed |Added Status|Signed Off |Patch doesn't apply CC||lu...@bywatersolutions.com --- Comment #58 from Lucas Gass --- patch no longer applies -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Liz Rea changed: What|Removed |Added CC||wizzy...@gmail.com --- Comment #57 from Liz Rea --- Hi, Wondering if we can get a QA here? :D Liz -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #56 from David Cook --- *** Bug 28980 has been marked as a duplicate of this bug. *** -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 David Nind changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=21309 -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #55 from Sam Lau --- (In reply to Jonathan Druart from comment #54) > Ok, thanks! Sorry for my laziness! > I usually expect from people to reply with a comment. Even if it's only "all > done!", at least I know it has been taken into account :) > No problem, sorry for the misunderstanding. I'll definitely start doing that in the future. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #54 from Jonathan Druart --- Ok, thanks! Sorry for my laziness! I usually expect from people to reply with a comment. Even if it's only "all done!", at least I know it has been taken into account :) Thanks again for your work! -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #53 from Sam Lau --- (In reply to Jonathan Druart from comment #52) > What about those 2 points? It seems that the new DB column is still part of > the patch set. Originally, there were two nesw DB columns, one for needs_password_reset and one to add it to the patron categories. I got rid of the first one per your request. The one that remains allows it to be a part patron categories if that's what you are referring to. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #52 from Jonathan Druart --- (In reply to Jonathan Druart from comment #18) > 4. Could this be achieve without adding a new column (needs_password_reset) > but instead compare dateenrolled and password_expiration_date? > If there are the same, then we display "It's your first login", otherwise > "Your password has expired". Am I missing something? > > 5. Not sure how this can behave properly with third-party using the REST API > to create patrons (for discussion only). What about those 2 points? It seems that the new DB column is still part of the patch set. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #51 from Sam Lau --- (In reply to Jonathan Druart from comment #50) > Did you address comment 18? Yes... did I miss something? -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #50 from Jonathan Druart --- Did you address comment 18? -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168418|0 |1 is obsolete|| --- Comment #49 from Laura Escamilla --- Created attachment 168457 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168457&action=edit Bug 33462: (Follow up) Fix Perl Tidy 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168417|0 |1 is obsolete|| --- Comment #48 from Laura Escamilla --- Created attachment 168456 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168456&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168416|0 |1 is obsolete|| --- Comment #47 from Laura Escamilla --- Created attachment 168455 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168455&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-off :) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168415|0 |1 is obsolete|| --- Comment #46 from Laura Escamilla --- Created attachment 168454 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168454&action=edit Bug 33462: Manual Schema Changes (define as boolean) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168414|0 |1 is obsolete|| --- Comment #45 from Laura Escamilla --- Created attachment 168453 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168453&action=edit Bug 33462: Update Schema Files (Don't push) 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added Attachment #168413|0 |1 is obsolete|| --- Comment #44 from Laura Escamilla --- Created attachment 168452 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168452&action=edit Bug 33462: DB Updates 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167986|0 |1 is obsolete|| --- Comment #43 from Sam Lau --- Created attachment 168418 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168418&action=edit Bug 33462: (Follow up) Fix Perl Tidy -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167985|0 |1 is obsolete|| --- Comment #42 from Sam Lau --- Created attachment 168417 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168417&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167984|0 |1 is obsolete|| --- Comment #41 from Sam Lau --- Created attachment 168416 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168416&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167983|0 |1 is obsolete|| --- Comment #40 from Sam Lau --- Created attachment 168415 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168415&action=edit Bug 33462: Manual Schema Changes (define as boolean) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167982|0 |1 is obsolete|| --- Comment #39 from Sam Lau --- Created attachment 168414 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168414&action=edit Bug 33462: Update Schema Files (Don't push) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167981|0 |1 is obsolete|| --- Comment #38 from Sam Lau --- Created attachment 168413 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168413&action=edit Bug 33462: DB Updates -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 David Nind changed: What|Removed |Added CC||da...@davidnind.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Laura Escamilla changed: What|Removed |Added CC||Laura.escamilla@bywatersolu ||tions.com Status|Needs Signoff |Patch doesn't apply --- Comment #37 from Laura Escamilla --- (In reply to Sam Lau from comment #36) > Should be good to go now, thanks for letting me know Lucas. Hi Sam, I'm seeing the following when I attempt to apply the patch: error: sha1 information is lacking or useless (admin/categories.pl). error: could not build fake ancestor Patch failed at 0001 Bug 33462: (Follow up) Fix Perl Tidy hint: Use 'git am --show-current-patch=diff' to see the failed patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-33462-Follow-up-Fix-Perl-Tidy-r7k5ya0s.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #36 from Sam Lau --- Should be good to go now, thanks for letting me know Lucas. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #35 from Sam Lau --- Created attachment 167986 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167986&action=edit Bug 33462: (Follow up) Fix Perl Tidy -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #34 from Sam Lau --- Created attachment 167985 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167985&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #33 from Sam Lau --- Created attachment 167984 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167984&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #32 from Sam Lau --- Created attachment 167983 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167983&action=edit Bug 33462: Manual Schema Changes (define as boolean) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #31 from Sam Lau --- Created attachment 167982 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167982&action=edit Bug 33462: Update Schema Files (Don't push) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #167582|0 |1 is obsolete|| Attachment #167583|0 |1 is obsolete|| Attachment #167584|0 |1 is obsolete|| Attachment #167585|0 |1 is obsolete|| Attachment #167586|0 |1 is obsolete|| Attachment #167587|0 |1 is obsolete|| --- Comment #30 from Sam Lau --- Created attachment 167981 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167981&action=edit Bug 33462: DB Updates -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Lucas changed: What|Removed |Added Keywords||Sandbox CC||lchirl...@ville-roubaix.fr Status|Needs Signoff |Patch doesn't apply --- Comment #29 from Lucas --- CONFLICT (content): Merge conflict in Koha/Schema/Result/Category.pm -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #28 from Sam Lau --- Created attachment 167587 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167587&action=edit Bug 33462: (Follow up) Fix Perl Tidy -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #154395|0 |1 is obsolete|| --- Comment #27 from Sam Lau --- Created attachment 167586 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167586&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #154397|0 |1 is obsolete|| --- Comment #26 from Sam Lau --- Created attachment 167585 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167585&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff' and a new column to the categories table 'force_password_reset_when_set_by_staff. To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #25 from Sam Lau --- Created attachment 167584 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167584&action=edit Bug 33462: Manual Schema Changes -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Attachment #154396|0 |1 is obsolete|| --- Comment #24 from Sam Lau --- Created attachment 167583 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167583&action=edit Bug 33462: Update Schema Files [DO NOT PUSH] -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #23 from Sam Lau --- Created attachment 167582 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=167582&action=edit Bug 33462: DB Updates -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #22 from Sam Lau --- Hi Christopher, Sorry for the delayed response. I was interning for ByWater last summer when I created this patch and haven't had a chance to look at it since then. However, I am back now and will try to fix things up ASAP. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Austin Shaver changed: What|Removed |Added CC||austins@ranchomiragelibrary ||.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Amanda Campbell changed: What|Removed |Added CC||acampb...@hmcpl.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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Barbara Johnson changed: What|Removed |Added CC||barbara.johnson@bedfordtx.g ||ov -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #21 from Christopher Brannon --- We seem to have lost Sam on this conversation. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #20 from Christopher Brannon --- Sam Lau, do you have any comments? -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #19 from Christopher Brannon --- (In reply to Jonathan Druart from comment #18) > 4. Could this be achieve without adding a new column (needs_password_reset) > but instead compare dateenrolled and password_expiration_date? > If there are the same, then we display "It's your first login", otherwise > "Your password has expired". Am I missing something? Not being the coder, as long as it can achieve the same result, and only applied when the account is staff entered, and not entered through online registration, I would say anything to reduce extras would be positive. But let's see what Sam has to say on this. -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Failed QA CC||jonathan.druart+koha@gmail. ||com --- Comment #18 from Jonathan Druart --- 1. Please run the QA script and fix the warning (tidy) 2. The manual changes (is_boolean) added to the schema file must be in a separate commit, or it will get lost when pushed 3. indentation is 4 spaces, not 2 (see changes in categories.tt) 4. Could this be achieve without adding a new column (needs_password_reset) but instead compare dateenrolled and password_expiration_date? If there are the same, then we display "It's your first login", otherwise "Your password has expired". Am I missing something? 5. Not sure how this can behave properly with third-party using the REST API to create patrons (for discussion only). -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Christopher Brannon changed: What|Removed |Added Status|Needs Signoff |Signed Off --- Comment #17 from Christopher Brannon --- Works as expected and is fantastic! Can't wait to have it in production -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Biblibre Sandboxes changed: What|Removed |Added Attachment #153689|0 |1 is obsolete|| --- Comment #16 from Biblibre Sandboxes --- Created attachment 154397 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154397&action=edit Bug 33462: Add ability to force password change for new patrons entered by staff This patch attempts to force a password change for new staff created patrons. This is done by setting the password_expiration_date to an expired date when adding a new patron. This patch adds a new system preference 'ForcePasswordResetWhenSetByStaff', a new column to the categories table 'force_password_reset_when_set_by_staff', and new columns to the borrowers, borrower_modification, and deletedborrowers table 'needs_password_reset' To test: 1) Apply patch, restart_all, updatedatabase, and also be sure to update schema. 2) Visit Administration->Sytem Preferences and search for 'EnableExpiredPasswordReset'. Make sure this is set to enable. Now search for 'ForcePasswordResetWhenSetByStaff'. This should be defaulted to 'Don't force'. 3) Keep that tab open and visit Administration->Patron categories. Click on edit on the Board category. Noitce that there is a now a 'Force new patron password reset' section. Notice that the by default, this is set to follow the ForcePasswordResetWhenSetByStaff system preference (currently set to don't force). Click on the dropdown and change it to 'Force'. Save changes 4) Click on the Patrons tab to visit members-home.pl and then click 'New Patron'. Select on Patron. Fill in the required information and also enter a password. 5) Submit this form and notice that the patron's password expiration date is set to never. This should be the case because the default for 'Force new patron password reset' follows the sys. pref. which is still set to 'Don't force' (You could have some expiry date in this step, but it should at least be set to a date that is not expired. this depends on whether or not you have a defalut password expiration date set in patron categories ) 6) Log into the OPAC with this patron and notice it works as expected and log in was successful. 7) Go back to the patron home page and click to add a new patron. This time select 'Board'. Once again fill out the required info, enter a password, and then save the form. 8) Notice that for this patron, the password expiration date is set for today's date. This is because we changed the setting for the 'Board' patron category to force. 9) Log into the OPAC with this patron. You should be redirected to a page with an error that says: "It's your first login! You need to reset your password." Click on the reset password link below this message. 10) You should be sent to a page where you can reset your password. Fill in the form and click 'Update password'. Attempt to sign into the OPAC with this new password. Everything works as expected. 11) Go back to the staff interface and view this patron's detail page. Notice the password expiration date is now set to what the default is in the patron category. 12) Edit this patrons information and set their password expiration date to yesterday. Go back to the OPAC and try to sign in with this patron again. Note that this time, you are also redirected but the message says "Error: Your password has expired!" 13) Go back to the staff interface and visit the sys. pref tab we left open. Set it to the 'Force' option and save changes. 14) Visit the patron home page and click add patron, now select the patron category again. Fill in required info and enter password. Submit form and note that the patron's password expiration date is set to today. Try to login to the OPAC with this patron, you should be redirected to the page with the error that says "Error: It's your first login! You need to reset your password." 15) Sign-off :) Signed-off-by: Christopher Brannon -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Biblibre Sandboxes changed: What|Removed |Added Attachment #153688|0 |1 is obsolete|| --- Comment #15 from Biblibre Sandboxes --- Created attachment 154396 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154396&action=edit Bug 33462: Update Schema Files [DO NOT PUSH] This updates the schema files for Borrower.pm, BorrowerModification.pm, Category.pm, and Deletedborrower.pm. Signed-off-by: Christopher Brannon -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Biblibre Sandboxes changed: What|Removed |Added Attachment #153687|0 |1 is obsolete|| --- Comment #14 from Biblibre Sandboxes --- Created attachment 154395 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=154395&action=edit Bug 33462: Unit Tests prove t/db_dependent/Koha/Patron/Category.t prove t/db_dependent/Koha/Patron.t Signed-off-by: Christopher Brannon -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #13 from Christopher Brannon --- Yes, I did use ByWater's sandbox. I'll take a look again. 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #12 from Sam Lau --- Everything looks like it is working good on PTFS Europe sandbox so maybe try that one out! :) -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 Sam Lau changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #11 from Sam Lau --- (In reply to Sam Lau from comment #10) > Hi Christopher, thanks for trying to test! Were you using ByWater's sandbox? I tried applying patch on the testing docker and everything was still applying cleanly. I then tried going to the sandbox (I used ByWater) and after applying it and I did end up getting the same message as you in the provision logs. I'm not sure what the issue is (not very familiar with the sandbox) but I asked Kyle Hall for help and we are both hoping the issue is fixed now. I would say give it another shot (I think Kyle created a sandbox for this bug) and if it still doesn't work let me know and then maybe try another sandbox? -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462 --- Comment #10 from Sam Lau --- Hi Christopher, thanks for trying to test! Were you using ByWater's sandbox? -- 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 33462] Force password change for new patrons entered by staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33462
Christopher Brannon changed:
What|Removed |Added
Status|Needs Signoff |Failed QA
--- Comment #9 from Christopher Brannon ---
I'm getting a merge failure in the sandbox on this bug.
TASK [Apply bug 33462 via git-bz in docker container]
**
fatal: [localhost -> koha-bug33462]: FAILED! => {"changed": true, "cmd":
"koha-shell -c 'cd /kohadevbox/koha && yes | git bz apply 33462' bug33462",
"delta": "0:00:03.143844", "end": "2023-08-04 15:49:21.744724", "msg":
"non-zero return code", "rc": 1, "start": "2023-08-04 15:49:18.600880",
"stderr": "error: Failed to merge in the changes.\nhint: Use 'git am
--show-current-patch=diff' to see the failed patch\nPatch left in
/tmp/Bug-33462-Unit-Tests-4t46gml6.patch", "stderr_lines": ["error: Failed to
merge in the changes.", "hint: Use 'git am --show-current-patch=diff' to see
the failed patch", "Patch left in /tmp/Bug-33462-Unit-Tests-4t46gml6.patch"],
"stdout": "\nBug 33462 - Force password change for new patrons entered by
staff\n\n153687 - Bug 33462: Unit Tests\n153688 - Bug 33462: Update Schema
Files [DO NOT PUSH]\n153689 - Bug 33462: Add ability to force password change
for new patrons entered by staff\n\nApply? [(y)es, (n)o, (i)nteractive]
Applying: Bug 33462: Unit Tests\nUsing index info to reconstruct a base
tree...\nM\tt/db_dependent/Koha/Patron.t\nFalling back to patching base and
3-way merge...\nAuto-merging t/db_dependent/Koha/Patron.t\nCONFLICT (content):
Merge conflict in t/db_dependent/Koha/Patron.t\nPatch failed at 0001 Bug 33462:
Unit Tests\nWhen you have resolved this problem run \"git bz apply
--continue\".\nIf you would prefer to skip this patch, instead run \"git bz
apply --skip\".\nTo restore the original branch and stop patching run \"git bz
apply --abort\".", "stdout_lines": ["", "Bug 33462 - Force password change for
new patrons entered by staff", "", "153687 - Bug 33462: Unit Tests", "153688 -
Bug 33462: Update Schema Files [DO NOT PUSH]", "153689 - Bug 33462: Add ability
to force password change for new patrons entered by staff", "", "Apply? [(y)es,
(n)o, (i)nteractive] Applying: Bug 33462: Unit Tests", "Using index info to
reconstruct a base tree...", "M\tt/db_dependent/Koha/Patron.t", "Falling back
to patching base and 3-way merge...", "Auto-merging
t/db_dependent/Koha/Patron.t", "CONFLICT (content): Merge conflict in
t/db_dependent/Koha/Patron.t", "Patch failed at 0001 Bug 33462: Unit Tests",
"When you have resolved this problem run \"git bz apply --continue\".", "If you
would prefer to skip this patch, instead run \"git bz apply --skip\".", "To
restore the original branch and stop patching run \"git bz apply --abort\"."]}
Sam, can you please check.
--
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/
