Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor 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).

Issue Links

Activity

Hide
valerian added a comment -

I've attached a patch to correct this

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

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

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

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

Show
Dan Marsden added a comment - Dakota - is that correct? - or is this still an issue?
Hide
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
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
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
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

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: