Passed
Push — 1.10.x ( 04397c...5e25f1 )
by Angel Fernando Quiroz
181:14 queued 130:00
created
main/admin/class_import.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -59,8 +59,8 @@
 block discarded – undo
59 59
 api_protect_admin_script();
60 60
 
61 61
 // setting breadcrumbs
62
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
63
-$interbreadcrumb[] = array ('url' => 'class_list.php', 'name' => get_lang('Classes'));
62
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
63
+$interbreadcrumb[] = array('url' => 'class_list.php', 'name' => get_lang('Classes'));
64 64
 
65 65
 // Database Table Definitions
66 66
 
Please login to merge, or discard this patch.
main/admin/skill_badge.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,14 +23,14 @@
 block discarded – undo
23 23
 
24 24
 $interbreadcrumb = array(
25 25
     array(
26
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/index.php',
26
+        'url' => api_get_path(WEB_CODE_PATH).'admin/index.php',
27 27
         'name' => get_lang('Administration')
28 28
     )
29 29
 );
30 30
 
31 31
 $toolbar = Display::toolbarButton(
32 32
     get_lang('ManageSkills'),
33
-    api_get_path(WEB_CODE_PATH) . 'admin/skill_list.php',
33
+    api_get_path(WEB_CODE_PATH).'admin/skill_list.php',
34 34
     'list',
35 35
     'primary',
36 36
     ['title' => get_lang('ManageSkills')]
Please login to merge, or discard this patch.
main/admin/user_import.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -215,8 +215,8 @@  discard block
 block discarded – undo
215 215
             // We are sure that the extra field exists.
216 216
             foreach ($extra_fields as $extras) {
217 217
                 if (isset($user[$extras[1]])) {
218
-                    $key 	= $extras[1];
219
-                    $value 	= $user[$extras[1]];
218
+                    $key = $extras[1];
219
+                    $value = $user[$extras[1]];
220 220
                     UserManager::update_extra_field_value($user_id, $key, $value);
221 221
                 }
222 222
             }
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
     global $current_tag;
253 253
     switch ($data) {
254 254
         case 'Contact':
255
-            $user = array ();
255
+            $user = array();
256 256
             break;
257 257
         default:
258 258
             $current_tag = $data;
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
     $allowed_file_mimetype = array('csv', 'xml');
346 346
     $error_kind_file = false;
347 347
 
348
-    $checkUniqueEmail = isset($_POST['check_unique_email']) ? $_POST['check_unique_email'] :null;
348
+    $checkUniqueEmail = isset($_POST['check_unique_email']) ? $_POST['check_unique_email'] : null;
349 349
 
350 350
     $uploadInfo = pathinfo($_FILES['import_file']['name']);
351 351
     $ext_import_file = $uploadInfo['extension'];
@@ -354,7 +354,7 @@  discard block
 block discarded – undo
354 354
         if (strcmp($file_type, 'csv') === 0 &&
355 355
             $ext_import_file == $allowed_file_mimetype[0]
356 356
         ) {
357
-            $users	= parse_csv_data($_FILES['import_file']['tmp_name']);
357
+            $users = parse_csv_data($_FILES['import_file']['tmp_name']);
358 358
             $errors = validate_data($users, $checkUniqueEmail);
359 359
             $error_kind_file = false;
360 360
         } elseif (strcmp($file_type, 'xml') === 0 && $ext_import_file == $allowed_file_mimetype[1]) {
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
     Display::display_error_message($error_message);
441 441
 }
442 442
 
443
-$form = new FormValidator('user_import','post','user_import.php');
443
+$form = new FormValidator('user_import', 'post', 'user_import.php');
444 444
 $form->addElement('header', '', $tool_name);
445 445
 $form->addElement('hidden', 'formSent');
446 446
 $form->addElement('file', 'import_file', get_lang('ImportFileLocation'));
Please login to merge, or discard this patch.
main/admin/dashboard_add_sessions_to_user.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
 
23 23
 // setting breadcrumbs
24 24
 $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
25
-$interbreadcrumb[] = array('url' => 'user_list.php','name' => get_lang('UserList'));
25
+$interbreadcrumb[] = array('url' => 'user_list.php', 'name' => get_lang('UserList'));
26 26
 
27 27
 // Database Table Definitions
28
-$tbl_session 			= Database::get_main_table(TABLE_MAIN_SESSION);
29
-$tbl_session_rel_user 	= Database::get_main_table(TABLE_MAIN_SESSION_USER);
28
+$tbl_session = Database::get_main_table(TABLE_MAIN_SESSION);
29
+$tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER);
30 30
 $tbl_session_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
31 31
 
32 32
 // Initializing variables
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 }
47 47
 
48 48
 $add_type = 'multiple';
49
-if (isset($_GET['add_type']) && $_GET['add_type']!='') {
49
+if (isset($_GET['add_type']) && $_GET['add_type'] != '') {
50 50
     $add_type = Security::remove_XSS($_REQUEST['add_type']);
51 51
 }
52 52
 
@@ -71,20 +71,20 @@  discard block
 block discarded – undo
71 71
         }
72 72
 
73 73
         if (api_is_multiple_url_enabled()) {
74
-            $sql 	= " SELECT s.id, s.name FROM $tbl_session s
74
+            $sql = " SELECT s.id, s.name FROM $tbl_session s
75 75
                         LEFT JOIN $tbl_session_rel_access_url a ON (s.id = a.session_id)
76 76
                         WHERE  s.name LIKE '$needle%' $without_assigned_sessions AND access_url_id = ".api_get_current_access_url_id()."";
77 77
         } else {
78 78
             $sql = "SELECT s.id, s.name FROM $tbl_session s
79 79
                     WHERE  s.name LIKE '$needle%' $without_assigned_sessions ";
80 80
         }
81
-        $rs	= Database::query($sql);
81
+        $rs = Database::query($sql);
82 82
         $return .= '<select class="form-control" id="origin" name="NoAssignedSessionsList[]" multiple="multiple" size="20">';
83
-        while($session = Database :: fetch_array($rs)) {
84
-            $return .= '<option value="'.$session['id'].'" title="'.htmlspecialchars($session['name'],ENT_QUOTES).'">'.$session['name'].'</option>';
83
+        while ($session = Database :: fetch_array($rs)) {
84
+            $return .= '<option value="'.$session['id'].'" title="'.htmlspecialchars($session['name'], ENT_QUOTES).'">'.$session['name'].'</option>';
85 85
         }
86 86
         $return .= '</select>';
87
-        $xajax_response->addAssign('ajax_list_sessions_multiple','innerHTML',api_utf8_encode($return));
87
+        $xajax_response->addAssign('ajax_list_sessions_multiple', 'innerHTML', api_utf8_encode($return));
88 88
     }
89 89
 
90 90
     return $xajax_response;
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 }
144 144
 </script>';
145 145
 
146
-$formSent=0;
146
+$formSent = 0;
147 147
 $firstLetterSession = isset($_POST['firstLetterSession']) ? $_POST['firstLetterSession'] : null;
148 148
 $errorMsg = '';
149 149
 $UserList = array();
@@ -168,12 +168,12 @@  discard block
 block discarded – undo
168 168
 // actions
169 169
 
170 170
 if ($user_info['status'] != SESSIONADMIN) {
171
-    $actionsLeft = '<a href="dashboard_add_users_to_user.php?user='.$user_id.'">' . Display::return_icon('add-user.png', get_lang('AssignUsers'), null, ICON_SIZE_MEDIUM ) . '</a>';
172
-    $actionsLeft .= '<a href="dashboard_add_courses_to_user.php?user='.$user_id.'">' . Display::return_icon('course-add.png', get_lang('AssignCourses'), null, ICON_SIZE_MEDIUM) . '</a>';
171
+    $actionsLeft = '<a href="dashboard_add_users_to_user.php?user='.$user_id.'">'.Display::return_icon('add-user.png', get_lang('AssignUsers'), null, ICON_SIZE_MEDIUM).'</a>';
172
+    $actionsLeft .= '<a href="dashboard_add_courses_to_user.php?user='.$user_id.'">'.Display::return_icon('course-add.png', get_lang('AssignCourses'), null, ICON_SIZE_MEDIUM).'</a>';
173 173
 }
174 174
 
175 175
 
176
-echo Display::toolbarAction('toolbar-dashboard', array( 0 => $actionsLeft, 1 => ''));
176
+echo Display::toolbarAction('toolbar-dashboard', array(0 => $actionsLeft, 1 => ''));
177 177
 
178 178
 echo Display::page_header(sprintf(get_lang('AssignSessionsToX'), api_get_person_name($user_info['firstname'], $user_info['lastname'])), null, 'h3');
179 179
 
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
 
183 183
 $without_assigned_sessions = '';
184 184
 if (count($assigned_sessions_id) > 0) {
185
-    $without_assigned_sessions = " AND s.id NOT IN (".implode(',',$assigned_sessions_id).") ";
185
+    $without_assigned_sessions = " AND s.id NOT IN (".implode(',', $assigned_sessions_id).") ";
186 186
 }
187 187
 
188 188
 $needle = '%';
@@ -203,9 +203,9 @@  discard block
 block discarded – undo
203 203
 		    WHERE  s.name LIKE '$needle%' $without_assigned_sessions
204 204
             ORDER BY s.name";
205 205
 }
206
-$result	= Database::query($sql);
206
+$result = Database::query($sql);
207 207
 ?>
208
-    <form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?user=<?php echo $user_id ?>" style="margin:0px;" <?php if($ajax_search){ echo ' onsubmit="valide();"';}?>>
208
+    <form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?user=<?php echo $user_id ?>" style="margin:0px;" <?php if ($ajax_search) { echo ' onsubmit="valide();"'; }?>>
209 209
         <input type="hidden" name="formSent" value="1" />
210 210
         
211 211
         <div class="row">
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
                         <?php
218 218
                         while ($enreg = Database::fetch_array($result)) {
219 219
                         ?>
220
-                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['name'],ENT_QUOTES).'"';?>>
220
+                            <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['name'], ENT_QUOTES).'"'; ?>>
221 221
                                 <?php echo $enreg['name']; ?>
222 222
                             </option>
223 223
                         <?php } ?>
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
             <div class="col-md-4">
228 228
                 <div class="code-course">
229 229
                     <?php if ($add_type == 'multiple') { ?>
230
-                    <p><?php echo get_lang('FirstLetterSession');?> :</p>
230
+                    <p><?php echo get_lang('FirstLetterSession'); ?> :</p>
231 231
                     <select class="selectpicker form-control" name="firstLetterSession" onchange = "xajax_search_sessions(this.value, 'multiple')">
232 232
                         <option value="%">--</option>
233 233
                             <?php  echo Display :: get_alphabet_options($firstLetterSession); ?>
@@ -282,9 +282,9 @@  discard block
 block discarded – undo
282 282
                  <select id='destination' name="SessionsList[]" multiple="multiple" size="20" style="width:320px;">
283 283
                         <?php
284 284
                         if (is_array($assigned_sessions_to_hrm)) {
285
-                            foreach($assigned_sessions_to_hrm as $enreg) {
285
+                            foreach ($assigned_sessions_to_hrm as $enreg) {
286 286
                                 ?>
287
-                                <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['name'],ENT_QUOTES).'"'; ?>>
287
+                                <option value="<?php echo $enreg['id']; ?>" <?php echo 'title="'.htmlspecialchars($enreg['name'], ENT_QUOTES).'"'; ?>>
288 288
                                     <?php echo $enreg['name'] ?>
289 289
                                 </option>
290 290
                             <?php }
Please login to merge, or discard this patch.
main/admin/ldap_users_list.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 }
31 31
 */
32 32
 
33
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
33
+$interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
34 34
 $tool_name = get_lang('SearchLDAPUsers');
35 35
 //Display :: display_header($tool_name); //cannot display now as we need to redirect
36 36
 //api_display_tool_title($tool_name);
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 if (isset ($_GET['action']))
39 39
 {
40 40
 	$check = Security::check_token('get');
41
-	if($check)
41
+	if ($check)
42 42
 	{
43 43
 		switch ($_GET['action'])
44 44
 		{
@@ -59,41 +59,41 @@  discard block
 block discarded – undo
59 59
 				break;
60 60
 			case 'lock' :
61 61
 				Display :: display_header($tool_name);
62
-				$message=lock_unlock_user('lock',$_GET['user_id']);
62
+				$message = lock_unlock_user('lock', $_GET['user_id']);
63 63
 				Display :: display_normal_message($message);
64 64
 				break;
65 65
 			case 'unlock';
66 66
 				Display :: display_header($tool_name);
67
-				$message=lock_unlock_user('unlock',$_GET['user_id']);
67
+				$message = lock_unlock_user('unlock', $_GET['user_id']);
68 68
 				Display :: display_normal_message($message);
69 69
 				break;
70 70
 			case 'add_user';
71
-				$id=$_GET['id'];
72
-				$UserList=array();
71
+				$id = $_GET['id'];
72
+				$UserList = array();
73 73
 				$userid_match_login = array();
74 74
 				foreach ($id as $user_id) {
75 75
 					$tmp = ldap_add_user($user_id);
76
-					$UserList[]= $tmp;
76
+					$UserList[] = $tmp;
77 77
 					$userid_match_login[$tmp] = $user_id;
78 78
 				}
79
-				if (isset($_GET['id_session']) && ($_GET['id_session'] == strval(intval($_GET['id_session']))) && ($_GET['id_session']>0)) {
79
+				if (isset($_GET['id_session']) && ($_GET['id_session'] == strval(intval($_GET['id_session']))) && ($_GET['id_session'] > 0)) {
80 80
 					ldap_add_user_to_session($UserList, $_GET['id_session']);
81 81
 					header('Location: resume_session.php?id_session='.$_GET['id_session']);
82 82
 				} else {
83 83
 					Display :: display_header($tool_name);
84
-					if(count($userid_match_login)>0)
84
+					if (count($userid_match_login) > 0)
85 85
 					{
86
-						$message=get_lang('LDAPUsersAddedOrUpdated').':<br />';
87
-						foreach($userid_match_login as $user_id => $login)
86
+						$message = get_lang('LDAPUsersAddedOrUpdated').':<br />';
87
+						foreach ($userid_match_login as $user_id => $login)
88 88
 						{
89 89
 							$message .= '- '.$login.'<br />';
90 90
 						}
91 91
 					}
92 92
 					else
93 93
 					{
94
-						$message=get_lang('NoUserAdded');
94
+						$message = get_lang('NoUserAdded');
95 95
 					}
96
-					Display :: display_normal_message($message,false);
96
+					Display :: display_normal_message($message, false);
97 97
 				}
98 98
 				break;
99 99
 			default :
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 if (isset ($_POST['action']))
114 114
 {
115 115
 	$check = Security::check_token('get');
116
-	if($check)
116
+	if ($check)
117 117
 	{
118 118
 		switch ($_POST['action'])
119 119
 		{
@@ -122,15 +122,15 @@  discard block
 block discarded – undo
122 122
 				$number_of_deleted_users = 0;
123 123
 				foreach ($_POST['id'] as $index => $user_id)
124 124
 				{
125
-					if($user_id != $_user['user_id'])
125
+					if ($user_id != $_user['user_id'])
126 126
 					{
127
-						if(UserManager :: delete_user($user_id))
127
+						if (UserManager :: delete_user($user_id))
128 128
 						{
129 129
 							$number_of_deleted_users++;
130 130
 						}
131 131
 					}
132 132
 				}
133
-				if($number_of_selected_users == $number_of_deleted_users)
133
+				if ($number_of_selected_users == $number_of_deleted_users)
134 134
 				{
135 135
 					Display :: display_normal_message(get_lang('SelectedUsersDeleted'));
136 136
 				}
@@ -142,17 +142,17 @@  discard block
 block discarded – undo
142 142
 			case 'add_user' :
143 143
 				$number_of_selected_users = count($_POST['id']);
144 144
 				$number_of_added_users = 0;
145
-				$UserList=array();
145
+				$UserList = array();
146 146
 				foreach ($_POST['id'] as $index => $user_id)
147 147
 				{
148
-					if($user_id != $_user['user_id'])
148
+					if ($user_id != $_user['user_id'])
149 149
 					{
150 150
 						$UserList[] = ldap_add_user($user_id);
151 151
 					}
152 152
 				}
153
-				if (isset($_GET['id_session']) && (trim($_GET['id_session'])!=""))
153
+				if (isset($_GET['id_session']) && (trim($_GET['id_session']) != ""))
154 154
 					addUserToSession($UserList, $_GET['id_session']);
155
-				if(count($UserList)>0)
155
+				if (count($UserList) > 0)
156 156
 				{
157 157
 					Display :: display_normal_message(count($UserList)." ".get_lang('LDAPUsersAdded'));
158 158
 				}
@@ -167,8 +167,8 @@  discard block
 block discarded – undo
167 167
 	}
168 168
 }
169 169
 
170
-$form = new FormValidator('advanced_search','get');
171
-$form->addText('keyword_username',get_lang('LoginName'),false);
170
+$form = new FormValidator('advanced_search', 'get');
171
+$form->addText('keyword_username', get_lang('LoginName'), false);
172 172
 if (api_is_western_name_order())
173 173
 {
174 174
 	$form->addText('keyword_firstname', get_lang('FirstName'), false);
@@ -176,20 +176,20 @@  discard block
 block discarded – undo
176 176
 }
177 177
 else
178 178
 {
179
-	$form->addText('keyword_lastname',get_lang('LastName'),false);
180
-	$form->addText('keyword_firstname',get_lang('FirstName'),false);
179
+	$form->addText('keyword_lastname', get_lang('LastName'), false);
180
+	$form->addText('keyword_firstname', get_lang('FirstName'), false);
181 181
 }
182 182
 if (isset($_GET['id_session']))
183
-	$form->addElement('hidden','id_session',$_GET['id_session']);
183
+	$form->addElement('hidden', 'id_session', $_GET['id_session']);
184 184
 
185 185
 $type = array();
186 186
 $type["all"] = get_lang('All');
187
-$type["employee"]  = get_lang('Teacher');
187
+$type["employee"] = get_lang('Teacher');
188 188
 $type["student"] = get_lang('Student');
189 189
 
190
-$form->addElement('select','keyword_type',get_lang('Status'),$type);
190
+$form->addElement('select', 'keyword_type', get_lang('Status'), $type);
191 191
 // Structure a rajouer ??
192
-$form->addElement('submit','submit',get_lang('Ok'));
192
+$form->addElement('submit', 'submit', get_lang('Ok'));
193 193
 //$defaults['keyword_active'] = 1;
194 194
 //$defaults['keyword_inactive'] = 1;
195 195
 //$form->setDefaults($defaults);
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 //$table->set_column_filter(5, 'email_filter');
226 226
 //$table->set_column_filter(5, 'active_filter');
227 227
 $table->set_column_filter(5, 'modify_filter');
228
-$table->set_form_actions(array ('add_user' => get_lang('AddLDAPUsers')));
228
+$table->set_form_actions(array('add_user' => get_lang('AddLDAPUsers')));
229 229
 $table->display();
230 230
 
231 231
 /*
Please login to merge, or discard this patch.
main/admin/filler.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 $nameTools = get_lang('PlatformAdmin');
23 23
 
24 24
 // setting breadcrumbs
25
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => $nameTools);
25
+$interbreadcrumb[] = array("url" => 'index.php', "name" => $nameTools);
26 26
 
27 27
 // setting the name of the tool
28 28
 $nameTools = get_lang('DataFiller');
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 Display::display_header($nameTools);
48 48
 
49 49
 $result = '';
50
-if (count($output)>0) {
50
+if (count($output) > 0) {
51 51
     $result = '<div class="filler-report">'."\n";
52 52
     $result .= '<h3>'.$output[0]['title'].'</h3>'."\n";
53 53
     $result .= '<table>';
@@ -58,15 +58,15 @@  discard block
 block discarded – undo
58 58
     }
59 59
     $result .= '</table>';
60 60
     $result .= '</div>';
61
-    Display::display_normal_message($result,false);
61
+    Display::display_normal_message($result, false);
62 62
 }
63 63
 ?>
64 64
 <div id="datafiller" class="actions">
65
-  <h4><?php echo Display::return_icon('bug.png',get_lang('DataFiller'),null,ICON_SIZE_MEDIUM).' '.get_lang('DataFiller')?></h4>
66
-  <div class="description"><?php echo get_lang('ThisSectionIsOnlyVisibleOnSourceInstalls');?></div>
65
+  <h4><?php echo Display::return_icon('bug.png', get_lang('DataFiller'), null, ICON_SIZE_MEDIUM).' '.get_lang('DataFiller')?></h4>
66
+  <div class="description"><?php echo get_lang('ThisSectionIsOnlyVisibleOnSourceInstalls'); ?></div>
67 67
   <ul class="fillers">
68
-    <li><a href="filler.php?fill=users"><?php echo Display::return_icon('user.png',get_lang('FillUsers'),null,ICON_SIZE_SMALL).' '.get_lang('FillUsers');?></a></li>
69
-    <li><a href="filler.php?fill=courses"><?php echo Display::return_icon('new-course.png',get_lang('FillCourses'),null,ICON_SIZE_SMALL).' '.get_lang('FillCourses');?></a></li>
68
+    <li><a href="filler.php?fill=users"><?php echo Display::return_icon('user.png', get_lang('FillUsers'), null, ICON_SIZE_SMALL).' '.get_lang('FillUsers'); ?></a></li>
69
+    <li><a href="filler.php?fill=courses"><?php echo Display::return_icon('new-course.png', get_lang('FillCourses'), null, ICON_SIZE_SMALL).' '.get_lang('FillCourses'); ?></a></li>
70 70
   </ul>
71 71
 </div>
72 72
 <?php
Please login to merge, or discard this patch.
main/admin/access_url_edit_course_category_to_url.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@  discard block
 block discarded – undo
24 24
 
25 25
 // Setting breadcrumbs
26 26
 $tool_name = get_lang('EditUserGroupToURL');
27
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
28
-$interbreadcrumb[] = array ('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs'));
27
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
28
+$interbreadcrumb[] = array('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs'));
29 29
 
30 30
 $add_type = 'multiple';
31 31
 if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') {
@@ -72,10 +72,10 @@  discard block
 block discarded – undo
72 72
 }
73 73
 </script>';
74 74
 
75
-$form_sent=0;
76
-$errorMsg='';
77
-$UserList=$SessionList = array();
78
-$users=$sessions = array();
75
+$form_sent = 0;
76
+$errorMsg = '';
77
+$UserList = $SessionList = array();
78
+$users = $sessions = array();
79 79
 
80 80
 if (isset($_POST['form_sent']) && $_POST['form_sent']) {
81 81
     $form_sent = $_POST['form_sent'];
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 
118 118
     foreach ($userGroups as $item) {
119 119
         if ($item['access_url_id'] == $access_url_id) {
120
-            $userGroupList[$item['id']] = $item ;
120
+            $userGroupList[$item['id']] = $item;
121 121
         }
122 122
     }
123 123
     $noUserGroupList = getCourseCategoryNotInList(array_keys($userGroupList));
@@ -153,11 +153,11 @@  discard block
 block discarded – undo
153 153
         if (!empty($access_url_id)) {
154 154
             if ($url_obj[0] == $access_url_id) {
155 155
                 $checked = 'selected=true';
156
-                $url_selected=$url_obj[1];
156
+                $url_selected = $url_obj[1];
157 157
             }
158 158
         }
159
-        if ($url_obj['active']==1) { ?>
160
-            <option <?php echo $checked;?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?>
159
+        if ($url_obj['active'] == 1) { ?>
160
+            <option <?php echo $checked; ?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?>
161 161
             </option>
162 162
         <?php
163 163
         }
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
   <td align="center"><b><?php echo get_lang('CourseCategoryInPlatform') ?> :</b>
181 181
   </td>
182 182
   <td></td>
183
-  <td align="center"><b><?php printf(get_lang('CourseCategoryListInX'),$url_selected); ?></b></td>
183
+  <td align="center"><b><?php printf(get_lang('CourseCategoryListInX'), $url_selected); ?></b></td>
184 184
 </tr>
185 185
 
186 186
 <tr>
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
   <td align="center">
226 226
   <select id="destination_users" name="course_list[]" multiple="multiple" size="15" style="width:380px;">
227 227
 <?php
228
-foreach($userGroupList as $item) {
228
+foreach ($userGroupList as $item) {
229 229
 ?>
230 230
 	<option value="<?php echo $item['id']; ?>">
231 231
         <?php echo $item['name']; ?>
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
 	<td colspan="3" align="center">
241 241
 		<br />
242 242
 		<?php
243
-		if(isset($_GET['add']))
243
+		if (isset($_GET['add']))
244 244
 			echo '<button class="save" onclick="valide()" >'.get_lang('Add').'</button>';
245 245
 		else
246 246
 			echo '<button class="save" onclick="valide()" >'.get_lang('Edit').'</button>';
Please login to merge, or discard this patch.
main/admin/add_users_to_group.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -16,14 +16,14 @@  discard block
 block discarded – undo
16 16
 api_protect_admin_script(true);
17 17
 
18 18
 // setting breadcrumbs
19
-$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
20
-$interbreadcrumb[] = array('url' => 'group_list.php','name' => get_lang('GroupList'));
19
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
20
+$interbreadcrumb[] = array('url' => 'group_list.php', 'name' => get_lang('GroupList'));
21 21
 
22 22
 // Database Table Definitions
23
-$tbl_group			= Database::get_main_table(TABLE_MAIN_GROUP);
24
-$tbl_user			= Database::get_main_table(TABLE_MAIN_USER);
25
-$tbl_group_rel_user	= Database::get_main_table(TABLE_USERGROUP_REL_USER);
26
-$tbl_user_rel_access_url= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
23
+$tbl_group = Database::get_main_table(TABLE_MAIN_GROUP);
24
+$tbl_user = Database::get_main_table(TABLE_MAIN_USER);
25
+$tbl_group_rel_user = Database::get_main_table(TABLE_USERGROUP_REL_USER);
26
+$tbl_user_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
27 27
 $needle = null;
28 28
 $user_anonymous = api_get_anonymous_id();
29 29
 
@@ -83,14 +83,14 @@  discard block
 block discarded – undo
83 83
         } else {
84 84
             $return_destination .= '<select id="destination_users" name="sessionUsersList[]" multiple="multiple" size="15" style="width:360px;"></select>';
85 85
         }
86
-        $xajax_response->addAssign('ajax_destination_list','innerHTML', api_utf8_encode($return_destination));
86
+        $xajax_response->addAssign('ajax_destination_list', 'innerHTML', api_utf8_encode($return_destination));
87 87
     } else {
88 88
         $return_destination .= '<select id="destination_users" name="sessionUsersList[]" multiple="multiple" size="15" style="width:360px;"></select>';
89
-        $xajax_response->addAssign('ajax_destination_list','innerHTML', api_utf8_encode($return_destination));
89
+        $xajax_response->addAssign('ajax_destination_list', 'innerHTML', api_utf8_encode($return_destination));
90 90
 
91 91
         if ($type == 'single') {
92
-            $return.= '';
93
-            $xajax_response->addAssign('ajax_list_users_single','innerHTML',api_utf8_encode($return));
92
+            $return .= '';
93
+            $xajax_response->addAssign('ajax_list_users_single', 'innerHTML', api_utf8_encode($return));
94 94
         } else {
95 95
             $return_origin .= '<select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" style="width:360px;"></select>';
96 96
             $xajax_response->addAssign('ajax_origin_list_multiple', 'innerHTML', api_utf8_encode($return_origin));
@@ -117,17 +117,17 @@  discard block
 block discarded – undo
117 117
                     }
118 118
                 }
119 119
                 $rs_single = Database::query($sql);
120
-                $i=0;
120
+                $i = 0;
121 121
                 while ($user = Database :: fetch_array($rs_single)) {
122 122
                     $i++;
123
-                    if ($i<=10) {
123
+                    if ($i <= 10) {
124 124
                         $person_name = api_get_person_name($user['firstname'], $user['lastname']);
125 125
                         $return .= '<a href="javascript: void(0);" onclick="javascript: add_user(\''.$user['user_id'].'\',\''.$person_name.' ('.$user['username'].')'.'\')">'.$person_name.' ('.$user['username'].')</a><br />';
126 126
                     } else {
127 127
                         $return .= '...<br />';
128 128
                     }
129 129
                 }
130
-                $xajax_response->addAssign('ajax_list_users_single','innerHTML',api_utf8_encode($return));
130
+                $xajax_response->addAssign('ajax_list_users_single', 'innerHTML', api_utf8_encode($return));
131 131
             } else {
132 132
                 $xajax_response->addAlert(get_lang('YouMustChooseARelationType'));
133 133
                 $xajax_response->addClear('user_to_add', 'value');
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
 </script>';
203 203
 
204 204
 $form_sent = 0;
205
-$errorMsg = $firstLetterUser = $firstLetterSession='';
205
+$errorMsg = $firstLetterUser = $firstLetterSession = '';
206 206
 $UserList = $SessionList = array();
207 207
 $users = $sessions = array();
208 208
 $noPHP_SELF = true;
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
         $rs_destination = Database::query($sql);
257 257
         if (Database::num_rows($rs_destination) > 0) {
258 258
             while ($row_destination_list = Database::fetch_array($rs_destination)) {
259
-                $sessionUsersList[$row_destination_list['user_id']] = $row_destination_list ;
259
+                $sessionUsersList[$row_destination_list['user_id']] = $row_destination_list;
260 260
             }
261 261
         }
262 262
     }
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
         $rs_destination = Database::query($sql);
341 341
         if (Database::num_rows($rs_destination) > 0) {
342 342
             while ($row_destination_list = Database::fetch_array($rs_destination)) {
343
-                $sessionUsersList[$row_destination_list['user_id']] = $row_destination_list ;
343
+                $sessionUsersList[$row_destination_list['user_id']] = $row_destination_list;
344 344
             }
345 345
         }
346 346
     }
@@ -359,20 +359,20 @@  discard block
 block discarded – undo
359 359
 	<?php echo $link_add_type_unique ?>&nbsp;|&nbsp;<?php echo $link_add_type_multiple ?>
360 360
 </div>
361 361
 
362
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $group_id; ?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?>>
362
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $group_id; ?>" style="margin:0px;" <?php if ($ajax_search) {echo ' onsubmit="valide();"'; }?>>
363 363
 <?php echo '<legend>'.$tool_name.' ('.$group_info['name'].')</legend>'; ?>
364
-<?php if ($add_type=='multiple') { ?>
364
+<?php if ($add_type == 'multiple') { ?>
365 365
 <select name="relation" id="relation" onchange="xajax_search_users(document.getElementById('firstLetterUser').value,'multiple',this.value)">
366 366
 <?php } else { ?>
367 367
 <select name="relation" id="relation" onchange="xajax_search_users(document.getElementById('user_to_add').value,'single',this.value);">
368 368
 <?php } ?>
369 369
 <option value=""><?php echo get_lang('SelectARelationType')?></option>
370
-<option value="<?php echo GROUP_USER_PERMISSION_ADMIN ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_ADMIN)?'selected=selected':'') ?> > <?php echo get_lang('Admin') ?></option>
371
-<option value="<?php echo GROUP_USER_PERMISSION_READER ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_READER)?'selected=selected':'') ?> > <?php echo get_lang('Reader') ?></option>
372
-<option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_PENDING_INVITATION)?'selected=selected':'') ?> > <?php echo get_lang('PendingInvitation') ?></option>
373
-<option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION_SENT_BY_USER ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_PENDING_INVITATION_SENT_BY_USER)?'selected=selected':'') ?> > <?php echo get_lang('WaitingForAdminResponse') ?></option>
374
-<option value="<?php echo GROUP_USER_PERMISSION_MODERATOR ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_MODERATOR)?'selected=selected':'') ?> > <?php echo get_lang('Moderator') ?></option>
375
-<option value="<?php echo GROUP_USER_PERMISSION_HRM ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation']==GROUP_USER_PERMISSION_HRM)?'selected=selected':'') ?> > <?php echo get_lang('Drh') ?></option>
370
+<option value="<?php echo GROUP_USER_PERMISSION_ADMIN ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_ADMIN) ? 'selected=selected' : '') ?> > <?php echo get_lang('Admin') ?></option>
371
+<option value="<?php echo GROUP_USER_PERMISSION_READER ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_READER) ? 'selected=selected' : '') ?> > <?php echo get_lang('Reader') ?></option>
372
+<option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_PENDING_INVITATION) ? 'selected=selected' : '') ?> > <?php echo get_lang('PendingInvitation') ?></option>
373
+<option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION_SENT_BY_USER ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_PENDING_INVITATION_SENT_BY_USER) ? 'selected=selected' : '') ?> > <?php echo get_lang('WaitingForAdminResponse') ?></option>
374
+<option value="<?php echo GROUP_USER_PERMISSION_MODERATOR ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_MODERATOR) ? 'selected=selected' : '') ?> > <?php echo get_lang('Moderator') ?></option>
375
+<option value="<?php echo GROUP_USER_PERMISSION_HRM ?>" <?php echo ((isset($_POST['relation']) && $_POST['relation'] == GROUP_USER_PERMISSION_HRM) ? 'selected=selected' : '') ?> > <?php echo get_lang('Drh') ?></option>
376 376
 </select>
377 377
 <input type="hidden" name="form_sent" value="1" />
378 378
 <input type="hidden" name="id" value="<?php echo $group_id ?>" />
@@ -391,7 +391,7 @@  discard block
 block discarded – undo
391 391
   <td>&nbsp;</td>
392 392
   <td align="center"><b><?php echo get_lang('UsersInGroup') ?> :</b></td>
393 393
 </tr>
394
-<?php if ($add_type=='multiple') { ?>
394
+<?php if ($add_type == 'multiple') { ?>
395 395
 <tr>
396 396
 <td align="center">
397 397
 <?php echo get_lang('FirstLetterUser'); ?> :
@@ -412,7 +412,7 @@  discard block
 block discarded – undo
412 412
   <td align="center">
413 413
   <div id="content_source">
414 414
   	  <?php
415
-  	  if (!($add_type=='multiple')) {
415
+  	  if (!($add_type == 'multiple')) {
416 416
   	  	?>
417 417
 		<input type="text" id="user_to_add" onkeyup="xajax_search_users(this.value,'single',document.getElementById('relation').value)" />
418 418
 		<div id="ajax_list_users_single"></div>
@@ -423,7 +423,7 @@  discard block
 block discarded – undo
423 423
 	  <select id="origin_users" name="nosessionUsersList[]" multiple="multiple" size="15" style="width:360px;">
424 424
 		<?php
425 425
 		if (!empty($nosessionUsersList)) {
426
-			foreach($nosessionUsersList as $enreg) {
426
+			foreach ($nosessionUsersList as $enreg) {
427 427
 			?>
428 428
 				<option value="<?php echo $enreg['user_id']; ?>"  > <?php echo $enreg['firstname'].' '.$enreg['lastname'].' ('.$enreg['username'].')'; ?></option>
429 429
 			<?php
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
   <select id="destination_users" name="sessionUsersList[]" multiple="multiple" size="15" style="width:360px;">
460 460
 	<?php
461 461
 	if (!empty($sessionUsersList)) {
462
-		foreach($sessionUsersList as $enreg) { ?>
462
+		foreach ($sessionUsersList as $enreg) { ?>
463 463
 			<option value="<?php echo $enreg['user_id']; ?>">
464 464
                 <?php echo $enreg['firstname'].' '.$enreg['lastname'].' ('.$enreg['username'].')'; ?>
465 465
             </option>
Please login to merge, or discard this patch.
main/admin/system_announcements.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
 
15 15
 // Setting the section (for the tabs).
16 16
 $this_section = SECTION_PLATFORM_ADMIN;
17
-$_SESSION['this_section']=$this_section;
17
+$_SESSION['this_section'] = $this_section;
18 18
 
19 19
 $action = isset($_GET['action']) ? $_GET['action'] : null;
20 20
 $action_todo = false;
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
 }
36 36
 
37 37
 if (!empty($action)) {
38
-    $interbreadcrumb[] = array ("url" => "system_announcements.php", "name" => get_lang('SystemAnnouncements'));
38
+    $interbreadcrumb[] = array("url" => "system_announcements.php", "name" => get_lang('SystemAnnouncements'));
39 39
     if ($action == 'add') {
40
-        $interbreadcrumb[] = array ("url" => '#', "name" => get_lang('AddAnnouncement'));
40
+        $interbreadcrumb[] = array("url" => '#', "name" => get_lang('AddAnnouncement'));
41 41
     }
42 42
     if ($action == 'edit') {
43
-        $interbreadcrumb[] = array ("url" => '#', "name" => get_lang('Edit'));
43
+        $interbreadcrumb[] = array("url" => '#', "name" => get_lang('Edit'));
44 44
     }
45 45
 } else {
46 46
     $tool_name = get_lang('SystemAnnouncements');
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 }
67 67
 
68 68
 // Actions
69
-switch($action) {
69
+switch ($action) {
70 70
     case 'make_visible':
71 71
     case 'make_invisible':
72 72
         $status = false;
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
         Display :: display_confirmation_message(get_lang('AnnouncementDeleted'));
82 82
         break;
83 83
     case 'delete_selected':
84
-        foreach($_POST['id'] as $index => $id) {
84
+        foreach ($_POST['id'] as $index => $id) {
85 85
             SystemAnnouncementManager :: delete_announcement($id);
86 86
         }
87 87
         Display :: display_confirmation_message(get_lang('AnnouncementDeleted'));
@@ -137,11 +137,11 @@  discard block
 block discarded – undo
137 137
     $language_list = api_get_languages();
138 138
     $language_list_with_keys = array();
139 139
     $language_list_with_keys['all'] = get_lang('All');
140
-    for ($i=0; $i<count($language_list['name']) ; $i++) {
140
+    for ($i = 0; $i < count($language_list['name']); $i++) {
141 141
         $language_list_with_keys[$language_list['folder'][$i]] = $language_list['name'][$i];
142 142
     }
143 143
 
144
-    $form->addElement('select', 'lang',get_lang('Language'), $language_list_with_keys);
144
+    $form->addElement('select', 'lang', get_lang('Language'), $language_list_with_keys);
145 145
     $form->addHtmlEditor(
146 146
         'content',
147 147
         get_lang('Content'),
@@ -157,9 +157,9 @@  discard block
 block discarded – undo
157 157
 
158 158
     $group = array();
159 159
 
160
-    $group[]= $form->createElement('checkbox', 'visible_teacher', null, get_lang('Teacher'));
161
-    $group[]= $form->createElement('checkbox', 'visible_student', null, get_lang('Student'));
162
-    $group[]= $form->createElement('checkbox', 'visible_guest', null, get_lang('Guest'));
160
+    $group[] = $form->createElement('checkbox', 'visible_teacher', null, get_lang('Teacher'));
161
+    $group[] = $form->createElement('checkbox', 'visible_student', null, get_lang('Student'));
162
+    $group[] = $form->createElement('checkbox', 'visible_guest', null, get_lang('Guest'));
163 163
 
164 164
     $form->addGroup($group, null, get_lang('Visible'), '');
165 165
 
@@ -182,15 +182,15 @@  discard block
 block discarded – undo
182 182
 
183 183
     $form->addElement('checkbox', 'send_mail', null, get_lang('SendMail'));
184 184
 
185
-    if (isset($_REQUEST['action']) && $_REQUEST['action']=='add') {
185
+    if (isset($_REQUEST['action']) && $_REQUEST['action'] == 'add') {
186 186
         $form->addElement('checkbox', 'add_to_calendar', null, get_lang('AddToCalendar'));
187
-        $text=get_lang('AddNews');
188
-        $class='add';
187
+        $text = get_lang('AddNews');
188
+        $class = 'add';
189 189
         $form->addElement('hidden', 'action', 'add');
190 190
 
191 191
     } elseif (isset($_REQUEST['action']) && $_REQUEST['action'] == 'edit') {
192
-        $text=get_lang('EditNews');
193
-        $class='save';
192
+        $text = get_lang('EditNews');
193
+        $class = 'save';
194 194
         $form->addElement('hidden', 'action', 'edit');
195 195
     }
196 196
     $form->addElement('checkbox', 'send_email_test', null, get_lang('SendOnlyAnEmailToMySelfToTest'));
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
 
281 281
 if ($show_announcement_list) {
282 282
     $announcements = SystemAnnouncementManager :: get_all_announcements();
283
-    $announcement_data = array ();
283
+    $announcement_data = array();
284 284
     foreach ($announcements as $index => $announcement) {
285 285
         $row = array();
286 286
         $row[] = $announcement->id;
@@ -288,9 +288,9 @@  discard block
 block discarded – undo
288 288
         $row[] = $announcement->title;
289 289
         $row[] = api_convert_and_format_date($announcement->date_start);
290 290
         $row[] = api_convert_and_format_date($announcement->date_end);
291
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_TEACHER."&action=". ($announcement->visible_teacher ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_teacher  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
292
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_STUDENT."&action=". ($announcement->visible_student  ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_student  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
293
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_GUEST."&action=". ($announcement->visible_guest ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_guest  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
291
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_TEACHER."&action=".($announcement->visible_teacher ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_teacher ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
292
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_STUDENT."&action=".($announcement->visible_student ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_student ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
293
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_GUEST."&action=".($announcement->visible_guest ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_guest ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
294 294
 
295 295
         $row[] = $announcement->lang;
296 296
         $row[] = "<a href=\"?action=edit&id=".$announcement->id."\">".Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL)."</a> <a href=\"?action=delete&id=".$announcement->id."\"  onclick=\"javascript:if(!confirm('".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES))."')) return false;\">".Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)."</a>";
Please login to merge, or discard this patch.