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

scorm debugging function coding standards

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.8.7, 2.9.1, 3.0
    • 2.8.8, 2.9.2
    • SCORM
    • MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • master_MDL-51015
    • 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.

    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

            danmarsden Dan Marsden
            danmarsden Dan Marsden
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Frédéric Massart Frédéric Massart
            Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Sujith Haridasan
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              14/Sep/15