From 6507b7772a7335a105a3dae2da99d5d2dab0a88d Mon Sep 17 00:00:00 2001 From: Scrutinizer Auto-Fixer Date: Fri, 17 Mar 2017 10:10:20 +0000 Subject: [PATCH] Scrutinizer Auto-Fixes This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com --- .../admin/access_url_add_usergroup_to_url.php | 8 +-- main/admin/access_url_check_user_session.php | 12 ++--- main/admin/access_url_edit_courses_to_url.php | 26 +++++----- main/admin/archive_cleanup.php | 2 +- main/admin/configure_extensions.php | 10 ++-- main/admin/course_intro_pdf_import.php | 10 ++-- main/admin/course_list.php | 26 +++++----- main/admin/course_request_accepted.php | 2 +- main/admin/extra_field_options.php | 30 +++++------ .../admin/ldap_import_students_to_session.php | 38 +++++++------- main/admin/skills_wheel.php | 2 +- main/admin/system_status.php | 2 +- main/admin/usergroup_export.php | 4 +- main/attendance/attendance_edit.php | 12 ++--- .../shibboleth/test/shibboleth_test.class.php | 2 +- main/blog/blog.php | 46 ++++++++--------- main/calendar/ical_export.php | 6 +-- main/course_description/add.php | 10 ++-- main/course_description/edit.php | 12 ++--- main/course_description/listing.php | 18 +++---- main/course_progress/thematic_plan.php | 8 +-- main/forum/newthread.php | 6 +-- main/gradebook/gradebook_add_link.php | 2 +- main/gradebook/lib/be/linkfactory.class.php | 4 +- main/gradebook/lib/fe/usertable.class.php | 10 ++-- main/inc/ajax/link.ajax.php | 2 +- main/inc/ajax/session.ajax.php | 4 +- main/inc/lib/AnnouncementEmail.php | 2 +- .../add_courses_to_session_functions.lib.php | 20 ++++---- main/inc/lib/extra_field_option.lib.php | 4 +- main/inc/lib/lp_item.lib.php | 2 +- main/inc/lib/security.lib.php | 6 +-- main/lp/aiccItem.class.php | 2 +- main/lp/blank.php | 2 +- main/lp/lp_ajax_save_item.php | 6 +-- main/lp/lp_edit_item.php | 8 +-- main/lp/lp_subscribe_users.php | 2 +- main/mySpace/course.php | 6 +-- main/mySpace/users.php | 16 +++--- main/session/session_import_drh.php | 4 +- main/social/group_add.php | 6 +-- main/social/group_invitation.php | 18 +++---- main/survey/ch_multiplechoice.php | 4 +- main/survey/ch_score.php | 2 +- main/survey/ch_yesno.php | 2 +- main/survey/link.php | 2 +- main/ticket/categories.php | 4 +- main/tracking/course_session_report.php | 50 +++++++++---------- main/user/class.php | 2 +- main/user/classes.php | 4 +- .../Component/CourseCopy/CourseBuilder.php | 2 +- .../CourseCopy/Resources/CourseSession.php | 2 +- .../CourseCopy/Resources/Glossary.php | 4 +- .../Component/CourseCopy/Resources/Link.php | 2 +- .../CourseCopy/Resources/LinkCategory.php | 4 +- 55 files changed, 251 insertions(+), 251 deletions(-) diff --git a/main/admin/access_url_add_usergroup_to_url.php b/main/admin/access_url_add_usergroup_to_url.php index 1b7ba022ce..3d85b9c71d 100755 --- a/main/admin/access_url_add_usergroup_to_url.php +++ b/main/admin/access_url_add_usergroup_to_url.php @@ -30,8 +30,8 @@ $tbl_user = Database:: get_main_table(TABLE_MAIN_USER); $tbl_course = Database:: get_main_table(TABLE_MAIN_COURSE); $tool_name = get_lang('AddUserGroupToURL'); -$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin')); -$interbreadcrumb[] = array ('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs')); +$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin')); +$interbreadcrumb[] = array('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs')); Display::display_header($tool_name); @@ -46,8 +46,8 @@ api_display_tool_title($tool_name); if (isset($_POST['form_sent']) && $_POST['form_sent']) { $form_sent = $_POST['form_sent']; - $userGroups = is_array($_POST['user_group_list']) ? $_POST['user_group_list'] : array() ; - $urlList = is_array($_POST['url_list']) ? $_POST['url_list'] : array() ; + $userGroups = is_array($_POST['user_group_list']) ? $_POST['user_group_list'] : array(); + $urlList = is_array($_POST['url_list']) ? $_POST['url_list'] : array(); $firstLetterUserGroup = $_POST['first_letter_user_group']; if ($form_sent == 1) { diff --git a/main/admin/access_url_check_user_session.php b/main/admin/access_url_check_user_session.php index de09b076a5..9d39cfd0ce 100755 --- a/main/admin/access_url_check_user_session.php +++ b/main/admin/access_url_check_user_session.php @@ -15,18 +15,18 @@ api_protect_admin_script(true); $tool_name = get_lang('SessionOverview'); -$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('PlatformAdmin')); -$interbreadcrumb[]=array('url' => 'session_list.php','name' => get_lang('SessionList')); +$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin')); +$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList')); // Database Table Definitions -$tbl_user = Database::get_main_table(TABLE_MAIN_USER); -$tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER); +$tbl_user = Database::get_main_table(TABLE_MAIN_USER); +$tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER); $table_access_url_user = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER); $url_id = api_get_current_access_url_id(); $action = $_GET['action']; -switch($action) { +switch ($action) { case 'add_user_to_url': $user_id = $_REQUEST['user_id']; $result = UrlManager::add_user_to_url($user_id, $url_id); @@ -58,7 +58,7 @@ if ($show_users_with_problems) { } -foreach($session_list as $session_item) { +foreach ($session_list as $session_item) { $session_id = $session_item['id']; $html .= '

'.$session_item['name'].'

'; diff --git a/main/admin/access_url_edit_courses_to_url.php b/main/admin/access_url_edit_courses_to_url.php index 16b741f519..68c39806ce 100755 --- a/main/admin/access_url_edit_courses_to_url.php +++ b/main/admin/access_url_edit_courses_to_url.php @@ -35,12 +35,12 @@ $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdm $interbreadcrumb[] = array('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs')); $add_type = 'multiple'; -if (isset($_REQUEST['add_type']) && $_REQUEST['add_type']!='') { +if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') { $add_type = Security::remove_XSS($_REQUEST['add_type']); } $access_url_id = 1; -if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id']!='') { +if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id'] != '') { $access_url_id = Security::remove_XSS($_REQUEST['access_url_id']); } @@ -157,23 +157,23 @@ $url_list = UrlManager::get_url_data();  | 

-
> + >
@@ -205,7 +205,7 @@ $url_list = UrlManager::get_url_data(); ?> @@ -258,7 +258,7 @@ $url_list = UrlManager::get_url_data();
'.get_lang('AddCoursesToURL').''; else echo ''; diff --git a/main/admin/archive_cleanup.php b/main/admin/archive_cleanup.php index 4f33d40f08..0df7afd6c9 100755 --- a/main/admin/archive_cleanup.php +++ b/main/admin/archive_cleanup.php @@ -16,7 +16,7 @@ $this_section = SECTION_PLATFORM_ADMIN; api_protect_admin_script(true); // setting breadcrumbs -$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin')); +$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin')); $form = new FormValidator('archive_cleanup_form', 'post', '', '', array(), FormValidator::LAYOUT_BOX); $form->addButtonSend(get_lang('ArchiveDirCleanupProceedButton')); diff --git a/main/admin/configure_extensions.php b/main/admin/configure_extensions.php index ce127f39f1..c746b5ef68 100755 --- a/main/admin/configure_extensions.php +++ b/main/admin/configure_extensions.php @@ -26,7 +26,7 @@ if (isset($_POST['activeExtension'])) { $rs = Database::query($sql); - if (Database::affected_rows($rs)>0){ + if (Database::affected_rows($rs) > 0) { $message = get_lang('ServiceActivated'); } @@ -76,18 +76,18 @@ $sql = 'SELECT variable FROM '.$tbl_settings_current.' WHERE variable LIKE "service_%" AND subkey="active" and selected_value="true"'; $rs = Database::query($sql); -while($row = Database::fetch_array($rs)){ +while ($row = Database::fetch_array($rs)) { $listActiveServices[] = $row['variable']; } // javascript to handle accordion behaviour $javascript_message = ''; -if(!empty($message)){ +if (!empty($message)) { $javascript_message = ' document.getElementById("message").style.display = "block"; var timer = setTimeout(hideMessage, 5000);'; } -$htmlHeadXtra[]= ' diff --git a/main/user/classes.php b/main/user/classes.php index 6261ae19f8..94d8d911e3 100755 --- a/main/user/classes.php +++ b/main/user/classes.php @@ -12,9 +12,9 @@ api_block_anonymous_users(); $this_section = SECTION_COURSES; -$interbreadcrumb[]= array ('url' =>'classes.php','name' => get_lang('Classes')); +$interbreadcrumb[] = array('url' =>'classes.php', 'name' => get_lang('Classes')); if (isset($_GET['id'])) { - $interbreadcrumb[]= array ('url' =>'#','name' => get_lang('Groups')); + $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('Groups')); } Display :: display_header($tool_name, 'Classes'); diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/CourseBuilder.php b/src/Chamilo/CourseBundle/Component/CourseCopy/CourseBuilder.php index da86c8ab61..8b1abd56b8 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/CourseBuilder.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/CourseBuilder.php @@ -569,7 +569,7 @@ class CourseBuilder $table_doc = Database:: get_course_table(TABLE_DOCUMENT); if (!empty($courseId) && !empty($session_id)) { - $session_id = intval($session_id); + $session_id = intval($session_id); if ($with_base_content) { $session_condition = api_get_session_condition( $session_id, diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/CourseSession.php b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/CourseSession.php index 44b8a475c9..56193f30e5 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/CourseSession.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/CourseSession.php @@ -18,7 +18,7 @@ class CourseSession extends Resource * @param int $id * @param string $title */ - public function __construct($id,$title) + public function __construct($id, $title) { parent::__construct($id, RESOURCE_SESSION_COURSE); $this->title = $title; diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Glossary.php b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Glossary.php index d8a97ed963..902e05c1f2 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Glossary.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Glossary.php @@ -22,9 +22,9 @@ class Glossary extends Resource * @param string $description * @param int $display_order */ - public function __construct($id,$name,$description,$display_order) + public function __construct($id, $name, $description, $display_order) { - parent::__construct($id,RESOURCE_GLOSSARY); + parent::__construct($id, RESOURCE_GLOSSARY); $this->glossary_id = $id; $this->name = $name; $this->description = $description; diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Link.php b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Link.php index 3845ba5ab8..cfb4fa9334 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Link.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/Link.php @@ -46,7 +46,7 @@ class Link extends Resource $category_id, $on_homepage ) { - parent::__construct($id,RESOURCE_LINK); + parent::__construct($id, RESOURCE_LINK); $this->title = $title; $this->url = $url; $this->description = $description; diff --git a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/LinkCategory.php b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/LinkCategory.php index 4212e49648..fd6ffca672 100644 --- a/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/LinkCategory.php +++ b/src/Chamilo/CourseBundle/Component/CourseCopy/Resources/LinkCategory.php @@ -29,9 +29,9 @@ class LinkCategory extends Resource * @param string $title * @param string $description */ - public function __construct($id,$title,$description,$display_order) + public function __construct($id, $title, $description, $display_order) { - parent::__construct($id,RESOURCE_LINKCATEGORY); + parent::__construct($id, RESOURCE_LINKCATEGORY); $this->title = $title; $this->description = $description; $this->display_order = $display_order;