Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-36504 enrol improvements 2.5 META
  3. MDL-36898

Course Contact Information Incorrectly Includes "Other Users"

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      1/ run phpunit tests
      2/ enrol some user as teacher into course
      3/ assign teacher role in course to some other user, but do not enrol them
      4/ verify only the one user is visible as contact in course/info.php and enrolment page

      Note: please note you may need to tweak the teacher role to include moodle/course:view, otherwise the role will not appear in the other users UI

      Show
      1/ run phpunit tests 2/ enrol some user as teacher into course 3/ assign teacher role in course to some other user, but do not enrol them 4/ verify only the one user is visible as contact in course/info.php and enrolment page Note: please note you may need to tweak the teacher role to include moodle/course:view, otherwise the role will not appear in the other users UI
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w02_MDL-36898_m25_ccontact

      Description

      On the course listing, course enrollment page and the course information page, the course contact information is incorrectly display "Other Users".


      The patch revives the deprecated $gethidden parameter in get_role_users(), this should be 100% backwards compatible, but it disqualifies it from the integration into stable branches.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                skodak Petr Skoda
                Reporter:
                jrchamp Jonathan Champ
                Integrator:
                Dan Poltawski
                Tester:
                Rajesh Taneja
                Participants:
                Component watchers:
                Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Jake Dallimore, Jun Pataleta
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13