Details

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

      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.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            mudrd8mz David Mudrak added a comment -

            Submitting for integration.

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

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

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

            Works Great
            Thanks for fixing this David.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Works Great Thanks for fixing this David.
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  28/Nov/11