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

SCORM module upgrade problems...

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8, 1.9
    • Fix Version/s: 1.8, 1.9
    • Component/s: SCORM
    • Labels:
      None
    • Database:
      Any
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      Today's modification to SCORM module (1.8.2.1) show one warning:

      (Warning: Invalid argument supplied for foreach() in mod/scorm/db/upgrade.php on line 255)

      if:

      $tracks = get_records('scorm_scoes_track');

      returns no rows. Should be changed to:

      if ($tracks = get_records('scorm_scoes_track')) {

      But that isn't important. The important thing is that such change hasn't been merged to HEAD (and I think it should!!).

      Please, merge everything from branch to HEAD per commit, moving the MERGED tag and so on... else we won't be providing a consistent upgrade path to moodlers.

      Review pending files to merge at:

      http://docs.moodle.org/en/Development:Unmerged_files#Files_with_MOODLE_18_MERGED_tag_not_updated_-_Wed_Mar_14_21:40:24_CET_2007

      Ciao

        Gliffy Diagrams

          Activity

          Hide
          upo Roberto Pinna added a comment -

          Done

          today i'll check all unmerged files.

          thanks,

          Ciao

          Show
          upo Roberto Pinna added a comment - Done today i'll check all unmerged files. thanks, Ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                31/Mar/07