Moodle
  1. Moodle
  2. MDL-28311

SCORM displays a lot of php notices for invalid packages

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.2
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      Create new SCORM using package from MDL-26678 - on Save of the settings page it will display a list of moodle warnings about missing resources in the manifest file. (not ugly PHP errors)

      This occurs when the manifest contains "links" to resources in the package (htm, doc, txt etc) that don't actually exist in the package.

      Show
      Create new SCORM using package from MDL-26678 - on Save of the settings page it will display a list of moodle warnings about missing resources in the manifest file. (not ugly PHP errors) This occurs when the manifest contains "links" to resources in the package (htm, doc, txt etc) that don't actually exist in the package.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-28311

      Description

      When adding an incomplete SCORM to Moodle a bunch of PHP errors are shown - needs tidying up.

      It might be nice to report to the user that their package has some issues - the package attached to MDL-26678 helps to reproduce this - MDL-26942 has a dump of the PHP errors that occur.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Dan Marsden added a comment -

            rebased against this weeks master

            Show
            Dan Marsden added a comment - rebased against this weeks master
            Hide
            Sam Hemelryk added a comment -

            Thanks Dan, this has been integrated now

            Show
            Sam Hemelryk added a comment - Thanks Dan, this has been integrated now
            Hide
            Andrew Davis added a comment -

            Works as described

            Show
            Andrew Davis added a comment - Works as described
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Sent upstream and closing, many thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: