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

scorm api.php was moved to loaddatamodel.php but accidentally reverted back

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.3
    • Component/s: SCORM
    • Labels:

      Description

      need to dump api.php and replace it with loaddatamodel.php - this was done a while back (in 1.9 branch before 2.0 release) but somewhere along the lines it was reverted and now loaddatamodel.php isn't used anymore and api.php has a bunch of fixes that need to be transferred to loaddatamodel.php

        Gliffy Diagrams

          Activity

          Hide
          chtombleson Christopher Tombleson added a comment -

          Fixed here:
          https://github.com/chtombleson/moodle/tree/master-MDL-30598

          Merged api.php into loaddatamodel.php

          Show
          chtombleson Christopher Tombleson added a comment - Fixed here: https://github.com/chtombleson/moodle/tree/master-MDL-30598 Merged api.php into loaddatamodel.php
          Hide
          danmarsden Dan Marsden added a comment -

          thanks Chris - have grabbed your change and fixed a few coding guideline issues with that file before I submit it for integration.

          This is only for master.

          Show
          danmarsden Dan Marsden added a comment - thanks Chris - have grabbed your change and fixed a few coding guideline issues with that file before I submit it for integration. This is only for master.
          Hide
          danmarsden Dan Marsden added a comment -

          submitting for integration - thanks for the help Chris!

          Show
          danmarsden Dan Marsden added a comment - submitting for integration - thanks for the help Chris!
          Hide
          poltawski Dan Poltawski added a comment -

          Integrated, thanks

          Show
          poltawski Dan Poltawski added a comment - Integrated, thanks
          Hide
          ankit_frenz Ankit Agarwal added a comment -

          Hi Dan,

          I am not sure exactly if its related but, around 40% of the time I tried to load the scorm it gives out an error
          "Unable to find a SCORM API. Session will not be logged."

          The cases when it doesn't pop any error, everything works fine and grades are pushed back without any issue.
          Please advice if to pass or fail this test.

          Thanks

          Show
          ankit_frenz Ankit Agarwal added a comment - Hi Dan, I am not sure exactly if its related but, around 40% of the time I tried to load the scorm it gives out an error "Unable to find a SCORM API. Session will not be logged." The cases when it doesn't pop any error, everything works fine and grades are pushed back without any issue. Please advice if to pass or fail this test. Thanks
          Hide
          danmarsden Dan Marsden added a comment -

          the strict standards stuff could be causing that - there's some code for peer-review that might help in MDL-32507 - try turning strict standards off to see if it's more reliable?

          Show
          danmarsden Dan Marsden added a comment - the strict standards stuff could be causing that - there's some code for peer-review that might help in MDL-32507 - try turning strict standards off to see if it's more reliable?
          Hide
          ankit_frenz Ankit Agarwal added a comment -

          Thanks Dan,
          I set debug to E_ALL and it is much better now. No more errors.

          Passing this
          Thanks

          Show
          ankit_frenz Ankit Agarwal added a comment - Thanks Dan, I set debug to E_ALL and it is much better now. No more errors. Passing this Thanks
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome?

          Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U

          Closing, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome? Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                25/Jun/12