@@ -167,9 +167,9 @@ |
||
167 | 167 | // possible : number_groups_left > 0 and is group member |
168 | 168 | $possible_users = array(); |
169 | 169 | foreach ($complete_user_list as $index => $user) { |
170 | - if ($user['number_groups_left'] > 0 || in_array($user['user_id'], $selected_users)) { |
|
170 | + if ($user['number_groups_left'] > 0 || in_array($user['user_id'], $selected_users)) { |
|
171 | 171 | $possible_users[$user['user_id']] = api_get_person_name($user['firstname'], $user['lastname']).' ('.$user['username'].')'; |
172 | - } |
|
172 | + } |
|
173 | 173 | } |
174 | 174 | |
175 | 175 | $group_members_element = $form->addElement( |
@@ -173,7 +173,7 @@ |
||
173 | 173 | ); |
174 | 174 | } |
175 | 175 | |
176 | - return $result; |
|
176 | + return $result; |
|
177 | 177 | } |
178 | 178 | |
179 | 179 | $table->set_header(0, '', false); |
@@ -628,7 +628,7 @@ |
||
628 | 628 | |
629 | 629 | foreach ($user_data as $key => $value) { |
630 | 630 | if (substr($key, 0, 6) === 'extra_') { //an extra field |
631 | - continue; |
|
631 | + continue; |
|
632 | 632 | } elseif (strpos($key, 'remove_extra_') !== false) { |
633 | 633 | } else { |
634 | 634 | if (in_array($key, $available_values_to_modify)) { |
@@ -87,7 +87,7 @@ discard block |
||
87 | 87 | $(this).parents(".message-content").animate({ opacity: "hide" }, "slow"); |
88 | 88 | $(".message-view").animate({ opacity: "show" }, "slow"); |
89 | 89 | }); |
90 | -EOF; |
|
90 | +eof; |
|
91 | 91 | } |
92 | 92 | |
93 | 93 | $tool_name = is_profile_editable() ? get_lang('ModifProfile') : get_lang('ViewProfile'); |
@@ -623,7 +623,7 @@ discard block |
||
623 | 623 | ); |
624 | 624 | |
625 | 625 | // build SQL query |
626 | - $sql = "UPDATE $table_user SET"; |
|
626 | + $sql = "update $table_user SET"; |
|
627 | 627 | unset($user_data['api_key_generate']); |
628 | 628 | |
629 | 629 | foreach ($user_data as $key => $value) { |
@@ -632,7 +632,8 @@ |
||
632 | 632 | unset($user_data['api_key_generate']); |
633 | 633 | |
634 | 634 | foreach ($user_data as $key => $value) { |
635 | - if (substr($key, 0, 6) === 'extra_') { //an extra field |
|
635 | + if (substr($key, 0, 6) === 'extra_') { |
|
636 | +//an extra field |
|
636 | 637 | continue; |
637 | 638 | } elseif (strpos($key, 'remove_extra_') !== false) { |
638 | 639 | } else { |
@@ -31,14 +31,14 @@ |
||
31 | 31 | * Redirect to the correct script to handle this type of upload |
32 | 32 | */ |
33 | 33 | switch ($toolFromSession) { |
34 | - case TOOL_LEARNPATH: |
|
35 | - require 'upload.scorm.php'; |
|
36 | - break; |
|
37 | - //the following cases need to be distinguished later on |
|
38 | - case TOOL_DROPBOX: |
|
39 | - case TOOL_STUDENTPUBLICATION: |
|
40 | - case TOOL_DOCUMENT: |
|
41 | - default: |
|
42 | - require 'upload.document.php'; |
|
43 | - break; |
|
34 | + case TOOL_LEARNPATH: |
|
35 | + require 'upload.scorm.php'; |
|
36 | + break; |
|
37 | + //the following cases need to be distinguished later on |
|
38 | + case TOOL_DROPBOX: |
|
39 | + case TOOL_STUDENTPUBLICATION: |
|
40 | + case TOOL_DOCUMENT: |
|
41 | + default: |
|
42 | + require 'upload.document.php'; |
|
43 | + break; |
|
44 | 44 | } |
@@ -133,7 +133,7 @@ |
||
133 | 133 | if (!empty($glossary_data['update_date'])) { |
134 | 134 | $glossary_data['update_date'] = Display::dateToStringAgoAndLongDate($glossary_data['update_date']); |
135 | 135 | } else { |
136 | - $glossary_data['update_date'] = ''; |
|
136 | + $glossary_data['update_date'] = ''; |
|
137 | 137 | } |
138 | 138 | |
139 | 139 | $form->addLabel(get_lang('CreationDate'), $glossary_data['insert_date']); |
@@ -33,7 +33,8 @@ |
||
33 | 33 | // Tracking |
34 | 34 | Event::event_access_tool(TOOL_GLOSSARY); |
35 | 35 | |
36 | -function sorter($item1, $item2) { |
|
36 | +function sorter($item1, $item2) |
|
37 | +{ |
|
37 | 38 | if ($item1[2] == $item2[2]) { |
38 | 39 | return 0; |
39 | 40 | } |
@@ -26,10 +26,10 @@ |
||
26 | 26 | class UserRepository extends EntityRepository |
27 | 27 | { |
28 | 28 | /** |
29 | - * @param string $keyword |
|
29 | + * @param string $keyword |
|
30 | 30 | * |
31 | - * @return mixed |
|
32 | - */ |
|
31 | + * @return mixed |
|
32 | + */ |
|
33 | 33 | public function searchUserByKeyword($keyword) |
34 | 34 | { |
35 | 35 | $qb = $this->createQueryBuilder('a'); |
@@ -173,7 +173,7 @@ |
||
173 | 173 | ); |
174 | 174 | } |
175 | 175 | |
176 | - return $result; |
|
176 | + return $result; |
|
177 | 177 | } |
178 | 178 | |
179 | 179 | $table->set_header(0, '', false); |
@@ -26,10 +26,10 @@ |
||
26 | 26 | class UserRepository extends EntityRepository |
27 | 27 | { |
28 | 28 | /** |
29 | - * @param string $keyword |
|
29 | + * @param string $keyword |
|
30 | 30 | * |
31 | - * @return mixed |
|
32 | - */ |
|
31 | + * @return mixed |
|
32 | + */ |
|
33 | 33 | public function searchUserByKeyword($keyword) |
34 | 34 | { |
35 | 35 | $qb = $this->createQueryBuilder('a'); |
@@ -277,7 +277,7 @@ discard block |
||
277 | 277 | * Formats a number depending of the number of decimals |
278 | 278 | * |
279 | 279 | * @param float $score |
280 | - * @return float the score formatted |
|
280 | + * @return boolean|string the score formatted |
|
281 | 281 | */ |
282 | 282 | public function format_score($score) |
283 | 283 | { |
@@ -391,7 +391,7 @@ discard block |
||
391 | 391 | |
392 | 392 | /** |
393 | 393 | * @param array $score |
394 | - * @return float|string |
|
394 | + * @return boolean|string |
|
395 | 395 | */ |
396 | 396 | private function display_simple_score($score) |
397 | 397 | { |
@@ -404,7 +404,7 @@ discard block |
||
404 | 404 | /** |
405 | 405 | * Returns "1" for array("100", "100"); |
406 | 406 | * @param array $score |
407 | - * @return float |
|
407 | + * @return boolean|string |
|
408 | 408 | */ |
409 | 409 | private function display_as_decimal($score) |
410 | 410 | { |