Permalink
Browse files

MDL-12614 - Bad regular expression matching was causing syntactically

invalid js when returning to previously attempted SCORM package.

Credit to Peter Chamberlin of ljcreate.com
  • Loading branch information...
1 parent 3ceeb19 commit 86ad0ba4593c3a1d080590c90b50b1f05b2aa7e6 poltawski committed Apr 30, 2008
Showing with 3 additions and 3 deletions.
  1. +3 −3 mod/scorm/datamodels/scorm_13.js.php
@@ -210,9 +210,9 @@ function SCORMapi1_3() {
$objectives = '';
foreach($userdata as $element => $value){
if (substr($element,0,14) == 'cmi.objectives') {
- preg_match('/\.(\d+)\./',$element,$matches);
- $element = preg_replace('/\.(\d+)\./',".N\$1.",$element);
- if ($matches[1] == $count) {
+ $element = preg_replace('/\.(\d+)\./', "_\$1.", $element);
+ preg_match('/_(\d+)\./', $element, $matches);
+ if (isset($matches[1]) && $matches[1] == $count) {
$count++;
$end = strpos($element,$matches[1])+strlen($matches[1]);
$subelement = substr($element,0,$end);

0 comments on commit 86ad0ba

Please sign in to comment.