From f3a4d3fef360efb98907277fdd178f87e082a26a Mon Sep 17 00:00:00 2001
From: Dan Marsden <dan@danmarsden.com>
Date: Mon, 20 Feb 2017 13:41:15 +1300
Subject: [PATCH] MDL-58015 mod_scorm: set orgidentifier correctly for popup.

---
 mod/scorm/view.php | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/mod/scorm/view.php b/mod/scorm/view.php
index 1a172964758eb..9cdbf764165d7 100644
--- a/mod/scorm/view.php
+++ b/mod/scorm/view.php
@@ -66,23 +66,22 @@
 
 $launch = false; // Does this automatically trigger a launch based on skipview.
 if (!empty($scorm->popup)) {
-    $orgidentifier = '';
-
     $scoid = 0;
     $orgidentifier = '';
 
     $result = scorm_get_toc($USER, $scorm, $cm->id, TOCFULLURL);
     // Set last incomplete sco to launch first.
     if (!empty($result->sco->id)) {
-        $scoid = $result->sco->id;
+        $sco = $result->sco;
     } else {
-        if ($sco = scorm_get_sco($scorm->launch, SCO_ONLY)) {
-            if (($sco->organization == '') && ($sco->launch == '')) {
-                $orgidentifier = $sco->identifier;
-            } else {
-                $orgidentifier = $sco->organization;
-            }
-            $scoid = $sco->id;
+        $sco = scorm_get_sco($scorm->launch, SCO_ONLY);
+    }
+    if (!empty($sco)) {
+        $scoid = $sco->id;
+        if (($sco->organization == '') && ($sco->launch == '')) {
+            $orgidentifier = $sco->identifier;
+        } else {
+            $orgidentifier = $sco->organization;
         }
     }