Quellcode durchsuchen

Minor - fix php warning

jmontoyaa vor 7 Jahren
Ursprung
Commit
e18aefdf0e
1 geänderte Dateien mit 17 neuen und 15 gelöschten Zeilen
  1. 17 15
      src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php

+ 17 - 15
src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php

@@ -1091,7 +1091,8 @@ class CourseRestorer
                 if (!empty($params['forum_image'])) {
                     $original_forum_image = $this->course->path.'upload/forum/images/'.$params['forum_image'];
                     if (file_exists($original_forum_image)) {
-                        $new_forum_image = api_get_path(SYS_COURSE_PATH).$this->destination_course_info['path'].'/upload/forum/images/'.$params['forum_image'];
+                        $new_forum_image = api_get_path(SYS_COURSE_PATH).
+                            $this->destination_course_info['path'].'/upload/forum/images/'.$params['forum_image'];
                         @copy($original_forum_image, $new_forum_image);
                     }
                 }
@@ -1101,23 +1102,24 @@ class CourseRestorer
                 if ($new_id) {
                     $sql = "UPDATE $table_forum SET forum_id = iid WHERE iid = $new_id";
                     Database::query($sql);
-                }
-
-                $this->course->resources[RESOURCE_FORUM][$id]->destination_id = $new_id;
+                    $this->course->resources[RESOURCE_FORUM][$id]->destination_id = $new_id;
 
-                $forum_topics = 0;
-                if (is_array($this->course->resources[RESOURCE_FORUMTOPIC])) {
-                    foreach ($this->course->resources[RESOURCE_FORUMTOPIC] as $topic_id => $topic) {
-                        if ($topic->obj->forum_id == $id) {
-                            $this->restore_topic($topic_id, $new_id, $sessionId);
-                            $forum_topics++;
+                    $forum_topics = 0;
+                    if (isset($this->course->resources[RESOURCE_FORUMTOPIC]) &&
+                        is_array($this->course->resources[RESOURCE_FORUMTOPIC])
+                    ) {
+                        foreach ($this->course->resources[RESOURCE_FORUMTOPIC] as $topic_id => $topic) {
+                            if ($topic->obj->forum_id == $id) {
+                                $this->restore_topic($topic_id, $new_id, $sessionId);
+                                $forum_topics++;
+                            }
                         }
                     }
-                }
-                if ($forum_topics > 0) {
-                    $sql = "UPDATE ".$table_forum." SET forum_threads = ".$forum_topics."
-                            WHERE c_id = {$this->destination_course_id} AND forum_id = ".(int) $new_id;
-                    Database::query($sql);
+                    if ($forum_topics > 0) {
+                        $sql = "UPDATE ".$table_forum." SET forum_threads = ".$forum_topics."
+                                WHERE c_id = {$this->destination_course_id} AND forum_id = ".(int) $new_id;
+                        Database::query($sql);
+                    }
                 }
             }
         }