Dan Marsden added a comment - 06/Dec/11 2:04 PM Note - CM -01 is still failing as next activity isn't launched after first activity is complete.

Dan Marsden added a comment - 25/Jun/12 5:40 AM this could do with a peer review too - I'll try to look at this a bit closer unless another dev beats me to it!

Rossiani Wijaya added a comment - 26/Jun/12 2:03 PM Hi Dan, Could you add testing instruction for this issue? Thanks

Rossiani Wijaya added a comment - 26/Jun/12 4:24 PM While reviewing this issue, I couldn't get it to passed the first page. I'm using the 'LMSTestPackage_CM-01.zip' Am I missing any setting?

Dan Marsden added a comment - 27/Jun/12 5:40 AM Hi Rosie - I probably need to write a decent set of testing info for SCORM 2004 packages - it only works in certain browsers, you are supposed to have the adl test engine running etc etc. If you wouldn't mind doing an initial look at the code around coding guidelines/structure etc that would be really useful at this stage. I'll try to throw a decent test doc explaining how to test a single package later this week thanks!

Rossiani Wijaya added a comment - 27/Jun/12 4:35 PM Hi Dan, I tried to setup adl test engine on my machine but it kept killing my machine. So I follow your suggestion to review the code itself. On mod/scorm/locallib.php, add break line between if statement. Then, on line 1703, is there any reason for re-set $scoes as $sco. If it is necessary, it should probably assigned as clone($scoes). On mod/scorm/module.js line 297 to 308, the conditional for "var scorm_skipprev = function(node) {}" could be simplified to: if (node.previousSibling && typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') { ... } else if (node.depth > 0 && typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') { .... } or if (typeof scoes_nav[mod_scorm_launch_sco].prevscoid != 'undefined') { var prevscoid = scoes_nav[mod_scorm_launch_sco].prevscoid; if (node.previousSibling) { node.previousSibling.title = scoes_nav[prevscoid].url; return node.previousSibling; }else if (node.depth > 0) { node.parent.title = scoes_nav[prevscoid].url; return node.parent; } } On mod/scorm/module.js line 325, the conditional for "var scorm_skipnext = function(node) {}", should the elseif statement check for "typeof scoes_nav [mod_scorm_launch_sco] .nextscoid != 'undefined'"? Let me know if you have any question.

Dan Marsden added a comment - 28/Jun/12 5:22 AM Thanks! - that's helpful feedback - Mayank can you please take a look at Rossiani's comments and make changes as required?

Dan Marsden added a comment - 13/Sep/12 3:59 PM Mayanks patch contains fixes for more than just this test - have reabased and moved his patch into MDL-35418 - will link all the tests that his patch fixes after testing it. Thanks Mayank.

Dan Marsden added a comment - 19/Oct/12 4:02 AM Moodle 2.4 now passes the CM-01 test due to the work in MDL-35418