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:
    • Rank:
      17702

      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

        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: