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

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

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

      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

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            mudrd8mz David Mudrak added a comment -

            Submitting patch for integration.

            Show
            mudrd8mz David Mudrak added a comment - Submitting patch for integration.
            Hide
            stronk7 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
            stronk7 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
            samhemelryk Sam Hemelryk added a comment -

            Tested. Passed. Thanks

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

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

            Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - git & cvs repositories have been populated with this solution. Many thanks for your collaboration, yay! Closing, ciao
            Hide
            dmonllao David Monllaó added a comment -
            Show
            dmonllao 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:
                  Fix Release Date:
                  10/Oct/11