Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.3
    • Fix Version/s: 2.1.5
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide
      1. Add a scorm package to a course and submit a few attempts from various students accounts
      2. Goto Scorm>Reports as admin/teacher
      3. Select 'all users' from the settings and save
      4. In top navigation bar (appears only when there are more than 20 users in total) select a firstname such that no user should be reported
      5. Make sure you still can see the preferences form and the First name/last name navigation bar
      6. Delete all attempts and change settings to "users with attempts" and save
      7. Make sure you still can see the preferences form
      Show
      Add a scorm package to a course and submit a few attempts from various students accounts Goto Scorm>Reports as admin/teacher Select 'all users' from the settings and save In top navigation bar (appears only when there are more than 20 users in total) select a firstname such that no user should be reported Make sure you still can see the preferences form and the First name/last name navigation bar Delete all attempts and change settings to "users with attempts" and save Make sure you still can see the preferences form
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE

      Description

      The structure of Scorm report is completely different in 2.1 and 2.2 onwards, hence the changes made in MDL-30652 needs to be back-ported to 2.1 as well

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            ankit_frenz Ankit Agarwal added a comment -

            @integrators
            21 only
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - @integrators 21 only Thanks
            Hide
            andyjdavis Andrew Davis added a comment -

            2 little things.

            This comment "// Show preferences form irrespective of attempts are there to report or not" sounds like it should probably be something like "// Show the preferences form irrespective of whether there are attempts to report or not"

            Also, the section that starts with "if ($candelete && !$download) {" is double indented.

            Show
            andyjdavis Andrew Davis added a comment - 2 little things. This comment "// Show preferences form irrespective of attempts are there to report or not" sounds like it should probably be something like "// Show the preferences form irrespective of whether there are attempts to report or not" Also, the section that starts with "if ($candelete && !$download) {" is double indented.
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Thanks for the review Andrew.
            I have made the changes.
            Sending to integration
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Thanks for the review Andrew. I have made the changes. Sending to integration Thanks
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Ankit, this has been integrated now.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Ankit, this has been integrated now.
            Hide
            abgreeve Adrian Greeve added a comment -

            This doesn't get locked any more when there are not users to report in the scorm results. I couldn't find any problems.
            Thanks Ankit.

            Show
            abgreeve Adrian Greeve added a comment - This doesn't get locked any more when there are not users to report in the scorm results. I couldn't find any problems. Thanks Ankit.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks!

            Closing as fixed, heading to zzzZZZzzz, niao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks! Closing as fixed, heading to zzzZZZzzz, niao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12