chamilo / chamilo-lms

main/admin/teachers_time_by_session_report.php

Spacing +15 added lines, -15 removed lines

@@ -12,7 +12,7 @@ 
 $cidReset = true;
 
 require_once '../inc/global.inc.php';
-require_once api_get_path(SYS_CODE_PATH) . 'work/work.lib.php';
+require_once api_get_path(SYS_CODE_PATH).'work/work.lib.php';
 
 if (!api_is_platform_admin(true) && !api_is_teacher()) {
     api_not_allowed(true);
@@ -47,7 +47,7 @@ 
 
     foreach ($sessionCourses as $sessionCourse) {
         $course = $sessionCourse->getCourse();
-        $coursesInfo[$course->getId()] =  $course->getCode();
+        $coursesInfo[$course->getId()] = $course->getCode();
         $criteria = Criteria::create()->where(
             Criteria::expr()->eq("status", Session::COACH)
         );
@@ -70,10 +70,10 @@ 
                 ];
             }
 
-            $usersInfo[$user->getId()][$course->getId() . '_number_of_students'] = null;
-            $usersInfo[$user->getId()][$course->getId() . '_number_of_works'] = null;
-            $usersInfo[$user->getId()][$course->getId() . '_last_work'] = null;
-            $usersInfo[$user->getId()][$course->getId() . '_time_spent_of_course'] = null;
+            $usersInfo[$user->getId()][$course->getId().'_number_of_students'] = null;
+            $usersInfo[$user->getId()][$course->getId().'_number_of_works'] = null;
+            $usersInfo[$user->getId()][$course->getId().'_last_work'] = null;
+            $usersInfo[$user->getId()][$course->getId().'_time_spent_of_course'] = null;
 
             if (!$session->hasCoachInCourseWithStatus($user, $course)) {
                 continue;
@@ -83,9 +83,9 @@ 
                 ->getRepository('ChamiloCourseBundle:CStudentPublication')
                 ->findByTeacher($user, $course, $session->getId());
 
-            $usersInfo[$user->getId()][$course->getId() . '_number_of_students'] = $sessionCourse->getNbrUsers();
-            $usersInfo[$user->getId()][$course->getId() . '_number_of_works'] = count($works);
-            $usersInfo[$user->getId()][$course->getId() . '_time_spent_of_course'] = api_time_to_hms(
+            $usersInfo[$user->getId()][$course->getId().'_number_of_students'] = $sessionCourse->getNbrUsers();
+            $usersInfo[$user->getId()][$course->getId().'_number_of_works'] = count($works);
+            $usersInfo[$user->getId()][$course->getId().'_time_spent_of_course'] = api_time_to_hms(
                 Tracking::get_time_spent_on_the_course($user->getId(), $course->getId(), $session->getId())
             );
 
@@ -97,7 +97,7 @@ 
 
             $lastFormattedDate = api_format_date($lastWork->getSentDate()->getTimestamp(), DATE_TIME_FORMAT_SHORT);
 
-            $usersInfo[$user->getId()][$course->getId() . '_last_work'] = api_format_date(
+            $usersInfo[$user->getId()][$course->getId().'_last_work'] = api_format_date(
                 $lastWork->getSentDate()->getTimestamp(),
                 DATE_TIME_FORMAT_SHORT
             );
@@ -106,7 +106,7 @@ 
 }
 
 if (isset($_GET['export']) && $session && ($coursesInfo && $usersInfo)) {
-    $fileName = get_lang('TeacherTimeReport') . ' ' . api_get_local_time();
+    $fileName = get_lang('TeacherTimeReport').' '.api_get_local_time();
 
     $dataToExport = [];
     $dataToExport[] = [$toolName, $session->getName()];
@@ -167,9 +167,9 @@ 
 }
 
 $this_section = SECTION_PLATFORM_ADMIN;
-$interbreadcrumb[] = ['url' => api_get_path(WEB_CODE_PATH) . 'mySpace/', 'name' => get_lang('Reporting')];
+$interbreadcrumb[] = ['url' => api_get_path(WEB_CODE_PATH).'mySpace/', 'name' => get_lang('Reporting')];
 $interbreadcrumb[] = [
-    'url' => api_get_path(WEB_CODE_PATH) . 'mySpace/session.php',
+    'url' => api_get_path(WEB_CODE_PATH).'mySpace/session.php',
     'name' => get_lang('FollowedSessions')
 ];
 
@@ -179,11 +179,11 @@ 
     $actions = [
         Display::url(
             Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), [], ICON_SIZE_MEDIUM),
-            api_get_self() . '?' . http_build_query(['export' => 'csv', 'session' => $session->getId()])
+            api_get_self().'?'.http_build_query(['export' => 'csv', 'session' => $session->getId()])
         ),
         Display::url(
             Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), [], ICON_SIZE_MEDIUM),
-            api_get_self() . '?' . http_build_query(['export' => 'xls', 'session' => $session->getId()])
+            api_get_self().'?'.http_build_query(['export' => 'xls', 'session' => $session->getId()])
         )
     ];
 }

main/group/group_category.php

Spacing +14 added lines, -14 removed lines

@@ -7,12 +7,12 @@ 
 
 require_once '../inc/global.inc.php';
 $this_section = SECTION_COURSES;
-$current_course_tool  = TOOL_GROUP;
+$current_course_tool = TOOL_GROUP;
 
 // Notice for unauthorized people.
 api_protect_course_script(true);
 
-if (!api_is_allowed_to_edit(false,true) ||
+if (!api_is_allowed_to_edit(false, true) ||
     !(isset ($_GET['id']) ||
     isset ($_POST['id']) ||
     isset ($_GET['action']) ||
@@ -59,7 +59,7 @@ 
             'groups_per_user' => 1,
             'doc_state' => GroupManager::TOOL_PRIVATE,
             'work_state' => GroupManager::TOOL_PRIVATE,
-            'wiki_state' => GroupManager::TOOL_PRIVATE ,
+            'wiki_state' => GroupManager::TOOL_PRIVATE,
             'chat_state' => GroupManager::TOOL_PRIVATE,
             'calendar_state' => GroupManager::TOOL_PRIVATE,
             'announcements_state'=> GroupManager::TOOL_PRIVATE,
@@ -113,7 +113,7 @@ 
 
     // Groups per user
     $possible_values = array();
-    for ($i = 1; $i <= 10; $i ++) {
+    for ($i = 1; $i <= 10; $i++) {
         $possible_values[$i] = $i;
     }
     $possible_values[GroupManager::GROUP_PER_MEMBER_NO_LIMIT] = get_lang('All');
@@ -138,7 +138,7 @@ 
 
     $form->addElement('html', '<div class="col-md-6">');
     // Description
-    $form->addElement('textarea', 'description', get_lang('Description'), array ('rows' => 6));
+    $form->addElement('textarea', 'description', get_lang('Description'), array('rows' => 6));
     $form->addElement('html', '</div>');
     $form->addElement('html', '</div>');
 } else {
@@ -158,7 +158,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('user.png', get_lang('GroupSelfRegistration')) . ' ' . get_lang('GroupSelfRegistration'),
+    Display::return_icon('user.png', get_lang('GroupSelfRegistration')).' '.get_lang('GroupSelfRegistration'),
     null,
     false
 );
@@ -172,7 +172,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('folder.png', get_lang('GroupDocument')) . ' ' . get_lang('GroupDocument'),
+    Display::return_icon('folder.png', get_lang('GroupDocument')).' '.get_lang('GroupDocument'),
     null,
     false
 );
@@ -186,7 +186,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('work.png', get_lang('GroupWork'), array(), ICON_SIZE_SMALL) . ' ' . get_lang('GroupWork'),
+    Display::return_icon('work.png', get_lang('GroupWork'), array(), ICON_SIZE_SMALL).' '.get_lang('GroupWork'),
     '',
     false);
 
@@ -199,7 +199,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('agenda.png', get_lang('GroupCalendar')) . ' ' . get_lang('GroupCalendar'),
+    Display::return_icon('agenda.png', get_lang('GroupCalendar')).' '.get_lang('GroupCalendar'),
     null,
     false
 );
@@ -216,7 +216,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('announce.png', get_lang('GroupAnnouncements')) . ' ' . get_lang('GroupAnnouncements'),
+    Display::return_icon('announce.png', get_lang('GroupAnnouncements')).' '.get_lang('GroupAnnouncements'),
     null,
     false
 );
@@ -230,7 +230,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('forum.png', get_lang('GroupForum')) . ' ' . get_lang('GroupForum'),
+    Display::return_icon('forum.png', get_lang('GroupForum')).' '.get_lang('GroupForum'),
     null,
     false
 );
@@ -244,7 +244,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('wiki.png', get_lang('GroupWiki')) . ' ' . get_lang('GroupWiki'),
+    Display::return_icon('wiki.png', get_lang('GroupWiki')).' '.get_lang('GroupWiki'),
     null,
     false
 );
@@ -258,7 +258,7 @@ 
 $form->addGroup(
     $group,
     '',
-    Display::return_icon('chat.png', get_lang('Chat')) . ' ' . get_lang('Chat'),
+    Display::return_icon('chat.png', get_lang('Chat')).' '.get_lang('Chat'),
     null,
     false
 );
@@ -333,7 +333,7 @@ 
 // actions bar
 echo '<div class="actions">';
 echo '<a href="group.php">'.
-    Display::return_icon('back.png', get_lang('BackToGroupList'),'',ICON_SIZE_MEDIUM).'</a>';
+    Display::return_icon('back.png', get_lang('BackToGroupList'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 $defaults = $category;

Indentation +10 added lines, -10 removed lines

@@ -18,7 +18,7 @@ 
     isset ($_GET['action']) ||
     isset ($_POST['action']))
 ) {
-	api_not_allowed();
+    api_not_allowed();
 }
 
 /**
@@ -86,30 +86,30 @@ 
 
 // Build the form
 if (isset($_GET['id'])) {
-	// Update settings of existing category
-	$action = 'update_settings';
+    // Update settings of existing category
+    $action = 'update_settings';
     $form = new FormValidator(
         'group_category',
         'post',
         api_get_self().'?id='.$category['id'].'&'.api_get_cidreq()
     );
-	$form->addElement('hidden', 'id');
+    $form->addElement('hidden', 'id');
 } else {
     // Checks if the field was created in the table Category. It creates it if is neccesary
     $table_category = Database :: get_course_table(TABLE_GROUP_CATEGORY);
     if (!Database::query("SELECT wiki_state FROM $table_category WHERE c_id = $course_id")) {
         Database::query("ALTER TABLE $table_category ADD wiki_state tinyint(3) UNSIGNED NOT NULL default '1' WHERE c_id = $course_id");
     }
-	// Create a new category
-	$action = 'add_category';
-	$form = new FormValidator('group_category');
+    // Create a new category
+    $action = 'add_category';
+    $form = new FormValidator('group_category');
 }
 
 // If categories allowed, show title & description field
 if (api_get_setting('allow_group_categories') == 'true') {
     $form->addElement('header', $nameTools);
     $form->addElement('html', '<div class="row"><div class="col-md-6">');
-	$form->addText('title', get_lang('Title'));
+    $form->addText('title', get_lang('Title'));
 
     // Groups per user
     $possible_values = array();
@@ -142,8 +142,8 @@ 
     $form->addElement('html', '</div>');
     $form->addElement('html', '</div>');
 } else {
-	$form->addElement('hidden', 'title');
-	$form->addElement('hidden', 'description');
+    $form->addElement('hidden', 'title');
+    $form->addElement('hidden', 'description');
 }
 
 $form->addElement('header', get_lang('DefaultSettingsForNewGroups'));

main/upload/upload.document.php

Indentation +21 added lines, -21 removed lines

@@ -47,9 +47,9 @@ 
  */
 //user has submitted a file
 if (isset($_FILES['user_upload'])) {
-	$upload_ok = process_uploaded_file($_FILES['user_upload']);
-	if ($upload_ok) {
-		//file got on the server without problems, now process it
+    $upload_ok = process_uploaded_file($_FILES['user_upload']);
+    if ($upload_ok) {
+        //file got on the server without problems, now process it
         $new_path = handle_uploaded_document(
             $_course,
             $_FILES['user_upload'],
@@ -61,21 +61,21 @@ 
             $_POST['unzip'],
             $_POST['if_exists']
         );
-    	$new_comment = isset($_POST['comment']) ? Database::escape_string(trim($_POST['comment'])) : '';
-    	$new_title = isset($_POST['title']) ? Database::escape_string(trim($_POST['title'])) : '';
+        $new_comment = isset($_POST['comment']) ? Database::escape_string(trim($_POST['comment'])) : '';
+        $new_title = isset($_POST['title']) ? Database::escape_string(trim($_POST['title'])) : '';
 
-    	if ($new_path && ($new_comment || $new_title))
-    	if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
-        	$table_document = Database::get_course_table(TABLE_DOCUMENT);
-        	$ct = '';
+        if ($new_path && ($new_comment || $new_title))
+        if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
+            $table_document = Database::get_course_table(TABLE_DOCUMENT);
+            $ct = '';
             if ($new_comment) {
                 $ct .= ", comment='$new_comment'";
             }
             if ($new_title) {
                 $ct .= ", title='$new_title'";
             }
-        	Database::query("UPDATE $table_document SET" . substr($ct, 1) ." WHERE id = '$docid'");
-    	}
+            Database::query("UPDATE $table_document SET" . substr($ct, 1) ." WHERE id = '$docid'");
+        }
         //check for missing images in html files
         $missing_files = check_for_missing_files($base_work_dir.$_POST['curdirpath'].$new_path);
         if ($missing_files) {
@@ -133,7 +133,7 @@ 
 //they want to create a directory
 if (isset($_POST['create_dir']) && $_POST['dirname']!='') {
     $added_slash = $path == '/' ? '' : '/';
-	$dir_name = $path.$added_slash.api_replace_dangerous_char($_POST['dirname']);
+    $dir_name = $path.$added_slash.api_replace_dangerous_char($_POST['dirname']);
     $created_dir = create_unexisting_directory(
         $_course,
         api_get_user_id(),
@@ -153,15 +153,15 @@ 
 }
 
 if (isset($_GET['createdir'])) {
-	//create the form that asks for the directory name
-	$new_folder_text = '<form action="'.api_get_self().'" method="POST">';
-	$new_folder_text .= '<input type="hidden" name="curdirpath" value="'.$path.'"/>';
-	$new_folder_text .= get_lang('NewDir') .' ';
-	$new_folder_text .= '<input type="text" name="dirname"/>';
-	$new_folder_text .= '<input type="submit" name="create_dir" value="'.get_lang('Ok').'"/>';
-	$new_folder_text .= '</form>';
-	//show the form
-	Display::display_normal_message($new_folder_text);
+    //create the form that asks for the directory name
+    $new_folder_text = '<form action="'.api_get_self().'" method="POST">';
+    $new_folder_text .= '<input type="hidden" name="curdirpath" value="'.$path.'"/>';
+    $new_folder_text .= get_lang('NewDir') .' ';
+    $new_folder_text .= '<input type="text" name="dirname"/>';
+    $new_folder_text .= '<input type="submit" name="create_dir" value="'.get_lang('Ok').'"/>';
+    $new_folder_text .= '</form>';
+    //show the form
+    Display::display_normal_message($new_folder_text);
 } else {	//give them a link to create a directory
 ?>
 	<p>

Spacing +18 added lines, -18 removed lines

@@ -11,9 +11,9 @@ 
  */
 
 $_course = api_get_course_info();
-$courseDir = $_course['path'] . "/document";
+$courseDir = $_course['path']."/document";
 $sys_course_path = api_get_path(SYS_COURSE_PATH);
-$base_work_dir = $sys_course_path . $courseDir;
+$base_work_dir = $sys_course_path.$courseDir;
 $noPHP_SELF = true;
 $max_filled_space = DocumentManager::get_course_quota();
 
@@ -35,12 +35,12 @@ 
  */
 $nameTools = get_lang('UplUploadDocument');
 $interbreadcrumb[] = array(
-    "url" => api_get_path(WEB_CODE_PATH)."document/document.php?curdirpath=" . urlencode($path) . '&'.api_get_cidreq(),
+    "url" => api_get_path(WEB_CODE_PATH)."document/document.php?curdirpath=".urlencode($path).'&'.api_get_cidreq(),
     "name" => $langDocuments
 );
 Display::display_header($nameTools, "Doc");
 //show the title
-api_display_tool_title($nameTools . $add_group_to_title);
+api_display_tool_title($nameTools.$add_group_to_title);
 
 /**
  * Process
@@ -74,7 +74,7 @@ 
             if ($new_title) {
                 $ct .= ", title='$new_title'";
             }
-        	Database::query("UPDATE $table_document SET" . substr($ct, 1) ." WHERE id = '$docid'");
+        	Database::query("UPDATE $table_document SET".substr($ct, 1)." WHERE id = '$docid'");
     	}
         //check for missing images in html files
         $missing_files = check_for_missing_files($base_work_dir.$_POST['curdirpath'].$new_path);
@@ -124,14 +124,14 @@ 
         replace_img_path_in_html_file(
             $_POST['img_file_path'],
             $paths_to_replace_in_file,
-            $base_work_dir . $_POST['related_file']
+            $base_work_dir.$_POST['related_file']
         );
         //update parent folders
         item_property_update_on_folder($_course, $_POST['curdirpath'], $_user['user_id']);
     }
 }
 //they want to create a directory
-if (isset($_POST['create_dir']) && $_POST['dirname']!='') {
+if (isset($_POST['create_dir']) && $_POST['dirname'] != '') {
     $added_slash = $path == '/' ? '' : '/';
 	$dir_name = $path.$added_slash.api_replace_dangerous_char($_POST['dirname']);
     $created_dir = create_unexisting_directory(
@@ -156,7 +156,7 @@ 
 	//create the form that asks for the directory name
 	$new_folder_text = '<form action="'.api_get_self().'" method="POST">';
 	$new_folder_text .= '<input type="hidden" name="curdirpath" value="'.$path.'"/>';
-	$new_folder_text .= get_lang('NewDir') .' ';
+	$new_folder_text .= get_lang('NewDir').' ';
 	$new_folder_text .= '<input type="text" name="dirname"/>';
 	$new_folder_text .= '<input type="submit" name="create_dir" value="'.get_lang('Ok').'"/>';
 	$new_folder_text .= '</form>';
@@ -167,7 +167,7 @@ 
 	<p>
         <a href="<?php echo api_get_self(); ?>?path=<?php echo $path; ?>&amp;createdir=1">
             <?php echo Display::return_icon('new_folder.gif'); ?>
-            <?php echo(get_lang('CreateDir'));?>
+            <?php echo(get_lang('CreateDir')); ?>
         </a>
     </p>
 <?php
@@ -190,11 +190,11 @@ 
 </td>
 </tr>
 <tr>
-<td><?php echo get_lang('Title');?></td>
+<td><?php echo get_lang('Title'); ?></td>
 <td><input type="text" size="20" name="title" style="width:300px;"></td>
 </tr>
 <tr>
-<td valign="top"><?php echo get_lang('Comment');?></td>
+<td valign="top"><?php echo get_lang('Comment'); ?></td>
 <td><textarea rows="3" cols="20" name="comment" wrap="virtual" style="width:300px;"></textarea></td>
 </tr>
 <tr>
@@ -202,21 +202,21 @@ 
 <?php echo get_lang('Options'); ?>
 </td>
 <td>
-- <input type="checkbox" name="unzip" value="1" onclick="check_unzip()"/> <?php echo(get_lang('Uncompress'));?><br/>
-- <?php echo (get_lang('UplWhatIfFileExists'));?><br/>
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="nothing" title="<?php echo (get_lang('UplDoNothingLong'));?>" checked="checked"/>  <?php echo (get_lang('UplDoNothing'));?><br/>
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="overwrite" title="<?php echo (get_lang('UplOverwriteLong'));?>"/> <?php echo (get_lang('UplOverwrite'));?><br/>
-&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="rename" title="<?php echo (get_lang('UplRenameLong'));?>"/> <?php echo (get_lang('UplRename'));?>
+- <input type="checkbox" name="unzip" value="1" onclick="check_unzip()"/> <?php echo(get_lang('Uncompress')); ?><br/>
+- <?php echo (get_lang('UplWhatIfFileExists')); ?><br/>
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="nothing" title="<?php echo (get_lang('UplDoNothingLong')); ?>" checked="checked"/>  <?php echo (get_lang('UplDoNothing')); ?><br/>
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="overwrite" title="<?php echo (get_lang('UplOverwriteLong')); ?>"/> <?php echo (get_lang('UplOverwrite')); ?><br/>
+&nbsp;&nbsp;&nbsp;<input type="radio" name="if_exists" value="rename" title="<?php echo (get_lang('UplRenameLong')); ?>"/> <?php echo (get_lang('UplRename')); ?>
 </td>
 </tr>
 </table>
 
-<input type="submit" value="<?php echo(get_lang('Ok'));?>">
+<input type="submit" value="<?php echo(get_lang('Ok')); ?>">
 </form>
 <!-- end upload form -->
 
  <!-- so they can get back to the documents   -->
- <p><?php echo (get_lang('Back'));?> <?php echo (get_lang('To'));?> <a href="document.php?curdirpath=<?php echo $path; ?>"><?php echo (get_lang('DocumentsOverview'));?></a></p>
+ <p><?php echo (get_lang('Back')); ?> <?php echo (get_lang('To')); ?> <a href="document.php?curdirpath=<?php echo $path; ?>"><?php echo (get_lang('DocumentsOverview')); ?></a></p>
 <?php
 
 Display::display_footer();

Braces +3 added lines, -2 removed lines

@@ -64,9 +64,10 @@
     	$new_comment = isset($_POST['comment']) ? Database::escape_string(trim($_POST['comment'])) : '';
     	$new_title = isset($_POST['title']) ? Database::escape_string(trim($_POST['title'])) : '';
 
-    	if ($new_path && ($new_comment || $new_title))
-    	if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
+    	if ($new_path && ($new_comment || $new_title)) {
+    	    	if (($docid = DocumentManager::get_document_id($_course, $new_path))) {
         	$table_document = Database::get_course_table(TABLE_DOCUMENT);
+    	}
         	$ct = '';
             if ($new_comment) {
                 $ct .= ", comment='$new_comment'";

main/exercise/adminhp.php

Spacing +15 added lines, -15 removed lines

@@ -7,12 +7,12 @@ 
 */
 require_once '../inc/global.inc.php';
 
-$this_section=SECTION_COURSES;
+$this_section = SECTION_COURSES;
 
 $_course = api_get_course_info();
 
 if (isset($_REQUEST["cancel"])) {
-    if ($_REQUEST["cancel"]==get_lang('Cancel')) {
+    if ($_REQUEST["cancel"] == get_lang('Cancel')) {
         header("Location: exercise.php");
         exit;
     }
@@ -20,16 +20,16 @@ 
 
 $newName = !empty($_REQUEST['newName']) ? $_REQUEST['newName'] : '';
 $hotpotatoesName = !empty($_REQUEST['hotpotatoesName']) ? Security::remove_XSS($_REQUEST['hotpotatoesName']) : '';
-$is_allowedToEdit=api_is_allowed_to_edit(null,true);
+$is_allowedToEdit = api_is_allowed_to_edit(null, true);
 
 // document path
-$documentPath=api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
+$documentPath = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
 
 // picture path
-$picturePath=$documentPath.'/images';
+$picturePath = $documentPath.'/images';
 
 // audio path
-$audioPath=$documentPath.'/audio';
+$audioPath = $documentPath.'/audio';
 
 // Database table definitions
 if (!$is_allowedToEdit) {
@@ -37,21 +37,21 @@ 
 }
 
 if (isset($_SESSION['gradebook'])) {
-    $gradebook=	$_SESSION['gradebook'];
+    $gradebook = $_SESSION['gradebook'];
 }
 
-if (!empty($gradebook) && $gradebook=='view') {
-    $interbreadcrumb[]= array (
+if (!empty($gradebook) && $gradebook == 'view') {
+    $interbreadcrumb[] = array(
         'url' => '../gradebook/'.$_SESSION['gradebook_dest'],
         'name' => get_lang('ToolGradebook'),
     );
 }
 
-$interbreadcrumb[]=array("url" => "exercise.php","name" => get_lang('Exercises'));
+$interbreadcrumb[] = array("url" => "exercise.php", "name" => get_lang('Exercises'));
 
 $nameTools = get_lang('adminHP');
 
-Display::display_header($nameTools,"Exercise");
+Display::display_header($nameTools, "Exercise");
 
 /** @todo probably wrong !!!! */
 require_once api_get_path(SYS_CODE_PATH).'/exercise/hotpotatoes.lib.php';
@@ -64,9 +64,9 @@ 
 
 <?php
 if (isset($newName)) {
-    if ($newName!="") {
+    if ($newName != "") {
         //alter database record for that test
-        SetComment($hotpotatoesName,$newName);
+        SetComment($hotpotatoesName, $newName);
         echo "<script language='Javascript' type='text/javascript'> window.location='exercise.php'; </script>";
     }
 }
@@ -78,10 +78,10 @@ 
 $lstrComment = '';
 $lstrComment = GetComment($hotpotatoesName);
 if ($lstrComment == '') {
-    $lstrComment = GetQuizName($hotpotatoesName,$documentPath);
+    $lstrComment = GetQuizName($hotpotatoesName, $documentPath);
 }
 if ($lstrComment == '') {
-    $lstrComment = basename($hotpotatoesName,$documentPath);
+    $lstrComment = basename($hotpotatoesName, $documentPath);
 }
 
 echo $lstrComment;

main/exercise/hotspot.class.php

Indentation +98 added lines, -98 removed lines

@@ -12,31 +12,31 @@ 
  **/
 class HotSpot extends Question
 {
-	public static $typePicture = 'hotspot.png';
-	public static $explanationLangVar = 'HotSpot';
+    public static $typePicture = 'hotspot.png';
+    public static $explanationLangVar = 'HotSpot';
 
     /**
      * HotSpot constructor.
      */
-	public function __construct()
-	{
-		parent::__construct();
-		$this->type = HOT_SPOT;
-	}
-
-	public function display()
-	{
-	}
-
-	/**
-	 * @param FormValidator $form
-	 * @param int $fck_config
-	 */
-	public function createForm (&$form, $fck_config=0)
-	{
-		parent::createForm($form, $fck_config);
-
-		if (!isset($_GET['editQuestion'])) {
+    public function __construct()
+    {
+        parent::__construct();
+        $this->type = HOT_SPOT;
+    }
+
+    public function display()
+    {
+    }
+
+    /**
+     * @param FormValidator $form
+     * @param int $fck_config
+     */
+    public function createForm (&$form, $fck_config=0)
+    {
+        parent::createForm($form, $fck_config);
+
+        if (!isset($_GET['editQuestion'])) {
             $form->addElement(
                 'file',
                 'imageUpload',
@@ -46,54 +46,54 @@ 
                 )
             );
 
-			// setting the save button here and not in the question class.php
-			// Saving a question
-			$form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
-			//$form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
-			$form->addRule('imageUpload', get_lang('OnlyImagesAllowed'), 'filetype', array ('jpg', 'jpeg', 'png', 'gif'));
-			$form->addRule('imageUpload', get_lang('NoImage'), 'uploadedfile');
-		} else {
-			// setting the save button here and not in the question class.php
-			// Editing a question
-			$form->addButtonUpdate(get_lang('ModifyExercise'), 'submitQuestion');
-		}
-	}
-
-	/**
-	 * @param FormValidator $form
-	 * @param null $objExercise
-	 * @return null|false
-	 */
-	public function processCreation($form, $objExercise = null)
-	{
-		$file_info = $form->getSubmitValue('imageUpload');
-		$_course = api_get_course_info();
-		parent::processCreation($form, $objExercise);
-
-		if(!empty($file_info['tmp_name'])) {
-			$this->uploadPicture($file_info['tmp_name'], $file_info['name']);
-			$documentPath  = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
-			$picturePath   = $documentPath.'/images';
-
-			// fixed width ang height
-			if (file_exists($picturePath.'/'.$this->picture)) {
-				$this->resizePicture('width', 800);
-				$this->save();
-			} else {
-				return false;
-			}
-		}
-	}
-
-	function createAnswersForm($form)
-	{
-		// nothing
-	}
-
-	function processAnswersCreation($form)
-	{
-		// nothing
-	}
+            // setting the save button here and not in the question class.php
+            // Saving a question
+            $form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
+            //$form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
+            $form->addRule('imageUpload', get_lang('OnlyImagesAllowed'), 'filetype', array ('jpg', 'jpeg', 'png', 'gif'));
+            $form->addRule('imageUpload', get_lang('NoImage'), 'uploadedfile');
+        } else {
+            // setting the save button here and not in the question class.php
+            // Editing a question
+            $form->addButtonUpdate(get_lang('ModifyExercise'), 'submitQuestion');
+        }
+    }
+
+    /**
+     * @param FormValidator $form
+     * @param null $objExercise
+     * @return null|false
+     */
+    public function processCreation($form, $objExercise = null)
+    {
+        $file_info = $form->getSubmitValue('imageUpload');
+        $_course = api_get_course_info();
+        parent::processCreation($form, $objExercise);
+
+        if(!empty($file_info['tmp_name'])) {
+            $this->uploadPicture($file_info['tmp_name'], $file_info['name']);
+            $documentPath  = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
+            $picturePath   = $documentPath.'/images';
+
+            // fixed width ang height
+            if (file_exists($picturePath.'/'.$this->picture)) {
+                $this->resizePicture('width', 800);
+                $this->save();
+            } else {
+                return false;
+            }
+        }
+    }
+
+    function createAnswersForm($form)
+    {
+        // nothing
+    }
+
+    function processAnswersCreation($form)
+    {
+        // nothing
+    }
 }
 
 /**
@@ -101,38 +101,38 @@ 
  */
 class HotSpotDelineation extends HotSpot
 {
-	public static $typePicture = 'hotspot-delineation.png';
-	public static $explanationLangVar = 'HotspotDelineation';
+    public static $typePicture = 'hotspot-delineation.png';
+    public static $explanationLangVar = 'HotspotDelineation';
 
     /**
      * HotSpotDelineation constructor.
      */
-	public function __construct()
-	{
-		parent::__construct();
-		$this -> type = HOT_SPOT_DELINEATION;
-
-	}
-
-	function createForm(&$form, $fck_config=0)
-	{
-		parent::createForm ($form, $fck_config);
-	}
-
-	function processCreation ($form, $objExercise = null)
-	{
-		$file_info = $form -> getSubmitValue('imageUpload');
-		parent::processCreation ($form, $objExercise);
-	}
-
-	function createAnswersForm ($form)
-	{
-		parent::createAnswersForm ($form);
-	}
-
-	function processAnswersCreation ($form)
-	{
-		parent::processAnswersCreation ($form);
-	}
+    public function __construct()
+    {
+        parent::__construct();
+        $this -> type = HOT_SPOT_DELINEATION;
+
+    }
+
+    function createForm(&$form, $fck_config=0)
+    {
+        parent::createForm ($form, $fck_config);
+    }
+
+    function processCreation ($form, $objExercise = null)
+    {
+        $file_info = $form -> getSubmitValue('imageUpload');
+        parent::processCreation ($form, $objExercise);
+    }
+
+    function createAnswersForm ($form)
+    {
+        parent::createAnswersForm ($form);
+    }
+
+    function processAnswersCreation ($form)
+    {
+        parent::processAnswersCreation ($form);
+    }
 }
 

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
 	 * @param FormValidator $form
 	 * @param int $fck_config
 	 */
-	public function createForm (&$form, $fck_config=0)
+	public function createForm(&$form, $fck_config = 0)
 	{
 		parent::createForm($form, $fck_config);
 
@@ -50,7 +50,7 @@ 
 			// Saving a question
 			$form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
 			//$form->addButtonSave(get_lang('GoToQuestion'), 'submitQuestion');
-			$form->addRule('imageUpload', get_lang('OnlyImagesAllowed'), 'filetype', array ('jpg', 'jpeg', 'png', 'gif'));
+			$form->addRule('imageUpload', get_lang('OnlyImagesAllowed'), 'filetype', array('jpg', 'jpeg', 'png', 'gif'));
 			$form->addRule('imageUpload', get_lang('NoImage'), 'uploadedfile');
 		} else {
 			// setting the save button here and not in the question class.php
@@ -70,7 +70,7 @@ 
 		$_course = api_get_course_info();
 		parent::processCreation($form, $objExercise);
 
-		if(!empty($file_info['tmp_name'])) {
+		if (!empty($file_info['tmp_name'])) {
 			$this->uploadPicture($file_info['tmp_name'], $file_info['name']);
 			$documentPath  = api_get_path(SYS_COURSE_PATH).$_course['path'].'/document';
 			$picturePath   = $documentPath.'/images';
@@ -114,25 +114,25 @@ 
 
 	}
 
-	function createForm(&$form, $fck_config=0)
+	function createForm(&$form, $fck_config = 0)
 	{
-		parent::createForm ($form, $fck_config);
+		parent::createForm($form, $fck_config);
 	}
 
-	function processCreation ($form, $objExercise = null)
+	function processCreation($form, $objExercise = null)
 	{
 		$file_info = $form -> getSubmitValue('imageUpload');
-		parent::processCreation ($form, $objExercise);
+		parent::processCreation($form, $objExercise);
 	}
 
-	function createAnswersForm ($form)
+	function createAnswersForm($form)
 	{
-		parent::createAnswersForm ($form);
+		parent::createAnswersForm($form);
 	}
 
-	function processAnswersCreation ($form)
+	function processAnswersCreation($form)
 	{
-		parent::processAnswersCreation ($form);
+		parent::processAnswersCreation($form);
 	}
 }
 

main/inc/lib/blog.lib.php

Doc Comments +1 added lines, -1 removed lines

@@ -1103,7 +1103,7 @@
      * @param String $type
      * @param Integer $blog_id
      * @param Integer $item_id
-     * @return array
+     * @return double
      */
     public static function display_rating($type, $blog_id, $item_id)
     {

Braces +26 added lines, -18 removed lines

@@ -1007,8 +1007,9 @@ 
             $blog_post_actions .= '</a>';
         }
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_rate'))
-            $rating_select = Blog::display_rating_form('post',$blog_id,$post_id);
+        if (api_is_allowed('BLOG_' . $blog_id, 'article_rate')) {
+                    $rating_select = Blog::display_rating_form('post',$blog_id,$post_id);
+        }
 
         $blog_post_text=stripslashes($blog_post_text);
 
@@ -1652,8 +1653,9 @@ 
 
                         $arrPermissions = array();
 
-                        while ($row = Database::fetch_array($result))
-                            $arrPermissions[] = $row['action'];
+                        while ($row = Database::fetch_array($result)) {
+                                                    $arrPermissions[] = $row['action'];
+                        }
 
                             echo '<tr>';
                             echo '<td style="text-align:right; vertical-align:top;">' . get_lang('TaskManager') . ':&nbsp;&nbsp;</td>';
@@ -2329,8 +2331,9 @@ 
         //Handle leap year
         $numberofdays = array (0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
 
-        if(($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0))
-            $numberofdays[2] = 29;
+        if(($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0)) {
+                    $numberofdays[2] = 29;
+        }
 
         //Get the first day of the month
         $dayone = getdate(mktime(0, 0, 0, $month, 1, $year));
@@ -2360,8 +2363,9 @@ 
         if( Database::num_rows($result) > 0) {
             while($blog_post = Database::fetch_array($result)) {
                 // If the day of this post is not yet in the array, add it.
-                if (!in_array($blog_post['post_day'], $posts))
-                    $posts[] = $blog_post['post_day'];
+                if (!in_array($blog_post['post_day'], $posts)) {
+                                    $posts[] = $blog_post['post_day'];
+                }
             }
         }
 
@@ -2400,8 +2404,9 @@ 
 
         echo "<tr>";
 
-        for($ii = 1; $ii < 8; $ii ++)
-            echo "<td class=\"weekdays\">", $DaysShort[$ii % 7], "</td>";
+        for($ii = 1; $ii < 8; $ii ++) {
+                    echo "<td class=\"weekdays\">", $DaysShort[$ii % 7], "</td>";
+        }
 
         echo "</tr>";
 
@@ -2411,8 +2416,9 @@ 
         while ($curday <= $numberofdays[$month]) {
             echo "<tr>";
             for ($ii = 0; $ii < 7; $ii ++) {
-                if (($curday == -1) && ($ii == $startdayofweek))
-                    $curday = 1;
+                if (($curday == -1) && ($ii == $startdayofweek)) {
+                                    $curday = 1;
+                }
 
                 if (($curday > 0) && ($curday <= $numberofdays[$month])) {
                     $bgcolor = $ii < 5 ? $class="class=\"days_week\"" : $class="class=\"days_weekend\"";
@@ -2426,10 +2432,11 @@ 
                     echo "<td " . $class.">";
 
                     // If there are posts on this day, create a filter link.
-                    if(in_array($curday, $posts))
-                        echo '<a href="blog.php?blog_id=' . $blog_id . '&filter=' . $year . '-' . $month . '-' . $curday . '&month=' . $month . '&year=' . $year . '" title="' . get_lang('ViewPostsOfThisDay') . '">' . $curday . '</a>';
-                    else
-                        echo $dayheader;
+                    if(in_array($curday, $posts)) {
+                                            echo '<a href="blog.php?blog_id=' . $blog_id . '&filter=' . $year . '-' . $month . '-' . $curday . '&month=' . $month . '&year=' . $year . '" title="' . get_lang('ViewPostsOfThisDay') . '">' . $curday . '</a>';
+                    } else {
+                                            echo $dayheader;
+                    }
 
                     if (count($tasks) > 0) {
                         if (isset($tasks[$curday]) && is_array($tasks[$curday])) {
@@ -2444,8 +2451,9 @@ 
 
                     echo "</td>";
                     $curday ++;
-                } else
-                    echo "<td>&nbsp;</td>";
+                } else {
+                                    echo "<td>&nbsp;</td>";
+                }
             }
             echo "</tr>";
         }

Indentation +92 added lines, -93 removed lines

@@ -6,7 +6,6 @@ 
  *
  * Contains several functions dealing with displaying,
  * editing,... of a blog
-
  * @package chamilo.blogs
  * @author Toon Keppens <toon@vi-host.net>
  * @author Julio Montoya - Cleaning code
@@ -335,7 +334,7 @@ 
                     // Storing the attachments if any
                     if ($result) {
                         $sql = 'INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size, blog_id,comment_id) '.
-                               "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$last_post_id."', '".intval($_FILES['user_upload']['size'])."',  '".$blog_id."', '0' )";
+                                "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$last_post_id."', '".intval($_FILES['user_upload']['size'])."',  '".$blog_id."', '0' )";
                         Database::query($sql);
                         $id = Database::insert_id();
                         if ($id) {
@@ -479,7 +478,7 @@ 
                     // Storing the attachments if any
                     if ($result) {
                         $sql='INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size,blog_id,comment_id) '.
-                             "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$post_id."', '".$_FILES['user_upload']['size']."',  '".$blog_id."', '".$last_id."'  )";
+                                "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$post_id."', '".$_FILES['user_upload']['size']."',  '".$blog_id."', '".$last_id."'  )";
                         Database::query($sql);
 
                         $id = Database::insert_id();
@@ -1398,10 +1397,10 @@ 
             echo '<span class="blogpost_title">' . get_lang('TaskList') . '</span><br />';
             echo "<table class=\"data_table\">";
             echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                     "<th width='240'><b>",get_lang('Title'),"</b></th>",
-                     "<th><b>",get_lang('Description'),"</b></th>",
-                     "<th><b>",get_lang('Color'),"</b></th>",
-                     "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                        "<th width='240'><b>",get_lang('Title'),"</b></th>",
+                        "<th><b>",get_lang('Description'),"</b></th>",
+                        "<th><b>",get_lang('Color'),"</b></th>",
+                        "<th width='50'><b>",get_lang('Modify'),"</b></th>",
                 "</tr>";
 
 
@@ -1433,14 +1432,14 @@ 
                 echo '<td width="50">';
                 echo '<a href="'.api_get_self().'?action=manage_tasks&blog_id='.$task['blog_id'].'&do=edit&task_id='.$task['task_id'].'">';
                 echo Display::return_icon('edit.png', get_lang('EditTask'));
-                      echo "</a>";
-                      echo '<a href="'.$delete_link.'"';
-                      echo $delete_confirm;
-                       echo '>';
+                        echo "</a>";
+                        echo '<a href="'.$delete_link.'"';
+                        echo $delete_confirm;
+                        echo '>';
                         echo Display::return_icon($delete_icon, $delete_title);
-                       echo "</a>";
-                     echo '</td>';
-                   echo '</tr>';
+                        echo "</a>";
+                        echo '</td>';
+                    echo '</tr>';
             }
             echo "</table>";
         }
@@ -1464,11 +1463,11 @@ 
         echo '<span class="blogpost_title">' . get_lang('AssignedTasks') . '</span><br />';
         echo "<table class=\"data_table\">";
         echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                 "<th width='240'><b>",get_lang('Member'),"</b></th>",
-                 "<th><b>",get_lang('Task'),"</b></th>",
-                 "<th><b>",get_lang('Description'),"</b></th>",
-                 "<th><b>",get_lang('TargetDate'),"</b></th>",
-                 "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                    "<th width='240'><b>",get_lang('Member'),"</b></th>",
+                    "<th><b>",get_lang('Task'),"</b></th>",
+                    "<th><b>",get_lang('Description'),"</b></th>",
+                    "<th><b>",get_lang('TargetDate'),"</b></th>",
+                    "<th width='50'><b>",get_lang('Modify'),"</b></th>",
             "</tr>";
 
         $course_id = api_get_course_int_id();
@@ -2316,7 +2315,7 @@ 
      * @param Integer $year: the 4-digit year indication e.g. 2005
      *
      * @return html code
-    */
+     */
     public static function display_minimonthcalendar($month, $year, $blog_id)
     {
         // Init
@@ -2609,34 +2608,34 @@ 
  */
 function get_blog_attachment($blog_id, $post_id=null,$comment_id=null)
 {
-	$blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
-
-	$blog_id = intval($blog_id);
-	$comment_id = intval($comment_id);
-	$post_id = intval($post_id);
-	$row=array();
-	$where='';
-	if (!empty ($post_id) && is_numeric($post_id)) {
-		$where.=' AND post_id ="'.$post_id.'" ';
-	}
-
-	if (!empty ($comment_id) && is_numeric($comment_id)) {
-		if (!empty ($post_id)) {
-			$where.= ' AND ';
-		}
-		$where.=' comment_id ="'.$comment_id.'" ';
-	}
+    $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
+
+    $blog_id = intval($blog_id);
+    $comment_id = intval($comment_id);
+    $post_id = intval($post_id);
+    $row=array();
+    $where='';
+    if (!empty ($post_id) && is_numeric($post_id)) {
+        $where.=' AND post_id ="'.$post_id.'" ';
+    }
+
+    if (!empty ($comment_id) && is_numeric($comment_id)) {
+        if (!empty ($post_id)) {
+            $where.= ' AND ';
+        }
+        $where.=' comment_id ="'.$comment_id.'" ';
+    }
 
     $course_id = api_get_course_int_id();
 
-	$sql = 'SELECT path, filename, comment FROM '. $blog_table_attachment.'
+    $sql = 'SELECT path, filename, comment FROM '. $blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
 
-	$result=Database::query($sql);
-	if (Database::num_rows($result)!=0) {
-		$row=Database::fetch_array($result);
-	}
-	return $row;
+    $result=Database::query($sql);
+    if (Database::num_rows($result)!=0) {
+        $row=Database::fetch_array($result);
+    }
+    return $row;
 }
 
 /**
@@ -2652,16 +2651,16 @@ 
     $post_id = null,
     $comment_id = null
 ) {
-	$_course = api_get_course_info();
-	$blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
-	$blog_id = intval($blog_id);
-	$comment_id = intval($comment_id);
-	$post_id = intval($post_id);
+    $_course = api_get_course_info();
+    $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
+    $blog_id = intval($blog_id);
+    $comment_id = intval($comment_id);
+    $post_id = intval($post_id);
 
     $course_id = api_get_course_int_id();
-	$where = null;
+    $where = null;
 
-	// delete files in DB
+    // delete files in DB
     if (!empty ($post_id) && is_numeric($post_id)) {
         $where .= ' AND post_id ="'.$post_id.'" ';
     }
@@ -2673,25 +2672,25 @@ 
         $where .= ' comment_id ="'.$comment_id.'" ';
     }
 
-	// delete all files in directory
-	$courseDir   = $_course['path'].'/upload/blog';
-	$sys_course_path = api_get_path(SYS_COURSE_PATH);
-	$updir = $sys_course_path.$courseDir;
+    // delete all files in directory
+    $courseDir   = $_course['path'].'/upload/blog';
+    $sys_course_path = api_get_path(SYS_COURSE_PATH);
+    $updir = $sys_course_path.$courseDir;
 
-	$sql = 'SELECT path FROM '.$blog_table_attachment.'
+    $sql = 'SELECT path FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
-	$result=Database::query($sql);
-
-	while ($row=Database::fetch_row($result)) {
-		$file=$updir.'/'.$row[0];
-		if (Security::check_abs_path($file,$updir) )
-		{
-			@ unlink($file);
-		}
-	}
-	$sql = 'DELETE FROM '. $blog_table_attachment.'
+    $result=Database::query($sql);
+
+    while ($row=Database::fetch_row($result)) {
+        $file=$updir.'/'.$row[0];
+        if (Security::check_abs_path($file,$updir) )
+        {
+            @ unlink($file);
+        }
+    }
+    $sql = 'DELETE FROM '. $blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND  blog_id ="'.intval($blog_id).'"  '.$where;
-	Database::query($sql);
+    Database::query($sql);
 }
 
 /**
@@ -2707,7 +2706,7 @@ 
     $course_id = $course_info['real_id'];
     $user_id = intval($user_id);
 
-	$sql = "SELECT DISTINCT blog.blog_id, post_id, title, full_text, post.date_creation
+    $sql = "SELECT DISTINCT blog.blog_id, post_id, title, full_text, post.date_creation
 			FROM $tbl_blogs blog
 			INNER JOIN  $tbl_blog_post post
 			ON (blog.blog_id = post.blog_id)
@@ -2716,19 +2715,19 @@ 
 				post.c_id = $course_id AND
 				author_id =  $user_id AND visibility = 1
 			ORDER BY post.date_creation DESC ";
-	$result = Database::query($sql);
-	$return_data = '';
-
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png',get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.= $row['full_text'];
-			$return_data.= '<br /><br />';
-		}
-	}
-	return $return_data;
+    $result = Database::query($sql);
+    $return_data = '';
+
+    if (Database::num_rows($result)!=0) {
+        while ($row=Database::fetch_array($result)) {
+            $return_data.=  '<div class="clear"></div><br />';
+            $return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png',get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
+            $return_data.=  '<br / >';
+            $return_data.= $row['full_text'];
+            $return_data.= '<br /><br />';
+        }
+    }
+    return $return_data;
 }
 
 /**
@@ -2745,7 +2744,7 @@ 
     $course_info = api_get_course_info($course_code);
     $course_id = $course_info['real_id'];
 
-	$sql = "SELECT DISTINCT blog.blog_id, comment_id, title, comment, comment.date_creation
+    $sql = "SELECT DISTINCT blog.blog_id, comment_id, title, comment, comment.date_creation
 			FROM $tbl_blogs blog INNER JOIN  $tbl_blog_comment comment
 			ON (blog.blog_id = comment.blog_id)
 			WHERE 	blog.c_id = $course_id AND
@@ -2753,17 +2752,17 @@ 
 					author_id =  $user_id AND
 					visibility = 1
 			ORDER BY blog_name";
-	$result = Database::query($sql);
-	$return_data = '';
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.=  $row['comment'];
-			$return_data.=  '<br />';
-		}
-	}
-	return $return_data;
+    $result = Database::query($sql);
+    $return_data = '';
+    if (Database::num_rows($result)!=0) {
+        while ($row=Database::fetch_array($result)) {
+            $return_data.=  '<div class="clear"></div><br />';
+            $return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
+            $return_data.=  '<br / >';
+            $return_data.=  $row['comment'];
+            $return_data.=  '<br />';
+        }
+    }
+    return $return_data;
 }
 

Spacing +370 added lines, -370 removed lines

@@ -29,8 +29,8 @@ 
             $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
 
             $sql = "SELECT blog_name
-                    FROM " . $tbl_blogs . "
-                    WHERE c_id = $course_id AND blog_id = " . intval($blog_id);
+                    FROM " . $tbl_blogs."
+                    WHERE c_id = $course_id AND blog_id = ".intval($blog_id);
 
             $result = Database::query($sql);
             $blog = Database::fetch_array($result);
@@ -82,9 +82,9 @@ 
                 ON (blogs_rel_user.user_id = user.user_id)
                 WHERE
                     blogs_rel_user.c_id = $course_id AND
-                    blogs_rel_user.blog_id = '" . (int)$blog_id."'";
+                    blogs_rel_user.blog_id = '".(int) $blog_id."'";
         $result = Database::query($sql);
-        $blog_members = array ();
+        $blog_members = array();
         while ($user = Database::fetch_array($result)) {
             $blog_members[$user['user_id']] = api_get_person_name(
                 $user['firstname'],
@@ -128,7 +128,7 @@ 
                 'blog_name' => $title,
                 'blog_subtitle' =>  $subtitle,
                 'date_creation' => $current_date,
-                'visibility' => 1 ,
+                'visibility' => 1,
                 'session_id' => $session_id,
             ];
             $this_blog_id = Database::insert($tbl_blogs, $params);
@@ -165,7 +165,7 @@ 
 
             // Put it on course homepage
             $sql = "INSERT INTO $tbl_tool (c_id, name, link, image, visibility, admin, address, added_tool, session_id, target)
-                    VALUES ($course_id, '".Database::escape_string($title)."','blog/blog.php?blog_id=".(int)$this_blog_id."','blog.gif','1','0','pastillegris.gif',0,'$session_id', '')";
+                    VALUES ($course_id, '".Database::escape_string($title)."','blog/blog.php?blog_id=".(int) $this_blog_id."','blog.gif','1','0','pastillegris.gif',0,'$session_id', '')";
             Database::query($sql);
 
             $toolId = Database::insert_id();
@@ -198,7 +198,7 @@ 
                 blog_subtitle = '".Database::escape_string($subtitle)."'
                 WHERE
                     c_id = $course_id AND
-                    blog_id ='".Database::escape_string((int)$blog_id)."'
+                    blog_id ='".Database::escape_string((int) $blog_id)."'
                 LIMIT 1";
         Database::query($sql);
 
@@ -214,7 +214,7 @@ 
         // Update course homepage link
         $sql = "UPDATE $tbl_tool SET
                 name = '".Database::escape_string($title)."'
-                WHERE c_id = $course_id AND link = 'blog/blog.php?blog_id=".(int)$blog_id."' 
+                WHERE c_id = $course_id AND link = 'blog/blog.php?blog_id=".(int) $blog_id."' 
                 LIMIT 1";
         Database::query($sql);
     }
@@ -257,7 +257,7 @@ 
         Database::query($sql);
 
         // Delete blog
-        $sql ="DELETE FROM $tbl_blogs WHERE c_id = $course_id AND blog_id ='".$blog_id."'";
+        $sql = "DELETE FROM $tbl_blogs WHERE c_id = $course_id AND blog_id ='".$blog_id."'";
         Database::query($sql);
 
         // Delete from course homepage
@@ -288,13 +288,13 @@ 
         $course_id = $_course['real_id'];
 
         $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
-        $upload_ok=true;
-        $has_attachment=false;
+        $upload_ok = true;
+        $has_attachment = false;
         $current_date = api_get_utc_datetime();
 
         if (!empty($_FILES['user_upload']['name'])) {
             $upload_ok = process_uploaded_file($_FILES['user_upload']);
-            $has_attachment=true;
+            $has_attachment = true;
         }
 
         if ($upload_ok) {
@@ -303,7 +303,7 @@ 
 
             // Create the post
             $sql = "INSERT INTO $tbl_blogs_posts (c_id, title, full_text, date_creation, blog_id, author_id )
-                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."','".$current_date."', '".(int)$blog_id."', '".(int)$_user['user_id']."');";
+                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."','".$current_date."', '".(int) $blog_id."', '".(int) $_user['user_id']."');";
 
             Database::query($sql);
             $last_post_id = Database::insert_id();
@@ -314,7 +314,7 @@ 
             }
 
             if ($has_attachment) {
-                $courseDir   = $_course['path'].'/upload/blog';
+                $courseDir = $_course['path'].'/upload/blog';
                 $sys_course_path = api_get_path(SYS_COURSE_PATH);
                 $updir = $sys_course_path.$courseDir;
 
@@ -365,9 +365,9 @@ 
 
         // Create the post
         $sql = "UPDATE $tbl_blogs_posts SET
-                title = '" . Database::escape_string($title)."',
+                title = '".Database::escape_string($title)."',
                 full_text = '" . Database::escape_string($full_text)."'
-                WHERE c_id = $course_id AND post_id ='".(int)$post_id."' AND blog_id ='".(int)$blog_id."'
+                WHERE c_id = $course_id AND post_id ='".(int) $post_id."' AND blog_id ='".(int) $blog_id."'
                 LIMIT 1 ";
         Database::query($sql);
     }
@@ -388,17 +388,17 @@ 
 
         // Delete ratings on this comment
         $sql = "DELETE FROM $tbl_blogs_rating
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND item_id = '".(int)$post_id."' AND rating_type = 'post'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND item_id = '".(int) $post_id."' AND rating_type = 'post'";
         Database::query($sql);
 
         // Delete the post
         $sql = "DELETE FROM $tbl_blogs_posts
-                WHERE c_id = $course_id AND post_id = '".(int)$post_id."'";
+                WHERE c_id = $course_id AND post_id = '".(int) $post_id."'";
         Database::query($sql);
 
         // Delete the comments
         $sql = "DELETE FROM $tbl_blogs_comments
-                WHERE c_id = $course_id AND post_id = '".(int)$post_id."' AND blog_id = '".(int)$blog_id."'";
+                WHERE c_id = $course_id AND post_id = '".(int) $post_id."' AND blog_id = '".(int) $blog_id."'";
         Database::query($sql);
 
         // Delete posts and attachments
@@ -425,7 +425,7 @@ 
     ) {
         $_user = api_get_user_info();
         $_course = api_get_course_info();
-        $blog_table_attachment 	= Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
+        $blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
 
         $upload_ok = true;
         $has_attachment = false;
@@ -434,7 +434,7 @@ 
 
         if (!empty($_FILES['user_upload']['name'])) {
             $upload_ok = process_uploaded_file($_FILES['user_upload']);
-            $has_attachment=true;
+            $has_attachment = true;
         }
 
         if ($upload_ok) {
@@ -443,7 +443,7 @@ 
 
             // Create the comment
             $sql = "INSERT INTO $tbl_blogs_comments (c_id, title, comment, author_id, date_creation, blog_id, post_id, parent_comment_id, task_id )
-                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."', '".(int)$_user['user_id']."','".$current_date."', '".(int)$blog_id."', '".(int)$post_id."', '".(int)$parent_id."', '".(int)$task_id."')";
+                    VALUES ($course_id, '".Database::escape_string($title)."', '".Database::escape_string($full_text)."', '".(int) $_user['user_id']."','".$current_date."', '".(int) $blog_id."', '".(int) $post_id."', '".(int) $parent_id."', '".(int) $task_id."')";
             Database::query($sql);
 
             // Empty post values, or they are shown on the page again
@@ -455,7 +455,7 @@ 
             }
 
             if ($has_attachment) {
-                $courseDir   = $_course['path'].'/upload/blog';
+                $courseDir = $_course['path'].'/upload/blog';
                 $sys_course_path = api_get_path(SYS_COURSE_PATH);
                 $updir = $sys_course_path.$courseDir;
 
@@ -466,19 +466,19 @@ 
                 );
 
                 // user's file name
-                $file_name =$_FILES['user_upload']['name'];
+                $file_name = $_FILES['user_upload']['name'];
 
                 if (!filter_extension($new_file_name)) {
                     Display :: display_error_message(get_lang('UplUnableToSaveFileFilteredExtension'));
                 } else {
                     $new_file_name = uniqid('');
-                    $new_path=$updir.'/'.$new_file_name;
-                    $result= @move_uploaded_file($_FILES['user_upload']['tmp_name'], $new_path);
+                    $new_path = $updir.'/'.$new_file_name;
+                    $result = @move_uploaded_file($_FILES['user_upload']['tmp_name'], $new_path);
                     $comment = Database::escape_string($file_comment);
 
                     // Storing the attachments if any
                     if ($result) {
-                        $sql='INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size,blog_id,comment_id) '.
+                        $sql = 'INSERT INTO '.$blog_table_attachment.'(c_id, filename,comment, path, post_id,size,blog_id,comment_id) '.
                              "VALUES ($course_id, '".Database::escape_string($file_name)."', '".$comment."', '".Database::escape_string($new_file_name)."' , '".$post_id."', '".$_FILES['user_upload']['size']."',  '".$blog_id."', '".$last_id."'  )";
                         Database::query($sql);
 
@@ -553,7 +553,7 @@ 
 
         // Create the task
         $sql = "INSERT INTO $tbl_blogs_tasks (c_id, blog_id, title, description, color, system_task)
-                VALUES ($course_id , '".(int)$blog_id."', '" . Database::escape_string($title)."', '" . Database::escape_string($description)."', '" . Database::escape_string($color)."', '0');";
+                VALUES ($course_id , '".(int) $blog_id."', '".Database::escape_string($title)."', '".Database::escape_string($description)."', '".Database::escape_string($color)."', '0');";
         Database::query($sql);
 
         $task_id = Database::insert_id();
@@ -563,13 +563,13 @@ 
             Database::query($sql);
         }
 
-        $tool = 'BLOG_' . $blog_id;
+        $tool = 'BLOG_'.$blog_id;
 
         if ($articleDelete == 'on') {
-            $sql = " INSERT INTO " . $tbl_tasks_permissions . " ( c_id,  task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = " INSERT INTO ".$tbl_tasks_permissions." ( c_id,  task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_delete'
                 )";
             Database::query($sql);
@@ -584,10 +584,10 @@ 
 
         if ($articleEdit == 'on') {
             $sql = "
-                INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+                INSERT INTO " . $tbl_tasks_permissions." (c_id, task_id, tool, action ) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_edit'
                 )";
             Database::query($sql);
@@ -601,10 +601,10 @@ 
 
         if ($commentsDelete == 'on') {
             $sql = "
-                INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+                INSERT INTO " . $tbl_tasks_permissions." (c_id, task_id, tool, action ) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_comments_delete'
                 )";
             Database::query($sql);
@@ -645,20 +645,20 @@ 
                     title = '".Database::escape_string($title)."',
                     description = '".Database::escape_string($description)."',
                     color = '".Database::escape_string($color)."'
-                WHERE c_id = $course_id AND task_id ='".(int)$task_id."' LIMIT 1";
+                WHERE c_id = $course_id AND task_id ='".(int) $task_id."' LIMIT 1";
         Database::query($sql);
 
-        $tool = 'BLOG_' . $blog_id;
+        $tool = 'BLOG_'.$blog_id;
 
-        $sql = "DELETE FROM " . $tbl_tasks_permissions . "
-                WHERE c_id = $course_id AND task_id = '" . (int)$task_id."'";
+        $sql = "DELETE FROM ".$tbl_tasks_permissions."
+                WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
         Database::query($sql);
 
         if ($articleDelete == 'on') {
-            $sql = "INSERT INTO " . $tbl_tasks_permissions . " ( c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = "INSERT INTO ".$tbl_tasks_permissions." ( c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_delete'
                 )";
             Database::query($sql);
@@ -671,10 +671,10 @@ 
         }
 
         if ($articleEdit == 'on') {
-            $sql = "INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = "INSERT INTO ".$tbl_tasks_permissions." (c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_edit'
                 )";
             Database::query($sql);
@@ -687,10 +687,10 @@ 
         }
 
         if ($commentsDelete == 'on') {
-            $sql = " INSERT INTO " . $tbl_tasks_permissions . " (c_id, task_id, tool, action) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($tool) . "',
+            $sql = " INSERT INTO ".$tbl_tasks_permissions." (c_id, task_id, tool, action) VALUES (
+                    '" . (int) $course_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($tool)."',
                     'article_comments_delete'
                 )";
             Database::query($sql);
@@ -715,7 +715,7 @@ 
 
         // Delete posts
         $sql = "DELETE FROM $tbl_blogs_tasks
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND task_id = '".(int)$task_id."'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND task_id = '".(int) $task_id."'";
         Database::query($sql);
     }
 
@@ -732,9 +732,9 @@ 
         $sql = "DELETE FROM $tbl_blogs_tasks_rel_user
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    task_id = '".(int)$task_id."' AND
-                    user_id = '".(int)$user_id."'";
+                    blog_id = '".(int) $blog_id."' AND
+                    task_id = '".(int) $task_id."' AND
+                    user_id = '".(int) $user_id."'";
         Database::query($sql);
     }
 
@@ -748,7 +748,7 @@ 
         $_user = api_get_user_info();
 
         $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
-        $tbl_blogs_tasks_rel_user 	= Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
+        $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
 
         $course_id = api_get_course_int_id();
@@ -765,7 +765,7 @@ 
                         task.c_id = $course_id AND
                         blog.c_id = $course_id AND
                         task_rel_user.c_id = $course_id AND
-                        task_rel_user.user_id = ".(int)$_user['user_id']."
+                        task_rel_user.user_id = ".(int) $_user['user_id']."
                     ORDER BY target_date ASC";
 
             $result = Database::query($sql);
@@ -773,7 +773,7 @@ 
             if (Database::num_rows($result) > 0) {
                 echo '<ul>';
                 while ($mytask = Database::fetch_array($result)) {
-                    echo '<li><a href="blog.php?action=execute_task&blog_id=' . $mytask['blog_id'] . '&task_id='.stripslashes($mytask['task_id']) . '" title="[Blog: '.stripslashes($mytask['blog_name']) . '] ' . get_lang('ExecuteThisTask') . '">'.stripslashes($mytask['title']) . '</a></li>';
+                    echo '<li><a href="blog.php?action=execute_task&blog_id='.$mytask['blog_id'].'&task_id='.stripslashes($mytask['task_id']).'" title="[Blog: '.stripslashes($mytask['blog_name']).'] '.get_lang('ExecuteThisTask').'">'.stripslashes($mytask['title']).'</a></li>';
                 }
                 echo '<ul>';
             } else {
@@ -797,7 +797,7 @@ 
 
         // Get blog properties
         $sql = "SELECT blog_name, visibility FROM $tbl_blogs
-                WHERE c_id = $course_id AND blog_id='".(int)$blog_id."'";
+                WHERE c_id = $course_id AND blog_id='".(int) $blog_id."'";
         $result = Database::query($sql);
         $blog = Database::fetch_array($result);
         $visibility = $blog['visibility'];
@@ -806,7 +806,7 @@ 
         if ($visibility == 1) {
             // Change visibility state, remove from course home.
             $sql = "UPDATE $tbl_blogs SET visibility = '0'
-                    WHERE c_id = $course_id AND blog_id ='".(int)$blog_id."' LIMIT 1";
+                    WHERE c_id = $course_id AND blog_id ='".(int) $blog_id."' LIMIT 1";
             Database::query($sql);
 
             $sql = "DELETE FROM $tbl_tool
@@ -815,11 +815,11 @@ 
         } else {
             // Change visibility state, add to course home.
             $sql = "UPDATE $tbl_blogs SET visibility = '1'
-                    WHERE c_id = $course_id AND blog_id ='".(int)$blog_id."' LIMIT 1";
+                    WHERE c_id = $course_id AND blog_id ='".(int) $blog_id."' LIMIT 1";
             Database::query($sql);
 
             $sql = "INSERT INTO $tbl_tool (c_id, name, link, image, visibility, admin, address, added_tool, target )
-                    VALUES ($course_id, '".Database::escape_string($title)."', 'blog/blog.php?blog_id=".(int)$blog_id."', 'blog.gif', '1', '0', 'pastillegris.gif', '0', '_self')";
+                    VALUES ($course_id, '".Database::escape_string($title)."', 'blog/blog.php?blog_id=".(int) $blog_id."', 'blog.gif', '1', '0', 'pastillegris.gif', '0', '_self')";
             Database::query($sql);
             $id = Database::insert_id();
 
@@ -850,14 +850,14 @@ 
                 FROM $tbl_blogs_posts post
                 INNER JOIN $tbl_users user
                 ON post.author_id = user.user_id
-                WHERE 	post.blog_id = '".(int)$blog_id."' AND
+                WHERE 	post.blog_id = '".(int) $blog_id."' AND
                         post.c_id = $course_id AND
                         $filter
-                ORDER BY post_id DESC LIMIT 0,".(int)$max_number_of_posts;
+                ORDER BY post_id DESC LIMIT 0,".(int) $max_number_of_posts;
         $result = Database::query($sql);
 
         // Display
-        if(Database::num_rows($result) > 0) {
+        if (Database::num_rows($result) > 0) {
             $limit = 200;
             while ($blog_post = Database::fetch_array($result)) {
                 // Get number of comments
@@ -865,8 +865,8 @@ 
                         FROM $tbl_blogs_comments
                         WHERE
                             c_id = $course_id AND
-                            blog_id = '".(int)$blog_id."' AND
-                            post_id = '" . (int)$blog_post['post_id']."'";
+                            blog_id = '".(int) $blog_id."' AND
+                            post_id = '" . (int) $blog_post['post_id']."'";
                 $tmp = Database::query($sql);
                 $blog_post_comments = Database::fetch_array($tmp);
 
@@ -877,11 +877,11 @@ 
 
                 // Create an introduction text (but keep FULL sentences)
                 $words = 0;
-                $blog_post_text_cut = cut($blog_post_text, $limit) ;
+                $blog_post_text_cut = cut($blog_post_text, $limit);
                 $words = strlen($blog_post_text);
 
                 if ($words >= $limit) {
-                    $readMoreLink = ' <div class="link" onclick="document.getElementById(\'blogpost_text_' . $blog_post_id . '\').style.display=\'block\'; document.getElementById(\'blogpost_introduction_' . $blog_post_id . '\').style.display=\'none\'">' . get_lang('ReadMore') . '</div>';
+                    $readMoreLink = ' <div class="link" onclick="document.getElementById(\'blogpost_text_'.$blog_post_id.'\').style.display=\'block\'; document.getElementById(\'blogpost_introduction_'.$blog_post_id.'\').style.display=\'none\'">'.get_lang('ReadMore').'</div>';
                     $introduction_text = $blog_post_text_cut;
                 } else {
                     $introduction_text = $blog_post_text;
@@ -891,27 +891,27 @@ 
                 $introduction_text = stripslashes($introduction_text);
 
                 echo '<div class="blogpost">';
-                echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >'.stripslashes($blog_post['title']) . '</a></span>';
-                echo '<span class="blogpost_date"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >' . $blog_post_date . '</a></span>';
-                echo '<div class="blogpost_introduction" id="blogpost_introduction_'.$blog_post_id.'">' . $introduction_text.$readMoreLink.'</div>';
-                echo '<div class="blogpost_text" id="blogpost_text_' . $blog_post_id . '" style="display: none">' . $blog_post_text . '</div>';
+                echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.stripslashes($blog_post['title']).'</a></span>';
+                echo '<span class="blogpost_date"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.$blog_post_date.'</a></span>';
+                echo '<div class="blogpost_introduction" id="blogpost_introduction_'.$blog_post_id.'">'.$introduction_text.$readMoreLink.'</div>';
+                echo '<div class="blogpost_text" id="blogpost_text_'.$blog_post_id.'" style="display: none">'.$blog_post_text.'</div>';
 
-                $file_name_array = get_blog_attachment($blog_id,$blog_post_id,0);
+                $file_name_array = get_blog_attachment($blog_id, $blog_post_id, 0);
 
                 if (!empty($file_name_array)) {
                     echo '<br /><br />';
-                    echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+                    echo Display::return_icon('attachment.gif', get_lang('Attachment'));
                     echo '<a href="download.php?file=';
                     echo $file_name_array['path'];
                     echo ' "> '.$file_name_array['filename'].' </a><br />';
                     echo '</span>';
                 }
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-                echo '<span class="blogpost_info">' . get_lang('Author') . ': ' . Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)) .' - <a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '#add_comment" title="' . get_lang('ReadPost') . '" >' . get_lang('Comments') . ': ' . $blog_post_comments['number_of_comments'] . '</a></span>';
+                echo '<span class="blogpost_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - <a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'#add_comment" title="'.get_lang('ReadPost').'" >'.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].'</a></span>';
                 echo '</div>';
             }
         } else {
-            if($filter == '1=1') {
+            if ($filter == '1=1') {
                 echo get_lang('NoArticles');
             } else {
                 echo get_lang('NoArticleMatches');
@@ -925,19 +925,19 @@ 
      * @param Integer $blog_id
      * @param String $query_string
      */
-    public static function display_search_results ($blog_id, $query_string)
+    public static function display_search_results($blog_id, $query_string)
     {
         // Init
         $query_string = Database::escape_string($query_string);
-        $query_string_parts = explode(' ',$query_string);
+        $query_string_parts = explode(' ', $query_string);
         $query_string = array();
         foreach ($query_string_parts as $query_part) {
-            $query_string[] = " full_text LIKE '%" . $query_part."%' OR title LIKE '%" . $query_part."%' ";
+            $query_string[] = " full_text LIKE '%".$query_part."%' OR title LIKE '%".$query_part."%' ";
         }
-        $query_string = '('.implode('OR',$query_string) . ')';
+        $query_string = '('.implode('OR', $query_string).')';
 
         // Display the posts
-        echo '<span class="blogpost_title">' . get_lang('SearchResults') . '</span>';
+        echo '<span class="blogpost_title">'.get_lang('SearchResults').'</span>';
         Blog::display_blog_posts($blog_id, $query_string);
     }
 
@@ -950,14 +950,14 @@ 
     public static function display_day_results($blog_id, $query_string)
     {
         $date_output = $query_string;
-        $date = explode('-',$query_string);
-        $query_string = ' DAYOFMONTH(date_creation) =' . intval($date[2]) . ' AND MONTH(date_creation) =' . intval($date[1]) . ' AND YEAR(date_creation) =' . intval($date[0]);
+        $date = explode('-', $query_string);
+        $query_string = ' DAYOFMONTH(date_creation) ='.intval($date[2]).' AND MONTH(date_creation) ='.intval($date[1]).' AND YEAR(date_creation) ='.intval($date[0]);
 
         // Put date in correct output format
         $date_output = api_format_date($date_output, DATE_FORMAT_LONG);
 
         // Display the posts
-        echo '<span class="blogpost_title">' . get_lang('PostsOf') . ': ' . $date_output . '</span>';
+        echo '<span class="blogpost_title">'.get_lang('PostsOf').': '.$date_output.'</span>';
         Blog::display_blog_posts($blog_id, $query_string);
     }
 
@@ -984,8 +984,8 @@ 
                     ON post.author_id = user.user_id
                 WHERE
                     post.c_id = $course_id AND
-                    post.blog_id = '".(int)$blog_id."' AND
-                    post.post_id = '".(int)$post_id."'
+                    post.blog_id = '".(int) $blog_id."' AND
+                    post.post_id = '".(int) $post_id."'
                 ORDER BY post_id DESC";
         $result = Database::query($sql);
         $blog_post = Database::fetch_array($result);
@@ -993,7 +993,7 @@ 
         // Get number of comments
         $sql = "SELECT COUNT(1) as number_of_comments
                 FROM $tbl_blogs_comments
-                WHERE c_id = $course_id AND blog_id = '".(int)$blog_id."' AND post_id = '".(int)$post_id."'";
+                WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."' AND post_id = '".(int) $post_id."'";
         $result = Database::query($sql);
         $blog_post_comments = Database::fetch_array($result);
 
@@ -1004,34 +1004,34 @@ 
 
         $task_id = (isset($_GET['task_id']) && is_numeric($_GET['task_id'])) ? intval($_GET['task_id']) : 0;
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_edit', $task_id)) {
-            $blog_post_actions .= '<a href="blog.php?action=edit_post&blog_id=' . $blog_id . '&post_id=' . $post_id . '&article_id=' . $blog_post['post_id'] . '&task_id=' . $task_id . '" title="' . get_lang('EditThisPost') . '">';
-            $blog_post_actions .=  Display::return_icon('edit.png');
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_edit', $task_id)) {
+            $blog_post_actions .= '<a href="blog.php?action=edit_post&blog_id='.$blog_id.'&post_id='.$post_id.'&article_id='.$blog_post['post_id'].'&task_id='.$task_id.'" title="'.get_lang('EditThisPost').'">';
+            $blog_post_actions .= Display::return_icon('edit.png');
             $blog_post_actions .= '</a>';
         }
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_delete', $task_id)) {
-            $blog_post_actions .= '<a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $post_id . '&do=delete_article&article_id=' . $blog_post['post_id'] . '&task_id=' . $task_id . '" title="' . get_lang('DeleteThisArticle') . '" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;">';
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_delete', $task_id)) {
+            $blog_post_actions .= '<a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$post_id.'&do=delete_article&article_id='.$blog_post['post_id'].'&task_id='.$task_id.'" title="'.get_lang('DeleteThisArticle').'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;">';
             $blog_post_actions .= Display::return_icon('delete.png');
             $blog_post_actions .= '</a>';
         }
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_rate'))
-            $rating_select = Blog::display_rating_form('post',$blog_id,$post_id);
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_rate'))
+            $rating_select = Blog::display_rating_form('post', $blog_id, $post_id);
 
-        $blog_post_text=stripslashes($blog_post_text);
+        $blog_post_text = stripslashes($blog_post_text);
 
         // Display post
         echo '<div class="blogpost">';
-        echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id=' . $blog_id . '&post_id=' . $blog_post['post_id'] . '" title="' . get_lang('ReadPost') . '" >'.stripslashes($blog_post['title']) . '</a></span>';
-        echo '<span class="blogpost_date">' . $blog_post_date . '</span>';
-        echo '<span class="blogpost_text">' . $blog_post_text . '</span><br />';
+        echo '<span class="blogpost_title"><a href="blog.php?action=view_post&blog_id='.$blog_id.'&post_id='.$blog_post['post_id'].'" title="'.get_lang('ReadPost').'" >'.stripslashes($blog_post['title']).'</a></span>';
+        echo '<span class="blogpost_date">'.$blog_post_date.'</span>';
+        echo '<span class="blogpost_text">'.$blog_post_text.'</span><br />';
 
         $file_name_array = get_blog_attachment($blog_id, $post_id);
 
         if (!empty($file_name_array)) {
             echo ' <br />';
-            echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+            echo Display::return_icon('attachment.gif', get_lang('Attachment'));
             echo '<a href="download.php?file=';
             echo $file_name_array['path'];
             echo ' "> '.$file_name_array['filename'].' </a>';
@@ -1042,20 +1042,20 @@ 
             echo '<br />';
         }
         $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-        echo '<span class="blogpost_info">'.get_lang('Author').': ' .Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - '.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].' - '.get_lang('Rating').': '.Blog::display_rating('post',$blog_id,$post_id).$rating_select.'</span>';
-        echo '<span class="blogpost_actions">' . $blog_post_actions . '</span>';
+        echo '<span class="blogpost_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username)).' - '.get_lang('Comments').': '.$blog_post_comments['number_of_comments'].' - '.get_lang('Rating').': '.Blog::display_rating('post', $blog_id, $post_id).$rating_select.'</span>';
+        echo '<span class="blogpost_actions">'.$blog_post_actions.'</span>';
         echo '</div>';
 
         // Display comments if there are any
-        if($blog_post_comments['number_of_comments'] > 0) {
+        if ($blog_post_comments['number_of_comments'] > 0) {
             echo '<div class="comments">';
-                echo '<span class="blogpost_title">' . get_lang('Comments') . '</span><br />';
+                echo '<span class="blogpost_title">'.get_lang('Comments').'</span><br />';
                 Blog::get_threaded_comments(0, 0, $blog_id, $post_id, $task_id);
             echo '</div>';
         }
 
         // Display comment form
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_comments_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_comments_add')) {
             Blog::display_new_comment_form($blog_id, $post_id, $blog_post['title']);
         }
     }
@@ -1083,16 +1083,16 @@ 
         $sql = "SELECT rating_id FROM $tbl_blogs_rating
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    item_id = '".(int)$item_id."' AND
+                    blog_id = '".(int) $blog_id."' AND
+                    item_id = '".(int) $item_id."' AND
                     rating_type = '".Database::escape_string($type)."' AND
-                    user_id = '".(int)$_user['user_id']."'";
+                    user_id = '".(int) $_user['user_id']."'";
         $result = Database::query($sql);
 
         // Add rating
         if (Database::num_rows($result) == 0) {
             $sql = "INSERT INTO $tbl_blogs_rating (c_id, blog_id, rating_type, item_id, user_id, rating )
-                    VALUES ($course_id, '".(int)$blog_id."', '".Database::escape_string($type)."', '".(int)$item_id."', '".(int)$_user['user_id']."', '".Database::escape_string($rating)."')";
+                    VALUES ($course_id, '".(int) $blog_id."', '".Database::escape_string($type)."', '".(int) $item_id."', '".(int) $_user['user_id']."', '".Database::escape_string($rating)."')";
             Database::query($sql);
 
             $id = Database::insert_id();
@@ -1123,8 +1123,8 @@ 
         $sql = "SELECT AVG(rating) as rating FROM $tbl_blogs_rating
                 WHERE
                     c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."' AND
-                    item_id = '".(int)$item_id."' AND
+                    blog_id = '".(int) $blog_id."' AND
+                    item_id = '".(int) $item_id."' AND
                     rating_type = '".Database::escape_string($type)."' ";
         $result = Database::query($sql);
         $result = Database::fetch_array($result);
@@ -1139,7 +1139,7 @@ 
      * @param Integer $blog_id
      * @param integer $post_id
      */
-    public static function display_rating_form ($type, $blog_id, $post_id, $comment_id = NULL)
+    public static function display_rating_form($type, $blog_id, $post_id, $comment_id = NULL)
     {
         $_user = api_get_user_info();
         $tbl_blogs_rating = Database::get_course_table(TABLE_BLOGS_RATING);
@@ -1149,14 +1149,14 @@ 
             // Check if the user has already rated this post
             $sql = "SELECT rating_id FROM $tbl_blogs_rating
                     WHERE c_id = $course_id AND
-                    blog_id = '".(int)$blog_id."'
-                    AND item_id = '".(int)$post_id."'
+                    blog_id = '".(int) $blog_id."'
+                    AND item_id = '".(int) $post_id."'
                     AND rating_type = '".Database::escape_string($type)."'
-                    AND user_id = '".(int)$_user['user_id']."'";
+                    AND user_id = '".(int) $_user['user_id']."'";
             $result = Database::query($sql);
             // Add rating
             if (Database::num_rows($result) == 0) {
-                return ' - ' . get_lang('RateThis') . ': <form method="get" action="blog.php" style="display: inline" id="frm_rating_' . $type . '_' . $post_id . '" name="frm_rating_' . $type . '_' . $post_id . '"><select name="rating" onchange="document.forms[\'frm_rating_' . $type . '_' . $post_id . '\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="' . $type . '" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="' . $blog_id . '" /><input type="hidden" name="post_id" value="' . $post_id . '" /></form>';
+                return ' - '.get_lang('RateThis').': <form method="get" action="blog.php" style="display: inline" id="frm_rating_'.$type.'_'.$post_id.'" name="frm_rating_'.$type.'_'.$post_id.'"><select name="rating" onchange="document.forms[\'frm_rating_'.$type.'_'.$post_id.'\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="'.$type.'" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="'.$blog_id.'" /><input type="hidden" name="post_id" value="'.$post_id.'" /></form>';
             } else {
                 return '';
             }
@@ -1165,14 +1165,14 @@ 
         if ($type = 'comment') {
             // Check if the user has already rated this comment
             $sql = "SELECT rating_id FROM $tbl_blogs_rating
-                    WHERE c_id = $course_id AND blog_id = '".(int)$blog_id ."'
-                    AND item_id = '".(int)$comment_id."'
+                    WHERE c_id = $course_id AND blog_id = '".(int) $blog_id."'
+                    AND item_id = '".(int) $comment_id."'
                     AND rating_type = '".Database::escape_string($type)."'
-                    AND user_id = '".(int)$_user['user_id']."'";
+                    AND user_id = '".(int) $_user['user_id']."'";
             $result = Database::query($sql);
 
             if (Database::num_rows($result) == 0) {
-                return ' - ' . get_lang('RateThis') . ': <form method="get" action="blog.php" style="display: inline" id="frm_rating_' . $type . '_' . $comment_id . '" name="frm_rating_' . $type . '_' . $comment_id . '"><select name="rating" onchange="document.forms[\'frm_rating_' . $type . '_' . $comment_id . '\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="' . $type . '" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="' . $blog_id . '" /><input type="hidden" name="post_id" value="' . $post_id . '" /><input type="hidden" name="comment_id" value="' . $comment_id . '" /></form>';
+                return ' - '.get_lang('RateThis').': <form method="get" action="blog.php" style="display: inline" id="frm_rating_'.$type.'_'.$comment_id.'" name="frm_rating_'.$type.'_'.$comment_id.'"><select name="rating" onchange="document.forms[\'frm_rating_'.$type.'_'.$comment_id.'\'].submit()"><option value="">-</option><option value="1">1</option><option value="2">2</option><option value="3">3</option><option value="4">4</option><option value="5">5</option><option value="6">6</option><option value="7">7</option><option value="8">8</option><option value="9">9</option><option value="10">10</option></select><input type="hidden" name="action" value="view_post" /><input type="hidden" name="type" value="'.$type.'" /><input type="hidden" name="do" value="rate" /><input type="hidden" name="blog_id" value="'.$blog_id.'" /><input type="hidden" name="post_id" value="'.$post_id.'" /><input type="hidden" name="comment_id" value="'.$comment_id.'" /></form>';
             } else {
                 return '';
             }
@@ -1207,11 +1207,11 @@ 
                 WHERE
                     comments.c_id = $course_id AND
                     parent_comment_id = $current AND
-                    comments.blog_id = '".(int)$blog_id."' AND
-                    comments.post_id = '".(int)$post_id."'";
+                    comments.blog_id = '".(int) $blog_id."' AND
+                    comments.post_id = '".(int) $post_id."'";
         $result = Database::query($sql);
 
-        while($comment = Database::fetch_array($result)) {
+        while ($comment = Database::fetch_array($result)) {
             // Select the children recursivly
             $tmp = "SELECT comments.*, user.lastname, user.firstname, user.username
                     FROM $tbl_blogs_comments comments
@@ -1220,8 +1220,8 @@ 
                     WHERE
                         comments.c_id = $course_id AND
                         comment_id = $current
-                        AND blog_id = '".(int)$blog_id."'
-                        AND post_id = '".(int)$post_id."'";
+                        AND blog_id = '".(int) $blog_id."'
+                        AND post_id = '".(int) $post_id."'";
             $tmp = Database::query($tmp);
             $tmp = Database::fetch_array($tmp);
             $parent_cat = $tmp['parent_comment_id'];
@@ -1246,22 +1246,22 @@ 
             }
 
             if (!is_null($comment['task_id'])) {
-                $border_color = ' border-left: 3px solid #' . $comment['color'];
+                $border_color = ' border-left: 3px solid #'.$comment['color'];
             }
 
             $comment_text = stripslashes($comment_text);
 
             // Output...
             $margin = $current_level * 30;
-            echo '<div class="blogpost_comment" style="margin-left: ' . $margin . 'px;' . $border_color . '">';
-                echo '<span class="blogpost_comment_title"><a href="#add_comment" onclick="document.getElementById(\'comment_parent_id\').value=\'' . $comment['comment_id'] . '\'; document.getElementById(\'comment_title\').value=\'Re: '.addslashes($comment['title']) . '\'" title="' . get_lang('ReplyToThisComment') . '" >'.stripslashes($comment['title']) . '</a></span>';
-                echo '<span class="blogpost_comment_date">' . $blog_comment_date . '</span>';
-                echo '<span class="blogpost_text">' . $comment_text . '</span>';
+            echo '<div class="blogpost_comment" style="margin-left: '.$margin.'px;'.$border_color.'">';
+                echo '<span class="blogpost_comment_title"><a href="#add_comment" onclick="document.getElementById(\'comment_parent_id\').value=\''.$comment['comment_id'].'\'; document.getElementById(\'comment_title\').value=\'Re: '.addslashes($comment['title']).'\'" title="'.get_lang('ReplyToThisComment').'" >'.stripslashes($comment['title']).'</a></span>';
+                echo '<span class="blogpost_comment_date">'.$blog_comment_date.'</span>';
+                echo '<span class="blogpost_text">'.$comment_text.'</span>';
 
-                $file_name_array = get_blog_attachment($blog_id,$post_id, $comment['comment_id']);
+                $file_name_array = get_blog_attachment($blog_id, $post_id, $comment['comment_id']);
                 if (!empty($file_name_array)) {
                     echo '<br /><br />';
-                    echo Display::return_icon('attachment.gif',get_lang('Attachment'));
+                    echo Display::return_icon('attachment.gif', get_lang('Attachment'));
                     echo '<a href="download.php?file=';
                     echo $file_name_array['path'];
                     echo ' "> '.$file_name_array['filename'].' </a>';
@@ -1271,7 +1271,7 @@ 
                 }
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $comment['username']), ENT_QUOTES);
                 echo '<span class="blogpost_comment_info">'.get_lang('Author').': '.Display::tag('span', api_get_person_name($comment['firstname'], $comment['lastname']), array('title'=>$username)).' - '.get_lang('Rating').': '.Blog::display_rating('comment', $blog_id, $comment['comment_id']).$rating_select.'</span>';
-                echo '<span class="blogpost_actions">' . $blog_comment_actions . '</span>';
+                echo '<span class="blogpost_actions">'.$blog_comment_actions.'</span>';
             echo '</div>';
 
             // Go further down the tree.
@@ -1287,11 +1287,11 @@ 
      */
     public static function display_form_new_post($blog_id)
     {
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_add')) {
             $form = new FormValidator(
                 'add_post',
                 'post',
-                api_get_path(WEB_CODE_PATH)."blog/blog.php?action=new_post&blog_id=" . $blog_id . "&" . api_get_cidreq(),
+                api_get_path(WEB_CODE_PATH)."blog/blog.php?action=new_post&blog_id=".$blog_id."&".api_get_cidreq(),
                 null,
                 array('enctype' => 'multipart/form-data')
             );
@@ -1335,8 +1335,8 @@ 
                 INNER JOIN $tbl_users user ON post.author_id = user.user_id
                 WHERE
                 post.c_id 			= $course_id AND
-                post.blog_id 		= '".(int)$blog_id ."'
-                AND post.post_id	= '".(int)$post_id."'
+                post.blog_id 		= '".(int) $blog_id."'
+                AND post.post_id	= '".(int) $post_id."'
                 ORDER BY post_id DESC";
         $result = Database::query($sql);
         $blog_post = Database::fetch_array($result);
@@ -1345,7 +1345,7 @@ 
         $form = new FormValidator(
             'edit_post',
             'post',
-            api_get_path(WEB_CODE_PATH).'blog/blog.php?action=edit_post&post_id=' . intval($_GET['post_id']) . '&blog_id=' . intval($blog_id) . '&article_id='.intval($_GET['article_id']).'&task_id='.intval($_GET['task_id'])
+            api_get_path(WEB_CODE_PATH).'blog/blog.php?action=edit_post&post_id='.intval($_GET['post_id']).'&blog_id='.intval($blog_id).'&article_id='.intval($_GET['article_id']).'&task_id='.intval($_GET['task_id'])
         );
 
         $form->addHeader(get_lang('EditPost'));
@@ -1377,31 +1377,31 @@ 
         global $charset;
         $course_id = api_get_course_int_id();
 
-        if (api_is_allowed('BLOG_' . $blog_id, 'article_add')) {
+        if (api_is_allowed('BLOG_'.$blog_id, 'article_add')) {
             $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
             $counter = 0;
             global $color2;
 
             echo '<div class="actions">';
-            echo '<a href="' .api_get_self(). '?action=manage_tasks&blog_id=' . $blog_id . '&do=add">';
+            echo '<a href="'.api_get_self().'?action=manage_tasks&blog_id='.$blog_id.'&do=add">';
             echo Display::return_icon('blog_newtasks.gif', get_lang('AddTasks'));
-            echo get_lang('AddTasks') . '</a> ';
-            echo '<a href="' .api_get_self(). '?action=manage_tasks&blog_id=' . $blog_id . '&do=assign">';
+            echo get_lang('AddTasks').'</a> ';
+            echo '<a href="'.api_get_self().'?action=manage_tasks&blog_id='.$blog_id.'&do=assign">';
             echo Display::return_icon('blog_task.gif', get_lang('AssignTasks'));
-            echo get_lang('AssignTasks') . '</a>';
+            echo get_lang('AssignTasks').'</a>';
             ?>
                 <a href="<?php echo api_get_self(); ?>?action=manage_rights&blog_id=<?php echo $blog_id ?>" title="<?php echo get_lang('ManageRights') ?>">
-                    <?php echo Display::return_icon('blog_admin_users.png', get_lang('RightsManager'),'',ICON_SIZE_SMALL). get_lang('RightsManager') ?></a>
+                    <?php echo Display::return_icon('blog_admin_users.png', get_lang('RightsManager'), '', ICON_SIZE_SMALL).get_lang('RightsManager') ?></a>
             <?php
             echo '</div>';
 
-            echo '<span class="blogpost_title">' . get_lang('TaskList') . '</span><br />';
+            echo '<span class="blogpost_title">'.get_lang('TaskList').'</span><br />';
             echo "<table class=\"data_table\">";
             echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                     "<th width='240'><b>",get_lang('Title'),"</b></th>",
-                     "<th><b>",get_lang('Description'),"</b></th>",
-                     "<th><b>",get_lang('Color'),"</b></th>",
-                     "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                     "<th width='240'><b>", get_lang('Title'), "</b></th>",
+                     "<th><b>", get_lang('Description'), "</b></th>",
+                     "<th><b>", get_lang('Color'), "</b></th>",
+                     "<th width='50'><b>", get_lang('Modify'), "</b></th>",
                 "</tr>";
 
 
@@ -1413,8 +1413,8 @@ 
                         description,
                         color,
                         system_task
-                    FROM " . $tbl_blogs_tasks . "
-                    WHERE c_id = $course_id AND blog_id = " . (int)$blog_id . "
+                    FROM " . $tbl_blogs_tasks."
+                    WHERE c_id = $course_id AND blog_id = ".(int) $blog_id."
                     ORDER BY system_task, title";
             $result = Database::query($sql);
 
@@ -1423,10 +1423,10 @@ 
                 $css_class = (($counter % 2) == 0) ? "row_odd" : "row_even";
                 $delete_icon = ($task['system_task'] == '1') ? "delete_na.png" : "delete.png";
                 $delete_title = ($task['system_task'] == '1') ? get_lang('DeleteSystemTask') : get_lang('DeleteTask');
-                $delete_link = ($task['system_task'] == '1') ? '#' : api_get_self() . '?action=manage_tasks&blog_id=' . $task['blog_id'] . '&do=delete&task_id=' . $task['task_id'];
-                $delete_confirm = ($task['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;"';
+                $delete_link = ($task['system_task'] == '1') ? '#' : api_get_self().'?action=manage_tasks&blog_id='.$task['blog_id'].'&do=delete&task_id='.$task['task_id'];
+                $delete_confirm = ($task['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;"';
 
-                echo '<tr class="' . $css_class . '" valign="top">';
+                echo '<tr class="'.$css_class.'" valign="top">';
                 echo '<td width="240">'.Security::remove_XSS($task['title']).'</td>';
                 echo '<td>'.Security::remove_XSS($task['description']).'</td>';
                 echo '<td><span style="background-color: #'.$task['color'].'">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</span></td>';
@@ -1452,23 +1452,23 @@ 
      *
      * @param Integer $blog_id
      */
-    public static function display_assigned_task_list ($blog_id)
+    public static function display_assigned_task_list($blog_id)
     {
         // Init
         $tbl_users = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
         $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
         $counter = 0;
-        global $charset,$color2;
+        global $charset, $color2;
 
-        echo '<span class="blogpost_title">' . get_lang('AssignedTasks') . '</span><br />';
+        echo '<span class="blogpost_title">'.get_lang('AssignedTasks').'</span><br />';
         echo "<table class=\"data_table\">";
         echo	"<tr bgcolor=\"$color2\" align=\"center\" valign=\"top\">",
-                 "<th width='240'><b>",get_lang('Member'),"</b></th>",
-                 "<th><b>",get_lang('Task'),"</b></th>",
-                 "<th><b>",get_lang('Description'),"</b></th>",
-                 "<th><b>",get_lang('TargetDate'),"</b></th>",
-                 "<th width='50'><b>",get_lang('Modify'),"</b></th>",
+                 "<th width='240'><b>", get_lang('Member'), "</b></th>",
+                 "<th><b>", get_lang('Task'), "</b></th>",
+                 "<th><b>", get_lang('Description'), "</b></th>",
+                 "<th><b>", get_lang('TargetDate'), "</b></th>",
+                 "<th width='50'><b>", get_lang('Modify'), "</b></th>",
             "</tr>";
 
         $course_id = api_get_course_int_id();
@@ -1480,17 +1480,17 @@ 
                 WHERE
                     task_rel_user.c_id = $course_id AND
                     task.c_id = $course_id AND
-                    task_rel_user.blog_id = '".(int)$blog_id."'
+                    task_rel_user.blog_id = '".(int) $blog_id."'
                 ORDER BY target_date ASC";
         $result = Database::query($sql);
 
         while ($assignment = Database::fetch_array($result)) {
             $counter++;
-            $css_class = (($counter % 2)==0) ? "row_odd" : "row_even";
+            $css_class = (($counter % 2) == 0) ? "row_odd" : "row_even";
             $delete_icon = ($assignment['system_task'] == '1') ? "delete_na.png" : "delete.png";
             $delete_title = ($assignment['system_task'] == '1') ? get_lang('DeleteSystemTask') : get_lang('DeleteTask');
-            $delete_link = ($assignment['system_task'] == '1') ? '#' : api_get_self() . '?action=manage_tasks&blog_id=' . $assignment['blog_id'] . '&do=delete&task_id=' . $assignment['task_id'];
-            $delete_confirm = ($assignment['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;"';
+            $delete_link = ($assignment['system_task'] == '1') ? '#' : api_get_self().'?action=manage_tasks&blog_id='.$assignment['blog_id'].'&do=delete&task_id='.$assignment['task_id'];
+            $delete_confirm = ($assignment['system_task'] == '1') ? '' : 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;"';
 
             $username = api_htmlentities(sprintf(get_lang('LoginX'), $assignment['username']), ENT_QUOTES);
 
@@ -1522,7 +1522,7 @@ 
      * @author Toon Keppens
      *
      */
-    public static function display_new_task_form ($blog_id)
+    public static function display_new_task_form($blog_id)
     {
         // Init
         $colors = array(
@@ -1544,7 +1544,7 @@ 
         );
 
         // form
-        echo '<form name="add_task" method="post" action="blog.php?action=manage_tasks&blog_id=' . $blog_id . '">';
+        echo '<form name="add_task" method="post" action="blog.php?action=manage_tasks&blog_id='.$blog_id.'">';
 
         // form title
         echo '<legend>'.get_lang('AddTask').'</legend>';
@@ -1552,7 +1552,7 @@ 
         // task title
         echo '	<div class="control-group">
                     <label class="control-label">
-                        <span class="form_required">*</span>' . get_lang('Title') . '
+                        <span class="form_required">*</span>' . get_lang('Title').'
                     </label>
                     <div class="controls">
                         <input name="task_name" type="text" size="70" />
@@ -1562,7 +1562,7 @@ 
         // task comment
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('Description') . '
+                        ' . get_lang('Description').'
                     </label>
                     <div class="controls">
                         <textarea name="task_description" cols="45"></textarea>
@@ -1572,18 +1572,18 @@ 
         // task management
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('TaskManager') . '
+                        ' . get_lang('TaskManager').'
                     </label>
                     <div class="controls">';
                 echo '<table class="data_table" cellspacing="0" style="border-collapse:collapse; width:446px;">';
                     echo '<tr>';
-                        echo '<th colspan="2" style="width:223px;">' . get_lang('ArticleManager') . '</th>';
-                        echo '<th width:223px;>' . get_lang('CommentManager') . '</th>';
+                        echo '<th colspan="2" style="width:223px;">'.get_lang('ArticleManager').'</th>';
+                        echo '<th width:223px;>'.get_lang('CommentManager').'</th>';
                     echo '</tr>';
                     echo '<tr>';
-                        echo '<th style="width:111px;"><label for="articleDelete">' . get_lang('Delete') . '</label></th>';
-                        echo '<th style="width:112px;"><label for="articleEdit">' . get_lang('Edit') . '</label></th>';
-                        echo '<th style="width:223px;"><label for="commentsDelete">' . get_lang('Delete') . '</label></th>';
+                        echo '<th style="width:111px;"><label for="articleDelete">'.get_lang('Delete').'</label></th>';
+                        echo '<th style="width:112px;"><label for="articleEdit">'.get_lang('Edit').'</label></th>';
+                        echo '<th style="width:223px;"><label for="commentsDelete">'.get_lang('Delete').'</label></th>';
                     echo '</tr>';
                     echo '<tr>';
                         echo '<td style="text-align:center;"><input id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
@@ -1598,13 +1598,13 @@ 
         // task color
         echo '	<div class="control-group">
                     <label class="control-label">
-                        ' . get_lang('Color') . '
+                        ' . get_lang('Color').'
                     </label>
                     <div class="controls">';
         echo '<select name="task_color" id="color" style="width: 150px; background-color: #eeeeee" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
                 foreach ($colors as $color) {
-                    $style = 'style="background-color: #' . $color . '"';
-                    echo '<option value="' . $color . '" ' . $style . '>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
+                    $style = 'style="background-color: #'.$color.'"';
+                    echo '<option value="'.$color.'" '.$style.'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
                 }
         echo '</select>';
         echo '		</div>
@@ -1615,7 +1615,7 @@ 
                     <div class="controls">
                             <input type="hidden" name="action" value="" />
                             <input type="hidden" name="new_task_submit" value="true" />
-                        <button class="save" type="submit" name="Submit">' . get_lang('Save') . '</button>
+                        <button class="save" type="submit" name="Submit">' . get_lang('Save').'</button>
                     </div>
                 </div>';
         echo '</form>';
@@ -1629,34 +1629,34 @@ 
      * @author Toon Keppens
      *
      */
-    public static function display_edit_task_form ($blog_id, $task_id) {
+    public static function display_edit_task_form($blog_id, $task_id) {
         $tbl_blogs_tasks = Database::get_course_table(TABLE_BLOGS_TASKS);
         $course_id = api_get_course_int_id();
 
-        $colors = array('FFFFFF','FFFF99','FFCC99','FF9933','FF6699','CCFF99','CC9966','66FF00', '9966FF', 'CF3F3F', '990033','669933','0033FF','003366','000000');
+        $colors = array('FFFFFF', 'FFFF99', 'FFCC99', 'FF9933', 'FF6699', 'CCFF99', 'CC9966', '66FF00', '9966FF', 'CF3F3F', '990033', '669933', '0033FF', '003366', '000000');
 
-        $sql = "SELECT blog_id, task_id, title, description, color FROM $tbl_blogs_tasks WHERE c_id = $course_id AND task_id = '".(int)$task_id."'";
+        $sql = "SELECT blog_id, task_id, title, description, color FROM $tbl_blogs_tasks WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
         $result = Database::query($sql);
         $task = Database::fetch_array($result);
 
         // Display
-        echo '<form name="edit_task" method="post" action="blog.php?action=manage_tasks&blog_id=' . $blog_id . '">
-                    <legend>' . get_lang('EditTask') . '</legend>
+        echo '<form name="edit_task" method="post" action="blog.php?action=manage_tasks&blog_id='.$blog_id.'">
+                    <legend>' . get_lang('EditTask').'</legend>
                     <table width="100%" border="0" cellspacing="2">
                         <tr>
-                       <td align="right">' . get_lang('Title') . ':&nbsp;&nbsp;</td>
-                       <td><input name="task_name" type="text" size="70" value="'.Security::remove_XSS($task['title']) . '" /></td>
+                       <td align="right">' . get_lang('Title').':&nbsp;&nbsp;</td>
+                       <td><input name="task_name" type="text" size="70" value="'.Security::remove_XSS($task['title']).'" /></td>
                         </tr>
                         <tr>
-                       <td align="right">' . get_lang('Description') . ':&nbsp;&nbsp;</td>
+                       <td align="right">' . get_lang('Description').':&nbsp;&nbsp;</td>
                        <td><textarea name="task_description" cols="45">'.Security::remove_XSS($task['description']).'</textarea></td>
                         </tr>';
 
                         /* edit by Kevin Van Den Haute (kevin@develop-it.be) */
                         $tbl_tasks_permissions = Database::get_course_table(TABLE_BLOGS_TASKS_PERMISSIONS);
 
-                        $sql = " SELECT id, action FROM " . $tbl_tasks_permissions . "
-                                 WHERE c_id = $course_id AND task_id = '" . (int)$task_id."'";
+                        $sql = " SELECT id, action FROM ".$tbl_tasks_permissions."
+                                 WHERE c_id = $course_id AND task_id = '".(int) $task_id."'";
                         $result = Database::query($sql);
 
                         $arrPermissions = array();
@@ -1665,22 +1665,22 @@ 
                             $arrPermissions[] = $row['action'];
 
                             echo '<tr>';
-                            echo '<td style="text-align:right; vertical-align:top;">' . get_lang('TaskManager') . ':&nbsp;&nbsp;</td>';
+                            echo '<td style="text-align:right; vertical-align:top;">'.get_lang('TaskManager').':&nbsp;&nbsp;</td>';
                             echo '<td>';
                                 echo '<table  class="data_table" cellspacing="0" style="border-collapse:collapse; width:446px;">';
                                     echo '<tr>';
-                                        echo '<th colspan="2" style="width:223px;">' . get_lang('ArticleManager') . '</th>';
-                                        echo '<th width:223px;>' . get_lang('CommentManager') . '</th>';
+                                        echo '<th colspan="2" style="width:223px;">'.get_lang('ArticleManager').'</th>';
+                                        echo '<th width:223px;>'.get_lang('CommentManager').'</th>';
                                     echo '</tr>';
                                     echo '<tr>';
-                                        echo '<th style="width:111px;"><label for="articleDelete">' . get_lang('Delete') . '</label></th>';
-                                        echo '<th style="width:112px;"><label for="articleEdit">' . get_lang('Edit') . '</label></th>';
-                                        echo '<th style="width:223px;"><label for="commentsDelete">' . get_lang('Delete') . '</label></th>';
+                                        echo '<th style="width:111px;"><label for="articleDelete">'.get_lang('Delete').'</label></th>';
+                                        echo '<th style="width:112px;"><label for="articleEdit">'.get_lang('Edit').'</label></th>';
+                                        echo '<th style="width:223px;"><label for="commentsDelete">'.get_lang('Delete').'</label></th>';
                                     echo '</tr>';
                                     echo '<tr>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_delete', $arrPermissions)) ? 'checked ' : '') . 'id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_edit', $arrPermissions)) ? 'checked ' : '') . 'id="articleEdit" name="chkArticleEdit" type="checkbox" /></td>';
-                                        echo '<td style="text-align:center;"><input ' . ((in_array('article_comments_delete', $arrPermissions)) ? 'checked ' : '') . 'id="commentsDelete" name="chkCommentsDelete" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_delete', $arrPermissions)) ? 'checked ' : '').'id="articleDelete" name="chkArticleDelete" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_edit', $arrPermissions)) ? 'checked ' : '').'id="articleEdit" name="chkArticleEdit" type="checkbox" /></td>';
+                                        echo '<td style="text-align:center;"><input '.((in_array('article_comments_delete', $arrPermissions)) ? 'checked ' : '').'id="commentsDelete" name="chkCommentsDelete" type="checkbox" /></td>';
                                     echo '</tr>';
                                 echo '</table>';
                             echo '</td>';
@@ -1688,13 +1688,13 @@ 
                         /* end of edit */
 
                         echo '<tr>
-                       <td align="right">' . get_lang('Color') . ':&nbsp;&nbsp;</td>
+                       <td align="right">' . get_lang('Color').':&nbsp;&nbsp;</td>
                        <td>
-                        <select name="task_color" id="color" style="width: 150px; background-color: #' . $task['color'] . '" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
+                        <select name="task_color" id="color" style="width: 150px; background-color: #' . $task['color'].'" onchange="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value" onkeypress="document.getElementById(\'color\').style.backgroundColor=\'#\'+document.getElementById(\'color\').value">';
                             foreach ($colors as $color) {
                                 $selected = ($color == $task['color']) ? ' selected' : '';
-                                $style = 'style="background-color: #' . $color . '"';
-                                echo '<option value="' . $color . '" ' . $style . ' ' . $selected . ' >&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
+                                $style = 'style="background-color: #'.$color.'"';
+                                echo '<option value="'.$color.'" '.$style.' '.$selected.' >&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</option>';
                             }
         echo '			   </select>
                           </td>
@@ -1703,9 +1703,9 @@ 
                             <td align="right">&nbsp;</td>
                             <td><br /><input type="hidden" name="action" value="" />
                             <input type="hidden" name="edit_task_submit" value="true" />
-                            <input type="hidden" name="task_id" value="' . $task['task_id'] . '" />
-                            <input type="hidden" name="blog_id" value="' . $task['blog_id'] . '" />
-                            <button class="save" type="submit" name="Submit">' . get_lang('Save') . '</button></td>
+                            <input type="hidden" name="task_id" value="' . $task['task_id'].'" />
+                            <input type="hidden" name="blog_id" value="' . $task['blog_id'].'" />
+                            <button class="save" type="submit" name="Submit">' . get_lang('Save').'</button></td>
                         </tr>
                     </table>
                 </form>';
@@ -1727,7 +1727,7 @@ 
                 FROM $tbl_users user
                 INNER JOIN $tbl_blogs_rel_user blogs_rel_user
                 ON user.user_id = blogs_rel_user.user_id
-                WHERE blogs_rel_user.c_id = $course_id AND blogs_rel_user.blog_id = '".(int)$blog_id."'";
+                WHERE blogs_rel_user.c_id = $course_id AND blogs_rel_user.blog_id = '".(int) $blog_id."'";
         $result = Database::query($sql);
 
         $options = array();
@@ -1746,7 +1746,7 @@ 
                 color,
                 system_task
             FROM $tbl_blogs_tasks
-            WHERE c_id = $course_id AND blog_id = " . (int)$blog_id . "
+            WHERE c_id = $course_id AND blog_id = ".(int) $blog_id."
             ORDER BY system_task, title";
         $result = Database::query($sql);
 
@@ -1794,7 +1794,7 @@ 
      */
     public static function display_edit_assigned_task_form($blog_id, $task_id, $user_id)
     {
-        $tbl_blogs_tasks_rel_user 	= Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
+        $tbl_blogs_tasks_rel_user = Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER);
 
         $course_id = api_get_course_int_id();
 
@@ -1803,9 +1803,9 @@ 
             SELECT target_date
             FROM $tbl_blogs_tasks_rel_user
             WHERE c_id = $course_id AND
-                  blog_id = '".(int)$blog_id."' AND
-                  user_id = '".(int)$user_id."' AND
-                  task_id = '".(int)$task_id."'";
+                  blog_id = '".(int) $blog_id."' AND
+                  user_id = '".(int) $user_id."' AND
+                  task_id = '".(int) $task_id."'";
         $result = Database::query($sql);
         $row = Database::fetch_assoc($result);
 
@@ -1840,11 +1840,11 @@ 
 
         $sql = "
             SELECT COUNT(*) as 'number'
-            FROM " . $tbl_blogs_tasks_rel_user . "
+            FROM " . $tbl_blogs_tasks_rel_user."
             WHERE c_id = $course_id AND
-            blog_id = " . (int)$blog_id . "
-            AND	user_id = " . (int)$user_id . "
-            AND	task_id = " . (int)$task_id . "
+            blog_id = ".(int) $blog_id."
+            AND	user_id = " . (int) $user_id."
+            AND	task_id = " . (int) $task_id."
         ";
 
         $result = Database::query($sql);
@@ -1852,18 +1852,18 @@ 
 
         if ($row['number'] == 0) {
             $sql = "
-                INSERT INTO " . $tbl_blogs_tasks_rel_user . " (
+                INSERT INTO " . $tbl_blogs_tasks_rel_user." (
                     c_id,
                     blog_id,
                     user_id,
                     task_id,
                     target_date
                 ) VALUES (
-                    '" . (int)$course_id . "',
-                    '" . (int)$blog_id . "',
-                    '" . (int)$user_id . "',
-                    '" . (int)$task_id . "',
-                    '" . Database::escape_string($target_date) . "'
+                    '" . (int) $course_id."',
+                    '" . (int) $blog_id."',
+                    '" . (int) $user_id."',
+                    '" . (int) $task_id."',
+                    '" . Database::escape_string($target_date)."'
                 )";
 
             Database::query($sql);
@@ -1893,12 +1893,12 @@ 
         $course_id = api_get_course_int_id();
 
         $sql = "SELECT COUNT(*) as 'number'
-                FROM " . $tbl_blogs_tasks_rel_user . "
+                FROM " . $tbl_blogs_tasks_rel_user."
                 WHERE
                     c_id = $course_id AND
-                    blog_id = " . (int)$blog_id . " AND
-                    user_id = " . (int)$user_id . " AND
-                    task_id = " . (int)$task_id . "
+                    blog_id = ".(int) $blog_id." AND
+                    user_id = " . (int) $user_id." AND
+                    task_id = " . (int) $task_id."
             ";
 
         $result = Database::query($sql);
@@ -1906,17 +1906,17 @@ 
 
         if ($row['number'] == 0 || ($row['number'] != 0 && $task_id == $old_task_id && $user_id == $old_user_id)) {
             $sql = "
-                UPDATE " . $tbl_blogs_tasks_rel_user . "
+                UPDATE " . $tbl_blogs_tasks_rel_user."
                 SET
-                    user_id = " . (int)$user_id . ",
-                    task_id = " . (int)$task_id . ",
-                    target_date = '" . Database::escape_string($target_date) . "'
+                    user_id = " . (int) $user_id.",
+                    task_id = " . (int) $task_id.",
+                    target_date = '" . Database::escape_string($target_date)."'
                 WHERE
                     c_id = $course_id AND
-                    blog_id = " . (int)$blog_id . " AND
-                    user_id = " . (int)$old_user_id . " AND
-                    task_id = " . (int)$old_task_id . " AND
-                    target_date = '" . Database::escape_string($old_target_date) . "'
+                    blog_id = ".(int) $blog_id." AND
+                    user_id = " . (int) $old_user_id." AND
+                    task_id = " . (int) $old_task_id." AND
+                    target_date = '" . Database::escape_string($old_target_date)."'
             ";
             Database::query($sql);
         }
@@ -1937,7 +1937,7 @@ 
 
 
         $sql = "SELECT title, description FROM $tbl_blogs_tasks
-                WHERE task_id = '".(int)$task_id."'
+                WHERE task_id = '".(int) $task_id."'
                 AND c_id = $course_id";
         $result = Database::query($sql);
         $row = Database::fetch_assoc($result);
@@ -1945,19 +1945,19 @@ 
         $sql = "SELECT post.*, user.lastname, user.firstname, user.username
                 FROM $tbl_blogs_posts post
                 INNER JOIN $tbl_users user ON post.author_id = user.user_id
-                WHERE post.blog_id = '".(int)$blog_id."' AND post.c_id = $course_id
+                WHERE post.blog_id = '".(int) $blog_id."' AND post.c_id = $course_id
                 ORDER BY post_id DESC
                 LIMIT 0, 100";
         $result = Database::query($sql);
 
         // Display
-        echo '<span class="blogpost_title">' . get_lang('SelectTaskArticle') . ' "' . stripslashes($row['title']) . '"</span>';
-        echo '<span style="font-style: italic;"">'.stripslashes($row['description']) . '</span><br><br>';
+        echo '<span class="blogpost_title">'.get_lang('SelectTaskArticle').' "'.stripslashes($row['title']).'"</span>';
+        echo '<span style="font-style: italic;"">'.stripslashes($row['description']).'</span><br><br>';
 
         if (Database::num_rows($result) > 0) {
-            while($blog_post = Database::fetch_array($result)) {
+            while ($blog_post = Database::fetch_array($result)) {
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $blog_post['username']), ENT_QUOTES);
-                echo '<a href="blog.php?action=execute_task&blog_id=' . $blog_id . '&task_id=' . $task_id . '&post_id=' . $blog_post['post_id'] . '#add_comment">'.stripslashes($blog_post['title']) . '</a>, ' . get_lang('WrittenBy') . ' ' . stripslashes(Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username))) . '<br />';
+                echo '<a href="blog.php?action=execute_task&blog_id='.$blog_id.'&task_id='.$task_id.'&post_id='.$blog_post['post_id'].'#add_comment">'.stripslashes($blog_post['title']).'</a>, '.get_lang('WrittenBy').' '.stripslashes(Display::tag('span', api_get_person_name($blog_post['firstname'], $blog_post['lastname']), array('title'=>$username))).'<br />';
             }
         } else {
             echo get_lang('NoArticles');
@@ -1981,12 +1981,12 @@ 
 
         // Subscribe the user
         $sql = "INSERT INTO $tbl_blogs_rel_user (c_id, blog_id, user_id )
-                VALUES ($course_id, '".(int)$blog_id."', '".(int)$user_id."');";
+                VALUES ($course_id, '".(int) $blog_id."', '".(int) $user_id."');";
         Database::query($sql);
 
         // Give this user basic rights
         $sql = "INSERT INTO $tbl_user_permissions (c_id, user_id,tool,action)
-                VALUES ($course_id, '".(int)$user_id."','BLOG_" . (int)$blog_id."','article_add')";
+                VALUES ($course_id, '".(int) $user_id."','BLOG_".(int) $blog_id."','article_add')";
         Database::query($sql);
 
         $id = Database::insert_id();
@@ -1996,7 +1996,7 @@ 
         }
 
         $sql = "INSERT INTO $tbl_user_permissions (c_id, user_id,tool,action)
-                VALUES ($course_id, '".(int)$user_id."','BLOG_" . (int)$blog_id."','article_comments_add')";
+                VALUES ($course_id, '".(int) $user_id."','BLOG_".(int) $blog_id."','article_comments_add')";
         Database::query($sql);
 
         $id = Database::insert_id();
@@ -2022,12 +2022,12 @@ 
 
         // Unsubscribe the user
         $sql = "DELETE FROM $tbl_blogs_rel_user
-                WHERE blog_id = '".(int)$blog_id."' AND user_id = '".(int)$user_id."'";
+                WHERE blog_id = '".(int) $blog_id."' AND user_id = '".(int) $user_id."'";
         Database::query($sql);
 
         // Remove this user's permissions.
         $sql = "DELETE FROM $tbl_user_permissions
-                WHERE user_id = '".(int)$user_id."'";
+                WHERE user_id = '".(int) $user_id."'";
         Database::query($sql);
     }
 
@@ -2048,7 +2048,7 @@ 
         $course_id = $_course['real_id'];
 
         $currentCourse = $_course['code'];
-        $tbl_users 			= Database::get_main_table(TABLE_MAIN_USER);
+        $tbl_users = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_blogs_rel_user = Database::get_course_table(TABLE_BLOGS_REL_USER);
 
         echo '<legend>'.get_lang('SubscribeMembers').'</legend>';
@@ -2063,12 +2063,12 @@ 
         $result = Database::query($sql);
 
         $blog_member_ids = array();
-        while($user = Database::fetch_array($result)) {
+        while ($user = Database::fetch_array($result)) {
             $blog_member_ids[] = $user['user_id'];
         }
 
         // Set table headers
-        $column_header[] = array ('', false, '');
+        $column_header[] = array('', false, '');
         if ($is_western_name_order) {
             $column_header[] = array(get_lang('FirstName'), true, '');
             $column_header[] = array(get_lang('LastName'), true, '');
@@ -2088,13 +2088,13 @@ 
 
         // Add users that are not in this blog to the list.
         foreach ($student_list as $key=>$user) {
-            if(isset($user['id_user'])) {
+            if (isset($user['id_user'])) {
                 $user['user_id'] = $user['id_user'];
             }
-            if(!in_array($user['user_id'],$blog_member_ids)) {
+            if (!in_array($user['user_id'], $blog_member_ids)) {
                 $a_infosUser = api_get_user_info($user['user_id']);
-                $row = array ();
-                $row[] = '<input type="checkbox" name="user[]" value="' . $a_infosUser['user_id'] . '" '.((isset($_GET['selectall']) && $_GET['selectall'] == "subscribe") ? ' checked="checked" ' : '') . '/>';
+                $row = array();
+                $row[] = '<input type="checkbox" name="user[]" value="'.$a_infosUser['user_id'].'" '.((isset($_GET['selectall']) && $_GET['selectall'] == "subscribe") ? ' checked="checked" ' : '').'/>';
                 $username = api_htmlentities(sprintf(get_lang('LoginX'), $a_infosUser["username"]), ENT_QUOTES);
                 if ($is_western_name_order) {
                     $row[] = $a_infosUser["firstname"];
@@ -2106,8 +2106,8 @@ 
                 $row[] = Display::icon_mailto_link($a_infosUser["email"]);
 
                 //Link to register users
-                if ($a_infosUser["user_id"] != $_SESSION['_user']['user_id']){
-                    $row[] = "<a class=\"btn btn-primary \" href=\"" .api_get_self()."?action=manage_members&blog_id=$blog_id&register=yes&user_id=" . $a_infosUser["user_id"]."\">" . get_lang('Register')."</a>";
+                if ($a_infosUser["user_id"] != $_SESSION['_user']['user_id']) {
+                    $row[] = "<a class=\"btn btn-primary \" href=\"".api_get_self()."?action=manage_members&blog_id=$blog_id&register=yes&user_id=".$a_infosUser["user_id"]."\">".get_lang('Register')."</a>";
                 } else {
                     $row[] = '';
                 }
@@ -2118,20 +2118,20 @@ 
         // Display
         $query_vars['action'] = 'manage_members';
         $query_vars['blog_id'] = $blog_id;
-        echo '<form method="post" action="blog.php?action=manage_members&blog_id=' . $blog_id . '">';
-            Display::display_sortable_table($column_header, $user_data,null,null,$query_vars);
+        echo '<form method="post" action="blog.php?action=manage_members&blog_id='.$blog_id.'">';
+            Display::display_sortable_table($column_header, $user_data, null, null, $query_vars);
             $link = '';
-            $link .= isset ($_GET['action']) ? 'action=' . Security::remove_XSS($_GET['action']) . '&' : '';
+            $link .= isset ($_GET['action']) ? 'action='.Security::remove_XSS($_GET['action']).'&' : '';
             $link .= "blog_id=$blog_id&";
 
-            echo '<a href="blog.php?' . $link . 'selectall=subscribe">' . get_lang('SelectAll') . '</a> - ';
-            echo '<a href="blog.php?' . $link . '">' . get_lang('UnSelectAll') . '</a> ';
-            echo get_lang('WithSelected') . ' : ';
+            echo '<a href="blog.php?'.$link.'selectall=subscribe">'.get_lang('SelectAll').'</a> - ';
+            echo '<a href="blog.php?'.$link.'">'.get_lang('UnSelectAll').'</a> ';
+            echo get_lang('WithSelected').' : ';
             echo '<select name="action">';
-            echo '<option value="select_subscribe">' . get_lang('Register') . '</option>';
+            echo '<option value="select_subscribe">'.get_lang('Register').'</option>';
             echo '</select>';
             echo '<input type="hidden" name="register" value="true" />';
-            echo '<button class="save" type="submit">' . get_lang('Ok') . '</button>';
+            echo '<button class="save" type="submit">'.get_lang('Ok').'</button>';
         echo '</form>';
     }
 
@@ -2144,7 +2144,7 @@ 
      *
      * @return false|null Form with sortable table with users to unsubcribe from a blog.
      */
-    public static function display_form_user_unsubscribe ($blog_id)
+    public static function display_form_user_unsubscribe($blog_id)
     {
         $_user = api_get_user_info();
         $is_western_name_order = api_is_western_name_order();
@@ -2157,34 +2157,34 @@ 
 
         $properties["width"] = "100%";
         //table column titles
-        $column_header[] = array ('', false, '');
+        $column_header[] = array('', false, '');
         if ($is_western_name_order) {
-            $column_header[] = array (get_lang('FirstName'), true, '');
-            $column_header[] = array (get_lang('LastName'), true, '');
+            $column_header[] = array(get_lang('FirstName'), true, '');
+            $column_header[] = array(get_lang('LastName'), true, '');
         } else {
-            $column_header[] = array (get_lang('LastName'), true, '');
-            $column_header[] = array (get_lang('FirstName'), true, '');
+            $column_header[] = array(get_lang('LastName'), true, '');
+            $column_header[] = array(get_lang('FirstName'), true, '');
         }
-        $column_header[] = array (get_lang('Email'), false, '');
-        $column_header[] = array (get_lang('TaskManager'), true, '');
-        $column_header[] = array (get_lang('UnRegister'), false, '');
+        $column_header[] = array(get_lang('Email'), false, '');
+        $column_header[] = array(get_lang('TaskManager'), true, '');
+        $column_header[] = array(get_lang('UnRegister'), false, '');
 
         $course_id = api_get_course_int_id();
 
         $sql = "SELECT user.user_id, user.lastname, user.firstname, user.email, user.username
                 FROM $tbl_users user INNER JOIN $tbl_blogs_rel_user blogs_rel_user
                 ON user.user_id = blogs_rel_user.user_id
-                WHERE blogs_rel_user.c_id = $course_id AND  blogs_rel_user.blog_id = '".(int)$blog_id."'";
+                WHERE blogs_rel_user.c_id = $course_id AND  blogs_rel_user.blog_id = '".(int) $blog_id."'";
 
         if (!($sql_result = Database::query($sql))) {
             return false;
         }
 
-        $user_data = array ();
+        $user_data = array();
 
         while ($myrow = Database::fetch_array($sql_result)) {
-            $row = array ();
-            $row[] = '<input type="checkbox" name="user[]" value="' . $myrow['user_id'] . '" '.((isset($_GET['selectall']) && $_GET['selectall'] == "unsubscribe") ? ' checked="checked" ' : '') . '/>';
+            $row = array();
+            $row[] = '<input type="checkbox" name="user[]" value="'.$myrow['user_id'].'" '.((isset($_GET['selectall']) && $_GET['selectall'] == "unsubscribe") ? ' checked="checked" ' : '').'/>';
             $username = api_htmlentities(sprintf(get_lang('LoginX'), $myrow["username"]), ENT_QUOTES);
             if ($is_western_name_order) {
                 $row[] = $myrow["firstname"];
@@ -2196,19 +2196,19 @@ 
             $row[] = Display::icon_mailto_link($myrow["email"]);
 
             $sql = "SELECT bt.title task
-                    FROM " . Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER) . " btu
-                    INNER JOIN " . Database::get_course_table(TABLE_BLOGS_TASKS) . " bt
+                    FROM " . Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER)." btu
+                    INNER JOIN " . Database::get_course_table(TABLE_BLOGS_TASKS)." bt
                     ON btu.task_id = bt.task_id
                     WHERE 	btu.c_id 	= $course_id  AND
                             bt.c_id 	= $course_id  AND
                             btu.blog_id = $blog_id AND
-                            btu.user_id = " . $myrow['user_id'];
+                            btu.user_id = ".$myrow['user_id'];
             $sql_res = Database::query($sql);
 
             $task = '';
 
-            while($r = Database::fetch_array($sql_res)) {
-                $task .= stripslashes($r['task']) . ', ';
+            while ($r = Database::fetch_array($sql_res)) {
+                $task .= stripslashes($r['task']).', ';
             }
             //echo $task;
             $task = (api_strlen(trim($task)) != 0) ? api_substr($task, 0, api_strlen($task) - 2) : get_lang('Reader');
@@ -2216,7 +2216,7 @@ 
             //Link to register users
 
             if ($myrow["user_id"] != $_user['user_id']) {
-                $row[] = "<a class=\"btn btn-primary\" href=\"" .api_get_self()."?action=manage_members&blog_id=$blog_id&unregister=yes&user_id=" . $myrow['user_id']."\">" . get_lang('UnRegister')."</a>";
+                $row[] = "<a class=\"btn btn-primary\" href=\"".api_get_self()."?action=manage_members&blog_id=$blog_id&unregister=yes&user_id=".$myrow['user_id']."\">".get_lang('UnRegister')."</a>";
             } else {
                 $row[] = '';
             }
@@ -2226,20 +2226,20 @@ 
 
         $query_vars['action'] = 'manage_members';
         $query_vars['blog_id'] = $blog_id;
-        echo '<form method="post" action="blog.php?action=manage_members&blog_id=' . $blog_id . '">';
-        Display::display_sortable_table($column_header, $user_data,null,null,$query_vars);
+        echo '<form method="post" action="blog.php?action=manage_members&blog_id='.$blog_id.'">';
+        Display::display_sortable_table($column_header, $user_data, null, null, $query_vars);
         $link = '';
-        $link .= isset ($_GET['action']) ? 'action=' . Security::remove_XSS($_GET['action']). '&' : '';
+        $link .= isset ($_GET['action']) ? 'action='.Security::remove_XSS($_GET['action']).'&' : '';
         $link .= "blog_id=$blog_id&";
 
-        echo '<a href="blog.php?' . $link . 'selectall=unsubscribe">' . get_lang('SelectAll') . '</a> - ';
-        echo '<a href="blog.php?' . $link . '">' . get_lang('UnSelectAll') . '</a> ';
-        echo get_lang('WithSelected') . ' : ';
+        echo '<a href="blog.php?'.$link.'selectall=unsubscribe">'.get_lang('SelectAll').'</a> - ';
+        echo '<a href="blog.php?'.$link.'">'.get_lang('UnSelectAll').'</a> ';
+        echo get_lang('WithSelected').' : ';
         echo '<select name="action">';
-        echo '<option value="select_unsubscribe">' . get_lang('UnRegister') . '</option>';
+        echo '<option value="select_unsubscribe">'.get_lang('UnRegister').'</option>';
         echo '</select>';
         echo '<input type="hidden" name="unregister" value="true" />';
-        echo '<button class="save" type="submit">' . get_lang('Ok') . '</button>';
+        echo '<button class="save" type="submit">'.get_lang('Ok').'</button>';
         echo '</form>';
     }
 
@@ -2249,7 +2249,7 @@ 
      *
      * @param Integer $blog_id
      */
-    public static function display_form_user_rights ($blog_id)
+    public static function display_form_user_rights($blog_id)
     {
         echo '<legend>'.get_lang('RightsManager').'</legend>';
         echo '<br />';
@@ -2270,7 +2270,7 @@ 
         $form = new FormValidator(
             'add_post',
             'post',
-            api_get_path(WEB_CODE_PATH)."blog/blog.php?action=view_post&blog_id=" . intval($blog_id)  . "&post_id=".intval($post_id)."&".api_get_cidreq(),
+            api_get_path(WEB_CODE_PATH)."blog/blog.php?action=view_post&blog_id=".intval($blog_id)."&post_id=".intval($post_id)."&".api_get_cidreq(),
             null,
             array('enctype' => 'multipart/form-data')
         );
@@ -2336,21 +2336,21 @@ 
         $course_id = api_get_course_int_id();
 
         //Handle leap year
-        $numberofdays = array (0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
+        $numberofdays = array(0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31);
 
-        if(($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0))
+        if (($year % 400 == 0) or ($year % 4 == 0 and $year % 100 <> 0))
             $numberofdays[2] = 29;
 
         //Get the first day of the month
         $dayone = getdate(mktime(0, 0, 0, $month, 1, $year));
-        $monthName = $MonthsLong[$month-1];
+        $monthName = $MonthsLong[$month - 1];
 
         //Start the week on monday
         $startdayofweek = $dayone['wday'] <> 0 ? ($dayone['wday'] - 1) : 6;
         $blogId = isset($_GET['blog_id']) ? intval($_GET['blog_id']) : null;
         $filter = isset($_GET['filter']) ? Security::remove_XSS($_GET['filter']) : null;
-        $backwardsURL = api_get_self()."?blog_id=" . $blogId."&filter=" . $filter."&month=". ($month == 1 ? 12 : $month -1)."&year=". ($month == 1 ? $year -1 : $year);
-        $forewardsURL = api_get_self()."?blog_id=" . $blogId."&filter=" . $filter."&month=". ($month == 12 ? 1 : $month +1)."&year=". ($month == 12 ? $year +1 : $year);
+        $backwardsURL = api_get_self()."?blog_id=".$blogId."&filter=".$filter."&month=".($month == 1 ? 12 : $month - 1)."&year=".($month == 1 ? $year - 1 : $year);
+        $forewardsURL = api_get_self()."?blog_id=".$blogId."&filter=".$filter."&month=".($month == 12 ? 1 : $month + 1)."&year=".($month == 12 ? $year + 1 : $year);
 
         // Get posts for this month
         $sql = "SELECT post.*, DAYOFMONTH(date_creation) as post_day, user.lastname, user.firstname
@@ -2359,15 +2359,15 @@ 
                 ON post.author_id = user.user_id
                 WHERE
                     post.c_id = $course_id AND
-                    post.blog_id = '".(int)$blog_id."' AND
-                    MONTH(date_creation) = '".(int)$month."' AND
-                    YEAR(date_creation) = '".(int)$year."'
+                    post.blog_id = '".(int) $blog_id."' AND
+                    MONTH(date_creation) = '".(int) $month."' AND
+                    YEAR(date_creation) = '".(int) $year."'
                 ORDER BY date_creation";
         $result = Database::query($sql);
 
         // We will create an array of days on which there are posts.
-        if( Database::num_rows($result) > 0) {
-            while($blog_post = Database::fetch_array($result)) {
+        if (Database::num_rows($result) > 0) {
+            while ($blog_post = Database::fetch_array($result)) {
                 // If the day of this post is not yet in the array, add it.
                 if (!in_array($blog_post['post_day'], $posts))
                     $posts[] = $blog_post['post_day'];
@@ -2384,9 +2384,9 @@ 
                     task_rel_user.c_id = $course_id AND
                     task.c_id = $course_id AND
                     blog.c_id = $course_id AND
-                    task_rel_user.user_id = '".(int)$_user['user_id']."' AND
-                    MONTH(target_date) = '".(int)$month."' AND
-                    YEAR(target_date) = '".(int)$year."'
+                    task_rel_user.user_id = '".(int) $_user['user_id']."' AND
+                    MONTH(target_date) = '".(int) $month."' AND
+                    YEAR(target_date) = '".(int) $year."'
                 ORDER BY target_date ASC";
             $result = Database::query($sql);
 
@@ -2409,7 +2409,7 @@ 
 
         echo "<tr>";
 
-        for($ii = 1; $ii < 8; $ii ++)
+        for ($ii = 1; $ii < 8; $ii++)
             echo "<td class=\"weekdays\">", $DaysShort[$ii % 7], "</td>";
 
         echo "</tr>";
@@ -2419,24 +2419,24 @@ 
 
         while ($curday <= $numberofdays[$month]) {
             echo "<tr>";
-            for ($ii = 0; $ii < 7; $ii ++) {
+            for ($ii = 0; $ii < 7; $ii++) {
                 if (($curday == -1) && ($ii == $startdayofweek))
                     $curday = 1;
 
                 if (($curday > 0) && ($curday <= $numberofdays[$month])) {
-                    $bgcolor = $ii < 5 ? $class="class=\"days_week\"" : $class="class=\"days_weekend\"";
+                    $bgcolor = $ii < 5 ? $class = "class=\"days_week\"" : $class = "class=\"days_weekend\"";
                     $dayheader = "$curday";
 
-                    if(($curday == $today['mday']) && ($year == $today['year']) && ($month == $today['mon'])) {
+                    if (($curday == $today['mday']) && ($year == $today['year']) && ($month == $today['mon'])) {
                         $dayheader = "$curday";
                         $class = "class=\"days_today\"";
                     }
 
-                    echo "<td " . $class.">";
+                    echo "<td ".$class.">";
 
                     // If there are posts on this day, create a filter link.
-                    if(in_array($curday, $posts))
-                        echo '<a href="blog.php?blog_id=' . $blog_id . '&filter=' . $year . '-' . $month . '-' . $curday . '&month=' . $month . '&year=' . $year . '" title="' . get_lang('ViewPostsOfThisDay') . '">' . $curday . '</a>';
+                    if (in_array($curday, $posts))
+                        echo '<a href="blog.php?blog_id='.$blog_id.'&filter='.$year.'-'.$month.'-'.$curday.'&month='.$month.'&year='.$year.'" title="'.get_lang('ViewPostsOfThisDay').'">'.$curday.'</a>';
                     else
                         echo $dayheader;
 
@@ -2444,7 +2444,7 @@ 
                         if (isset($tasks[$curday]) && is_array($tasks[$curday])) {
                             // Add tasks to calendar
                             foreach ($tasks[$curday] as $task) {
-                                echo '<a href="blog.php?action=execute_task&blog_id=' . $task['blog_id'] . '&task_id='.stripslashes($task['task_id']) . '" title="' . $task['title'] . ' : ' . get_lang('InBlog') . ' : ' . $task['blog_name'] . ' - ' . get_lang('ExecuteThisTask') . '">';
+                                echo '<a href="blog.php?action=execute_task&blog_id='.$task['blog_id'].'&task_id='.stripslashes($task['task_id']).'" title="'.$task['title'].' : '.get_lang('InBlog').' : '.$task['blog_name'].' - '.get_lang('ExecuteThisTask').'">';
                                 echo Display::return_icon('blog_task.gif', get_lang('ExecuteThisTask'));
                                 echo '</a>';
                             }
@@ -2452,7 +2452,7 @@ 
                     }
 
                     echo "</td>";
-                    $curday ++;
+                    $curday++;
                 } else
                     echo "<td>&nbsp;</td>";
             }
@@ -2490,7 +2490,7 @@ 
     public static function display_edit_blog_form($blog_id)
     {
         $course_id = api_get_course_int_id();
-        $blog_id= intval($blog_id);
+        $blog_id = intval($blog_id);
         $tbl_blogs = Database::get_course_table(TABLE_BLOGS);
 
         $sql = "SELECT blog_id, blog_name, blog_subtitle
@@ -2505,7 +2505,7 @@ 
             $blog['blog_subtitle'] = Security::remove_XSS($_POST['blog_subtitle']);
         }
 
-        $form = new FormValidator('edit_blog', 'post','blog_admin.php?action=edit&blog_id='.intval($_GET['blog_id']));
+        $form = new FormValidator('edit_blog', 'post', 'blog_admin.php?action=edit&blog_id='.intval($_GET['blog_id']));
         $form->addElement('header', get_lang('EditBlog'));
         $form->addElement('text', 'blog_name', get_lang('Title'));
         $form->addElement('textarea', 'blog_subtitle', get_lang('SubTitle'));
@@ -2541,8 +2541,8 @@ 
         $result = Database::query($sql);
         $list_info = array();
         if (Database::num_rows($result)) {
-            while ($row_project=Database::fetch_row($result)) {
-                $list_info[]=$row_project;
+            while ($row_project = Database::fetch_row($result)) {
+                $list_info[] = $row_project;
             }
         }
 
@@ -2554,35 +2554,35 @@ 
                 // Validation when belongs to a session
                 $session_img = api_get_session_image($info_log[4], $_user['status']);
 
-                $url_start_blog = 'blog.php' ."?". "blog_id=".$info_log[3]. "&".api_get_cidreq();
+                $url_start_blog = 'blog.php'."?"."blog_id=".$info_log[3]."&".api_get_cidreq();
                 $title = $info_log[0];
                         $image = Display::return_icon('blog.png', $title);
-                $list_name = '<div style="float: left; width: 35px; height: 22px;"><a href="'.$url_start_blog.'">' . $image . '</a></div><a href="'.$url_start_blog.'">' .$title. '</a>' . $session_img;
+                $list_name = '<div style="float: left; width: 35px; height: 22px;"><a href="'.$url_start_blog.'">'.$image.'</a></div><a href="'.$url_start_blog.'">'.$title.'</a>'.$session_img;
 
                 $list_body_blog[] = $list_name;
                 $list_body_blog[] = $info_log[1];
 
-                $visibility_icon=($info_log[2]==0) ? 'invisible' : 'visible';
-                $visibility_info=($info_log[2]==0) ? 'Visible' : 'Invisible';
-                $my_image = '<a href="' .api_get_self(). '?action=edit&blog_id=' . $info_log[3] . '">';
-                                $my_image.= Display::return_icon('edit.png', get_lang('EditBlog'));
+                $visibility_icon = ($info_log[2] == 0) ? 'invisible' : 'visible';
+                $visibility_info = ($info_log[2] == 0) ? 'Visible' : 'Invisible';
+                $my_image = '<a href="'.api_get_self().'?action=edit&blog_id='.$info_log[3].'">';
+                                $my_image .= Display::return_icon('edit.png', get_lang('EditBlog'));
 
-                $my_image.= "</a>";
-                $my_image.= '<a href="' .api_get_self(). '?action=delete&blog_id=' . $info_log[3] . '" ';
-                $my_image.= 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset)). '\')) return false;" >';
-                                $my_image.= Display::return_icon('delete.png', get_lang('DeleteBlog'));
+                $my_image .= "</a>";
+                $my_image .= '<a href="'.api_get_self().'?action=delete&blog_id='.$info_log[3].'" ';
+                $my_image .= 'onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset)).'\')) return false;" >';
+                                $my_image .= Display::return_icon('delete.png', get_lang('DeleteBlog'));
 
-                $my_image.= "</a>";
-                $my_image.= '<a href="' .api_get_self(). '?action=visibility&blog_id=' . $info_log[3] . '">';
-                                $my_image.= Display::return_icon($visibility_icon . '.gif', get_lang($visibility_info));
+                $my_image .= "</a>";
+                $my_image .= '<a href="'.api_get_self().'?action=visibility&blog_id='.$info_log[3].'">';
+                                $my_image .= Display::return_icon($visibility_icon.'.gif', get_lang($visibility_info));
 
-                $my_image.= "</a>";
-                $list_body_blog[]=$my_image;
-                $list_content_blog[]=$list_body_blog;
+                $my_image .= "</a>";
+                $list_body_blog[] = $my_image;
+                $list_content_blog[] = $list_body_blog;
                 $list_body_blog = array();
             }
 
-            $table = new SortableTableFromArrayConfig($list_content_blog, 1,20,'project');
+            $table = new SortableTableFromArrayConfig($list_content_blog, 1, 20, 'project');
             $table->set_header(0, get_lang('Title'));
             $table->set_header(1, get_lang('SubTitle'));
             $table->set_header(2, get_lang('Modify'));
@@ -2607,34 +2607,34 @@ 
  * @author Julio Montoya
  * @version avril 2008, dokeos 1.8.5
  */
-function get_blog_attachment($blog_id, $post_id=null,$comment_id=null)
+function get_blog_attachment($blog_id, $post_id = null, $comment_id = null)
 {
 	$blog_table_attachment = Database::get_course_table(TABLE_BLOGS_ATTACHMENT);
 
 	$blog_id = intval($blog_id);
 	$comment_id = intval($comment_id);
 	$post_id = intval($post_id);
-	$row=array();
-	$where='';
+	$row = array();
+	$where = '';
 	if (!empty ($post_id) && is_numeric($post_id)) {
-		$where.=' AND post_id ="'.$post_id.'" ';
+		$where .= ' AND post_id ="'.$post_id.'" ';
 	}
 
 	if (!empty ($comment_id) && is_numeric($comment_id)) {
 		if (!empty ($post_id)) {
-			$where.= ' AND ';
+			$where .= ' AND ';
 		}
-		$where.=' comment_id ="'.$comment_id.'" ';
+		$where .= ' comment_id ="'.$comment_id.'" ';
 	}
 
     $course_id = api_get_course_int_id();
 
-	$sql = 'SELECT path, filename, comment FROM '. $blog_table_attachment.'
+	$sql = 'SELECT path, filename, comment FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
 
-	$result=Database::query($sql);
-	if (Database::num_rows($result)!=0) {
-		$row=Database::fetch_array($result);
+	$result = Database::query($sql);
+	if (Database::num_rows($result) != 0) {
+		$row = Database::fetch_array($result);
 	}
 	return $row;
 }
@@ -2674,22 +2674,22 @@ 
     }
 
 	// delete all files in directory
-	$courseDir   = $_course['path'].'/upload/blog';
+	$courseDir = $_course['path'].'/upload/blog';
 	$sys_course_path = api_get_path(SYS_COURSE_PATH);
 	$updir = $sys_course_path.$courseDir;
 
 	$sql = 'SELECT path FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND blog_id ="'.intval($blog_id).'"  '.$where;
-	$result=Database::query($sql);
+	$result = Database::query($sql);
 
-	while ($row=Database::fetch_row($result)) {
-		$file=$updir.'/'.$row[0];
-		if (Security::check_abs_path($file,$updir) )
+	while ($row = Database::fetch_row($result)) {
+		$file = $updir.'/'.$row[0];
+		if (Security::check_abs_path($file, $updir))
 		{
 			@ unlink($file);
 		}
 	}
-	$sql = 'DELETE FROM '. $blog_table_attachment.'
+	$sql = 'DELETE FROM '.$blog_table_attachment.'
 	        WHERE c_id = '.$course_id.' AND  blog_id ="'.intval($blog_id).'"  '.$where;
 	Database::query($sql);
 }
@@ -2719,13 +2719,13 @@ 
 	$result = Database::query($sql);
 	$return_data = '';
 
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png',get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.= $row['full_text'];
-			$return_data.= '<br /><br />';
+	if (Database::num_rows($result) != 0) {
+		while ($row = Database::fetch_array($result)) {
+			$return_data .= '<div class="clear"></div><br />';
+			$return_data .= '<div class="actions" style="margin-left:5px;margin-right:5px;">'.Display::return_icon('blog_article.png', get_lang('BlogPosts')).' '.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.$my_course_id.' " >'.get_lang('SeeBlog').'</a></div></div>';
+			$return_data .= '<br / >';
+			$return_data .= $row['full_text'];
+			$return_data .= '<br /><br />';
 		}
 	}
 	return $return_data;
@@ -2755,13 +2755,13 @@ 
 			ORDER BY blog_name";
 	$result = Database::query($sql);
 	$return_data = '';
-	if (Database::num_rows($result)!=0) {
-		while ($row=Database::fetch_array($result)) {
-			$return_data.=  '<div class="clear"></div><br />';
-			$return_data.=  '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
-			$return_data.=  '<br / >';
-			$return_data.=  $row['comment'];
-			$return_data.=  '<br />';
+	if (Database::num_rows($result) != 0) {
+		while ($row = Database::fetch_array($result)) {
+			$return_data .= '<div class="clear"></div><br />';
+			$return_data .= '<div class="actions" style="margin-left:5px;margin-right:5px;">'.$row['title'].'&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<div style="float:right;margin-top:-18px"><a href="../blog/blog.php?blog_id='.$row['blog_id'].'&gidReq=&cidReq='.Security::remove_XSS($course_code).' " >'.get_lang('SeeBlog').'</a></div></div>';
+			$return_data .= '<br / >';
+			$return_data .= $row['comment'];
+			$return_data .= '<br />';
 		}
 	}
 	return $return_data;

main/inc/lib/grade_model.lib.php

Doc Comments +2 added lines, -2 removed lines

@@ -259,10 +259,10 @@
     }
 
     /**
-     * @param $form
+     * @param CatForm $form
      * @param string $name
      * @param null $default_value
-     * @return bool
+     * @return false|null
      */
     public function fill_grade_model_select_in_form(&$form, $name = 'gradebook_model_id', $default_value = null)
     {

Indentation +12 added lines, -12 removed lines

@@ -51,15 +51,15 @@ 
     /**
      * Displays the title + grid
      */
-	public function display()
+    public function display()
     {
-		// action links
-		echo '<div class="actions" style="margin-bottom:20px">';
+        // action links
+        echo '<div class="actions" style="margin-bottom:20px">';
         echo '<a href="grade_models.php">'.Display::return_icon('back.png',get_lang('Back'),'', ICON_SIZE_MEDIUM).'</a>';
-		echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
-		echo '</div>';
+        echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
+        echo '</div>';
         echo Display::grid_html('grade_model');
-	}
+    }
 
     /**
      * Returns a Form validator Obj
@@ -117,7 +117,7 @@ 
         }
 
         $form->addElement('hidden', 'maxvalue', '100');
-		$form->addElement('hidden', 'minvalue', '0');
+        $form->addElement('hidden', 'minvalue', '0');
         $renderer = & $form->defaultRenderer();
 
         $component_array = array();
@@ -192,7 +192,7 @@ 
         // Setting the rules
         $form->addRule('name', get_lang('ThisFieldIsRequired'), 'required');
 
-		return $form;
+        return $form;
     }
 
     /**
@@ -255,7 +255,7 @@ 
      */
     public function delete($id)
     {
-	    parent::delete($id);
+        parent::delete($id);
     }
 
     /**
@@ -309,11 +309,11 @@ 
     /**
      * GradeModelComponents constructor.
      */
-	public function __construct()
+    public function __construct()
     {
         parent::__construct();
         $this->table = Database::get_main_table(TABLE_GRADE_MODEL_COMPONENTS);
-	}
+    }
 
     /**
      * @param array $params
@@ -322,7 +322,7 @@ 
      */
     public function save($params, $show_query = false)
     {
-	    $id = parent::save($params, $show_query);
+        $id = parent::save($params, $show_query);
 
         return $id;
     }

Spacing +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct()
     {
         parent::__construct();
-        $this->table =  Database::get_main_table(TABLE_GRADE_MODEL);
+        $this->table = Database::get_main_table(TABLE_GRADE_MODEL);
     }
 
     /**
@@ -55,7 +55,7 @@ 
     {
 		// action links
 		echo '<div class="actions" style="margin-bottom:20px">';
-        echo '<a href="grade_models.php">'.Display::return_icon('back.png',get_lang('Back'),'', ICON_SIZE_MEDIUM).'</a>';
+        echo '<a href="grade_models.php">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
 		echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
 		echo '</div>';
         echo Display::grid_html('grade_model');
@@ -129,7 +129,7 @@ 
             $form->addElement('hidden', 'components['.$i.'][id]', null);
 
             $template_percentage =
-            '<div id=' . $i . ' style="display: '.(($i<=$nr_items)?'inline':'none').';" class="form-group">                
+            '<div id='.$i.' style="display: '.(($i <= $nr_items) ? 'inline' : 'none').';" class="form-group">                
                 <label for="" class="col-sm-2 control-label">
                     {label}
                 </label>
@@ -143,11 +143,11 @@ 
 
             $template_title =
             '&nbsp{element} <!-- BEGIN error --> <span class="form_error">{error}</span><!-- END error -->
-             <a href="javascript:plusItem(' . ($counter+1) . ')">
-                '.Display::return_icon('add.png', get_lang('Add'), ['id' => 'plus-' . ($counter+1), 'style' => 'display: '.(($counter>=$nr_items) ? 'inline':'none') ]).'
+             <a href="javascript:plusItem(' . ($counter + 1).')">
+                '.Display::return_icon('add.png', get_lang('Add'), ['id' => 'plus-'.($counter + 1), 'style' => 'display: '.(($counter >= $nr_items) ? 'inline' : 'none')]).'
             </a>
-            <a href="javascript:minItem(' . ($counter) . ')">
-                '.Display::return_icon('delete.png', get_lang('Delete'), ['id' => 'min-' . ($counter), 'style' => 'display: '.(($counter>=$nr_items) ? 'inline':'none') ]).'
+            <a href="javascript:minItem(' . ($counter).')">
+                '.Display::return_icon('delete.png', get_lang('Delete'), ['id' => 'min-'.($counter), 'style' => 'display: '.(($counter >= $nr_items) ? 'inline' : 'none')]).'
             </a>
             </div></div>';
             $renderer->setElementTemplate($template_title, 'components['.$i.'][title]');

main/attendance/attendance_add.php

Spacing +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
 
 // error messages
 if (isset($error)) {
-    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'),false);
+    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'), false);
 }
 
 if (!isset($error)) {
@@ -28,7 +28,7 @@ 
 $form->addElement('hidden', 'sec_token', $token);
 
 $form->addText('title', get_lang('Title'), true);
-$form->applyFilter('title','html_filter');
+$form->applyFilter('title', 'html_filter');
 $form->addHtmlEditor(
     'description',
     get_lang('Description'),
@@ -41,7 +41,7 @@ 
 if ((api_get_session_id() != 0 && Gradebook::is_active()) || api_get_session_id() == 0) {
     $form->addButtonAdvancedSettings('id_qualify');
 
-    $form->addElement('html','<div id="id_qualify_options" style="display:none">');
+    $form->addElement('html', '<div id="id_qualify_options" style="display:none">');
 
     // Qualify Attendance for gradebook option
     $form->addElement(
@@ -51,7 +51,7 @@ 
         get_lang('QualifyAttendanceGradebook'),
         'onclick="javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}"'
     );
-    $form->addElement('html','<div id="options_field" style="display:none">');
+    $form->addElement('html', '<div id="options_field" style="display:none">');
 
     GradebookUtils::load_gradebook_select_in_tool($form);
 
@@ -64,8 +64,8 @@ 
         'value="0.00" Style="width:40px" onfocus="javascript: this.select();"'
     );
     $form->applyFilter('attendance_weight', 'html_filter');
-    $form->addElement('html','</div>');
-    $form->addElement('html','</div>');
+    $form->addElement('html', '</div>');
+    $form->addElement('html', '</div>');
 }
 $form->addButtonCreate(get_lang('Save'));
 $form->display();

main/attendance/attendance_edit.php

Spacing +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
 
 // error messages
 if (isset($error)) {
-    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'),false);
+    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'), false);
 }
 
 if (!isset($error)) {
@@ -49,18 +49,18 @@ 
 if (Gradebook::is_active()) {
     if (!empty($attendance_qualify_title) || !empty($attendance_weight)) {
         $form->addButtonAdvancedSettings('id_qualify');
-        $form->addElement('html','<div id="id_qualify_options" style="display:block">');
+        $form->addElement('html', '<div id="id_qualify_options" style="display:block">');
         $form->addElement(
             'checkbox',
             'attendance_qualify_gradebook',
             '',
             get_lang('QualifyAttendanceGradebook'),
-            array('checked'=>'true','onclick'=>'javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}')
+            array('checked'=>'true', 'onclick'=>'javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}')
         );
-        $form->addElement('html','<div id="options_field" style="display:block">');
+        $form->addElement('html', '<div id="options_field" style="display:block">');
     } else {
         $form->addButtonAdvancedSettings('id_qualify');
-        $form->addElement('html','<div id="id_qualify_options" style="display:none">');
+        $form->addElement('html', '<div id="id_qualify_options" style="display:none">');
         $form->addElement(
             'checkbox',
             'attendance_qualify_gradebook',
@@ -68,7 +68,7 @@ 
             get_lang('QualifyAttendanceGradebook'),
             'onclick="javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}"'
         );
-        $form->addElement('html','<div id="options_field" style="display:none">');
+        $form->addElement('html', '<div id="options_field" style="display:none">');
     }
     GradebookUtils::load_gradebook_select_in_tool($form);
     $form->addElement('text', 'attendance_qualify_title', get_lang('TitleColumnGradebook'));