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), Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

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