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

          Attachments

            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