Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-43488

Self registration requires city and country fields

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6.2, 2.7
    • Component/s: Authentication
    • Labels:
    • Story Points (Obsolete):
      2
    • Sprint:
      BACKEND Sprint 8

      Description

      When a new user fills in the registration form for email-based self registration, the city and country fields are required.

      Replication steps:

      1. Navigate to Site admin > Plugins > Authentication > Manage authentication
      2. Ensure "Email-based self-registration" is enabled (default)
      3. Enable " Self registration" (registerauth)
      4. Open a new browser for the student
      5. Click the "Log in" link in top-right corner
      6. Click "Create a new account"

      Expected behaviour: The city and country fields should be optional.

      Actual result: The city and country fields are required.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              dobedobedoh Andrew Nicols added a comment -

              Looks good to me. Submitting for integration.

              Show
              dobedobedoh Andrew Nicols added a comment - Looks good to me. Submitting for integration.
              Hide
              poltawski Dan Poltawski added a comment -

              Sorry Petr - We can't remove strings from the stable branches. It will break for people using 2.6.0.

              http://docs.moodle.org/dev/Peer_reviewing_checklist#Language

              Show
              poltawski Dan Poltawski added a comment - Sorry Petr - We can't remove strings from the stable branches. It will break for people using 2.6.0. http://docs.moodle.org/dev/Peer_reviewing_checklist#Language
              Hide
              cibot CiBoT added a comment -

              Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

              Show
              cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
              Hide
              skodak Petr Skoda added a comment -

              fixed, resubmitting - now strings are dropped in master only

              Show
              skodak Petr Skoda added a comment - fixed, resubmitting - now strings are dropped in master only
              Hide
              damyon Damyon Wiese added a comment -

              Thanks Petr, looks good to me.

              Integrated to 26 and master.

              Show
              damyon Damyon Wiese added a comment - Thanks Petr, looks good to me. Integrated to 26 and master.
              Hide
              andyjdavis Andrew Davis added a comment -

              Works as described. Passing.

              Show
              andyjdavis Andrew Davis added a comment - Works as described. Passing.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Well done is better than well said.

              Closing, big thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Well done is better than well said. Closing, big thanks!

                People

                • Votes:
                  2 Vote for this issue
                  Watchers:
                  6 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Mar/14