Moodle
  1. Moodle
  2. MDL-28017

scorm updatetime setting is invalid

    Details

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

          Activity

          Hide
          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
          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
          Eloy Lafuente (stronk7) added a comment -

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

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

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

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

          change made as requested - thanks.

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

          Thanks Dan, all branches integrated!

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

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

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

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

          Show
          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: