Moodle
  1. Moodle
  2. MDL-30009

SCORM reports- bug with started on/last accessed when showing all users

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      Create a new SCORM - here's one:
      http://moodle.org/mod/data/view.php?d=50&rid=1655&filter=1
      in a course with multiple students log in as one and enter the SCORM to get a grade.
      as the teacher (or admin) view the SCORM reports page and choose to show "all users" - then use the order functions in the table to make sure some users appear under the user that has a grade - make sure all start/last accessed dates are blank for users that have not entered the SCORM.

      Show
      Create a new SCORM - here's one: http://moodle.org/mod/data/view.php?d=50&rid=1655&filter=1 in a course with multiple students log in as one and enter the SCORM to get a grade. as the teacher (or admin) view the SCORM reports page and choose to show "all users" - then use the order functions in the table to make sure some users appear under the user that has a grade - make sure all start/last accessed dates are blank for users that have not entered the SCORM.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-30009
    • Rank:
      19553

      Description

      When showing all users in the SCORM reports page /scorm/report.php if the user doesn't have an attempt and the user listed before it does, the started on and last accessed on dates are copied from the previous record instead of showing a blank statrted on and last accessed date.

        Issue Links

          Activity

          Hide
          Sam Hemelryk added a comment -

          Thanks Dan - this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Dan - this has been integrated now
          Hide
          Rossiani Wijaya added a comment -

          Hi Dan,

          The choose to show function always set back to 'all users'.

          To reproduce:
          on report page, set show to 'user with no attempts only' and save preference. Then click on 'surname' to reorder user by their surname. The report page is refresh by showing list of user wit attempts and show selection is set back to 'all users'

          The patch fixed the started and last accessed date for user attempt.

          Show
          Rossiani Wijaya added a comment - Hi Dan, The choose to show function always set back to 'all users'. To reproduce: on report page, set show to 'user with no attempts only' and save preference. Then click on 'surname' to reorder user by their surname. The report page is refresh by showing list of user wit attempts and show selection is set back to 'all users' The patch fixed the started and last accessed date for user attempt.
          Hide
          Dan Marsden added a comment -

          Thanks Rosie - I noticed some weird behaviour there yesterday too but hadn't got round to creating a bug for it yet - feel free to create a new one -thanks!

          Show
          Dan Marsden added a comment - Thanks Rosie - I noticed some weird behaviour there yesterday too but hadn't got round to creating a bug for it yet - feel free to create a new one -thanks!
          Hide
          Rossiani Wijaya added a comment -

          Hi Dan,

          I created an issue to fix scorm preference report (MDL-30028).

          Your patch works great.

          Test passed.

          Show
          Rossiani Wijaya added a comment - Hi Dan, I created an issue to fix scorm preference report ( MDL-30028 ). Your patch works great. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And this has been sent upstream (already available @ git and cvs repos). Many, many thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - And this has been sent upstream (already available @ git and cvs repos). Many, many thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: