From 1e3118a924723430656465d7cf5665e96ce3326f Mon Sep 17 00:00:00 2001
From: Cristian Fasanando <cristian.fasanando@beeznest.com>
Date: Mon, 6 Apr 2009 22:01:03 +0200
Subject: [PATCH] [svn r19572] Minor - Fixed bug of loop when you add a section
 like first item in a learning path - see FS#3988

---
 main/newscorm/lp_controller.php |  4 ++--
 main/newscorm/lp_view.php       | 18 ++----------------
 2 files changed, 4 insertions(+), 18 deletions(-)

diff --git a/main/newscorm/lp_controller.php b/main/newscorm/lp_controller.php
index 25d979bff4..56b96247bc 100644
--- a/main/newscorm/lp_controller.php
+++ b/main/newscorm/lp_controller.php
@@ -724,10 +724,10 @@ switch($action)
 		else
 		{			
 			if($debug > 0){error_log('New LP - Trying to set current item to ' . $_REQUEST['item_id'], 0);}
-			/*if ( !empty($_REQUEST['item_id']) )
+			if ( !empty($_REQUEST['item_id']) )
             {
                 $_SESSION['oLP']->set_current_item($_REQUEST['item_id']);
-            }*/            
+            }            
 			require('lp_view.php');
 		}
 		break;		
diff --git a/main/newscorm/lp_view.php b/main/newscorm/lp_view.php
index 55dd51f5a5..749e143a29 100644
--- a/main/newscorm/lp_view.php
+++ b/main/newscorm/lp_view.php
@@ -79,23 +79,9 @@ $htmlHeadXtra[] = '<script language="JavaScript" type="text/javascript">
 
 $_SESSION['oLP']->error = '';
 $lp_type = $_SESSION['oLP']->get_type();
-
-if (api_is_anonymous()) {
-		$lp_item_id = $_SESSION['oLP']->get_first_item_id();
-		if ( $lp_item_id != 0)
-			$_SESSION['oLP']->set_current_item($lp_item_id); // already done by lp_controller.php
-		else 
- 			$lp_item_id = $_SESSION['oLP']->get_current_item_id();
-
-	} else {
-		//$lp_item_id = $_SESSION['oLP']->get_current_item_id();
-		$lp_item_id = $_SESSION['oLP']->get_first_item_id();	
-	}
-	
-//$lp_item_id = $_SESSION['oLP']->get_current_item_id();
-
+$lp_item_id = $_SESSION['oLP']->get_current_item_id();
 //$lp_item_id = learnpath::escape_string($_GET['item_id']);
-$_SESSION['oLP']->set_current_item($lp_item_id); // already done by lp_controller.php
+//$_SESSION['oLP']->set_current_item($lp_item_id); // already done by lp_controller.php
 
 //Prepare variables for the test tool (just in case) - honestly, this should disappear later on
 $_SESSION['scorm_view_id'] = $_SESSION['oLP']->get_view_id();