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

          Attachments

            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