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

SCORM 2004 courses trigger PHP notices, when being launched after a first completion of all LOs, which prevents e.g. IE to correctly show the content after the TOC rendering

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.0.3
    • Component/s: SCORM
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      Found on evaluating MDL-26863 and then MDL-26880.
      Note: FF correctly show the TOC and launch the item, even with the notices.

        Gliffy Diagrams

          Activity

          Hide
          matteo Matteo Scaramuccia added a comment - - edited

          Patch proposal: https://github.com/scara/moodle/commit/36c0108a2f9f7ac6168598c3ec1d3d84bf25e9f0
          Test case:

          1. Apply MDL-26880 proposed fix, otherwise no one will be able to store any SCORM 2004 tracking data into the database;
          2. Apply the sequence of calls suggested as MDL-26880 test case, to create a completion status into the database;
          3. Enter for the second time with FF or IE: using IE, before applying the suggested patch above, will not show the content on the right

          Note: the above patch applies to vanilla Moodle code and not to a being MDL-26880 patched one. It should be dependent on that as this bug is dependent on MDL-26880: not sure if this is the right way to proceed in terms of Core Dev workflow.

          Show
          matteo Matteo Scaramuccia added a comment - - edited Patch proposal: https://github.com/scara/moodle/commit/36c0108a2f9f7ac6168598c3ec1d3d84bf25e9f0 Test case: Apply MDL-26880 proposed fix, otherwise no one will be able to store any SCORM 2004 tracking data into the database; Apply the sequence of calls suggested as MDL-26880 test case, to create a completion status into the database; Enter for the second time with FF or IE: using IE, before applying the suggested patch above, will not show the content on the right Note: the above patch applies to vanilla Moodle code and not to a being MDL-26880 patched one. It should be dependent on that as this bug is dependent on MDL-26880 : not sure if this is the right way to proceed in terms of Core Dev workflow.
          Hide
          danmarsden Dan Marsden added a comment -

          Pull request submitted - thanks Matteo

          Show
          danmarsden Dan Marsden added a comment - Pull request submitted - thanks Matteo
          Hide
          tsala Helen Foster added a comment -

          Matteo, thanks for your report and patch and Dan, thanks for fixing it. The fix is included in the latest 2.0.2+.

          Show
          tsala Helen Foster added a comment - Matteo, thanks for your report and patch and Dan, thanks for fixing it. The fix is included in the latest 2.0.2+.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                5/May/11