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

First Access and Last Access dates for SCOs are the same in User -> Complete Report

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 1.9.5
    • Component/s: SCORM
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      This issue was first posted a while back, but not resolved for the user reports. (MDL-16184)

      The SCORM report uses the scorm_get_sco_runtime function from mod/scorm/locallib.php to get the "Started on" date. It actually queries this value from the database (scorm_scoes_track WHERE element='x.start.time').

      The user report, however, gets it's "First access" value from the scorm_user_complete function in mod/scorm/lib.php. Oddly, this value is first set to the current time and then reset to the timemodified if the current time is greater than the timemodified (which it always would be).

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            valerian valerian added a comment -

            I've attached a patch to correct this

            Show
            valerian valerian added a comment - I've attached a patch to correct this
            Hide
            valerian valerian added a comment -

            This issue can be set as resolved :
            already committed by Piers Harding in Moodle CVS (see MDL-16184)

            Show
            valerian valerian added a comment - This issue can be set as resolved : already committed by Piers Harding in Moodle CVS (see MDL-16184 )
            Hide
            danmarsden Dan Marsden added a comment -

            Dakota - is that correct? - or is this still an issue?

            Show
            danmarsden Dan Marsden added a comment - Dakota - is that correct? - or is this still an issue?
            Hide
            rmeske Ron Meske added a comment -

            I have tested this as part of MDL-16184. The first access date no longer updates upon each launch. However, please see my feedback on MDL-16184 for a potential new issue.

            Show
            rmeske Ron Meske added a comment - I have tested this as part of MDL-16184 . The first access date no longer updates upon each launch. However, please see my feedback on MDL-16184 for a potential new issue.
            Hide
            danmarsden Dan Marsden added a comment -

            flagging this one as fixed in 1.9.5 as per the commit from Piers - other issue Ron mentions will be dealt with on that other bug. - thanks!

            Show
            danmarsden Dan Marsden added a comment - flagging this one as fixed in 1.9.5 as per the commit from Piers - other issue Ron mentions will be dealt with on that other bug. - thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  13/May/09