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

scorm updatetime setting is invalid

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1, 2.2
    • Fix Version/s: 2.0.4, 2.1.1
    • Component/s: SCORM
    • Labels:

      Description

      admin SCORM setting - updatetime doesn't seem to be used - think it was supposed to define when to run the SCORM cron event for scorms that are set to UPDATE_EVERYDAY

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            danmarsden Dan Marsden added a comment -

            removing this setting - it was added incorrectly to 2.0 but the code to manage the setting was never implemented

            Show
            danmarsden Dan Marsden added a comment - removing this setting - it was added incorrectly to 2.0 but the code to manage the setting was never implemented
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Ho, sorry, but this requires upgrade script with deletion from config (unset_config). Reopening, thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Ho, sorry, but this requires upgrade script with deletion from config (unset_config). Reopening, thanks!
            Hide
            danmarsden Dan Marsden added a comment -

            really? - ok.... will submit new patch.

            Show
            danmarsden Dan Marsden added a comment - really? - ok.... will submit new patch.
            Hide
            danmarsden Dan Marsden added a comment -

            change made as requested - thanks.

            Show
            danmarsden Dan Marsden added a comment - change made as requested - thanks.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Thanks Dan, all branches integrated!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Thanks Dan, all branches integrated!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Tested all branches. Variable went out on upgrade and the setting is not available anymore. Pass.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Tested all branches. Variable went out on upgrade and the setting is not available anymore. Pass.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Congratulations - this fix has just been released in the weeklies.

            Show
            samhemelryk Sam Hemelryk added a comment - Congratulations - this fix has just been released in the weeklies.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11