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

Regression: groups/grouping support in SCORM 2.0 not working

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.2
    • Fix Version/s: 2.2.1
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide
      1. Enroll a few students (more than two ) to a course
      2. Create two separate groups and divide the students into the two groups
      3. Submit atleast one attempt to a scorm from each of the student
      4. Now go to scorm>reports>basic as admin and make sure you get no errors
      5. Now change the group from the drop down many and make sure you see the users from currently selected group only
      6. Repeat above two steps with scorm>report>interactions
      Show
      Enroll a few students (more than two ) to a course Create two separate groups and divide the students into the two groups Submit atleast one attempt to a scorm from each of the student Now go to scorm>reports>basic as admin and make sure you get no errors Now change the group from the drop down many and make sure you see the users from currently selected group only Repeat above two steps with scorm>report>interactions
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-28282-master

      Description

      during my review of MDL-27523 I found groups and grouping support in the SCORM reports are broken due to the patch made in MDL-21555

      we should fix this back up.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            brigittebaquero Brigitte Baquero added a comment -

            Any one know how can I resolve this? ... It is critical for me

            Show
            brigittebaquero Brigitte Baquero added a comment - Any one know how can I resolve this? ... It is critical for me
            Hide
            danmarsden Dan Marsden added a comment -

            if it's critical enough that you have funding to pay someone to do the work - you could contract your local Moodle Partner to do the work - otherwise you'll need to wait for someone with the time to look at it and propose a fix.

            I work for the Moodle Partner Catalyst IT

            Show
            danmarsden Dan Marsden added a comment - if it's critical enough that you have funding to pay someone to do the work - you could contract your local Moodle Partner to do the work - otherwise you'll need to wait for someone with the time to look at it and propose a fix. I work for the Moodle Partner Catalyst IT
            Hide
            danmarsden Dan Marsden added a comment -

            hehe. - just looked at your profile and saw you were from Nivel! - whoops - Hi!

            flick me an e-mail to discuss

            Show
            danmarsden Dan Marsden added a comment - hehe. - just looked at your profile and saw you were from Nivel! - whoops - Hi! flick me an e-mail to discuss
            Hide
            danmarsden Dan Marsden added a comment -

            looks good, +1 to integrate - thanks Ankit!

            Show
            danmarsden Dan Marsden added a comment - looks good, +1 to integrate - thanks Ankit!
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            @ testers
            Please wait till MDL-28588 to be integrated before testing this.

            Show
            ankit_frenz Ankit Agarwal added a comment - @ testers Please wait till MDL-28588 to be integrated before testing this.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys this has been integrated now at the same time as MDL-28588.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys this has been integrated now at the same time as MDL-28588 .
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Passed! "Beauty, Narelle!"

            Show
            salvetore Michael de Raadt added a comment - Test result: Passed! "Beauty, Narelle!"
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Yes, you did it!

            Now your code is part of the best weeklies released ever, many thanks!

            Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Yes, you did it! Now your code is part of the best weeklies released ever, many thanks! Closing, ciao

              People

              • Votes:
                2 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Jan/12