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

      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.

        Gliffy Diagrams

          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: