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

          Attachments

            Issue Links

              Activity

              Hide
              mudrd8mz David Mudrák added a comment -

              Submitting for integration.

              Show
              mudrd8mz David Mudrák 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