Guillaume Viguier 15 years ago
parent
commit
a84a4094ac
2 changed files with 11 additions and 10 deletions
  1. 2 1
      documentation/credits.html
  2. 9 9
      main/newscorm/learnpath.class.php

+ 2 - 1
documentation/credits.html

@@ -69,6 +69,7 @@ Thomas De Praetere (thomas.depraetere@dokeos.com)
   <ul>
     <li> Juan Carlos Raña Trabado - considerable work on the wiki integration and improvements, and the FCKEditor/Mimetex extension as well as *many* FCKEditor's file manager and other modules improvements, as well as a continuous stream of recommendations that helped shape 1.8.5 and 1.8.6.*</li>
     <li> Ivan Tcholakov - various bugfixes in 1.8.6.*, very accurate bug reporting contributions and a considerable work in the integration of a new version of FCKEditor, along with a bunch of useful plugins and an exceptional work on internationalization</li>
+    <li> Hubert Borderiou - various bugfixes in 1.8.7 and very accurate bug reporting contributions </li>
   </ul>
   </li>
 </ol>
@@ -516,4 +517,4 @@ of information requests and organizing great community events</li>
 	<a href="http://jigsaw.w3.org/css-validator/">
 		<img src="http://jigsaw.w3.org/css-validator/images/vcss-blue" style="margin: 1em; float: right;" alt="Valid CSS" />
 	</a>
-</body></html>
+</body></html>

+ 9 - 9
main/newscorm/learnpath.class.php

@@ -5045,7 +5045,7 @@ class learnpath {
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang('FirstPosition') . '</option>';
 
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -5269,7 +5269,7 @@ class learnpath {
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang('FirstPosition') . '</option>';
 
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -5488,7 +5488,7 @@ class learnpath {
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang('FirstPosition') . '</option>';
 
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -5683,7 +5683,7 @@ class learnpath {
 		$return .= "\t\t\t\t" . '<select id="idPosition" name="previous" size="1">';
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang('FirstPosition') . '</option>';
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -5895,7 +5895,7 @@ class learnpath {
 
 		//POSITION
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$s_selected_position = $arrLP[$i]['id'];
 				elseif ($action == 'add') $s_selected_position = $arrLP[$i]['id'];
@@ -6141,7 +6141,7 @@ class learnpath {
 
 		//POSITION
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$s_selected_position = $arrLP[$i]['id'];
 				elseif ($action == 'add') $s_selected_position = $arrLP[$i]['id'];
@@ -6445,7 +6445,7 @@ class learnpath {
 		$return .= "\t\t\t\t" . '<select id="idPosition" name="previous" style="width:100%;" size="1" class="learnpath_item_form">';
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang("FirstPosition") . '</option>';
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -6661,7 +6661,7 @@ class learnpath {
 		$return .= "\t\t\t\t" . '<select id="idPosition" name="previous" style="width:100%;" size="1" class="learnpath_item_form">';
 		$return .= "\t\t\t\t\t" . '<option class="top" value="0">' . get_lang("FirstPosition") . '</option>';
 		for ($i = 0; $i < count($arrLP); $i++) {
-			if ($arrLP[$i]['parent_item_id'] == $parent && $arrLP[$i]['id'] != $id) {
+			if ($arrLP[$i]['parent_item_id'] == $parent_item_id && $arrLP[$i]['id'] != $id) {
 				if ($extra_info['previous_item_id'] == $arrLP[$i]['id'])
 					$selected = 'selected="selected" ';
 				elseif ($action == 'add') $selected = 'selected="selected" ';
@@ -8443,4 +8443,4 @@ if (!function_exists('trim_value')) {
 	function trim_value(& $value) {
 		$value = trim($value);
 	}
-}
+}