Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-49613

count_role_users gives unexpected results

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.7.7, 2.8.5
    • 2.7.8, 2.8.6
    • Roles / Access
    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • Hide

      Run the lib/tests/accesslib_test.php unit tests. They should all pass.

      If they are run without the last commit in the attached branches the new unit test for count_role_users will fail.

      This should be tested against all databases.

      Show
      Run the lib/tests/accesslib_test.php unit tests. They should all pass. If they are run without the last commit in the attached branches the new unit test for count_role_users will fail. This should be tested against all databases.

    Description

      The count_role_users function in lib/accesslib.php gives the wrong results if a user has the same role at multiple levels in a context path.

      For example a user is a coursecreator on a course and the category that the course is in, in this case they would be counted twice, rather than being recognised as the same user.

      Attachments

        Issue Links

          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-22309 get_role_users() with more than one role assigned to user

          • Critical - Crashes server, loss of data, severe memory leak
          • Closed

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-43130 Users assigned context role via multiple components are counted multiple times

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              nmagill Neill Magill
              nmagill Neill Magill
              Andrew Lyons Andrew Lyons
              Adrian Greeve Adrian Greeve
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/May/15