Moodle
  1. Moodle
  2. MDL-15035

Wrong message displayed when not part of a group and trying to view participants with course set to separate groups.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.6, 1.7.4, 1.8.5, 1.9.1
    • Fix Version/s: 1.6.6, 1.7.4, 1.8.5, 1.9.1
    • Component/s: Course
    • Labels:
      None
    • Affected Branches:
      MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Rank:
      30871

      Description

      If you view participants from /user/index.php in a course that has separate groups set and forced, and you are not part of a group, you get the error message:

      "Sorry, but you need to be part of a group to see this forum."

      This appears to have been there since 1.6. The comments around it refer to a forum as well.

      This may also be associated with bug number 9847.

        Issue Links

          Activity

          Hide
          Ken Wilson added a comment -

          Sorry - this is not resolved as the same error msg appears. There are 2 notingroup language strings - one for forums and one in moodle.php. We need to use the one in moodle.php. Patch attached.

          Show
          Ken Wilson added a comment - Sorry - this is not resolved as the same error msg appears. There are 2 notingroup language strings - one for forums and one in moodle.php. We need to use the one in moodle.php. Patch attached.
          Hide
          Mike Churchward added a comment -

          Ken. The patch you attached is the same fix that was applied in all of the versions in CVS. Why did you re-open this bug?

          Show
          Mike Churchward added a comment - Ken. The patch you attached is the same fix that was applied in all of the versions in CVS. Why did you re-open this bug?
          Hide
          Ken Wilson added a comment -

          Sorry Mike - you're correct that it's already committed to CVS. The reason it was reopened as that I couldn't find it in the code, but have now. Closing. Thanks.

          Show
          Ken Wilson added a comment - Sorry Mike - you're correct that it's already committed to CVS. The reason it was reopened as that I couldn't find it in the code, but have now. Closing. Thanks.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: