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

            Assignee:
            jleyva Juan Leyva
            Reporter:
            jleyva Juan Leyva
            Peer reviewer:
            Dani Palou
            Integrator:
            Dan Poltawski
            Tester:
            Jake Dallimore
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

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