Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Component/s: SCORM
    • Testing Instructions:
      Hide

      This is a pain to test - it must be done on a windows machine using the 2004 test harness. thanks to Mayank we should have a new automated Test harness that allows testing a bit easier very soon...

      but if you really want to test this despite the fact it's only going in head....

      download the CM-01 Test Package and ADL 2004 Test Suite
      run through the test and make sure this doesn't appear in the logs:
      ERROR: Evaluating GetValue("cmi.scaled_passing_score") - Expected: "0.8" - LMS Returned: ""

      NOTE: there are still a range of other items causing the CM-01 to Fail - please see MDL-28740 for further detail.

      Show
      This is a pain to test - it must be done on a windows machine using the 2004 test harness. thanks to Mayank we should have a new automated Test harness that allows testing a bit easier very soon... but if you really want to test this despite the fact it's only going in head.... download the CM-01 Test Package and ADL 2004 Test Suite run through the test and make sure this doesn't appear in the logs: ERROR: Evaluating GetValue("cmi.scaled_passing_score") - Expected: "0.8" - LMS Returned: "" NOTE: there are still a range of other items causing the CM-01 to Fail - please see MDL-28740 for further detail.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-28541

      Description

      The ADL SCORM 2004 CM-01 test failed for a range of issues including:

      • cmi.scaled_passing_score - In activity 2 the GetValue("cmi.scaled_passing_score") returned null, expected - 0.8

      If you want to execute test CM-01 you can download the CM-01 Test Package and ADL 2004 Test Suite

      =====DETAILED LOG======
      Start ADL LMS Test Content Package CM-01
      Testing Activity 1
      Activity 1 was launched as expected
      ERROR: Did the LMS provide a disabled UI device that corresponds to the Continue Navigation Event? - Expected: Yes - Answered: No
      ERROR: Did the LMS provide an enabled Table Of Contents that includes Activity 2 as selectable? - Expected: No - Answered: Yes
      Evaluating Initialize("") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      ERROR: Evaluating GetValue("cmi.max_time_allowed") - Expected: "P5Y6M4DT12H30M58S" - LMS Returned: ""
      ERROR: Evaluating GetLastError("") - Expected: "0" - LMS Returned: "403"
      Evaluating SetValue("adl.nav.request", "continue") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      Evaluating SetValue("cmi.exit", "normal") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      Testing Activity 2
      Activity 2 was launched as expected
      Evaluating Initialize("") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      ERROR: Evaluating GetValue("cmi.scaled_passing_score") - Expected: "0.8" - LMS Returned: ""
      ERROR: Evaluating GetLastError("") - Expected: "0" - LMS Returned: "403"
      Evaluating SetValue("adl.nav.request", "continue") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      Evaluating SetValue("cmi.exit", "normal") - Expected: "true" - LMS Returned: "true"
      Evaluating GetLastError("") - Expected: "0" - LMS Returned: "0"
      Testing Activity 3
      ERROR: Expected Activity 3 - LMS Launched Activity 2

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Mayank Gupta added a comment - - edited

            [1] is a patch that fixes cmi.max_time_allowed - changing from $userdata->maxtimeallowed to $userdadta->attemptAbsoluteDurationLimit.
            This should fix - MDL-10707 as well.

            [1] - https://github.com/mayankgupta/moodle/commit/6a2b3cb

            I will be happy to modify it, if required.

            Thanks,
            Mayank.

            Show
            Mayank Gupta added a comment - - edited [1] is a patch that fixes cmi.max_time_allowed - changing from $userdata->maxtimeallowed to $userdadta->attemptAbsoluteDurationLimit. This should fix - MDL-10707 as well. [1] - https://github.com/mayankgupta/moodle/commit/6a2b3cb I will be happy to modify it, if required. Thanks, Mayank.
            Hide
            Dan Marsden added a comment -

            Hi Mayank - how are you going with the fix for cmi.scaled_passing_sco ?

            thanks,

            Dan

            Show
            Dan Marsden added a comment - Hi Mayank - how are you going with the fix for cmi.scaled_passing_sco ? thanks, Dan
            Hide
            Mayank Gupta added a comment -

            Hi Dan - I have added the code in mod/scorm/api.php to fetch the value of minnormalizedmeasure from the database table - scorm_seq_objective.
            Did not commit it, as thought of committing the fix for Activity 3 with this one.

            The patch [1] should fix cmi.scaled_passing_score

            [1] - https://github.com/mayankgupta/moodle/commit/2313993232a543802f534aed06d9d25a1e5e8384

            Thanks,
            Mayank

            Show
            Mayank Gupta added a comment - Hi Dan - I have added the code in mod/scorm/api.php to fetch the value of minnormalizedmeasure from the database table - scorm_seq_objective. Did not commit it, as thought of committing the fix for Activity 3 with this one. The patch [1] should fix cmi.scaled_passing_score [1] - https://github.com/mayankgupta/moodle/commit/2313993232a543802f534aed06d9d25a1e5e8384 Thanks, Mayank
            Hide
            Dan Marsden added a comment -

            converting this bug to track only the patch for cmi.scaled_passing_score to get this patch in - all other CM-01 failures should be tracked on MDL-28740 - thanks!

            Show
            Dan Marsden added a comment - converting this bug to track only the patch for cmi.scaled_passing_score to get this patch in - all other CM-01 failures should be tracked on MDL-28740 - thanks!
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Passing this based in comments from Dan about it:

            [22:01:25] <dan.marsden> cool - I would pass MDL-28541 without bothering to test too much - it only affects SCORM 2004 (which we don't officially support anyway) and it's only going in master...and it only adds an extra param to be available to the scorm 2004 api. (as long as it meets coding guidelines etc)

            Show
            Eloy Lafuente (stronk7) added a comment - Passing this based in comments from Dan about it: [22:01:25] <dan.marsden> cool - I would pass MDL-28541 without bothering to test too much - it only affects SCORM 2004 (which we don't officially support anyway) and it's only going in master...and it only adds an extra param to be available to the scorm 2004 api. (as long as it meets coding guidelines etc)
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Closing, this is now part of Moodle. Big thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Closing, this is now part of Moodle. Big thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: