Moodle
  1. Moodle
  2. MDL-27107

Profile field "confirmed" set to 0 for incoming roaming users

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: MNet
    • Labels:
    • Testing Instructions:
      Hide

      Needs two Moodle 2.x sites networked via MNet, one (A) acting as identity provider, one (B) as service provider.

      1. Login as admin at the identity provider site (A)
      2. At the Peers settings page for the second site, make sure the field "confirmed" is not selected among "Fields to export" (you may need to uncheck "Use the default settings instead")
      3. Still at (A), log in as a user who has not roamed to the second site yet (eg create a fresh new user)
      4. Roam to the site (B)
      5. Log out and log in as admin at (B)
      6. Filter all users with "confirmed" set to "no"
      7. TEST: make sure the new remote user is not listed here (hence is confirmed)

      Show
      Needs two Moodle 2.x sites networked via MNet, one (A) acting as identity provider, one (B) as service provider. 1. Login as admin at the identity provider site (A) 2. At the Peers settings page for the second site, make sure the field "confirmed" is not selected among "Fields to export" (you may need to uncheck "Use the default settings instead") 3. Still at (A), log in as a user who has not roamed to the second site yet (eg create a fresh new user) 4. Roam to the site (B) 5. Log out and log in as admin at (B) 6. Filter all users with "confirmed" set to "no" 7. TEST: make sure the new remote user is not listed here (hence is confirmed)
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-27107-mnet-confirmed
    • Rank:
      16800

      Description

      It may happen that the profile field "confirmed" is set to 0 for incoming roaming users. It leads to removal of the records. See http://moodle.org/mod/forum/discuss.php?d=171512 for the full report. Due to this data-loss risk, triaged as Critical

        Issue Links

          Activity

          Hide
          David Mudrak added a comment -

          Submitting patch for integration.

          Show
          David Mudrak added a comment - Submitting patch for integration.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated thanks!

          Side note: It surprised me a bit to see the "auth" column in both lists, when it's set to "mnet". FYC, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated thanks! Side note: It surprised me a bit to see the "auth" column in both lists, when it's set to "mnet". FYC, ciao
          Hide
          Sam Hemelryk added a comment -

          Tested. Passed. Thanks

          Show
          Sam Hemelryk added a comment - Tested. Passed. Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          git & cvs repositories have been populated with this solution. Many thanks for your collaboration, yay!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - git & cvs repositories have been populated with this solution. Many thanks for your collaboration, yay! Closing, ciao
          Hide
          David Monllaó added a comment -
          Show
          David Monllaó added a comment - Related issue: http://tracker.moodle.org/browse/MDL-31823

            People

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

              Dates

              • Created:
                Updated:
                Resolved: