https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20859

David Nind <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Text to go in the|                            |This enhancement adds SMS
      release notes|                            |number and SMS provider
                   |                            |fields to the OPAC
                   |                            |registration page, allowing
                   |                            |patrons to provide these
                   |                            |when self-registering.
                   |                            |
                   |                            |Whether they appear on the
                   |                            |registration forms is
                   |                            |determined by a combination
                   |                            |of:
                   |                            |-
                   |                            |PatronSelfRegistrationBorro
                   |                            |werMandatoryField system
                   |                            |preference
                   |                            |-
                   |                            |PatronSelfRegistrationBorro
                   |                            |werUnwantedField
                   |                            |- SMSSendDriver system
                   |                            |preference
                   |                            |- SMS providers set up at
                   |                            |Administration > Additional
                   |                            |parameters > SMS cellular
                   |                            |providers
                   |                            |
                   |                            |Note that:
                   |                            |- if SMSSendDriver is not
                   |                            |set (even if SMS providers
                   |                            |are set up), the fields are
                   |                            |not shown.
                   |                            |- if SMSSendDriver and SMS
                   |                            |providers are set up, then
                   |                            |the fields are shown and
                   |                            |are mandatory
                 CC|                            |[email protected]

--- Comment #25 from David Nind <[email protected]> ---
Note that if I add the SMS fields to both
PatronSelfRegistrationBorrowerMandatoryField and
PatronSelfRegistrationBorrowerUnwantedField, the SMS fields show up on the
self-registration form.

Even if I complete all the required fields I get:

"You have not filled out all required fields. Please fill in all missing fields
and resubmit."

I couldn't work out how to make the fields optional - that is, have them there,
but not make it mandatory to add information to them.

Testing notes (using KTD):
1. Step 2: Add some dummy SMS providers at Administration > Additional
parameters > SMS cellular providers
2. Tested making fields mandatory using
PatronSelfRegistrationBorrowerMandatoryField system preference - should be
marked as required, and you can't submit the self-registration form unless
completed:
- SMS provider ID (internal) (sms_provider_id)
- Mobile phone number (smsalertnumber)
3. Test hiding the fields using PatronSelfRegistrationBorrowerUnwantedField
system preference - should not be shown on the self-registration form and you
can successfully submit.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to