Minor - fix php notices, warnings

Author: jmontoyaa

main/lp/learnpathItem.class.php

@@ -1920,7 +1920,11 @@ public function fixAbusiveTime($time)
  }
  }*/
  // PL --- Porcentaje lección (tiempo leccion / tiempo total curso)
- $pl = $timeLp / $timeTotalCourse;
+ $pl = 0;
+ if (!empty($timeTotalCourse)) {
+ $pl = $timeLp / $timeTotalCourse;
+ }
+
  /*
  * TL: Tiempo que pone en una lección
  * TT : tiempo total que pone Teresa (suma tiempos lecciones curso)

main/lp/lp_view.php

@@ -576,7 +576,10 @@
  }
 
  // PL --- Porcentaje lección (tiempo leccion / tiempo total curso)
- $pl = $timeLp / $timeTotalCourse;
+ $pl = 0;
+ if (!empty($timeTotalCourse)) {
+ $pl = $timeLp / $timeTotalCourse;
+ }
 
  /*
  * TL: Tiempo que pone en una lección

src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php

@@ -2874,7 +2874,7 @@ public function restore_learnpaths($session_id = 0, $respect_base_content = fals
  // Updating prerequisites
  foreach ($old_prerequisite as $key => $my_old_prerequisite) {
  if ($my_old_prerequisite != '') {
- $sql = "UPDATE ".$table_item." SET prerequisite = '".$my_old_prerequisite."'
+ $sql = "UPDATE $table_item SET prerequisite = '".$my_old_prerequisite."'
  WHERE c_id = ".$this->destination_course_id." AND id = '".$key."' ";
  Database::query($sql);
  }
@@ -2883,7 +2883,7 @@ public function restore_learnpaths($session_id = 0, $respect_base_content = fals
  // Updating refs
  foreach ($old_refs as $key => $my_old_ref) {
  if ($my_old_ref != '') {
- $sql = "UPDATE ".$table_item." SET ref = '".$my_old_ref."'
+ $sql = "UPDATE $table_item SET ref = '".$my_old_ref."'
  WHERE c_id = ".$this->destination_course_id." AND id = '".$key."' ";
  Database::query($sql);
  }
@@ -2892,10 +2892,10 @@ public function restore_learnpaths($session_id = 0, $respect_base_content = fals
  foreach ($parent_item_ids as $new_item_id => $parent_item_old_id) {
  $parent_new_id = 0;
  if ($parent_item_old_id != 0) {
- $parent_new_id = $new_item_ids[$parent_item_old_id];
+ $parent_new_id = isset($new_item_ids[$parent_item_old_id]) ? $new_item_ids[$parent_item_old_id] : 0;
  }
- $sql = "UPDATE ".$table_item." SET parent_item_id = '".$parent_new_id."'
- WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'";
+ $sql = "UPDATE $table_item SET parent_item_id = '$parent_new_id'
+ WHERE c_id = ".$this->destination_course_id." AND id = $new_item_id";
  Database::query($sql);
  }
 
@@ -2904,17 +2904,17 @@ public function restore_learnpaths($session_id = 0, $respect_base_content = fals
  if ($previous_item_old_id != 0) {
  $previous_new_id = isset($new_item_ids[$previous_item_old_id]) ? $new_item_ids[$previous_item_old_id] : '';
  }
- $sql = "UPDATE ".$table_item." SET previous_item_id = '".$previous_new_id."'
+ $sql = "UPDATE $table_item SET previous_item_id = $previous_new_id
  WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'";
  Database::query($sql);
  }
 
  foreach ($next_item_ids as $new_item_id => $next_item_old_id) {
  $next_new_id = 0;
  if ($next_item_old_id != 0) {
- $next_new_id = $new_item_ids[$next_item_old_id];
+ $next_new_id = isset($new_item_ids[$next_item_old_id]) ? $new_item_ids[$next_item_old_id] : 0;
  }
- $sql = "UPDATE ".$table_item." SET next_item_id = '".$next_new_id."'
+ $sql = "UPDATE $table_item SET next_item_id = $next_new_id
  WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'";
  Database::query($sql);
  }
@@ -2924,8 +2924,8 @@ public function restore_learnpaths($session_id = 0, $respect_base_content = fals
  if ($prerequisite_old_id != 0) {
  $prerequisite_new_id = $new_item_ids[$prerequisite_old_id];
  }
- $sql = "UPDATE ".$table_item." SET prerequisite = '".$prerequisite_new_id."'
- WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'";
+ $sql = "UPDATE $table_item SET prerequisite = $prerequisite_new_id
+ WHERE c_id = ".$this->destination_course_id." AND id = $new_item_id";
  Database::query($sql);
  }
  $this->course->resources[RESOURCE_LEARNPATH][$id]->destination_id = $new_lp_id;