@@ -73,10 +73,11 @@ discard block |
||
73 | 73 | $data = curl_exec( $ch ); |
74 | 74 | curl_close( $ch ); |
75 | 75 | |
76 | - if($data) |
|
77 | - return (new SimpleXMLElement($data)); |
|
78 | - else |
|
79 | - return false; |
|
76 | + if($data) { |
|
77 | + return (new SimpleXMLElement($data)); |
|
78 | + } else { |
|
79 | + return false; |
|
80 | + } |
|
80 | 81 | } |
81 | 82 | return (simplexml_load_file($url)); |
82 | 83 | } |
@@ -85,11 +86,9 @@ discard block |
||
85 | 86 | /* Process required params and throw errors if we don't get values */ |
86 | 87 | if ((isset($param)) && ($param != '')) { |
87 | 88 | return $param; |
88 | - } |
|
89 | - elseif (!isset($param)) { |
|
89 | + } elseif (!isset($param)) { |
|
90 | 90 | throw new Exception('Missing parameter.'); |
91 | - } |
|
92 | - else { |
|
91 | + } else { |
|
93 | 92 | throw new Exception(''.$param.' is required.'); |
94 | 93 | } |
95 | 94 | } |
@@ -99,8 +98,7 @@ discard block |
||
99 | 98 | /* Don't know if we'll use this one, but let's build it in case. */ |
100 | 99 | if ((isset($param)) && ($param != '')) { |
101 | 100 | return $param; |
102 | - } |
|
103 | - else { |
|
101 | + } else { |
|
104 | 102 | $param = ''; |
105 | 103 | return $param; |
106 | 104 | } |
@@ -137,8 +135,9 @@ discard block |
||
137 | 135 | '&duration='.urlencode($creationParams['duration']); |
138 | 136 | //'&meta_category='.urlencode($creationParams['meta_category']); |
139 | 137 | $welcomeMessage = $creationParams['welcomeMsg']; |
140 | - if(trim($welcomeMessage)) |
|
141 | - $params .= '&welcome='.urlencode($welcomeMessage); |
|
138 | + if(trim($welcomeMessage)) { |
|
139 | + $params .= '&welcome='.urlencode($welcomeMessage); |
|
140 | + } |
|
142 | 141 | // Return the complete URL: |
143 | 142 | return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) ); |
144 | 143 | } |
@@ -165,8 +164,8 @@ discard block |
||
165 | 164 | $xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams)); |
166 | 165 | |
167 | 166 | if ($xml) { |
168 | - if($xml->meetingID) |
|
169 | - return array( |
|
167 | + if($xml->meetingID) { |
|
168 | + return array( |
|
170 | 169 | 'returncode' => $xml->returncode, |
171 | 170 | 'message' => $xml->message, |
172 | 171 | 'messageKey' => $xml->messageKey, |
@@ -176,14 +175,14 @@ discard block |
||
176 | 175 | 'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded, |
177 | 176 | 'createTime' => $xml->createTime |
178 | 177 | ); |
179 | - else |
|
180 | - return array( |
|
178 | + } else { |
|
179 | + return array( |
|
181 | 180 | 'returncode' => $xml->returncode, |
182 | 181 | 'message' => $xml->message, |
183 | 182 | 'messageKey' => $xml->messageKey |
184 | 183 | ); |
185 | - } |
|
186 | - else { |
|
184 | + } |
|
185 | + } else { |
|
187 | 186 | return null; |
188 | 187 | } |
189 | 188 | } |
@@ -253,8 +252,7 @@ discard block |
||
253 | 252 | 'message' => $xml->message, |
254 | 253 | 'messageKey' => $xml->messageKey |
255 | 254 | ); |
256 | - } |
|
257 | - else { |
|
255 | + } else { |
|
258 | 256 | return null; |
259 | 257 | } |
260 | 258 | |
@@ -288,8 +286,7 @@ discard block |
||
288 | 286 | 'returncode' => $xml->returncode, |
289 | 287 | 'running' => $xml->running // -- Returns true/false. |
290 | 288 | ); |
291 | - } |
|
292 | - else { |
|
289 | + } else { |
|
293 | 290 | return null; |
294 | 291 | } |
295 | 292 | |
@@ -317,8 +314,7 @@ discard block |
||
317 | 314 | 'returncode' => $xml->returncode |
318 | 315 | ); |
319 | 316 | return $result; |
320 | - } |
|
321 | - elseif ($xml->messageKey == 'noMeetings') { |
|
317 | + } elseif ($xml->messageKey == 'noMeetings') { |
|
322 | 318 | /* No meetings on server, so return just this info: */ |
323 | 319 | $result = array( |
324 | 320 | 'returncode' => $xml->returncode, |
@@ -326,8 +322,7 @@ discard block |
||
326 | 322 | 'message' => $xml->message |
327 | 323 | ); |
328 | 324 | return $result; |
329 | - } |
|
330 | - else { |
|
325 | + } else { |
|
331 | 326 | // In this case, we have success and meetings. First return general response: |
332 | 327 | $result = array( |
333 | 328 | 'returncode' => $xml->returncode, |
@@ -348,8 +343,7 @@ discard block |
||
348 | 343 | } |
349 | 344 | return $result; |
350 | 345 | } |
351 | - } |
|
352 | - else { |
|
346 | + } else { |
|
353 | 347 | return null; |
354 | 348 | } |
355 | 349 | |
@@ -388,8 +382,7 @@ discard block |
||
388 | 382 | 'message' => $xml->message |
389 | 383 | ); |
390 | 384 | return $result; |
391 | - } |
|
392 | - else { |
|
385 | + } else { |
|
393 | 386 | // In this case, we have success and meeting info: |
394 | 387 | $result = array( |
395 | 388 | 'returncode' => $xml->returncode, |
@@ -418,8 +411,7 @@ discard block |
||
418 | 411 | } |
419 | 412 | return $result; |
420 | 413 | } |
421 | - } |
|
422 | - else { |
|
414 | + } else { |
|
423 | 415 | return null; |
424 | 416 | } |
425 | 417 | |
@@ -463,8 +455,7 @@ discard block |
||
463 | 455 | 'message' => $xml->message |
464 | 456 | ); |
465 | 457 | return $result; |
466 | - } |
|
467 | - else { |
|
458 | + } else { |
|
468 | 459 | // In this case, we have success and recording info: |
469 | 460 | $result = array( |
470 | 461 | 'returncode' => $xml->returncode, |
@@ -495,8 +486,7 @@ discard block |
||
495 | 486 | } |
496 | 487 | return $result; |
497 | 488 | } |
498 | - } |
|
499 | - else { |
|
489 | + } else { |
|
500 | 490 | return null; |
501 | 491 | } |
502 | 492 | } |
@@ -529,8 +519,7 @@ discard block |
||
529 | 519 | 'returncode' => $xml->returncode, |
530 | 520 | 'published' => $xml->published // -- Returns true/false. |
531 | 521 | ); |
532 | - } |
|
533 | - else { |
|
522 | + } else { |
|
534 | 523 | return null; |
535 | 524 | } |
536 | 525 | |
@@ -562,8 +551,7 @@ discard block |
||
562 | 551 | 'returncode' => $xml->returncode, |
563 | 552 | 'deleted' => $xml->deleted // -- Returns true/false. |
564 | 553 | ); |
565 | - } |
|
566 | - else { |
|
554 | + } else { |
|
567 | 555 | return null; |
568 | 556 | } |
569 | 557 |
@@ -118,8 +118,11 @@ discard block |
||
118 | 118 | $time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id)); |
119 | 119 | $last_connection = Tracking :: get_last_connection_date($teacher_id); |
120 | 120 | |
121 | - if ($i%2 == 0) $class_tr = 'row_odd'; |
|
122 | - else $class_tr = 'row_even'; |
|
121 | + if ($i%2 == 0) { |
|
122 | + $class_tr = 'row_odd'; |
|
123 | + } else { |
|
124 | + $class_tr = 'row_even'; |
|
125 | + } |
|
123 | 126 | |
124 | 127 | $teachers_table .= ' |
125 | 128 | <tr class="'.$class_tr.'"> |
@@ -174,8 +177,11 @@ discard block |
||
174 | 177 | $username = $teacher['username']; |
175 | 178 | $time_on_platform = api_time_to_hms(Tracking :: get_time_spent_on_the_platform($teacher_id,true)); |
176 | 179 | |
177 | - if ($i%2 == 0) $class_tr = 'row_odd'; |
|
178 | - else $class_tr = 'row_even'; |
|
180 | + if ($i%2 == 0) { |
|
181 | + $class_tr = 'row_odd'; |
|
182 | + } else { |
|
183 | + $class_tr = 'row_even'; |
|
184 | + } |
|
179 | 185 | $teachers_table .= '<tr class="'.$class_tr.'"> |
180 | 186 | <td>'.api_get_person_name($firstname,$lastname).' ('.$username.')</td> |
181 | 187 | <td align="right">'.$time_on_platform.'</td> |
@@ -116,8 +116,11 @@ |
||
116 | 116 | |
117 | 117 | $count_courses_in_session = count(Tracking::get_courses_list_from_session($session_id)); |
118 | 118 | |
119 | - if ($i%2 == 0) $class_tr = 'row_odd'; |
|
120 | - else $class_tr = 'row_even'; |
|
119 | + if ($i%2 == 0) { |
|
120 | + $class_tr = 'row_odd'; |
|
121 | + } else { |
|
122 | + $class_tr = 'row_even'; |
|
123 | + } |
|
121 | 124 | |
122 | 125 | $sessions_table .= '<tr class="'.$class_tr.'"> |
123 | 126 | <td>'.$title.'</td> |
@@ -5,8 +5,12 @@ |
||
5 | 5 | * @author Pavel Vlasov |
6 | 6 | */ |
7 | 7 | |
8 | -if (!defined('DOKU_INC')) die(); |
|
9 | -if (!defined('DOKU_PLUGIN')) define('DOKU_PLUGIN', DOKU_INC . 'lib/plugins/'); |
|
8 | +if (!defined('DOKU_INC')) { |
|
9 | + die(); |
|
10 | +} |
|
11 | +if (!defined('DOKU_PLUGIN')) { |
|
12 | + define('DOKU_PLUGIN', DOKU_INC . 'lib/plugins/'); |
|
13 | +} |
|
10 | 14 | require_once (DOKU_PLUGIN . 'action.php'); |
11 | 15 | |
12 | 16 | class action_plugin_jcapture extends DokuWiki_Action_Plugin { |
@@ -5,7 +5,9 @@ |
||
5 | 5 | parse_str($HTTP_RAW_POST_DATA, $_POST); |
6 | 6 | } |
7 | 7 | |
8 | - if(!defined('DOKU_INC')) define('DOKU_INC',dirname(__FILE__).'/../../../'); |
|
8 | + if(!defined('DOKU_INC')) { |
|
9 | + define('DOKU_INC',dirname(__FILE__).'/../../../'); |
|
10 | + } |
|
9 | 11 | require_once(DOKU_INC.'inc/init.php'); |
10 | 12 | require_once(DOKU_INC.'inc/common.php'); |
11 | 13 | require_once(DOKU_INC.'inc/pageutils.php'); |
@@ -209,13 +209,15 @@ discard block |
||
209 | 209 | $action = $_REQUEST['action']; |
210 | 210 | switch ($action) { |
211 | 211 | case 'assign': |
212 | - if (api_is_platform_admin() && isset($_GET['ticket_id'])) |
|
213 | - TicketManager::assign_ticket_user($_GET['ticket_id'], $_POST['admins']); |
|
212 | + if (api_is_platform_admin() && isset($_GET['ticket_id'])) { |
|
213 | + TicketManager::assign_ticket_user($_GET['ticket_id'], $_POST['admins']); |
|
214 | + } |
|
214 | 215 | $ticket['ticket']['assigned_last_user'] = $_POST['admins']; |
215 | 216 | break; |
216 | 217 | case 'unassign': |
217 | - if (api_is_platform_admin() && isset($_GET['ticket_id'])) |
|
218 | - TicketManager::assign_ticket_user($_GET['ticket_id'], 0); |
|
218 | + if (api_is_platform_admin() && isset($_GET['ticket_id'])) { |
|
219 | + TicketManager::assign_ticket_user($_GET['ticket_id'], 0); |
|
220 | + } |
|
219 | 221 | $ticket['ticket']['assigned_last_user'] = 0; |
220 | 222 | break; |
221 | 223 | default: |
@@ -335,7 +337,7 @@ discard block |
||
335 | 337 | if ($isAdmin) { |
336 | 338 | $message['message'].='<br/><b>' . $plugin->get_lang('AttendedBy') . ': ' . $message['user_created'] . " - " . api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone()) . "</b>"; |
337 | 339 | } |
338 | - }else { |
|
340 | + } else { |
|
339 | 341 | $message['message'].='<b>' . get_lang('Sent') . ': ' . api_convert_and_format_date(api_get_local_time($message['sys_insert_datetime']), DATE_TIME_FORMAT_LONG, _api_get_timezone()) . "</b>"; |
340 | 342 | } |
341 | 343 |
@@ -143,16 +143,19 @@ |
||
143 | 143 | $action = $_GET['action']; |
144 | 144 | switch ($action) { |
145 | 145 | case 'assign': |
146 | - if ($isAdmin && isset($_GET['ticket_id'])) |
|
147 | - TicketManager::assign_ticket_user($_GET['ticket_id'], $user_id); |
|
146 | + if ($isAdmin && isset($_GET['ticket_id'])) { |
|
147 | + TicketManager::assign_ticket_user($_GET['ticket_id'], $user_id); |
|
148 | + } |
|
148 | 149 | break; |
149 | 150 | case 'unassign': |
150 | - if ($isAdmin && isset($_GET['ticket_id'])) |
|
151 | - TicketManager::assign_ticket_user($_GET['ticket_id'], 0); |
|
151 | + if ($isAdmin && isset($_GET['ticket_id'])) { |
|
152 | + TicketManager::assign_ticket_user($_GET['ticket_id'], 0); |
|
153 | + } |
|
152 | 154 | break; |
153 | 155 | case 'alert': |
154 | - if (!$isAdmin && isset($_GET['ticket_id'])) |
|
155 | - TicketManager::send_alert($_GET['ticket_id'], $user_id); |
|
156 | + if (!$isAdmin && isset($_GET['ticket_id'])) { |
|
157 | + TicketManager::send_alert($_GET['ticket_id'], $user_id); |
|
158 | + } |
|
156 | 159 | break; |
157 | 160 | case 'export': |
158 | 161 | $data = array( |
@@ -289,8 +289,9 @@ |
||
289 | 289 | |
290 | 290 | if ($start_date != '' || $end_date != '') { |
291 | 291 | $sql .= " HAVING "; |
292 | - if ($start_date != '') |
|
293 | - $sql .= " access_date >= '$start_date' "; |
|
292 | + if ($start_date != '') { |
|
293 | + $sql .= " access_date >= '$start_date' "; |
|
294 | + } |
|
294 | 295 | if ($end_date != '') { |
295 | 296 | $sql = ($start_date == '') ? $sql : ($sql . " AND "); |
296 | 297 | $sql .= " access_date <= '$end_date' "; |
@@ -123,8 +123,7 @@ discard block |
||
123 | 123 | if ($has_content) |
124 | 124 | { |
125 | 125 | echo '<div class="list">'; |
126 | - } |
|
127 | - else |
|
126 | + } else |
|
128 | 127 | { |
129 | 128 | echo '<div>'; |
130 | 129 | } |
@@ -132,8 +131,7 @@ discard block |
||
132 | 131 | if (RegisterCourseWidget::factory()->run()) |
133 | 132 | { |
134 | 133 | $result = true; |
135 | - } |
|
136 | - else |
|
134 | + } else |
|
137 | 135 | { |
138 | 136 | $result = $this->action_display(); |
139 | 137 | } |
@@ -337,8 +335,7 @@ discard block |
||
337 | 335 | if (api_is_anonymous()) |
338 | 336 | { |
339 | 337 | $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD; |
340 | - } |
|
341 | - else |
|
338 | + } else |
|
342 | 339 | { |
343 | 340 | $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD . ' OR '; |
344 | 341 | $course_fiter .= 'visibility = ' . COURSE_VISIBILITY_OPEN_PLATFORM . ' OR '; |