From c2bfb16132c1bc2d2350afd9234adb97946bc669 Mon Sep 17 00:00:00 2001 From: Julio Montoya Date: Thu, 27 May 2010 18:37:21 +0200 Subject: [PATCH] Minor - Cosmetic changes --- main/inc/lib/add_course.lib.inc.php | 10 +++----- .../add_courses_to_session_functions.lib.php | 5 ++-- main/inc/lib/array.lib.php | 10 ++++---- main/inc/lib/blog.lib.php | 23 +++++++------------ main/inc/lib/exercise_show_functions.lib.php | 2 -- main/survey/survey.php | 2 +- 6 files changed, 18 insertions(+), 34 deletions(-) mode change 100644 => 100755 main/inc/lib/exercise_show_functions.lib.php diff --git a/main/inc/lib/add_course.lib.inc.php b/main/inc/lib/add_course.lib.inc.php index 51702cb93a..ee3e481c42 100755 --- a/main/inc/lib/add_course.lib.inc.php +++ b/main/inc/lib/add_course.lib.inc.php @@ -5,7 +5,7 @@ * This is the course creation library for Chamilo. * Include/require it in your code for using its functionality. * - * @package dokeos.library + * @package chamilo.library * @todo clean up horrible structure, script is unwieldy, for example easier way to deal with * different tool visibility settings: ALL_TOOLS_INVISIBLE, ALL_TOOLS_VISIBLE, CORE_TOOLS_VISIBLE... */ @@ -14,9 +14,7 @@ require_once api_get_path(LIBRARY_PATH).'database.lib.php'; require_once api_get_path(LIBRARY_PATH).'mail.lib.inc.php'; /* -============================================================================== FUNCTIONS -============================================================================== */ /** @@ -32,12 +30,10 @@ require_once api_get_path(LIBRARY_PATH).'mail.lib.inc.php'; * @param integer Expiration delay in unix timestamp * @return true if the course creation was succesful, false otherwise. */ -function create_course($wanted_code, $title, $tutor_name, $category_code, $course_language, $course_admin_id, $db_prefix, $firstExpirationDelay) -{ +function create_course($wanted_code, $title, $tutor_name, $category_code, $course_language, $course_admin_id, $db_prefix, $firstExpirationDelay) { $keys = define_course_keys($wanted_code, "", $db_prefix); - if(sizeof($keys)) - { + if(sizeof($keys)) { $visual_code = $keys["currentCourseCode"]; $code = $keys["currentCourseId"]; $db_name = $keys["currentCourseDbName"]; diff --git a/main/inc/lib/add_courses_to_session_functions.lib.php b/main/inc/lib/add_courses_to_session_functions.lib.php index 75e2df5b64..dd83cbad5c 100755 --- a/main/inc/lib/add_courses_to_session_functions.lib.php +++ b/main/inc/lib/add_courses_to_session_functions.lib.php @@ -5,10 +5,9 @@ require_once (api_get_path(LIBRARY_PATH).'xajax/xajax.inc.php'); //require_once (api_get_path(SYS_CODE_PATH).'admin/add_courses_to_session.php'); -class AddCourseToSession{ +class AddCourseToSession { - public function search_courses($needle,$type) - { + public function search_courses($needle,$type) { global $tbl_course, $tbl_session_rel_course, $id_session; $xajax_response = new XajaxResponse(); diff --git a/main/inc/lib/array.lib.php b/main/inc/lib/array.lib.php index 833c10bcd9..9f0263cc22 100755 --- a/main/inc/lib/array.lib.php +++ b/main/inc/lib/array.lib.php @@ -1,8 +1,7 @@ * @author Julio Montoya - Cleaning code * @@ -22,15 +22,13 @@ class Blog { * @return String Blog Title */ public static function get_blog_title ($blog_id) { - if(is_numeric($blog_id)) - { + if(is_numeric($blog_id)) { // init $tbl_blogs = Database::get_course_table(TABLE_BLOGS); - $sql = " - SELECT blog_name - FROM " . $tbl_blogs . " - WHERE blog_id = " . Database::escape_string((int)$blog_id); + $sql = "SELECT blog_name + FROM " . $tbl_blogs . " + WHERE blog_id = " . intval($blog_id); $result = Database::query($sql); $blog = Database::fetch_array($result); @@ -50,10 +48,9 @@ class Blog { public static function get_blog_subtitle ($blog_id) { // init $tbl_blogs = Database::get_course_table(TABLE_BLOGS); - $sql = "SELECT blog_subtitle FROM $tbl_blogs WHERE blog_id ='".Database::escape_string((int)$blog_id)."'"; + $sql = "SELECT blog_subtitle FROM $tbl_blogs WHERE blog_id ='".intval($blog_id)."'"; $result = Database::query($sql); $blog = Database::fetch_array($result); - return stripslashes($blog['blog_subtitle']); } @@ -73,8 +70,7 @@ class Blog { $tbl_blogs_rel_user = Database::get_course_table(TABLE_BLOGS_REL_USER); // Get blog members - $sql = " - SELECT + $sql = "SELECT user.user_id, user.firstname, user.lastname @@ -82,11 +78,8 @@ class Blog { INNER JOIN " . $tbl_users . " user ON blogs_rel_user.user_id = user.user_id WHERE blogs_rel_user.blog_id = '" . Database::escape_string((int)$blog_id)."'"; $result = Database::query($sql); - $blog_members = array (); - - while($user = Database::fetch_array($result)) - { + while($user = Database::fetch_array($result)) { $blog_members[$user['user_id']] = api_get_person_name($user['firstname'], $user['lastname']); } diff --git a/main/inc/lib/exercise_show_functions.lib.php b/main/inc/lib/exercise_show_functions.lib.php old mode 100644 new mode 100755 index 17329b8953..8145e23e64 --- a/main/inc/lib/exercise_show_functions.lib.php +++ b/main/inc/lib/exercise_show_functions.lib.php @@ -1,7 +1,6 @@