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

HttpRequest 404 Error Handling

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.4
    • Fix Version/s: 1.8.6, 1.9.1
    • Component/s: SCORM
    • Labels:
      None
    • Environment:
      Windows XP SP2 + IE7
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      File: /mod/scorm/api.php
      Include: /mod/scorm/datamodels/scorm_12.js.php
      Line: 546
      Error: Object doesn't support this property or method

      The value of variable 'result' is 404 (numeric). The call to split this number by line-break fails when the return code is the single number indicating file not found. This can be fixed by casting result to a string, leaving the errorCode variable undefined.

      results = String(result).split("\n");

      Note: The cause of this issue is the failure of the POST HTTP request, causing page redirection to return the response code 404 (even though an actual page is reached). I suspect this may be caused by session variable expiry, since the referenced /mod/scorm/datamodel.php file does exist. The error is more common when the server is under load, typically indicated by a longer time taking to return from the DoRequest() method.

      The return value of StoreData() function appears not to be used - which can result in the silent loss of student results!

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Peter! I've applied the fix to prevent the error, I think we need to file another bug for the issue of checking the status of this request and indeed trying to find where the 404's are coming from..

            Show
            poltawski Dan Poltawski added a comment - Thanks Peter! I've applied the fix to prevent the error, I think we need to file another bug for the issue of checking the status of this request and indeed trying to find where the 404's are coming from..
            Hide
            skodak Petr Skoda added a comment -

            reviewed, closing

            Show
            skodak Petr Skoda added a comment - reviewed, closing

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  15/May/08