[Koha-bugs] [Bug 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Kyle M Hall changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=23151 -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Kyle M Hall changed: What|Removed |Added Assignee|oleon...@myacpl.org |k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Kyle M Hall changed: What|Removed |Added Status|NEW |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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 --- Comment #1 from Kyle M Hall --- Created attachment 105908 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105908&action=edit Bug 25769: Patron self modification triggers change request for date of birth to null If PatronSelfModificationBorrowerUnwantedField contains dateofbirth, all patron self modification requests will attempt to set dateofbirth to null instead of ignoring the field. Test Plan: 1) Add dateofbirth to PatronSelfModificationBorrowerUnwantedField 2) Run a patron self modification 3) Note the request sets dateofbirth to null on the staff side 4) Apply this patch 5) Restart all the things! 6) Run another self-modification 7) Note of the staff side that dateofbirth is unaffected -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Emmi Takkinen changed: What|Removed |Added Attachment #105908|0 |1 is obsolete|| CC||emmi.takki...@outlook.com --- Comment #2 from Emmi Takkinen --- Created attachment 105938 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105938&action=edit Bug 25769: Patron self modification triggers change request for date of birth to null If PatronSelfModificationBorrowerUnwantedField contains dateofbirth, all patron self modification requests will attempt to set dateofbirth to null instead of ignoring the field. Test Plan: 1) Add dateofbirth to PatronSelfModificationBorrowerUnwantedField 2) Run a patron self modification 3) Note the request sets dateofbirth to null on the staff side 4) Apply this patch 5) Restart all the things! 6) Run another self-modification 7) Note of the staff side that dateofbirth is unaffected Signed-off-by: Emmi Takkinen -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Emmi Takkinen 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Kyle M Hall changed: What|Removed |Added Severity|enhancement |major -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 --- Comment #3 from Martin Renvoize --- Comment on attachment 105938 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105938 Bug 25769: Patron self modification triggers change request for date of birth to null Review of attachment 105938: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=25769&attachment=105938) - ::: opac/opac-memberentry.pl @@ +484,5 @@ > > +if ( defined $borrower{'dateofbirth'} ) { > +my $dob_dt; > +$dob_dt = eval { dt_from_string( $borrower{'dateofbirth'} ); } > +if ( $borrower{'dateofbirth'} ); You're now testing if $borrower{'dateofbirth'} is defined above.. do you still need to test it here too? -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europe ||.com Status|Signed Off |Failed QA --- Comment #4 from Martin Renvoize --- Failing QA whilst I wait for a reply. -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Martin Renvoize changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 --- Comment #5 from Kyle M Hall --- (In reply to Martin Renvoize from comment #3) > Comment on attachment 105938 [details] [review] > Bug 25769: Patron self modification triggers change request for date of > birth to null > > Review of attachment 105938 [details] [review]: > - > > ::: opac/opac-memberentry.pl > @@ +484,5 @@ > > > > +if ( defined $borrower{'dateofbirth'} ) { > > +my $dob_dt; > > +$dob_dt = eval { dt_from_string( $borrower{'dateofbirth'} ); } > > +if ( $borrower{'dateofbirth'} ); > > You're now testing if $borrower{'dateofbirth'} is defined above.. do you > still need to test it here too? I suppose not! Followup will be forthcoming. -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Kyle M Hall changed: What|Removed |Added Status|Failed QA |Signed Off --- Comment #6 from Kyle M Hall --- (In reply to Martin Renvoize from comment #3) > Comment on attachment 105938 [details] [review] > Bug 25769: Patron self modification triggers change request for date of > birth to null > > Review of attachment 105938 [details] [review]: > - > > ::: opac/opac-memberentry.pl > @@ +484,5 @@ > > > > +if ( defined $borrower{'dateofbirth'} ) { > > +my $dob_dt; > > +$dob_dt = eval { dt_from_string( $borrower{'dateofbirth'} ); } > > +if ( $borrower{'dateofbirth'} ); > > You're now testing if $borrower{'dateofbirth'} is defined above.. do you > still need to test it here too? After examining the code, yes we do still need it. This new check tests to see if the dateofbirth param is part of the form. If it *is*, it still may be blank. Without this check, we'd set the dateofbirth to the current date. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 --- Comment #7 from Martin Renvoize --- (In reply to Kyle M Hall from comment #6) > (In reply to Martin Renvoize from comment #3) > > Comment on attachment 105938 [details] [review] [review] > > Bug 25769: Patron self modification triggers change request for date of > > birth to null > > > > Review of attachment 105938 [details] [review] [review]: > > - > > > > ::: opac/opac-memberentry.pl > > @@ +484,5 @@ > > > > > > +if ( defined $borrower{'dateofbirth'} ) { > > > +my $dob_dt; > > > +$dob_dt = eval { dt_from_string( $borrower{'dateofbirth'} ); } > > > +if ( $borrower{'dateofbirth'} ); > > > > You're now testing if $borrower{'dateofbirth'} is defined above.. do you > > still need to test it here too? > > After examining the code, yes we do still need it. This new check tests to > see if the dateofbirth param is part of the form. If it *is*, it still may > be blank. Without this check, we'd set the dateofbirth to the current date. Ah, of course.. good catch. -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Martin Renvoize changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #9 from Martin Renvoize --- Works as expected, passes qa scripts. Passing 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Martin Renvoize changed: What|Removed |Added Attachment #105938|0 |1 is obsolete|| --- Comment #8 from Martin Renvoize --- Created attachment 106146 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106146&action=edit Bug 25769: Patron self modification triggers change request for date of birth to null If PatronSelfModificationBorrowerUnwantedField contains dateofbirth, all patron self modification requests will attempt to set dateofbirth to null instead of ignoring the field. Test Plan: 1) Add dateofbirth to PatronSelfModificationBorrowerUnwantedField 2) Run a patron self modification 3) Note the request sets dateofbirth to null on the staff side 4) Apply this patch 5) Restart all the things! 6) Run another self-modification 7) Note of the staff side that dateofbirth is unaffected Signed-off-by: Emmi Takkinen Signed-off-by: Martin Renvoize -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||20.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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 --- Comment #10 from Jonathan Druart --- Pushed to master for 20.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Status|Pushed to master|Pushed to stable Version(s)|20.11.00|20.11.00, 20.05.02 released in|| --- Comment #11 from Lucas Gass --- backported to 20.05.x for 20.05.02 -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Aleisha Amohia changed: What|Removed |Added CC||alei...@catalyst.net.nz Version(s)|20.11.00, 20.05.02 |20.11.00, 20.05.02, released in||19.11.08 Status|Pushed to stable|Pushed to oldstable --- Comment #12 from Aleisha Amohia --- backported to 19.11.x for 19.11.08 -- 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 25769] Patron self modification triggers change request for date of birth to null
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25769 Victor Grousset/tuxayo changed: What|Removed |Added Version(s)|20.11.00, 20.05.02, |20.11.00, 20.05.02, released in|19.11.08|19.11.08, 19.05.13 CC||vic...@tuxayo.net Status|Pushed to oldstable |Pushed to oldoldstable --- Comment #13 from Victor Grousset/tuxayo --- Backported to 19.05.x branch for 19.05.13 -- 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/