Completed
Pull Request — 1.11.x (#1256)
by José
419:50 queued 379:50
created
main/messages/download.php 2 patches
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -50,11 +50,11 @@  discard block
 block discarded – undo
50 50
 $message_uid = '';
51 51
 $message_type = array('inbox','outbox');
52 52
 if (in_array($_GET['type'],$message_type)) {
53
-	if ($_GET['type'] == 'inbox') {
54
-		$message_uid = $row_users['user_receiver_id'];
55
-	} else {
56
-		$message_uid = $row_users['user_sender_id'];
57
-	}
53
+    if ($_GET['type'] == 'inbox') {
54
+        $message_uid = $row_users['user_receiver_id'];
55
+    } else {
56
+        $message_uid = $row_users['user_sender_id'];
57
+    }
58 58
 }
59 59
 
60 60
 // allow to the correct user for download this file
@@ -62,19 +62,19 @@  discard block
 block discarded – undo
62 62
 $userGroup = new UserGroup();
63 63
 
64 64
 if (!empty($row_users['group_id'])) {
65
-	$users_group = $userGroup->get_all_users_by_group($row_users['group_id']);
66
-	if (!in_array($current_uid,array_keys($users_group))) {
67
-		$not_allowed_to_edit = true;
68
-	}
65
+    $users_group = $userGroup->get_all_users_by_group($row_users['group_id']);
66
+    if (!in_array($current_uid,array_keys($users_group))) {
67
+        $not_allowed_to_edit = true;
68
+    }
69 69
 } else {
70
-	if ($current_uid != $message_uid) {
71
-		$not_allowed_to_edit = true;
72
-	}
70
+    if ($current_uid != $message_uid) {
71
+        $not_allowed_to_edit = true;
72
+    }
73 73
 }
74 74
 
75 75
 if ($not_allowed_to_edit) {
76
-	api_not_allowed();
77
-	exit;
76
+    api_not_allowed();
77
+    exit;
78 78
 }
79 79
 
80 80
 // set the path directory file
@@ -85,14 +85,14 @@  discard block
 block discarded – undo
85 85
         true
86 86
     );
87 87
 } else {
88
-	$path_user_info['dir'] = UserManager::getUserPathById($message_uid, 'system');
88
+    $path_user_info['dir'] = UserManager::getUserPathById($message_uid, 'system');
89 89
 }
90 90
 
91 91
 $full_file_name = $path_user_info['dir'].'message_attachments/'.$file_url;
92 92
 
93 93
 if (Security::check_abs_path($full_file_name, $path_user_info['dir'].'message_attachments/')) {
94 94
     // launch event
95
-	Event::event_download($file_url);
95
+    Event::event_download($file_url);
96 96
     DocumentManager::file_send_for_download($full_file_name,TRUE, $title);
97 97
 }
98 98
 exit;
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -48,8 +48,8 @@  discard block
 block discarded – undo
48 48
 
49 49
 // get message user id for inbox/outbox
50 50
 $message_uid = '';
51
-$message_type = array('inbox','outbox');
52
-if (in_array($_GET['type'],$message_type)) {
51
+$message_type = array('inbox', 'outbox');
52
+if (in_array($_GET['type'], $message_type)) {
53 53
 	if ($_GET['type'] == 'inbox') {
54 54
 		$message_uid = $row_users['user_receiver_id'];
55 55
 	} else {
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
 if (!empty($row_users['group_id'])) {
65 65
 	$users_group = $userGroup->get_all_users_by_group($row_users['group_id']);
66
-	if (!in_array($current_uid,array_keys($users_group))) {
66
+	if (!in_array($current_uid, array_keys($users_group))) {
67 67
 		$not_allowed_to_edit = true;
68 68
 	}
69 69
 } else {
@@ -93,6 +93,6 @@  discard block
 block discarded – undo
93 93
 if (Security::check_abs_path($full_file_name, $path_user_info['dir'].'message_attachments/')) {
94 94
     // launch event
95 95
 	Event::event_download($file_url);
96
-    DocumentManager::file_send_for_download($full_file_name,TRUE, $title);
96
+    DocumentManager::file_send_for_download($full_file_name, TRUE, $title);
97 97
 }
98 98
 exit;
Please login to merge, or discard this patch.
main/messages/outbox.php 2 patches
Indentation   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -9,20 +9,20 @@  discard block
 block discarded – undo
9 9
 api_block_anonymous_users();
10 10
 
11 11
 if (isset($_GET['messages_page_nr'])) {
12
-	if (api_get_setting('allow_social_tool')=='true' &&
12
+    if (api_get_setting('allow_social_tool')=='true' &&
13 13
         api_get_setting('allow_message_tool')=='true'
14 14
     ) {
15
-		$social_link = '';
16
-		if ($_REQUEST['f']=='social') {
17
-			$social_link = '&f=social';
18
-		}
19
-		header('Location:outbox.php?pager='.Security::remove_XSS($_GET['messages_page_nr']).$social_link.'');
20
-		exit;
21
-	}
15
+        $social_link = '';
16
+        if ($_REQUEST['f']=='social') {
17
+            $social_link = '&f=social';
18
+        }
19
+        header('Location:outbox.php?pager='.Security::remove_XSS($_GET['messages_page_nr']).$social_link.'');
20
+        exit;
21
+    }
22 22
 }
23 23
 
24 24
 if (api_get_setting('allow_message_tool')!='true'){
25
-	api_not_allowed();
25
+    api_not_allowed();
26 26
 }
27 27
 //jquery thickbox already called from main/inc/header.inc.php
28 28
 
@@ -54,13 +54,13 @@  discard block
 block discarded – undo
54 54
 		MAIN CODE
55 55
 */
56 56
 if (isset($_GET['f']) && $_GET['f']=='social') {
57
-	$this_section = SECTION_SOCIAL;
58
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/social/home.php','name' => get_lang('Social'));
59
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
57
+    $this_section = SECTION_SOCIAL;
58
+    $interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/social/home.php','name' => get_lang('Social'));
59
+    $interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
60 60
 } else {
61
-	$this_section = SECTION_MYPROFILE;
62
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/auth/profile.php','name' => get_lang('Profile'));
63
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
61
+    $this_section = SECTION_MYPROFILE;
62
+    $interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/auth/profile.php','name' => get_lang('Profile'));
63
+    $interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
64 64
 }
65 65
 
66 66
 $actions = '';
@@ -83,8 +83,8 @@  discard block
 block discarded – undo
83 83
 
84 84
 if (isset($info_delete_outbox[0]) && trim($info_delete_outbox[0])=='delete') {
85 85
     for ($i = 1; $i <= $count_delete_outbox; $i++) {
86
-		MessageManager::delete_message_by_user_sender(api_get_user_id(),$info_delete_outbox[$i]);
87
-	}
86
+        MessageManager::delete_message_by_user_sender(api_get_user_id(),$info_delete_outbox[$i]);
87
+    }
88 88
     $message_box=get_lang('SelectedMessagesDeleted').
89 89
         '&nbsp
90 90
         <br><a href="../social/index.php?#remote-tab-3">'.
Please login to merge, or discard this patch.
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@  discard block
 block discarded – undo
9 9
 api_block_anonymous_users();
10 10
 
11 11
 if (isset($_GET['messages_page_nr'])) {
12
-	if (api_get_setting('allow_social_tool')=='true' &&
13
-        api_get_setting('allow_message_tool')=='true'
12
+	if (api_get_setting('allow_social_tool') == 'true' &&
13
+        api_get_setting('allow_message_tool') == 'true'
14 14
     ) {
15 15
 		$social_link = '';
16
-		if ($_REQUEST['f']=='social') {
16
+		if ($_REQUEST['f'] == 'social') {
17 17
 			$social_link = '&f=social';
18 18
 		}
19 19
 		header('Location:outbox.php?pager='.Security::remove_XSS($_GET['messages_page_nr']).$social_link.'');
@@ -51,46 +51,46 @@  discard block
 block discarded – undo
51 51
 */
52 52
 if (isset($_GET['f']) && $_GET['f'] === 'social') {
53 53
 	$this_section = SECTION_SOCIAL;
54
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/social/home.php','name' => get_lang('Social'));
55
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
54
+	$interbreadcrumb[] = array('url' => api_get_path(WEB_PATH).'main/social/home.php', 'name' => get_lang('Social'));
55
+	$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Outbox'));
56 56
 } else {
57 57
 	$this_section = SECTION_MYPROFILE;
58
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/auth/profile.php','name' => get_lang('Profile'));
59
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
58
+	$interbreadcrumb[] = array('url' => api_get_path(WEB_PATH).'main/auth/profile.php', 'name' => get_lang('Profile'));
59
+	$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Outbox'));
60 60
 }
61 61
 
62 62
 $actions = '';
63 63
 if (api_get_setting('extended_profile') == 'true') {
64 64
     if (api_get_setting('allow_social_tool') == 'true' && api_get_setting('allow_message_tool') == 'true') {
65
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/social/profile.php">'.
65
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/social/profile.php">'.
66 66
             Display::return_icon('shared_profile.png', get_lang('ViewSharedProfile')).'</a>';
67 67
     }
68 68
     if (api_get_setting('allow_message_tool') == 'true') {
69 69
         //echo '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.Display::return_icon('inbox.png').' '.get_lang('Messages').'</a>';
70
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php">'.
71
-            Display::return_icon('message_new.png',get_lang('ComposeMessage')).'</a>';
72
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.
73
-            Display::return_icon('inbox.png',get_lang('Inbox')).'</a>';
74
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/outbox.php">'.
75
-            Display::return_icon('outbox.png',get_lang('Outbox')).'</a>';
70
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php">'.
71
+            Display::return_icon('message_new.png', get_lang('ComposeMessage')).'</a>';
72
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.
73
+            Display::return_icon('inbox.png', get_lang('Inbox')).'</a>';
74
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/outbox.php">'.
75
+            Display::return_icon('outbox.png', get_lang('Outbox')).'</a>';
76 76
     }
77 77
 }
78 78
 
79 79
 
80
-$info_delete_outbox =array();
81
-$info_delete_outbox = isset($_GET['form_delete_outbox']) ? explode(',',$_GET['form_delete_outbox']) : '';
80
+$info_delete_outbox = array();
81
+$info_delete_outbox = isset($_GET['form_delete_outbox']) ? explode(',', $_GET['form_delete_outbox']) : '';
82 82
 $count_delete_outbox = count($info_delete_outbox) - 1;
83 83
 
84 84
 if (isset($info_delete_outbox[0]) && trim($info_delete_outbox[0]) == 'delete') {
85 85
     for ($i = 1; $i <= $count_delete_outbox; $i++) {
86
-		MessageManager::delete_message_by_user_sender(api_get_user_id(),$info_delete_outbox[$i]);
86
+		MessageManager::delete_message_by_user_sender(api_get_user_id(), $info_delete_outbox[$i]);
87 87
 	}
88
-    $message_box=get_lang('SelectedMessagesDeleted').
88
+    $message_box = get_lang('SelectedMessagesDeleted').
89 89
         '&nbsp
90 90
         <br><a href="../social/index.php?#remote-tab-3">'.
91 91
         get_lang('BackToOutbox').
92 92
         '</a>';
93
-    Display::display_normal_message(api_xml_http_response_encode($message_box),false);
93
+    Display::display_normal_message(api_xml_http_response_encode($message_box), false);
94 94
     exit;
95 95
 }
96 96
 
@@ -117,23 +117,23 @@  discard block
 block discarded – undo
117 117
 }
118 118
 //MAIN CONTENT
119 119
 if ($action == 'delete') {
120
-    $delete_list_id=array();
120
+    $delete_list_id = array();
121 121
     if (isset($_POST['out'])) {
122
-        $delete_list_id=$_POST['out'];
122
+        $delete_list_id = $_POST['out'];
123 123
     }
124 124
     if (isset($_POST['id'])) {
125
-        $delete_list_id=$_POST['id'];
125
+        $delete_list_id = $_POST['id'];
126 126
     }
127 127
     for ($i = 0; $i < count($delete_list_id); $i++) {
128 128
         MessageManager::delete_message_by_user_sender(api_get_user_id(), $delete_list_id[$i]);
129 129
     }
130
-    $delete_list_id=array();
130
+    $delete_list_id = array();
131 131
     $social_right_content .= MessageManager::outbox_display($keyword);
132 132
 } elseif ($action == 'deleteone') {
133 133
     $delete_list_id = array();
134 134
     $id = Security::remove_XSS($_GET['id']);
135 135
     MessageManager::delete_message_by_user_sender(api_get_user_id(), $id);
136
-    $delete_list_id=array();
136
+    $delete_list_id = array();
137 137
     $social_right_content .= MessageManager::outbox_display($keyword);
138 138
 } else {
139 139
     $social_right_content .= MessageManager::outbox_display($keyword);
Please login to merge, or discard this patch.
main/attendance/attendance_controller.php 3 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -636,11 +636,11 @@
 block discarded – undo
636 636
             }
637 637
             $formToDisplay = $form->returnForm();
638 638
         } else {
639
-           if (!empty($sessionId)) {
640
-               $sessionInfo = api_get_session_info($sessionId);
641
-               $startDate = $sessionInfo['access_start_date'];
642
-               $endDate = $sessionInfo['access_end_date'];
643
-           }
639
+            if (!empty($sessionId)) {
640
+                $sessionInfo = api_get_session_info($sessionId);
641
+                $startDate = $sessionInfo['access_start_date'];
642
+                $endDate = $sessionInfo['access_end_date'];
643
+            }
644 644
         }
645 645
 
646 646
         $attendance = new Attendance();
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
                     $attendance->set_attendance_qualify_title($_POST['attendance_qualify_title']);
61 61
                     $attendance->set_attendance_weight($_POST['attendance_weight']);
62 62
                     $link_to_gradebook = false;
63
-                    if ( isset($_POST['attendance_qualify_gradebook']) && $_POST['attendance_qualify_gradebook'] == 1 ) {
63
+                    if (isset($_POST['attendance_qualify_gradebook']) && $_POST['attendance_qualify_gradebook'] == 1) {
64 64
                         $link_to_gradebook = true;
65 65
                     }
66 66
                     $attendance->category_id = $_POST['category_id'];
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
                     ) {
113 113
                         $link_to_gradebook = true;
114 114
                     }
115
-                    $attendance->attendance_edit($attendance_id,$link_to_gradebook);
115
+                    $attendance->attendance_edit($attendance_id, $link_to_gradebook);
116 116
                     Security::clear_token();
117 117
                     header('location:index.php?action=attendance_list&'.api_get_cidreq());
118 118
                     exit;
@@ -543,17 +543,17 @@  discard block
 block discarded – undo
543 543
             }
544 544
         }
545 545
         $max_cols_per_page = 12; //10 dates + 2 name and number
546
-        $max_dates_per_page = $max_dates_per_page_original = $max_cols_per_page - 2;//10
546
+        $max_dates_per_page = $max_dates_per_page_original = $max_cols_per_page - 2; //10
547 547
         $rows = count($data_table);
548 548
 
549 549
         if ($cols > $max_cols_per_page) {
550
-            $number_tables = round(($cols-2)/$max_dates_per_page);
550
+            $number_tables = round(($cols - 2) / $max_dates_per_page);
551 551
             $headers = $data_table[0];
552 552
             $all = array();
553 553
             $tables = array();
554 554
             $changed = 1;
555 555
 
556
-            for ($i= 0; $i <= $rows; $i++) {
556
+            for ($i = 0; $i <= $rows; $i++) {
557 557
                 $row = isset($data_table[$i]) ? $data_table[$i] : null;
558 558
                 $key = 1;
559 559
                 $max_dates_per_page = 10;
@@ -564,7 +564,7 @@  discard block
 block discarded – undo
564 564
                     foreach ($item as $value) {
565 565
                         if ($count_j >= $max_dates_per_page) {
566 566
                             $key++;
567
-                            $max_dates_per_page = $max_dates_per_page_original*$key;
567
+                            $max_dates_per_page = $max_dates_per_page_original * $key;
568 568
                             //magic hack
569 569
                             $tables[$key][$i][] = $tables[1][$i][0];
570 570
                             $tables[$key][$i][] = $tables[1][$i][1];
@@ -622,8 +622,8 @@  discard block
 block discarded – undo
622 622
             $form = new FormValidator(
623 623
                 'search',
624 624
                 'post',
625
-                api_get_self() . '?' . api_get_cidreq(
626
-                ) . '&action=calendar_logins'
625
+                api_get_self().'?'.api_get_cidreq(
626
+                ).'&action=calendar_logins'
627 627
             );
628 628
             $form->addDateRangePicker('range', get_lang('DateRange'));
629 629
             $form->addButton('submit', get_lang('Submit'));
Please login to merge, or discard this patch.
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -529,10 +529,11 @@
 block discarded – undo
529 529
                 $result['full_name'] = api_get_person_name($user['firstname'], $user['lastname']);
530 530
                 foreach ($data_array['attendant_calendar'] as $class_day) {
531 531
                     if ($class_day['done_attendance'] == 1) {
532
-                        if ($data_users_presence[$user['user_id']][$class_day['id']]['presence'] == 1)
533
-                            $result[$class_day['id']] = get_lang('UserAttendedSymbol');
534
-                        else
535
-                            $result[$class_day['id']] = '<span style="color:red">'.get_lang('UserNotAttendedSymbol').'</span>';
532
+                        if ($data_users_presence[$user['user_id']][$class_day['id']]['presence'] == 1) {
533
+                                                    $result[$class_day['id']] = get_lang('UserAttendedSymbol');
534
+                        } else {
535
+                                                    $result[$class_day['id']] = '<span style="color:red">'.get_lang('UserNotAttendedSymbol').'</span>';
536
+                        }
536 537
                     } else {
537 538
                         $result[$class_day['id']] = " ";
538 539
                     }
Please login to merge, or discard this patch.
main/attendance/attendance_list.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
 if (api_is_allowed_to_edit(null, true)) {
14 14
     echo '<div class="actions">';
15 15
     echo '<a href="index.php?'.api_get_cidreq().'&action=attendance_add">'.
16
-        Display::return_icon('new_attendance_list.png', get_lang('CreateANewAttendance'),'',ICON_SIZE_MEDIUM).'</a>';
16
+        Display::return_icon('new_attendance_list.png', get_lang('CreateANewAttendance'), '', ICON_SIZE_MEDIUM).'</a>';
17 17
     echo '</div>';
18 18
 }
19 19
 $attendance = new Attendance();
Please login to merge, or discard this patch.
main/attendance/calendar_logins.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 // See AttendanceController::calendarLogins function
5 5
 echo '<div class="actions">';
6 6
 echo '<a href="index.php?'.api_get_cidreq().'&action=calendar_list">'.
7
-    Display::return_icon('back.png',get_lang('AttendanceCalendar'),'',ICON_SIZE_MEDIUM).'</a>';
7
+    Display::return_icon('back.png', get_lang('AttendanceCalendar'), '', ICON_SIZE_MEDIUM).'</a>';
8 8
 echo '</div>';
9 9
 
10 10
 echo $form;
Please login to merge, or discard this patch.
main/attendance/attendance_calendar.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -14,15 +14,15 @@  discard block
 block discarded – undo
14 14
     echo '<div class="actions">';
15 15
     if ($action == 'calendar_add') {
16 16
         echo '<a href="index.php?'.api_get_cidreq().'&action=calendar_list&attendance_id='.$attendance_id.'">'.
17
-            Display::return_icon('back.png',get_lang('AttendanceCalendar'),'',ICON_SIZE_MEDIUM).'</a>';
17
+            Display::return_icon('back.png', get_lang('AttendanceCalendar'), '', ICON_SIZE_MEDIUM).'</a>';
18 18
     } else {
19 19
         echo '<a href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list&attendance_id='.$attendance_id.'">'.
20
-            Display::return_icon('back.png',get_lang('AttendanceSheet'),'',ICON_SIZE_MEDIUM).'</a>';
20
+            Display::return_icon('back.png', get_lang('AttendanceSheet'), '', ICON_SIZE_MEDIUM).'</a>';
21 21
         if (api_is_allowed_to_edit()) {
22 22
             echo '<a href="index.php?'.api_get_cidreq().'&action=calendar_add&attendance_id='.$attendance_id.'">'.
23
-                Display::return_icon('add.png',get_lang('AddDateAndTime'),'',ICON_SIZE_MEDIUM).'</a>';
23
+                Display::return_icon('add.png', get_lang('AddDateAndTime'), '', ICON_SIZE_MEDIUM).'</a>';
24 24
             echo '<a onclick="javascript:if(!confirm(\''.get_lang('AreYouSureToDeleteAllDates').'\')) return false;" href="index.php?'.api_get_cidreq().'&action=calendar_all_delete&attendance_id='.$attendance_id.'">'.
25
-                Display::return_icon('clean.png',get_lang('CleanCalendar'),'',ICON_SIZE_MEDIUM).'</a>';
25
+                Display::return_icon('clean.png', get_lang('CleanCalendar'), '', ICON_SIZE_MEDIUM).'</a>';
26 26
         }
27 27
     }
28 28
     echo '</div>';
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
     $form = new FormValidator(
59 59
         'attendance_calendar_add',
60 60
         'POST',
61
-        'index.php?action=calendar_add&attendance_id='.$attendance_id.'&' . api_get_cidreq(),
61
+        'index.php?action=calendar_add&attendance_id='.$attendance_id.'&'.api_get_cidreq(),
62 62
         ''
63 63
     );
64 64
     $form->addElement('header', get_lang('AddADateTime'));
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
         'weekly' => get_lang('RepeatWeekly'),
96 96
         'monthlyByDate' => get_lang('RepeatMonthlyByDate')
97 97
     );
98
-    $form->addElement('select', 'repeat_type', get_lang('RepeatType') , $a_repeat_type);
98
+    $form->addElement('select', 'repeat_type', get_lang('RepeatType'), $a_repeat_type);
99 99
 
100 100
     $form->addElement('date_picker', 'end_date_time', get_lang('RepeatEnd'), array('form_name'=>'attendance_calendar_add'));
101 101
     $defaults['end_date_time'] = date('Y-m-d');
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
                 $form = new FormValidator(
132 132
                     'attendance_calendar_edit',
133 133
                     'POST',
134
-                    'index.php?action=calendar_edit&attendance_id=' . $attendance_id . '&calendar_id=' . $calendar_id . '&' . api_get_cidreq(),
134
+                    'index.php?action=calendar_edit&attendance_id='.$attendance_id.'&calendar_id='.$calendar_id.'&'.api_get_cidreq(),
135 135
                     ''
136 136
                 );
137 137
                 $form->addDateTimePicker('date_time', array(get_lang('Date')), array('form_name'=>'attendance_calendar_edit'), 5);
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
                     get_lang('DateTime'),
148 148
                         null,
149 149
                         ICON_SIZE_MEDIUM
150
-                ).' '.substr($calendar['date_time'], 0, strlen($calendar['date_time'])- 3) .'&nbsp;';
150
+                ).' '.substr($calendar['date_time'], 0, strlen($calendar['date_time']) - 3).'&nbsp;';
151 151
 
152 152
                 if (isset($calendar['groups']) && !empty($calendar['groups'])) {
153 153
                     foreach ($calendar['groups'] as $group) {
Please login to merge, or discard this patch.
main/attendance/attendance_add.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
 
13 13
 // error messages
14 14
 if (isset($error)) {
15
-    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'),false);
15
+    Display::display_error_message(get_lang('FormHasErrorsPleaseComplete'), false);
16 16
 }
17 17
 
18 18
 if (!isset($error)) {
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 $form->addElement('hidden', 'sec_token', $token);
29 29
 
30 30
 $form->addText('title', get_lang('Title'), true);
31
-$form->applyFilter('title','html_filter');
31
+$form->applyFilter('title', 'html_filter');
32 32
 $form->addHtmlEditor(
33 33
     'description',
34 34
     get_lang('Description'),
@@ -41,21 +41,21 @@  discard block
 block discarded – undo
41 41
 if ((api_get_session_id() != 0 && Gradebook::is_active()) || api_get_session_id() == 0) {
42 42
     $form->addButtonAdvancedSettings('id_qualify');
43 43
 
44
-    $form->addElement('html','<div id="id_qualify_options" style="display:none">');
44
+    $form->addElement('html', '<div id="id_qualify_options" style="display:none">');
45 45
 
46 46
     // Qualify Attendance for gradebook option
47
-    $form->addElement('checkbox', 'attendance_qualify_gradebook', '', get_lang('QualifyAttendanceGradebook'),'onclick="javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}"');
48
-    $form->addElement('html','<div id="options_field" style="display:none">');
47
+    $form->addElement('checkbox', 'attendance_qualify_gradebook', '', get_lang('QualifyAttendanceGradebook'), 'onclick="javascript: if(this.checked){document.getElementById(\'options_field\').style.display = \'block\';}else{document.getElementById(\'options_field\').style.display = \'none\';}"');
48
+    $form->addElement('html', '<div id="options_field" style="display:none">');
49 49
 
50 50
     GradebookUtils::load_gradebook_select_in_tool($form);
51 51
 
52 52
     $form->addElement('text', 'attendance_qualify_title', get_lang('TitleColumnGradebook'));
53 53
     $form->applyFilter('attendance_qualify_title', 'html_filter');
54
-    $form->addElement('text', 'attendance_weight', get_lang('QualifyWeight'),'value="0.00" Style="width:40px" onfocus="javascript: this.select();"');
54
+    $form->addElement('text', 'attendance_weight', get_lang('QualifyWeight'), 'value="0.00" Style="width:40px" onfocus="javascript: this.select();"');
55 55
     $form->applyFilter('attendance_weight', 'html_filter');
56
-    $form->addElement('html','</div>');
56
+    $form->addElement('html', '</div>');
57 57
 
58
-    $form->addElement('html','</div>');
58
+    $form->addElement('html', '</div>');
59 59
 }
60 60
 $form->addButtonCreate(get_lang('Save'));
61 61
 $form->display();
Please login to merge, or discard this patch.
main/course_home/vertical_activity.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@
 block discarded – undo
91 91
         $content .= '<div class="courseadminview-activity-3col"><span class="viewcaption">'.get_lang('SessionData').'</span>
92 92
             <table width="100%">';
93 93
                 $content .= CourseHome::show_session_data($session_id);
94
-             $content .=  '</table></div>';
94
+                $content .=  '</table></div>';
95 95
     }
96 96
 
97 97
     $content .=  '<div class="Authoringview">';
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     // Show message to confirm that a tool it to be hidden from available tools
38 38
     // visibility 0,1->2
39 39
     if (!empty($_GET['askDelete'])) {
40
-        $content .='<div id="toolhide">'.get_lang('DelLk').'<br />&nbsp;&nbsp;&nbsp;
40
+        $content .= '<div id="toolhide">'.get_lang('DelLk').'<br />&nbsp;&nbsp;&nbsp;
41 41
             <a href="'.api_get_self().'">'.get_lang('No').'</a>&nbsp;|&nbsp;
42 42
             <a href="'.api_get_self().'?delete=yes&id='.$id.'">'.get_lang('Yes').'</a>
43 43
         </div>';
@@ -55,14 +55,14 @@  discard block
 block discarded – undo
55 55
 
56 56
 // Start of tools for CourseAdmins (teachers/tutors)
57 57
 if (api_is_allowed_to_edit(null, true) && !api_is_coach()) {
58
-    $content .=  '<div class="courseadminview" style="border:0px; margin-top: 0px;padding:5px;">
58
+    $content .= '<div class="courseadminview" style="border:0px; margin-top: 0px;padding:5px;">
59 59
     <div class="normal-message" id="id_normal_message" style="display:none">';
60
-        $content .=  '<img src="'.api_get_path(WEB_PATH).'main/inc/lib/javascript/indicator.gif"/>&nbsp;&nbsp;';
61
-        $content .=  get_lang('PleaseStandBy');
60
+        $content .= '<img src="'.api_get_path(WEB_PATH).'main/inc/lib/javascript/indicator.gif"/>&nbsp;&nbsp;';
61
+        $content .= get_lang('PleaseStandBy');
62 62
 
63
-    $content .=  '</div>
63
+    $content .= '</div>
64 64
         <div class="confirmation-message" id="id_confirmation_message" style="display:none"></div></div>';
65
-    $content .=  '<div id="activity-3col">';
65
+    $content .= '<div id="activity-3col">';
66 66
 
67 67
     if (api_get_setting('show_session_data') == 'true' && $session_id > 0) {
68 68
         $content .= '<div class="courseadminview-activity-3col"><span class="viewcaption">'.get_lang('SessionData').'</span>
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
         $content .= '<div class="courseadminview-activity-3col"><span class="viewcaption">'.get_lang('SessionData').'</span>
91 91
             <table width="100%">';
92 92
                 $content .= CourseHome::show_session_data($session_id);
93
-             $content .=  '</table></div>';
93
+             $content .= '</table></div>';
94 94
     }
95 95
 
96
-    $content .=  '<div class="Authoringview">';
96
+    $content .= '<div class="Authoringview">';
97 97
                 $my_list = CourseHome::get_tools_category(TOOL_STUDENT_VIEW);
98 98
                 $content .= CourseHome::show_tools_category($my_list);
99 99
     $content .= '</div>';
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
         $order_tool_list = array();
107 107
         foreach ($my_list as $key => $new_tool) {
108 108
             $tool_name = CourseHome::translate_tool_name($new_tool);
109
-            $order_tool_list [$key]= $tool_name;
109
+            $order_tool_list [$key] = $tool_name;
110 110
         }
111 111
         natsort($order_tool_list);
112 112
         $my_temp_tool_array = array();
Please login to merge, or discard this patch.
main/course_home/activity.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
     // Show message to confirm that a tool it to be hidden from available tools
23 23
     // visibility 0,1->2
24 24
     if (!empty($_GET['askDelete'])) {
25
-        $content .='<div id="toolhide">'.get_lang('DelLk').'<br />&nbsp;&nbsp;&nbsp;
25
+        $content .= '<div id="toolhide">'.get_lang('DelLk').'<br />&nbsp;&nbsp;&nbsp;
26 26
             <a href="'.api_get_self().'">'.get_lang('No').'</a>&nbsp;|&nbsp;
27 27
             <a href="'.api_get_self().'?delete=yes&id='.$id.'">'.get_lang('Yes').'</a>
28 28
         </div>';
@@ -49,11 +49,11 @@  discard block
 block discarded – undo
49 49
 
50 50
 // Start of tools for CourseAdmins (teachers/tutors)
51 51
 if ($session_id == 0 && api_is_course_admin() && api_is_allowed_to_edit(null, true)) {
52
-    $content .=  '<div class="alert alert-success" style="border:0px; margin-top: 0px;padding:0px;">
52
+    $content .= '<div class="alert alert-success" style="border:0px; margin-top: 0px;padding:0px;">
53 53
 		<div class="normal-message" id="id_normal_message" style="display:none">';
54
-    $content .=  '<img src="'.api_get_path(WEB_PATH).'main/inc/lib/javascript/indicator.gif"/>&nbsp;&nbsp;';
55
-    $content .=  get_lang('PleaseStandBy');
56
-    $content .=  '</div>
54
+    $content .= '<img src="'.api_get_path(WEB_PATH).'main/inc/lib/javascript/indicator.gif"/>&nbsp;&nbsp;';
55
+    $content .= get_lang('PleaseStandBy');
56
+    $content .= '</div>
57 57
 		<div class="alert alert-success" id="id_confirmation_message" style="display:none"></div>
58 58
 	</div>';
59 59
 
@@ -73,18 +73,18 @@  discard block
 block discarded – undo
73 73
 
74 74
     $my_list = CourseHome::get_tools_category(TOOL_AUTHORING);
75 75
     $items = CourseHome::show_tools_category($my_list);
76
-    $content .= return_block(get_lang('Authoring'),  $items, 'course-tools-author');
76
+    $content .= return_block(get_lang('Authoring'), $items, 'course-tools-author');
77 77
 
78 78
     $my_list = CourseHome::get_tools_category(TOOL_INTERACTION);
79 79
     $list2 = CourseHome::get_tools_category(TOOL_COURSE_PLUGIN);
80 80
 
81
-    $my_list = array_merge($my_list,$list2);
82
-    $items =  CourseHome::show_tools_category($my_list);
83
-    $content .= return_block(get_lang('Interaction'),  $items, 'course-tools-interaction');
81
+    $my_list = array_merge($my_list, $list2);
82
+    $items = CourseHome::show_tools_category($my_list);
83
+    $content .= return_block(get_lang('Interaction'), $items, 'course-tools-interaction');
84 84
 
85 85
     $my_list = CourseHome::get_tools_category(TOOL_ADMIN_PLATFORM);
86 86
     $items = CourseHome::show_tools_category($my_list);
87
-    $content .= return_block(get_lang('Administration'),  $items , 'course-tools-administration');
87
+    $content .= return_block(get_lang('Administration'), $items, 'course-tools-administration');
88 88
 
89 89
 } elseif (api_is_coach()) {
90 90
     $content .= $pluginExtra;
@@ -94,10 +94,10 @@  discard block
 block discarded – undo
94 94
 			<span class="viewcaption">'.get_lang('SessionData').'</span>
95 95
 			<table class="course_activity_home">';
96 96
         $content .= CourseHome::show_session_data($session_id);
97
-        $content .=  '</table></div></div>';
97
+        $content .= '</table></div></div>';
98 98
     }
99 99
 
100
-    $content .=  '<div class="row">';
100
+    $content .= '<div class="row">';
101 101
     $my_list = CourseHome::get_tools_category(TOOL_STUDENT_VIEW);
102 102
     $content .= CourseHome::show_tools_category($my_list);
103 103
     $content .= '</div>';
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 {
180 180
     $html = '<div class="row">
181 181
                 <div class="col-xs-12 col-md-12">
182
-                    <div class="title-tools">' . $title . '</div>
182
+                    <div class="title-tools">' . $title.'</div>
183 183
                 </div>
184 184
             </div>
185 185
             <div class="row '.$class.'">'.$content.'</div>';
Please login to merge, or discard this patch.