diff --git a/main/coursecopy/copy_course_session_selected.php b/main/coursecopy/copy_course_session_selected.php index 8a7c90b7c2..908c73c85f 100644 --- a/main/coursecopy/copy_course_session_selected.php +++ b/main/coursecopy/copy_course_session_selected.php @@ -8,10 +8,6 @@ * @author Angel Fernando Quiroz Campos Code conventions * @package chamilo.backup */ -/** - * Code - */ -/* INIT SECTION */ // Language files that need to be included $language_file = array('coursebackup', 'admin'); diff --git a/main/inc/lib/course.lib.php b/main/inc/lib/course.lib.php index acdfe97aec..401fcafc29 100755 --- a/main/inc/lib/course.lib.php +++ b/main/inc/lib/course.lib.php @@ -3585,6 +3585,12 @@ class CourseManager return $html; } + /** + * @param string $main_content + * @param string $sub_content + * @param string $sub_sub_content + * @return string + */ public static function course_item_parent($main_content, $sub_content, $sub_sub_content = null) { return '
'.$main_content.$sub_content.$sub_sub_content.'
'; @@ -3597,7 +3603,7 @@ class CourseManager * in the sense that any user clicking them is registered as a student * @param int User id * @param bool Whether to show the document quick-loader or not - * @return void + * @return string */ public static function display_special_courses($user_id, $load_dirs = false) { @@ -3702,7 +3708,7 @@ class CourseManager * @uses display_courses_in_category() to display the courses themselves * @param int user id * @param bool Whether to show the document quick-loader or not - * @return void + * @return string */ public static function display_courses($user_id, $load_dirs = false) { @@ -3740,7 +3746,7 @@ class CourseManager * class userportal-course-item. * @param int User category id * @param bool Whether to show the document quick-loader or not - * @return void + * @return string */ public static function display_courses_in_category($user_category_id, $load_dirs = false) { @@ -3748,7 +3754,7 @@ class CourseManager // Table definitions $TABLECOURS = Database :: get_main_table(TABLE_MAIN_COURSE); $TABLECOURSUSER = Database :: get_main_table(TABLE_MAIN_COURSE_USER); - $TABLE_ACCESS_URL_REL_COURSE = Database :: get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE); + $TABLE_ACCESS_URL_REL_COURSE = Database :: get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE); $current_url_id = api_get_current_access_url_id(); // Get course list auto-register @@ -3862,18 +3868,23 @@ class CourseManager $course_title_url = ''; if ($course_info['visibility'] != COURSE_VISIBILITY_CLOSED || $course['status'] == COURSEMANAGER) { - $course_title_url = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/?id_session=0'; + $course_title_url = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/index.php?id_session=0'; $course_title = Display::url($course_info['title'], $course_title_url); } else { - $course_title = $course_info['title']." ".Display::tag('span',get_lang('CourseClosed'), array('class'=>'item_closed')); + $course_title = $course_info['title']." ".Display::tag('span', get_lang('CourseClosed'), array('class'=>'item_closed')); } // Start displaying the course block itself if (api_get_setting('display_coursecode_in_courselist') == 'true') { $course_title .= ' ('.$course_info['visual_code'].') '; } + if (api_get_setting('display_teacher_in_courselist') == 'true') { - $teachers = CourseManager::get_teacher_list_from_course_code_to_string($course['code'], self::USER_SEPARATOR, true); + $teachers = CourseManager::get_teacher_list_from_course_code_to_string( + $course['code'], + self::USER_SEPARATOR, + true + ); } $params['link'] = $course_title_url; @@ -3885,11 +3896,11 @@ class CourseManager $params['notifications'] = $show_notification; } - $is_subcontent = true; + $isSubcontent = true; if (empty($user_category_id)) { - $is_subcontent = false; + $isSubcontent = false; } - $html .= self::course_item_html($params, $is_subcontent); + $html .= self::course_item_html($params, $isSubcontent); } return $html; @@ -3914,18 +3925,23 @@ class CourseManager } /** - * Get the course id based on the original id and field name in the extra fields. Returns 0 if course was not found + * Get the course id based on the original id and field name in the extra fields. + * Returns 0 if course was not found * - * @param string Original course code - * @param string Original field name + * @param string $original_course_id_value Original course code + * @param string $original_course_id_name Original field name * @return int Course id */ public static function get_course_id_from_original_id($original_course_id_value, $original_course_id_name) { $t_cfv = Database::get_main_table(TABLE_MAIN_COURSE_FIELD_VALUES); $table_field = Database::get_main_table(TABLE_MAIN_COURSE_FIELD); - $sql_course = "SELECT course_code FROM $table_field cf INNER JOIN $t_cfv cfv ON cfv.field_id=cf.id WHERE field_variable='$original_course_id_name' AND field_value='$original_course_id_value'"; - $res = Database::query($sql_course); + $sql = "SELECT course_code FROM $table_field cf + INNER JOIN $t_cfv cfv ON cfv.field_id=cf.id + WHERE + field_variable='$original_course_id_name' AND + field_value='$original_course_id_value'"; + $res = Database::query($sql); $row = Database::fetch_object($res); if ($row != false) { return $row->course_code; diff --git a/main/inc/lib/formvalidator/FormValidator.class.php b/main/inc/lib/formvalidator/FormValidator.class.php index 23d94aff11..b23df384e8 100755 --- a/main/inc/lib/formvalidator/FormValidator.class.php +++ b/main/inc/lib/formvalidator/FormValidator.class.php @@ -556,12 +556,19 @@ EOT; * @return string $return_value HTML code of the form * * @author Patrick Cool , Ghent University, august 2006 + * @author Julio Montoya */ public function return_form() { $error = false; $addDateLibraries = false; - $dateElementTypes = array('date_range_picker', 'date_time_picker', 'date_picker', 'datepicker', 'datetimepicker'); + $dateElementTypes = array( + 'date_range_picker', + 'date_time_picker', + 'date_picker', + 'datepicker', + 'datetimepicker' + ); /** @var HTML_QuickForm_element $element */ foreach ($this->_elements as $element) { if (in_array($element->getType(), $dateElementTypes)) { diff --git a/main/inc/lib/userportal.lib.php b/main/inc/lib/userportal.lib.php index d42f0a2cf5..84944005a6 100755 --- a/main/inc/lib/userportal.lib.php +++ b/main/inc/lib/userportal.lib.php @@ -1047,7 +1047,7 @@ class IndexManager $date_session_start = $session['date_start']; $date_session_end = $session['date_end']; $days_access_before_beginning = $session['nb_days_access_before_beginning']; - $days_access_after_end = $session['nb_days_access_after_end']; + $days_access_after_end = $session['nb_days_access_after_end']; $session_now = time(); $count_courses_session = 0; @@ -1076,7 +1076,10 @@ class IndexManager } } } - if ($session_now > $allowed_time && $days_access_after_end > $dif_time_after - 1) { + + if ($session_now > $allowed_time && + $days_access_after_end > $dif_time_after - 1 + ) { // Read only and accessible. $atLeastOneCourseIsVisible = true; @@ -1104,7 +1107,12 @@ class IndexManager if ($count_courses_session > 0) { $params = array(); $session_box = Display::get_session_title_box($session_id); - $params['icon'] = Display::return_icon('window_list.png', $session_box['title'], array('id' => 'session_img_'.$session_id), ICON_SIZE_LARGE); + $params['icon'] = Display::return_icon( + 'window_list.png', + $session_box['title'], + array('id' => 'session_img_' . $session_id), + ICON_SIZE_LARGE + ); $extra_info = !empty($session_box['coach']) ? $session_box['coach'] : null; $extra_info .= !empty($session_box['coach']) ? ' - '.$session_box['dates'] : $session_box['dates']; $extra_info .= isset($session_box['duration']) ? ' '.$session_box['duration'] : null; @@ -1113,7 +1121,15 @@ class IndexManager $session_link = $session_box['title']; $params['link'] = null; } else { - $session_link = Display::tag('a', $session_box['title'], array('href'=>api_get_path(WEB_CODE_PATH).'session/index.php?session_id='.$session_id)); + $session_link = Display::tag( + 'a', + $session_box['title'], + array( + 'href' => api_get_path( + WEB_CODE_PATH + ) . 'session/index.php?session_id=' . $session_id + ) + ); $params['link'] = api_get_path(WEB_CODE_PATH).'session/index.php?session_id='.$session_id; } @@ -1130,15 +1146,20 @@ class IndexManager // $params['extra'] .= $html_courses_session; } - $params['description'] = isset($session_box['description']) ? $session_box['description'] : null; + $params['description'] = isset($session_box['description']) ? $session_box['description'] : null; $parentInfo = CourseManager::course_item_html($params, true); - if (isset($_configuration['show_simple_session_info']) && $_configuration['show_simple_session_info']) { + if (isset($_configuration['show_simple_session_info']) && + $_configuration['show_simple_session_info'] + ) { $params['title'] = $session_box['title']; $parentInfo = CourseManager::course_item_html_no_icon($params); } - $sessions_with_no_category .= CourseManager::course_item_parent($parentInfo, $html_courses_session); + $sessions_with_no_category .= CourseManager::course_item_parent( + $parentInfo, + $html_courses_session + ); } } } else {