MDL-44753 SCORM: make correct selection of first sco - also fixes MDL-44896
authorDan Marsden <dan@danmarsden.com>
Wed, 2 Apr 2014 07:50:21 +0000 (20:50 +1300)
committerDan Marsden <dan@danmarsden.com>
Wed, 2 Apr 2014 07:50:21 +0000 (20:50 +1300)
mod/scorm/datamodels/scormlib.php

index b8b14e9..f2d62ae 100644 (file)
@@ -705,7 +705,7 @@ function scorm_parse_scorm(&$scorm, $manifest) {
         $scorm->launch = $defaultorgid;
     } else if (!empty($defaultorgid) && isset($newscoes[$defaultorgid]) && empty($newscoes[$defaultorgid]->launch)) {
         // The launch is probably the default org so we need to find the first launchable item inside this org.
-        $sqlselect = 'scorm = ? AND sortorder > ? AND '.$DB->sql_isnotempty('scorm_scoes', 'launch', false, true);
+        $sqlselect = 'scorm = ? AND sortorder >= ? AND '.$DB->sql_isnotempty('scorm_scoes', 'launch', false, true);
         // We use get_records here as we need to pass a limit in the query that works cross db.
         $scoes = $DB->get_records_select('scorm_scoes', $sqlselect, array($scorm->id, $firstinorg), 'sortorder', 'id', 0, 1);
         if (!empty($scoes)) {