Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.9.14
    • Fix Version/s: 1.9.15
    • Component/s: MNet
    • Labels:
    • Testing Instructions:
      Hide

      Note: The solution was already tested and confirmed to work at http://moodle.org/mod/forum/discuss.php?d=171512#p819293

      Needs one recent Moodle 2.1.x site and a patched 1.9 site networked together via MNet. The 2.1.x site acting as identity provider, the 1.9 as service provider.

      1. In 2.1.x, log in as a user who has not roamed to the second 1.9 site yet (eg create a fresh new user)
      2. Roam to the 1.9 site
      3. Log out and log in as admin at 1.9
      4. Filter all users with "confirmed" set to "no"
      5. TEST: make sure the new remote user is not listed here (hence is confirmed)

      (alternatively look into the "user" table in 1.9 and check the value of the 'confirmed' field)

      Show
      Note: The solution was already tested and confirmed to work at http://moodle.org/mod/forum/discuss.php?d=171512#p819293 Needs one recent Moodle 2.1.x site and a patched 1.9 site networked together via MNet. The 2.1.x site acting as identity provider, the 1.9 as service provider. 1. In 2.1.x, log in as a user who has not roamed to the second 1.9 site yet (eg create a fresh new user) 2. Roam to the 1.9 site 3. Log out and log in as admin at 1.9 4. Filter all users with "confirmed" set to "no" 5. TEST: make sure the new remote user is not listed here (hence is confirmed) (alternatively look into the "user" table in 1.9 and check the value of the 'confirmed' field)
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE
    • Pull from Repository:
    • Rank:
      16802

      Description

      There is a regression of MDL-27107. As we forgot to update 1.9 branch, Moodle 1.9 sites still expect the "confirmed" field incoming from the remote Moodle 2.x sites. We should apply the same logic onto MOODLE_19_STABLE so that the most recent 1.9.x works well with the recent 2.x sites.

        Issue Links

          Activity

          Hide
          David Mudrak added a comment -

          Submitting for integration.

          Show
          David Mudrak added a comment - Submitting for integration.
          Hide
          Sam Hemelryk added a comment -

          Thanks for cleaning that up David - it has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks for cleaning that up David - it has been integrated now
          Hide
          Rajesh Taneja added a comment -

          Works Great
          Thanks for fixing this David.

          Show
          Rajesh Taneja added a comment - Works Great Thanks for fixing this David.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And this has been sent upstream (already available @ git and cvs repos). Many, many thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - And this has been sent upstream (already available @ git and cvs repos). Many, many thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: