[Bug 30664] New: PatronSelfRegistrationVerifyByEmail should use AutoEmailPrimaryAddress
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 Bug ID: 30664 Summary: PatronSelfRegistrationVerifyByEmail should use AutoEmailPrimaryAddress Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: dcook@prosentient.com.au QA Contact: testopia@bugs.koha-community.org At the moment, PatronSelfRegistrationVerifyByEmail forces you to use the "email" field for email verification regardless of the value of AutoEmailPrimaryAddress. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 --- Comment #1 from David Cook <dcook@prosentient.com.au> --- Admittedly this is challenging. If "AutoEmailPrimaryAddress" is set to "OFF", we don't know what field to make mandatory, since "OFF" uses "first_valid_email_address". If we knew the order of fields used in "first_valid_email_address" (using code logic and not hard-coding assumptions), I suppose we could make mandatory the "first" field checked in "first_valid_email_address"... I suppose we could also create a new system preference like "PatronSelfRegistrationVerifyEmailField"... that would require the fewest changes... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|PatronSelfRegistrationVerif |PatronSelfRegistrationVerif |yByEmail should use |yByEmail shouldn't make |AutoEmailPrimaryAddress |"email" field mandatory -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |dcook@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|dcook@prosentient.com.au |oleonard@myacpl.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|PatronSelfRegistrationVerif |PatronSelfRegistration |yByEmail shouldn't make |fields shouldn't make |"email" field mandatory |"email" field mandatory -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=30665 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 Andrew Fuerste-Henry <andrew@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com --- Comment #2 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Perhaps we make whichever field is indicated by the syspref mandatory? And if the syspref is off and we're using first_valid_email_address, then we make the form require a value in at least one of the email fields? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30664 --- Comment #3 from David Cook <dcook@prosentient.com.au> --- (In reply to Andrew Fuerste-Henry from comment #2)
Perhaps we make whichever field is indicated by the syspref mandatory? And if the syspref is off and we're using first_valid_email_address, then we make the form require a value in at least one of the email fields?
Agreed although that'll also take the most work heh. I don't think I have a work mandate to do that work, unfortunately. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org