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

auth_email_signup_user is not correctly setting the policyagreed field

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Go to Site administration > Security > Site policies
      2. Set the "Site policy" to a URL
      3. Enable sign-up in your site via the e-mail auth plugin
      4. Create a new user via this WS:

        curl 'http://localhost/m/stable_master/lib/ajax/service.php' --data-binary '[{"index":0,"methodname":"auth_email_signup_user","args":{"username": "u2abcde", "password": "moodle", "email": "myrealemail@aaagmail.com", "firstname": "asb", "lastname": "asdfasdf", "city": "Barcelona", "country": "ES", "redirect": "http://localhost/m/stable_master/admin/tool/mobile/launch.php?service=moodle_mobile_app&passport=xyz&confirmed=1"}}]' | python -m "json.tool"

      5. Check in the database that the user was created and the policyagreed field is set to 1
      Show
      Go to Site administration > Security > Site policies Set the "Site policy" to a URL Enable sign-up in your site via the e-mail auth plugin Create a new user via this WS: curl 'http://localhost/m/stable_master/lib/ajax/service.php' --data-binary ' [{"index":0,"methodname":"auth_email_signup_user","args":{"username": "u2abcde", "password": "moodle", "email": "myrealemail@aaagmail.com", "firstname": "asb", "lastname": "asdfasdf", "city": "Barcelona", "country": "ES", "redirect": "http://localhost/m/stable_master/admin/tool/mobile/launch.php?service=moodle_mobile_app&passport=xyz&confirmed=1"}}] ' | python -m "json.tool" Check in the database that the user was created and the policyagreed field is set to 1
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-57361-master

      Description

      It happens that we are not requiring it as a parameter of the WS (to be consistent with what decided in other issues where there were user interface fields blocking the form), but in this case, the form field was matching the policyagreed field in the user table.

      So this is going to require a fix for 3.2 and possibly a new parameter for master (to be discussed).

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/Mar/17