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

          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

                  Agile