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

Bugs in several modules when accessing groups in recent activity report

    XMLWordPrintable

Details

    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • wip-MDL-40843-master
    • Hide

      run behat tests from MDL-44955 (some of them unfortunately fail with seg fault close to the end)

      The only module that I did not cover with behat is chat:

      1. Create chat activity in separate group mode, create groups, enrol several students and assign them to the groups.
      2. Open two browser windows and login as two students in the same group, open chat window, start typing
      3. Make sure students see each other's presence in recent activity block (this is not reflected in the recent activity report)
      4. Open two browser windows and login as two students in different groups, open chat window, start typing
      5. Make sure students don't see other person's presence in recent activity block
      Show
      run behat tests from MDL-44955 (some of them unfortunately fail with seg fault close to the end) The only module that I did not cover with behat is chat: Create chat activity in separate group mode, create groups, enrol several students and assign them to the groups. Open two browser windows and login as two students in the same group, open chat window, start typing Make sure students see each other's presence in recent activity block (this is not reflected in the recent activity report) Open two browser windows and login as two students in different groups, open chat window, start typing Make sure students don't see other person's presence in recent activity block
    • BACKEND Sprint 11

    Description

      While working on MDL-34397 I was reviewing all cases of writing to course_modinfo objects (returned by get_fast_modinfo()). I found copy-pasted code for accessing module groups in following files:

      • course/recent.php
      • mod/assign/lib.php
      • mod/assignment/lib.php
      • mod/chat/lib.php
      • mod/feedback/lib.php
      • mod/forum/lib.php
      • mod/quiz/lib.php
      • mod/wiki/lib.php
      • mod/workshop/lib.php

      Of which assign, assignment, feedback, quiz and workshop access the $modinfo->groups incorrectly - this array is indexed by groupping id but acceed with module id as a key instead.

      Also mod_glossary accesses groups incorrectly

      Will try to understand the error it can cause and create tests.

      Creating separate issue for it because it may need fixing in stable versions too.

      Attachments

        Issue Links

          blocks

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-44955 Behat tests for recent activity

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

          Improvement - An enhancement to an existing Moodle feature. MDL-43742 Create API for recent activity

          • Minor - Minor loss of function where workaround is possible
          • Open
          Discovered while testing

          Improvement - An enhancement to an existing Moodle feature. MDL-44794 get_groups(), get_groups_all() and groups_get_user_groups are not covered by unit tests

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-29731 Recent activities block doesn't show new forum posts when forum is set to "separate groups"

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-30781 Make course/lib.php::print_recent_activity() and associated callbacks deprecated

          • Minor - Minor loss of function where workaround is possible
          • Open
          is blocked by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-44362 Wrong id in recent activity

          • Major - Major loss of function, incorrect output
          • Closed

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-40619 Simple behat test for quiz

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

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-40872 Show recent submissions setting does not work in assignment

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

          Activity

            People

              marina Marina Glancy
              marina Marina Glancy
              Andrew Davis Andrew Davis
              Damyon Wiese Damyon Wiese
              Frédéric Massart Frédéric Massart
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona), Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona), Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Tim Hunt, Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Stevani Andolo, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                12/May/14