From cc3883c87f33105b1b0d5f04b6f206a0f94d17e1 Mon Sep 17 00:00:00 2001 From: jmontoyaa Date: Tue, 15 May 2018 11:57:48 +0200 Subject: [PATCH] Minor - Format code, fix php warning --- main/group/member_settings.php | 7 ++++--- .../CourseBundle/Component/CourseCopy/CourseRestorer.php | 4 ++-- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/main/group/member_settings.php b/main/group/member_settings.php index 1f98431c9a..7c06ca2e78 100755 --- a/main/group/member_settings.php +++ b/main/group/member_settings.php @@ -197,17 +197,18 @@ $form->addButtonSave(get_lang('SaveSettings')); if ($form->validate()) { $values = $form->exportValues(); + // Storing the users (we first remove all users and then add only those who were selected) - GroupManager:: unsubscribe_all_users($current_group); + GroupManager::unsubscribe_all_users($current_group); if (isset($_POST['group_members']) && count($_POST['group_members']) > 0) { - GroupManager:: subscribe_users( + GroupManager::subscribe_users( $values['group_members'], $current_group ); } // Returning to the group area (note: this is inconsistent with the rest of chamilo) - $cat = GroupManager :: get_category_from_group($current_group['iid']); + $cat = GroupManager::get_category_from_group($current_group['iid']); if (isset($_POST['group_members']) && count($_POST['group_members']) > $max_member && $max_member != GroupManager::MEMBER_PER_GROUP_NO_LIMIT diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php b/src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php index 07eb4cc207..991d3bbe98 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/CourseRestorer.php @@ -2854,10 +2854,10 @@ class CourseRestorer foreach ($previous_item_ids as $new_item_id => $previous_item_old_id) { $previous_new_id = 0; if ($previous_item_old_id != 0) { - $previous_new_id = $new_item_ids[$previous_item_old_id]; + $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."' - WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'"; + WHERE c_id = ".$this->destination_course_id." AND id = '".$new_item_id."'"; Database::query($sql); }