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

scorm debugging function coding standards

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8.7, 2.9.1, 3.0
    • Fix Version/s: 2.8.8, 2.9.2
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      Test that the SCORM debugger can be enabled and disabled.
      Debugging settings are located at Settings > Site Administration > Plugins > Activities > Scorm.
      Check the checkbox for Activate API debug...

      Set the api mask. You can use the mask to enable debugging under certain conditions. For example, if you are logged in using the admin user (username admin) you can set the api mask to: admin.* Users not logged in as admin will not see the debugging log. The "Default" api mask is .*

      First test using default api mask, then try the variations like username.* *.scormname
      Try to break it by entering invalid chars and making sure that invalid chars just result in debugging not showing.

      Show
      Test that the SCORM debugger can be enabled and disabled. Debugging settings are located at Settings > Site Administration > Plugins > Activities > Scorm. Check the checkbox for Activate API debug... Set the api mask. You can use the mask to enable debugging under certain conditions. For example, if you are logged in using the admin user (username admin) you can set the api mask to: admin.* Users not logged in as admin will not see the debugging log. The "Default" api mask is .* First test using default api mask, then try the variations like username.* *.scormname Try to break it by entering invalid chars and making sure that invalid chars just result in debugging not showing.
    • Affected Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull Master Branch:
      master_MDL-51015

      Description

      The SCORM debugging function uses eval() in a way that just seems weird to me. We should remove the use of eval() there.

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/Sep/15