Completed
Push — 1.11.x ( ac8a66...46c8b4 )
by José
126:29 queued 89:25
created
main/webservices/webservice_user.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -20,12 +20,12 @@  discard block
 block discarded – undo
20 20
 	 */
21 21
 	protected function changeUserActiveState($user_id_field_name, $user_id_value, $state) {
22 22
 		$user_id = $this->getUserId($user_id_field_name, $user_id_value);
23
-		if($user_id instanceof WSError) {
23
+		if ($user_id instanceof WSError) {
24 24
 			return $user_id;
25 25
 		} else {
26
-			if($state == 0) {
26
+			if ($state == 0) {
27 27
 				UserManager::disable($user_id);
28
-			} else if($state == 1) {
28
+			} else if ($state == 1) {
29 29
 				UserManager::enable($user_id);
30 30
 			}
31 31
 		}
@@ -40,11 +40,11 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	protected function changeUsersActiveState($users, $state) {
42 42
 		$results = array();
43
-		foreach($users as $user) {
43
+		foreach ($users as $user) {
44 44
 			$result_tmp = array();
45 45
 			$result_op = $this->changeUserActiveState($user['user_id_field_name'], $user['user_id_value'], $state);
46 46
 			$result_tmp['user_id_value'] = $user['user_id_value'];
47
-			if($result_op instanceof WSError) {
47
+			if ($result_op instanceof WSError) {
48 48
 				// Return the error in the results
49 49
 				$result_tmp['result'] = $result_op->toArray();
50 50
 			} else {
@@ -64,12 +64,12 @@  discard block
 block discarded – undo
64 64
 	 */
65 65
 	public function DisableUser($secret_key, $user_id_field_name, $user_id_value) {
66 66
 		$verifKey = $this->verifyKey($secret_key);
67
-		if($verifKey instanceof WSError) {
67
+		if ($verifKey instanceof WSError) {
68 68
 			// Let the implementation handle it
69 69
 			$this->handleError($verifKey);
70 70
 		} else {
71 71
 			$result = $this->changeUserActiveState($user_id_field_name, $user_id_value, 0);
72
-			if($result instanceof WSError) {
72
+			if ($result instanceof WSError) {
73 73
 				$this->handleError($result);
74 74
 			}
75 75
 		}
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 	 */
86 86
 	public function DisableUsers($secret_key, $users) {
87 87
 		$verifKey = $this->verifyKey($secret_key);
88
-		if($verifKey instanceof WSError) {
88
+		if ($verifKey instanceof WSError) {
89 89
 			// Let the implementation handle it
90 90
 			$this->handleError($verifKey);
91 91
 		} else {
@@ -102,11 +102,11 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	public function EnableUser($secret_key, $user_id_field_name, $user_id_value) {
104 104
 		$verifKey = $this->verifyKey($secret_key);
105
-		if($verifKey instanceof WSError) {
105
+		if ($verifKey instanceof WSError) {
106 106
 			$this->handleError($verifKey);
107 107
 		} else {
108 108
 			$result = $this->changeUserActiveState($user_id_field_name, $user_id_value, 1);
109
-			if($result instanceof WSError) {
109
+			if ($result instanceof WSError) {
110 110
 				$this->handleError($result);
111 111
 			}
112 112
 		}
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 	 */
123 123
 	public function EnableUsers($secret_key, $users) {
124 124
 		$verifKey = $this->verifyKey($secret_key);
125
-		if($verifKey instanceof WSError) {
125
+		if ($verifKey instanceof WSError) {
126 126
 			// Let the implementation handle it
127 127
 			$this->handleError($verifKey);
128 128
 		} else {
@@ -139,10 +139,10 @@  discard block
 block discarded – undo
139 139
 	 */
140 140
 	protected function deleteUserHelper($user_id_field_name, $user_id_value) {
141 141
 		$user_id = $this->getUserId($user_id_field_name, $user_id_value);
142
-		if($user_id instanceof WSError) {
142
+		if ($user_id instanceof WSError) {
143 143
 			return $user_id;
144 144
 		} else {
145
-			if(!UserManager::delete_user($user_id)) {
145
+			if (!UserManager::delete_user($user_id)) {
146 146
 				return new WSError(101, "There was a problem while deleting this user");
147 147
 			} else {
148 148
 				return true;
@@ -159,11 +159,11 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	public function DeleteUser($secret_key, $user_id_field_name, $user_id_value) {
161 161
 		$verifKey = $this->verifyKey($secret_key);
162
-		if($verifKey instanceof WSError) {
162
+		if ($verifKey instanceof WSError) {
163 163
 			$this->handleError($verifKey);
164 164
 		} else {
165 165
 			$result = $this->deleteUserHelper($user_id_field_name, $user_id_value);
166
-			if($result instanceof WSError) {
166
+			if ($result instanceof WSError) {
167 167
 				$this->handleError($result);
168 168
 			}
169 169
 		}
@@ -179,15 +179,15 @@  discard block
 block discarded – undo
179 179
 	 */
180 180
 	public function DeleteUsers($secret_key, $users) {
181 181
 		$verifKey = $this->verifyKey($secret_key);
182
-		if($verifKey instanceof WSError) {
182
+		if ($verifKey instanceof WSError) {
183 183
 			$this->handleError($verifKey);
184 184
 		} else {
185 185
 			$results = array();
186
-			foreach($users as $user) {
186
+			foreach ($users as $user) {
187 187
 				$result_tmp = array();
188 188
 				$result_op = $this->deleteUserHelper($user['user_id_field_name'], $user['user_id_value']);
189 189
 				$result_tmp['user_id_value'] = $user['user_id_value'];
190
-				if($result_op instanceof WSError) {
190
+				if ($result_op instanceof WSError) {
191 191
 					// Return the error in the results
192 192
 					$result_tmp['result'] = $result_op->toArray();
193 193
 				} else {
@@ -223,11 +223,11 @@  discard block
 block discarded – undo
223 223
 
224 224
 		// Add the original user id field name and value to the extra fields if needed
225 225
 		$extras_associative = array();
226
-		if($user_id_field_name != "chamilo_user_id") {
226
+		if ($user_id_field_name != "chamilo_user_id") {
227 227
 			$extras_associative[$user_id_field_name] = $user_id_value;
228 228
 		}
229 229
                 if (!empty($extras)) {
230
-                    foreach($extras as $extra) {
230
+                    foreach ($extras as $extra) {
231 231
                         $extras_associative[$extra['field_name']] = $extra['field_value'];
232 232
                     }
233 233
                 }
@@ -273,11 +273,11 @@  discard block
 block discarded – undo
273 273
 	public function CreateUser($secret_key, $firstname, $lastname, $status, $login, $password, $encrypt_method, $user_id_field_name, $user_id_value, $visibility = 1, $email = '', $language = 'english', $phone = '', $expiration_date = '0000-00-00 00:00:00', $extras = array()) {
274 274
 		// First, verify the secret key
275 275
 		$verifKey = $this->verifyKey($secret_key);
276
-		if($verifKey instanceof WSError) {
276
+		if ($verifKey instanceof WSError) {
277 277
 			$this->handleError($verifKey);
278 278
 		} else {
279 279
 			$result = $this->createUserHelper($firstname, $lastname, $status, $login, $password, $encrypt_method, $user_id_field_name, $user_id_value, $visibility, $email, $language, $phone, $expiration_date, $extras);
280
-			if($result instanceof WSError) {
280
+			if ($result instanceof WSError) {
281 281
 				$this->handleError($result);
282 282
 			} else {
283 283
 				return $result;
@@ -294,17 +294,17 @@  discard block
 block discarded – undo
294 294
 	 */
295 295
 	public function CreateUsers($secret_key, $users) {
296 296
 		$verifKey = $this->verifyKey($secret_key);
297
-		if($verifKey instanceof WSError) {
297
+		if ($verifKey instanceof WSError) {
298 298
 			$this->handleError($verifKey);
299 299
 		} else {
300 300
 			$results = array();
301
-			foreach($users as $user) {
301
+			foreach ($users as $user) {
302 302
 				$result_tmp = array();
303 303
                 // re-initialize variables just in case
304 304
                 $firstname = $lastname = $status = $login = $password = $encrypt_method = $user_id_field_name = $user_id_value = $visibility = $email = $language = $phone = $expiration_date = $extras = null;
305 305
 				extract($user);
306 306
 				$result = $this->createUserHelper($firstname, $lastname, $status, $login, $password, $encrypt_method, $user_id_field_name, $user_id_value, $visibility, $email, $language, $phone, $expiration_date, $extras);
307
-				if($result instanceof WSError) {
307
+				if ($result instanceof WSError) {
308 308
 					$result_tmp['result'] = $result->toArray();
309 309
 					$result_tmp['user_id_value'] = $user_id_value;
310 310
 					$result_tmp['user_id_generated'] = 0;
@@ -353,10 +353,10 @@  discard block
 block discarded – undo
353 353
 		$extras
354 354
 	) {
355 355
 		$user_id = $this->getUserId($user_id_field_name, $user_id_value);
356
-		if($user_id instanceof WSError) {
356
+		if ($user_id instanceof WSError) {
357 357
 			return $user_id;
358 358
 		} else {
359
-			if($password == '') {
359
+			if ($password == '') {
360 360
 				$password = null;
361 361
 			}
362 362
 			$user_info = api_get_user_info($user_id);
@@ -417,19 +417,19 @@  discard block
 block discarded – undo
417 417
 	public function EditUser($secret_key, $user_id_field_name, $user_id_value, $firstname, $lastname, $status, $loginname, $password, $encrypt_method, $email, $language, $phone, $expiration_date, $extras) {
418 418
 		// First, verify the secret key
419 419
 		$verifKey = $this->verifyKey($secret_key);
420
-		if($verifKey instanceof WSError) {
420
+		if ($verifKey instanceof WSError) {
421 421
 			$this->handleError($verifKey);
422 422
 		} else {
423 423
 
424 424
 			$extras_associative = array();
425 425
 			if (!empty($extras)) {
426
-				foreach($extras as $extra) {
426
+				foreach ($extras as $extra) {
427 427
 					$extras_associative[$extra['field_name']] = $extra['field_value'];
428 428
 				}
429 429
 			}
430 430
 
431 431
 			$result = $this->editUserHelper($user_id_field_name, $user_id_value, $firstname, $lastname, $status, $loginname, $password, $encrypt_method, $email, $language, $phone, $expiration_date, $extras_associative);
432
-			if($result instanceof WSError) {
432
+			if ($result instanceof WSError) {
433 433
 				$this->handleError($result);
434 434
 			}
435 435
 		}
@@ -445,18 +445,18 @@  discard block
 block discarded – undo
445 445
 	 */
446 446
 	public function EditUsers($secret_key, $users) {
447 447
 		$verifKey = $this->verifyKey($secret_key);
448
-		if($verifKey instanceof WSError) {
448
+		if ($verifKey instanceof WSError) {
449 449
 			$this->handleError($verifKey);
450 450
 		} else {
451 451
 			$results = array();
452
-			foreach($users as $user) {
452
+			foreach ($users as $user) {
453 453
 				$result_tmp = array();
454 454
                 // re-initialize variables just in case
455 455
                 $user_id_field_name = $user_id_value = $firstname = $lastname = $status = $loginname = $password = $encrypt_method = $email = $language = $phone = $expiration_date = $extras = null;
456 456
 				extract($user);
457 457
 				$result_op = $this->editUserHelper($user_id_field_name, $user_id_value, $firstname, $lastname, $status, $loginname, $password, $encrypt_method, $email, $language, $phone, $expiration_date, $extras);
458 458
 				$result_tmp['user_id_value'] = $user['user_id_value'];
459
-				if($result_op instanceof WSError) {
459
+				if ($result_op instanceof WSError) {
460 460
 					// Return the error in the results
461 461
 					$result_tmp['result'] = $result_op->toArray();
462 462
 				} else {
Please login to merge, or discard this patch.
main/admin/access_url_edit_course_category_to_url.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 
114 114
     foreach ($userGroups as $item) {
115 115
         if ($item['access_url_id'] == $access_url_id) {
116
-            $userGroupList[$item['id']] = $item ;
116
+            $userGroupList[$item['id']] = $item;
117 117
         }
118 118
     }
119 119
     $noUserGroupList = CourseCategory::getCourseCategoryNotInList(array_keys($userGroupList));
@@ -151,11 +151,11 @@  discard block
 block discarded – undo
151 151
         if (!empty($access_url_id)) {
152 152
             if ($url_obj[0] == $access_url_id) {
153 153
                 $checked = 'selected=true';
154
-                $url_selected=$url_obj[1];
154
+                $url_selected = $url_obj[1];
155 155
             }
156 156
         }
157
-        if ($url_obj['active']==1) { ?>
158
-            <option <?php echo $checked;?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?>
157
+        if ($url_obj['active'] == 1) { ?>
158
+            <option <?php echo $checked; ?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?>
159 159
             </option>
160 160
         <?php
161 161
         }
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
   <td align="center"><b><?php echo get_lang('CourseCategoryInPlatform') ?> :</b>
172 172
   </td>
173 173
   <td></td>
174
-  <td align="center"><b><?php printf(get_lang('CourseCategoryListInX'),$url_selected); ?></b></td>
174
+  <td align="center"><b><?php printf(get_lang('CourseCategoryListInX'), $url_selected); ?></b></td>
175 175
 </tr>
176 176
 
177 177
 <tr>
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
   <td align="center">
209 209
   <select id="destination_users" name="course_list[]" multiple="multiple" size="15" style="width:380px;">
210 210
 <?php
211
-foreach($userGroupList as $item) {
211
+foreach ($userGroupList as $item) {
212 212
 ?>
213 213
 	<option value="<?php echo $item['id']; ?>">
214 214
         <?php echo $item['name']; ?>
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
 	<td colspan="3" align="center">
224 224
 		<br />
225 225
 		<?php
226
-		if(isset($_GET['add']))
226
+		if (isset($_GET['add']))
227 227
 			echo '<button class="save" onclick="valide()" >'.get_lang('Add').'</button>';
228 228
 		else
229 229
 			echo '<button class="save" onclick="valide()" >'.get_lang('Edit').'</button>';
Please login to merge, or discard this patch.
main/admin/access_url_edit_courses_to_url.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
 $interbreadcrumb[] = array('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs'));
36 36
 
37 37
 $add_type = 'multiple';
38
-if (isset($_REQUEST['add_type']) && $_REQUEST['add_type']!='') {
38
+if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') {
39 39
 	$add_type = Security::remove_XSS($_REQUEST['add_type']);
40 40
 }
41 41
 
42 42
 $access_url_id = 1;
43
-if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id']!='') {
43
+if (isset($_REQUEST['access_url_id']) && $_REQUEST['access_url_id'] != '') {
44 44
 	$access_url_id = Security::remove_XSS($_REQUEST['access_url_id']);
45 45
 }
46 46
 
@@ -157,23 +157,23 @@  discard block
 block discarded – undo
157 157
 		<?php echo $link_add_type_unique ?>&nbsp;|&nbsp;<?php echo $link_add_type_multiple ?>
158 158
 	</div>
159 159
 	<br /><br />
160
-	<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>" style="margin:0px;" <?php if($ajax_search){echo ' onsubmit="valide();"';}?> >
160
+	<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>" style="margin:0px;" <?php if ($ajax_search) {echo ' onsubmit="valide();"'; }?> >
161 161
 		<?php echo get_lang('SelectUrl').' : '; ?>
162 162
 		<select name="access_url_id" onchange="javascript:send();">
163 163
 			<option value="0">-- <?php echo get_lang('SelectUrl')?> -- </option>
164 164
 			<?php
165
-			$url_selected='';
165
+			$url_selected = '';
166 166
 			foreach ($url_list as $url_obj) {
167 167
 				$checked = '';
168 168
 				if (!empty($access_url_id)) {
169
-					if ($url_obj[0]==$access_url_id) {
169
+					if ($url_obj[0] == $access_url_id) {
170 170
 						$checked = 'selected=true';
171
-						$url_selected=$url_obj[1];
171
+						$url_selected = $url_obj[1];
172 172
 					}
173 173
 				}
174
-				if ($url_obj['active']==1) {
174
+				if ($url_obj['active'] == 1) {
175 175
 					?>
176
-					<option <?php echo $checked;?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?></option>
176
+					<option <?php echo $checked; ?> value="<?php echo $url_obj[0]; ?>"> <?php echo $url_obj[1]; ?></option>
177 177
 				<?php
178 178
 				}
179 179
 			}
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
 				<td align="center">
197 197
 					<div id="content_source">
198 198
 						<?php
199
-						if($ajax_search) {
199
+						if ($ajax_search) {
200 200
 							?>
201 201
 							<input type="text" id="course_to_add" onkeyup="xajax_search_courses(this.value,document.formulaire.access_url_id.options[document.formulaire.access_url_id.selectedIndex].value)" />
202 202
 							<div id="ajax_list_courses"></div>
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 							?>
206 206
 							<select id="origin_users" name="no_course_list[]" multiple="multiple" size="15" style="width:380px;">
207 207
 								<?php
208
-								foreach($no_course_list as $no_course) {
208
+								foreach ($no_course_list as $no_course) {
209 209
 									?>
210 210
 									<option value="<?php echo $no_course['id']; ?>"><?php echo $no_course['title'].' ('.$no_course['code'].')'; ?></option>
211 211
 								<?php
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
 				</td>
221 221
 				<td width="10%" valign="middle" align="center">
222 222
 					<?php
223
-					if($ajax_search) {
223
+					if ($ajax_search) {
224 224
 						?>
225 225
 						<button class="btn btn-default" type="button" onclick="remove_item(document.getElementById('destination_users'))" >
226 226
                             <em class="fa fa-arrow-left"></em>
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
 				<td align="center">
244 244
 					<select id="destination_users" name="course_list[]" multiple="multiple" size="15" style="width:380px;">
245 245
 						<?php
246
-						foreach($course_list as $course) {
246
+						foreach ($course_list as $course) {
247 247
 							$courseInfo = api_get_course_info_by_id($course['id']);
248 248
 							?>
249 249
 							<option value="<?php echo $course['id']; ?>"><?php echo $course['title'].' ('.$courseInfo['code'].')'; ?></option>
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
 				<td colspan="3" align="center">
259 259
 					<br />
260 260
 					<?php
261
-					if(isset($_GET['add']))
261
+					if (isset($_GET['add']))
262 262
 						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('AddCoursesToURL').'</button>';
263 263
 					else
264 264
 						echo '<button class="btn btn-default" onclick="valide()" >'.get_lang('EditCoursesToURL').'</button>';
Please login to merge, or discard this patch.
main/exercise/export/aiken/aiken_import.inc.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
         $dir .= '/';
26 26
 
27 27
     do {
28
-        $path = $dir . $prefix . mt_rand(0, 9999999);
28
+        $path = $dir.$prefix.mt_rand(0, 9999999);
29 29
     } while (!mkdir($path, $mode));
30 30
 
31 31
     return $path;
@@ -39,8 +39,8 @@  discard block
 block discarded – undo
39 39
 {
40 40
     $name_tools = get_lang('ImportAikenQuiz');
41 41
     $form  = '<div class="actions">';
42
-    $form .= '<a href="exercise.php?show=test&'.api_get_cidreq().'">' .
43
-        Display :: return_icon('back.png', get_lang('BackToExercisesList'),'',ICON_SIZE_MEDIUM).'</a>';
42
+    $form .= '<a href="exercise.php?show=test&'.api_get_cidreq().'">'.
43
+        Display :: return_icon('back.png', get_lang('BackToExercisesList'), '', ICON_SIZE_MEDIUM).'</a>';
44 44
     $form .= '</div>';
45 45
     $form_validator = new FormValidator(
46 46
         'aiken_upload',
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 function aiken_import_exercise($file)
94 94
 {
95 95
     global $exercise_info;
96
-    $archive_path = api_get_path(SYS_ARCHIVE_PATH) . 'aiken';
96
+    $archive_path = api_get_path(SYS_ARCHIVE_PATH).'aiken';
97 97
     $baseWorkDir = $archive_path;
98 98
 
99 99
     if (!is_dir($baseWorkDir)) {
@@ -130,9 +130,9 @@  discard block
 block discarded – undo
130 130
 
131 131
     // Parse every subdirectory to search txt question files
132 132
     while (false !== ($file = readdir($exerciseHandle))) {
133
-        if (is_dir($baseWorkDir . '/' . $file) && $file != "." && $file != "..") {
133
+        if (is_dir($baseWorkDir.'/'.$file) && $file != "." && $file != "..") {
134 134
             //find each manifest for each question repository found
135
-            $questionHandle = opendir($baseWorkDir . '/' . $file);
135
+            $questionHandle = opendir($baseWorkDir.'/'.$file);
136 136
             while (false !== ($questionFile = readdir($questionHandle))) {
137 137
                 if (preg_match('/.txt$/i', $questionFile)) {
138 138
                     $result = aiken_parse_file(
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
         }
217 217
 
218 218
         // Delete the temp dir where the exercise was unzipped
219
-        my_delete($baseWorkDir . $uploadPath);
219
+        my_delete($baseWorkDir.$uploadPath);
220 220
         $operation = $last_exercise_id;
221 221
     }
222 222
 
@@ -239,8 +239,8 @@  discard block
 block discarded – undo
239 239
 function aiken_parse_file(&$exercise_info, $exercisePath, $file, $questionFile) {
240 240
     global $questionTempDir;
241 241
 
242
-    $questionTempDir = $exercisePath . '/' . $file . '/';
243
-    $questionFilePath = $questionTempDir . $questionFile;
242
+    $questionTempDir = $exercisePath.'/'.$file.'/';
243
+    $questionFilePath = $questionTempDir.$questionFile;
244 244
 
245 245
     if (!is_file($questionFilePath)) {
246 246
         return 'FileNotFound';
@@ -252,13 +252,13 @@  discard block
 block discarded – undo
252 252
     $answers_array = array();
253 253
     $new_question = true;
254 254
     foreach ($data as $line => $info) {
255
-        if ($question_index > 0 && $new_question == true && preg_match('/^(\r)?\n/',$info)) {
255
+        if ($question_index > 0 && $new_question == true && preg_match('/^(\r)?\n/', $info)) {
256 256
             // double empty line
257 257
             continue;
258 258
         }
259 259
         $new_question = false;
260 260
         //make sure it is transformed from iso-8859-1 to utf-8 if in that form
261
-        if (!mb_check_encoding($info,'utf-8') && mb_check_encoding($info,'iso-8859-1')) {
261
+        if (!mb_check_encoding($info, 'utf-8') && mb_check_encoding($info, 'iso-8859-1')) {
262 262
             $info = utf8_encode($info);
263 263
         }
264 264
         $exercise_info['question'][$question_index]['type'] = 'MCUA';
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
         } elseif (preg_match('/^ETIQUETAS:\s?([A-Z])\s?/', $info, $matches)) {
291 291
             //TAGS for chamilo >= 1.10 (Spanish e-ducativa format)
292 292
             $exercise_info['question'][$question_index]['answer_tags'] = explode(',', $matches[1]);
293
-        } elseif (preg_match('/^(\r)?\n/',$info)) {
293
+        } elseif (preg_match('/^(\r)?\n/', $info)) {
294 294
             //moving to next question (tolerate \r\n or just \n)
295 295
             if (empty($exercise_info['question'][$question_index]['correct_answers'])) {
296 296
                 error_log('Aiken: Error in question index '.$question_index.': no correct answer defined');
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
                     $exercise_info['question'][$question_index]['title'] = $info;
311 311
                 } else {
312 312
                     //Question itself (use a 100-chars long title and a larger description)
313
-                    $exercise_info['question'][$question_index]['title'] = trim(substr($info, 0, 100)) . '...';
313
+                    $exercise_info['question'][$question_index]['title'] = trim(substr($info, 0, 100)).'...';
314 314
                     $exercise_info['question'][$question_index]['description'] = $info;
315 315
                 }
316 316
             } else {
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
     }
321 321
     $total_questions = count($exercise_info['question']);
322 322
     $total_weight = (!empty($_POST['total_weight'])) ? intval($_POST['total_weight']) : 20;
323
-    foreach  ($exercise_info['question'] as $key => $question) {
323
+    foreach ($exercise_info['question'] as $key => $question) {
324 324
         $exercise_info['question'][$key]['weighting'][current(array_keys($exercise_info['question'][$key]['weighting']))] = $total_weight / $total_questions;
325 325
     }
326 326
     return true;
Please login to merge, or discard this patch.
main/exercise/aiken.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     if (isset($_POST['submit'])) {
33 33
         $id = aiken_import_file($_FILES['userFile']);
34 34
         if (is_numeric($id) && !empty($id)) {
35
-            header('Location: admin.php?' . api_get_cidreq() . '&exerciseId=' . $id);
35
+            header('Location: admin.php?'.api_get_cidreq().'&exerciseId='.$id);
36 36
             exit;
37 37
         }
38 38
     }
Please login to merge, or discard this patch.
main/admin/configure_extensions.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 
27 27
 			$rs = Database::query($sql);
28 28
 
29
-			if (Database::affected_rows($rs)>0){
29
+			if (Database::affected_rows($rs) > 0) {
30 30
 				$message = get_lang('ServiceActivated');
31 31
 			}
32 32
 
@@ -76,18 +76,18 @@  discard block
 block discarded – undo
76 76
 		WHERE variable LIKE "service_%" AND subkey="active" and selected_value="true"';
77 77
 
78 78
 $rs = Database::query($sql);
79
-while($row = Database::fetch_array($rs)){
79
+while ($row = Database::fetch_array($rs)) {
80 80
 	$listActiveServices[] = $row['variable'];
81 81
 }
82 82
 
83 83
 // javascript to handle accordion behaviour
84 84
 $javascript_message = '';
85
-if(!empty($message)){
85
+if (!empty($message)) {
86 86
 	$javascript_message = '
87 87
 	document.getElementById("message").style.display = "block";
88 88
 	var timer = setTimeout(hideMessage, 5000);';
89 89
 }
90
-$htmlHeadXtra[]= '<script>
90
+$htmlHeadXtra[] = '<script>
91 91
 var listeDiv;
92 92
 var extensionsHeader = new Array();
93 93
 var extensionsContent = new Array();
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
 ?>
136 136
 <div id="message" style="display: none">
137 137
 	<?php
138
-	if(!empty($message))
138
+	if (!empty($message))
139 139
 		Display::display_normal_message($message)
140 140
 	?>
141 141
 </div>
Please login to merge, or discard this patch.
main/admin/skill_badge_create.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -22,16 +22,16 @@  discard block
 block discarded – undo
22 22
 $objSkill = new Skill();
23 23
 $skill = $objSkill->get($skillId);
24 24
 
25
-$htmlHeadXtra[] = '<link  href="'. api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/media/css/core.css" rel="stylesheet">';
25
+$htmlHeadXtra[] = '<link  href="'.api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/media/css/core.css" rel="stylesheet">';
26 26
 
27 27
 // Add badge studio paths
28 28
 
29 29
 $badgeStudio = [
30
-    'core' => api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/',
31
-    'media' => api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/media/',
32
-    'templates' => api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/media/images/templates/',
33
-    'masks' => api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/media/images/masks/',
34
-    'script_js' => '<script src="'. api_get_path(WEB_LIBRARY_JS_PATH) .'badge-studio/media/js/studio.js?"></script>'
30
+    'core' => api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/',
31
+    'media' => api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/media/',
32
+    'templates' => api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/media/images/templates/',
33
+    'masks' => api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/media/images/masks/',
34
+    'script_js' => '<script src="'.api_get_path(WEB_LIBRARY_JS_PATH).'badge-studio/media/js/studio.js?"></script>'
35 35
 ];
36 36
 
37 37
 
@@ -59,10 +59,10 @@  discard block
 block discarded – undo
59 59
         if ($existsBadgesDirectory) {
60 60
 
61 61
             if (!empty($skill['icon'])) {
62
-                $iconFileAbsolutePath = $badgePath . $skill['icon'];
62
+                $iconFileAbsolutePath = $badgePath.$skill['icon'];
63 63
 
64 64
                 if (Security::check_abs_path($iconFileAbsolutePath, $badgePath)) {
65
-                    unlink($badgePath . $skill['icon']);
65
+                    unlink($badgePath.$skill['icon']);
66 66
                 }
67 67
             }
68 68
 
@@ -85,24 +85,24 @@  discard block
 block discarded – undo
85 85
 
86 86
     $objSkill->update($params);
87 87
 
88
-    header('Location: ' . api_get_path(WEB_CODE_PATH) . 'admin/skill_badge_list.php');
88
+    header('Location: '.api_get_path(WEB_CODE_PATH).'admin/skill_badge_list.php');
89 89
     exit;
90 90
 }
91 91
 
92 92
 $interbreadcrumb = array(
93 93
     array(
94
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/index.php',
94
+        'url' => api_get_path(WEB_CODE_PATH).'admin/index.php',
95 95
         'name' => get_lang('Administration')
96 96
     ),
97 97
     array(
98
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/skill_badge.php',
98
+        'url' => api_get_path(WEB_CODE_PATH).'admin/skill_badge.php',
99 99
         'name' => get_lang('Badges')
100 100
     )
101 101
 );
102 102
 
103 103
 $toolbar = Display::toolbarButton(
104 104
     get_lang('ManageSkills'),
105
-    api_get_path(WEB_CODE_PATH) . 'admin/skill_list.php',
105
+    api_get_path(WEB_CODE_PATH).'admin/skill_list.php',
106 106
     'list',
107 107
     'primary',
108 108
     ['title' => get_lang('ManageSkills')]
Please login to merge, or discard this patch.
main/social/group_view.php 1 patch
Spacing   +35 added lines, -35 removed lines patch added patch discarded remove patch
@@ -86,31 +86,31 @@  discard block
 block discarded – undo
86 86
 
87 87
 </script>';
88 88
 
89
-$allowed_views = array('mygroups','newest','pop');
89
+$allowed_views = array('mygroups', 'newest', 'pop');
90 90
 $content = null;
91 91
 
92
-if (isset($_GET['view']) && in_array($_GET['view'],$allowed_views)) {
92
+if (isset($_GET['view']) && in_array($_GET['view'], $allowed_views)) {
93 93
     if ($_GET['view'] == 'mygroups') {
94
-        $interbreadcrumb[]= array ('url' =>'groups.php','name' => get_lang('Groups'));
95
-        $interbreadcrumb[]= array ('url' =>'#','name' => get_lang('MyGroups'));
96
-    } else if ( $_GET['view'] == 'newest') {
97
-        $interbreadcrumb[]= array ('url' =>'groups.php','name' => get_lang('Groups'));
98
-        $interbreadcrumb[]= array ('url' =>'#','name' => get_lang('Newest'));
99
-    } else  {
100
-        $interbreadcrumb[]= array ('url' =>'groups.php','name' => get_lang('Groups'));
101
-        $interbreadcrumb[]= array ('url' =>'#','name' => get_lang('Popular'));
94
+        $interbreadcrumb[] = array('url' =>'groups.php', 'name' => get_lang('Groups'));
95
+        $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('MyGroups'));
96
+    } else if ($_GET['view'] == 'newest') {
97
+        $interbreadcrumb[] = array('url' =>'groups.php', 'name' => get_lang('Groups'));
98
+        $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('Newest'));
99
+    } else {
100
+        $interbreadcrumb[] = array('url' =>'groups.php', 'name' => get_lang('Groups'));
101
+        $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('Popular'));
102 102
     }
103 103
 } else {
104
-    $interbreadcrumb[]= array ('url' =>'groups.php','name' => get_lang('Groups'));
104
+    $interbreadcrumb[] = array('url' =>'groups.php', 'name' => get_lang('Groups'));
105 105
     if (!isset($_GET['id'])) {
106
-        $interbreadcrumb[]= array ('url' =>'#','name' => get_lang('GroupList'));
106
+        $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('GroupList'));
107 107
     } else {
108 108
         //$interbreadcrumb[]= array ('url' =>'#','name' => get_lang('Group'));
109 109
     }
110 110
 }
111 111
 
112 112
 // getting group information
113
-$group_id	= isset($_GET['id']) ? intval($_GET['id']) : null;
113
+$group_id = isset($_GET['id']) ? intval($_GET['id']) : null;
114 114
 $relation_group_title = '';
115 115
 $role = 0;
116 116
 
@@ -120,9 +120,9 @@  discard block
 block discarded – undo
120 120
     $group_info = $usergroup->get($group_id);
121 121
 
122 122
 
123
-    $interbreadcrumb[]= array ('url' =>'#','name' => $group_info['name']);
123
+    $interbreadcrumb[] = array('url' =>'#', 'name' => $group_info['name']);
124 124
 
125
-    if (isset($_GET['action']) && $_GET['action']=='leave') {
125
+    if (isset($_GET['action']) && $_GET['action'] == 'leave') {
126 126
         $user_leaved = intval($_GET['u']);
127 127
         // I can "leave me myself"
128 128
         if (api_get_user_id() == $user_leaved) {
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
     }
137 137
 
138 138
     // add a user to a group if its open
139
-    if (isset($_GET['action']) && $_GET['action']=='join') {
139
+    if (isset($_GET['action']) && $_GET['action'] == 'join') {
140 140
         // we add a user only if is a open group
141 141
         $user_join = intval($_GET['u']);
142 142
         if (api_get_user_id() == $user_join && !empty($group_id)) {
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
 $group_info = $usergroup->get($group_id);
166 166
 
167 167
 //Loading group information
168
-if (isset($_GET['status']) && $_GET['status']=='sent') {
168
+if (isset($_GET['status']) && $_GET['status'] == 'sent') {
169 169
     $social_right_content .= Display::return_message(get_lang('MessageHasBeenSent'), 'confirmation', false);
170 170
 }
171 171
 
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
 
175 175
 if (!$is_group_member && $group_info['visibility'] == GROUP_PERMISSION_CLOSED) {
176 176
     if ($role == GROUP_USER_PERMISSION_PENDING_INVITATION_SENT_BY_USER) {
177
-        $social_right_content .=  Display::return_message(get_lang('YouAlreadySentAnInvitation'));
177
+        $social_right_content .= Display::return_message(get_lang('YouAlreadySentAnInvitation'));
178 178
     }
179 179
 }
180 180
 
@@ -185,19 +185,19 @@  discard block
 block discarded – undo
185 185
         ) {
186 186
             $social_right_content .= '<div class="group-tool">';
187 187
             $social_right_content .= '<div class="pull-right">';
188
-            $social_right_content .=  '<a class="btn btn-default btn-sm" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.
188
+            $social_right_content .= '<a class="btn btn-default btn-sm" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.
189 189
                 get_lang('JoinGroup').'</a>';
190
-            $social_right_content .=  '</div>';
191
-            $social_right_content .=  '</div>';
190
+            $social_right_content .= '</div>';
191
+            $social_right_content .= '</div>';
192 192
         } elseif ($role == GROUP_USER_PERMISSION_PENDING_INVITATION) {
193 193
             $social_right_content .= '<div class="group-tool">';
194 194
             $social_right_content .= '<div class="pull-right">';
195
-            $social_right_content .=  '<a class="btn btn-default btn-sm" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.
195
+            $social_right_content .= '<a class="btn btn-default btn-sm" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.
196 196
                     Display::returnFontAwesomeIcon('envelope').' '.
197 197
                 get_lang('YouHaveBeenInvitedJoinNow').'</a>';
198 198
         }
199
-        $social_right_content .=  '</div>';
200
-        $social_right_content .=  '</div>';
199
+        $social_right_content .= '</div>';
200
+        $social_right_content .= '</div>';
201 201
     }
202 202
     $content = MessageManager::display_messages_for_group($group_id);
203 203
     if ($is_group_member) {
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
                     'action' => 'add_message_group'
212 212
                 ]);
213 213
             $create_thread_link = Display::url(
214
-                Display::returnFontAwesomeIcon('commenting') . ' ' .
214
+                Display::returnFontAwesomeIcon('commenting').' '.
215 215
                 get_lang('YouShouldCreateATopic'),
216 216
                 $createThreadUrl,
217 217
                 [
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
                     'action' => 'add_message_group',
232 232
                 ]);
233 233
             $create_thread_link = Display::url(
234
-                Display::returnFontAwesomeIcon('commenting') . ' ' .
234
+                Display::returnFontAwesomeIcon('commenting').' '.
235 235
                 get_lang('NewTopic'),
236 236
                 $createThreadUrl,
237 237
                 [
@@ -271,21 +271,21 @@  discard block
 block discarded – undo
271 271
         foreach ($members as $member) {
272 272
             // if is a member
273 273
             if (in_array($member['relation_type'],
274
-                array(GROUP_USER_PERMISSION_ADMIN, GROUP_USER_PERMISSION_READER,GROUP_USER_PERMISSION_MODERATOR))
274
+                array(GROUP_USER_PERMISSION_ADMIN, GROUP_USER_PERMISSION_READER, GROUP_USER_PERMISSION_MODERATOR))
275 275
             ) {
276 276
                 //add icons
277 277
                 if ($member['relation_type'] == GROUP_USER_PERMISSION_ADMIN) {
278
-                    $icon= Display::return_icon('social_group_admin.png', get_lang('Admin'));
278
+                    $icon = Display::return_icon('social_group_admin.png', get_lang('Admin'));
279 279
                 } elseif ($member['relation_type'] == GROUP_USER_PERMISSION_MODERATOR) {
280
-                    $icon= Display::return_icon('social_group_moderator.png', get_lang('Moderator'));
280
+                    $icon = Display::return_icon('social_group_moderator.png', get_lang('Moderator'));
281 281
                 } else {
282
-                    $icon= '';
282
+                    $icon = '';
283 283
                 }
284 284
 
285 285
                 $userPicture = UserManager::getUserPicture($member['id']);
286 286
                 $member_content .= '<div class="col-md-3">';
287 287
                 $member_content .= '<div class="items-user">';
288
-                $member_name = Display::url(api_get_person_name(cut($member['user_info']['firstname'],15),cut($member['user_info']['lastname'],15)).'&nbsp;'.$icon, $member['user_info']['profile_url']);
288
+                $member_name = Display::url(api_get_person_name(cut($member['user_info']['firstname'], 15), cut($member['user_info']['lastname'], 15)).'&nbsp;'.$icon, $member['user_info']['profile_url']);
289 289
                 $member_content .= Display::div('<img class="img-circle" src="'.$userPicture.'"/>', array('class' => 'avatar'));
290 290
                 $member_content .= Display::div($member_name, array('class' => 'name'));
291 291
                 $member_content .= '</div>';
@@ -297,10 +297,10 @@  discard block
 block discarded – undo
297 297
     }
298 298
 
299 299
     if (!empty($create_thread_link)) {
300
-        $create_thread_link =  Display::div($create_thread_link, array('class'=>'pull-right'));
300
+        $create_thread_link = Display::div($create_thread_link, array('class'=>'pull-right'));
301 301
     }
302 302
     $headers = array(get_lang('Discussions'), get_lang('Members'));
303
-    $socialForum = Display::tabs($headers, array($content, $member_content),'tabs');
303
+    $socialForum = Display::tabs($headers, array($content, $member_content), 'tabs');
304 304
 
305 305
 } else {
306 306
     // if I already sent an invitation message
@@ -312,9 +312,9 @@  discard block
 block discarded – undo
312 312
         )
313 313
     )
314 314
     ) {
315
-        $social_right_content .=  '<a class="btn" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.get_lang('JoinGroup').'</a>';
315
+        $social_right_content .= '<a class="btn" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.get_lang('JoinGroup').'</a>';
316 316
     } elseif ($role == GROUP_USER_PERMISSION_PENDING_INVITATION) {
317
-        $social_right_content .=  '<a class="btn" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.get_lang('YouHaveBeenInvitedJoinNow').'</a>';
317
+        $social_right_content .= '<a class="btn" href="group_view.php?id='.$group_id.'&action=join&u='.api_get_user_id().'">'.get_lang('YouHaveBeenInvitedJoinNow').'</a>';
318 318
     }
319 319
 }
320 320
 
Please login to merge, or discard this patch.
main/social/group_invitation.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
 
19 19
 // Database Table Definitions
20 20
 $tbl_user = Database::get_main_table(TABLE_MAIN_USER);
21
-$tbl_group_rel_user	= Database::get_main_table(TABLE_USERGROUP_REL_USER);
21
+$tbl_group_rel_user = Database::get_main_table(TABLE_USERGROUP_REL_USER);
22 22
 
23 23
 // setting the name of the tool
24 24
 $tool_name = get_lang('SubscribeUsersToGroup');
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
     }
41 41
 }
42 42
 
43
-$interbreadcrumb[] = array('url' =>'groups.php','name' => get_lang('Groups'));
43
+$interbreadcrumb[] = array('url' =>'groups.php', 'name' => get_lang('Groups'));
44 44
 $interbreadcrumb[] = array('url' => 'group_view.php?id='.$group_id, 'name' => $group_info['name']);
45 45
 $interbreadcrumb[] = array('url' =>'#', 'name' => get_lang('SubscribeUsersToGroup'));
46 46
 
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 
75 75
         if (is_array($user_list) && count($user_list) > 0) {
76 76
             //send invitation message
77
-            foreach ($user_list as $user_id){
77
+            foreach ($user_list as $user_id) {
78 78
                 $result = MessageManager::send_message(
79 79
                     $user_id,
80 80
                     $title,
@@ -110,8 +110,8 @@  discard block
 block discarded – undo
110 110
 
111 111
             if (!isset($group_friend_list[$group_id]) ||
112 112
                 isset($group_friend_list[$group_id]) &&
113
-                $group_friend_list[$group_id]['relation_type'] == '' ) {
114
-                $Users[$friend['friend_user_id']]= array(
113
+                $group_friend_list[$group_id]['relation_type'] == '') {
114
+                $Users[$friend['friend_user_id']] = array(
115 115
                     'user_id' => $friend['friend_user_id'],
116 116
                     'firstname' => $friend['firstName'],
117 117
                     'lastname' => $friend['lastName'],
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
                 );
121 121
             }
122 122
         } else {
123
-            $Users[$friend['friend_user_id']]= array(
123
+            $Users[$friend['friend_user_id']] = array(
124 124
                 'user_id' => $friend['friend_user_id'],
125 125
                 'firstname' =>$friend['firstName'],
126 126
                 'lastname' => $friend['lastName'],
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
     }
132 132
 }
133 133
 
134
-if (is_array($Users) && count($Users) > 0 ) {
134
+if (is_array($Users) && count($Users) > 0) {
135 135
     foreach ($Users as $user) {
136 136
         if ($user['group_id'] != $group_id) {
137 137
             $nosessionUsersList[$user['user_id']] = api_get_person_name(
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 }
144 144
 
145 145
 $social_left_content = SocialManager::show_social_menu('invite_friends', $group_id);
146
-$social_right_content =  '<h3 class="group-title">'.Security::remove_XSS($group_info['name'], STUDENT, true).'</h3>';
146
+$social_right_content = '<h3 class="group-title">'.Security::remove_XSS($group_info['name'], STUDENT, true).'</h3>';
147 147
 
148 148
 if (count($nosessionUsersList) == 0) {
149 149
     $friends = SocialManager::get_friends(api_get_user_id());
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
     array(GROUP_USER_PERMISSION_PENDING_INVITATION)
180 180
 );
181 181
 
182
-if (is_array($members) && count($members)>0) {
182
+if (is_array($members) && count($members) > 0) {
183 183
     foreach ($members as &$member) {
184 184
         $image = UserManager::getUserPicture($member['id']);
185 185
         $member['image'] = '<img class="img-circle" src="'.$image.'"  width="50px" height="50px"  />';
Please login to merge, or discard this patch.