@@ -720,11 +720,11 @@ discard block |
||
720 | 720 | return false; |
721 | 721 | } |
722 | 722 | |
723 | - /** |
|
724 | - * @param int $itemId |
|
725 | - * @param int $fieldId |
|
726 | - * @return array |
|
727 | - */ |
|
723 | + /** |
|
724 | + * @param int $itemId |
|
725 | + * @param int $fieldId |
|
726 | + * @return array |
|
727 | + */ |
|
728 | 728 | public function getAllValuesByItemAndField($itemId, $fieldId) |
729 | 729 | { |
730 | 730 | $fieldId = intval($fieldId); |
@@ -847,7 +847,6 @@ discard block |
||
847 | 847 | /** |
848 | 848 | * Deletes all values from an item |
849 | 849 | * @param int $itemId (session id, course id, etc) |
850 | - |
|
851 | 850 | * @assert (-1,-1) == null |
852 | 851 | */ |
853 | 852 | public function deleteValuesByItem($itemId) |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | break; |
217 | 217 | } |
218 | 218 | |
219 | - $fileName = ExtraField::FIELD_TYPE_FILE_IMAGE . "_{$params['item_id']}.png"; |
|
219 | + $fileName = ExtraField::FIELD_TYPE_FILE_IMAGE."_{$params['item_id']}.png"; |
|
220 | 220 | |
221 | 221 | if (!file_exists($fileDir)) { |
222 | 222 | mkdir($fileDir, $dirPermissions, true); |
@@ -226,15 +226,15 @@ discard block |
||
226 | 226 | |
227 | 227 | //Crop the image to adjust 16:9 ratio |
228 | 228 | $crop = new Image($value['tmp_name']); |
229 | - $crop->crop($params['extra_' . $field_variable . '_crop_result']); |
|
229 | + $crop->crop($params['extra_'.$field_variable.'_crop_result']); |
|
230 | 230 | |
231 | 231 | $imageExtraField = new Image($value['tmp_name']); |
232 | 232 | $imageExtraField->resize(400); |
233 | - $imageExtraField->send_image($fileDir . $fileName, -1, 'png'); |
|
233 | + $imageExtraField->send_image($fileDir.$fileName, -1, 'png'); |
|
234 | 234 | $newParams = array( |
235 | 235 | 'item_id' => $params['item_id'], |
236 | 236 | 'field_id' => $extraFieldInfo['id'], |
237 | - 'value' => $fileDirStored . $fileName, |
|
237 | + 'value' => $fileDirStored.$fileName, |
|
238 | 238 | 'comment' => $comment |
239 | 239 | ); |
240 | 240 | |
@@ -260,18 +260,18 @@ discard block |
||
260 | 260 | } |
261 | 261 | |
262 | 262 | $cleanedName = api_replace_dangerous_char($value['name']); |
263 | - $fileName = ExtraField::FIELD_TYPE_FILE . "_{$params['item_id']}_$cleanedName"; |
|
263 | + $fileName = ExtraField::FIELD_TYPE_FILE."_{$params['item_id']}_$cleanedName"; |
|
264 | 264 | if (!file_exists($fileDir)) { |
265 | 265 | mkdir($fileDir, $dirPermissions, true); |
266 | 266 | } |
267 | 267 | |
268 | 268 | if ($value['error'] == 0) { |
269 | - moveUploadedFile($value, $fileDir . $fileName); |
|
269 | + moveUploadedFile($value, $fileDir.$fileName); |
|
270 | 270 | |
271 | 271 | $new_params = array( |
272 | 272 | 'item_id' => $params['item_id'], |
273 | 273 | 'field_id' => $extraFieldInfo['id'], |
274 | - 'value' => $fileDirStored . $fileName |
|
274 | + 'value' => $fileDirStored.$fileName |
|
275 | 275 | ); |
276 | 276 | |
277 | 277 | if ($this->type !== 'session' && $this->type !== 'course') { |
@@ -411,7 +411,7 @@ discard block |
||
411 | 411 | */ |
412 | 412 | if (false) { |
413 | 413 | global $app; |
414 | - switch($this->type) { |
|
414 | + switch ($this->type) { |
|
415 | 415 | case 'question': |
416 | 416 | $extraFieldValue = new ChamiloLMS\Entity\QuestionFieldValues(); |
417 | 417 | $extraFieldValue->setUserId(api_get_user_id()); |
@@ -473,7 +473,7 @@ discard block |
||
473 | 473 | */ |
474 | 474 | if (false) { |
475 | 475 | global $app; |
476 | - switch($this->type) { |
|
476 | + switch ($this->type) { |
|
477 | 477 | case 'question': |
478 | 478 | $extraFieldValue = $app['orm.ems']['db_write']->getRepository('ChamiloLMS\Entity\QuestionFieldValues')->find($field_values['id']); |
479 | 479 | $extraFieldValue->setUserId(api_get_user_id()); |
@@ -75,7 +75,7 @@ discard block |
||
75 | 75 | * This function is used with $extraField->addElements() |
76 | 76 | * @param array $params array for the insertion into the *_field_values table |
77 | 77 | * @param bool $showQuery |
78 | - * @return mixed false on empty params, void otherwise |
|
78 | + * @return false|null false on empty params, void otherwise |
|
79 | 79 | * @assert (array()) === false |
80 | 80 | */ |
81 | 81 | public function saveFieldValues($params, $showQuery = false) |
@@ -531,7 +531,7 @@ discard block |
||
531 | 531 | * @param int $item_id Item ID (It could be a session_id, course_id or user_id) |
532 | 532 | * @param int $field_id Field ID (the ID from the *_field table) |
533 | 533 | * @param bool $transform Whether to transform the result to a human readable strings |
534 | - * @return mixed A structured array with the field_id and field_value, or false on error |
|
534 | + * @return string A structured array with the field_id and field_value, or false on error |
|
535 | 535 | * @assert (-1,-1) === false |
536 | 536 | */ |
537 | 537 | public function get_values_by_handler_and_field_id($item_id, $field_id, $transform = false) |
@@ -624,7 +624,7 @@ discard block |
||
624 | 624 | * @param int $item_id Item ID from the original table |
625 | 625 | * @param string $field_variable The name of the field we are looking for |
626 | 626 | * @param bool $transform |
627 | - * @param bool $allVisibility |
|
627 | + * @param bool $visibility |
|
628 | 628 | * |
629 | 629 | * @return mixed Array of results, or false on error or not found |
630 | 630 | * @assert (-1,'') === false |
@@ -686,7 +686,7 @@ discard block |
||
686 | 686 | * @param string $field_value Data we are looking for in the given field |
687 | 687 | * @param bool $transform Whether to transform the result to a human readable strings |
688 | 688 | * @param bool $last Whether to return the last element or simply the first one we get |
689 | - * @return mixed Give the ID if found, or false on failure or not found |
|
689 | + * @return string|null Give the ID if found, or false on failure or not found |
|
690 | 690 | * @assert (-1,-1) === false |
691 | 691 | */ |
692 | 692 | public function get_item_id_from_field_variable_and_field_value( |
@@ -99,7 +99,7 @@ discard block |
||
99 | 99 | } |
100 | 100 | foreach ($data as $row) { |
101 | 101 | $string = implode("</td><td>", $row); |
102 | - $string = '<tr><td>' . $string . '</td></tr>'; |
|
102 | + $string = '<tr><td>'.$string.'</td></tr>'; |
|
103 | 103 | if ($encoding != 'utf-8') { |
104 | 104 | $string = api_convert_encoding($string, $encoding, $systemEncoding); |
105 | 105 | } |
@@ -173,7 +173,7 @@ discard block |
||
173 | 173 | fwrite($handle, '<'.$wrapper_tagname.'>'); |
174 | 174 | } |
175 | 175 | $s = self::_export_complex_table_xml_helper($data); |
176 | - fwrite($handle,$s); |
|
176 | + fwrite($handle, $s); |
|
177 | 177 | if (!is_null($wrapper_tagname)) { |
178 | 178 | fwrite($handle, '</'.$wrapper_tagname.'>'."\n"); |
179 | 179 | } |
@@ -195,10 +195,10 @@ discard block |
||
195 | 195 | } |
196 | 196 | $string = ''; |
197 | 197 | foreach ($data as $row) { |
198 | - $string .= "\n".str_repeat("\t",$level).'<'.$row['name'].'>'; |
|
198 | + $string .= "\n".str_repeat("\t", $level).'<'.$row['name'].'>'; |
|
199 | 199 | if (is_array($row['value'])) { |
200 | - $string .= self::_export_complex_table_xml_helper($row['value'],$level+1)."\n"; |
|
201 | - $string .= str_repeat("\t",$level).'</'.$row['name'].'>'; |
|
200 | + $string .= self::_export_complex_table_xml_helper($row['value'], $level + 1)."\n"; |
|
201 | + $string .= str_repeat("\t", $level).'</'.$row['name'].'>'; |
|
202 | 202 | } else { |
203 | 203 | $string .= $row['value']; |
204 | 204 | $string .= '</'.$row['name'].'>'; |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | * @param array $data |
39 | 39 | * @param string $filename |
40 | 40 | * |
41 | - * @return mixed csv file | false if no data to export |
|
41 | + * @return false|null csv file | false if no data to export |
|
42 | 42 | */ |
43 | 43 | public static function arrayToCsv($data, $filename = 'export') |
44 | 44 | { |
@@ -160,7 +160,7 @@ discard block |
||
160 | 160 | * @param string Name of common tag to place each line in |
161 | 161 | * @param string Name of the root element. A root element should always be given. |
162 | 162 | * @param string Encoding in which the data is provided |
163 | - * @return void Prompts the user for a file download |
|
163 | + * @return boolean Prompts the user for a file download |
|
164 | 164 | */ |
165 | 165 | public static function export_complex_table_xml( |
166 | 166 | $data, |
@@ -286,7 +286,7 @@ discard block |
||
286 | 286 | * @param string $name |
287 | 287 | * @param string $format |
288 | 288 | * |
289 | - * @return bool |
|
289 | + * @return false|null |
|
290 | 290 | */ |
291 | 291 | public static function htmlToOdt($html, $name, $format = 'odt') |
292 | 292 | { |
@@ -115,13 +115,13 @@ discard block |
||
115 | 115 | } |
116 | 116 | |
117 | 117 | /** |
118 | - * Export tabular data to XML-file |
|
119 | - * @param array Simple array of data to put in XML |
|
120 | - * @param string Name of file to be given to the user |
|
121 | - * @param string Name of common tag to place each line in |
|
122 | - * @param string Name of the root element. A root element should always be given. |
|
123 | - * @param string Encoding in which the data is provided |
|
124 | - */ |
|
118 | + * Export tabular data to XML-file |
|
119 | + * @param array Simple array of data to put in XML |
|
120 | + * @param string Name of file to be given to the user |
|
121 | + * @param string Name of common tag to place each line in |
|
122 | + * @param string Name of the root element. A root element should always be given. |
|
123 | + * @param string Encoding in which the data is provided |
|
124 | + */ |
|
125 | 125 | public static function arrayToXml( |
126 | 126 | $data, |
127 | 127 | $filename = 'export', |
@@ -200,7 +200,7 @@ discard block |
||
200 | 200 | foreach ($data as $row) { |
201 | 201 | $string .= "\n".str_repeat("\t",$level).'<'.$row['name'].'>'; |
202 | 202 | if (is_array($row['value'])) { |
203 | - $string .= self::_export_complex_table_xml_helper($row['value'],$level+1)."\n"; |
|
203 | + $string .= self::_export_complex_table_xml_helper($row['value'],$level+1)."\n"; |
|
204 | 204 | $string .= str_repeat("\t",$level).'</'.$row['name'].'>'; |
205 | 205 | } else { |
206 | 206 | $string .= $row['value']; |
@@ -108,24 +108,24 @@ |
||
108 | 108 | */ |
109 | 109 | function utf8_sort($array) |
110 | 110 | { |
111 | - $old_locale = setlocale(LC_ALL, null); |
|
112 | - $code = api_get_language_isocode(); |
|
113 | - $locale_list = array($code.'.utf8', 'en.utf8','en_US.utf8','en_GB.utf8'); |
|
114 | - $try_sort = false; |
|
111 | + $old_locale = setlocale(LC_ALL, null); |
|
112 | + $code = api_get_language_isocode(); |
|
113 | + $locale_list = array($code.'.utf8', 'en.utf8','en_US.utf8','en_GB.utf8'); |
|
114 | + $try_sort = false; |
|
115 | 115 | |
116 | - foreach($locale_list as $locale) { |
|
117 | - $my_local = setlocale(LC_COLLATE, $locale); |
|
118 | - if ($my_local) { |
|
119 | - $try_sort = true; |
|
120 | - break; |
|
121 | - } |
|
122 | - } |
|
116 | + foreach($locale_list as $locale) { |
|
117 | + $my_local = setlocale(LC_COLLATE, $locale); |
|
118 | + if ($my_local) { |
|
119 | + $try_sort = true; |
|
120 | + break; |
|
121 | + } |
|
122 | + } |
|
123 | 123 | |
124 | - if ($try_sort) { |
|
125 | - uasort($array, 'strcoll'); |
|
126 | - } |
|
127 | - setlocale(LC_COLLATE, $old_locale); |
|
128 | - return $array; |
|
124 | + if ($try_sort) { |
|
125 | + uasort($array, 'strcoll'); |
|
126 | + } |
|
127 | + setlocale(LC_COLLATE, $old_locale); |
|
128 | + return $array; |
|
129 | 129 | } |
130 | 130 | |
131 | 131 | /** |
@@ -110,10 +110,10 @@ discard block |
||
110 | 110 | { |
111 | 111 | $old_locale = setlocale(LC_ALL, null); |
112 | 112 | $code = api_get_language_isocode(); |
113 | - $locale_list = array($code.'.utf8', 'en.utf8','en_US.utf8','en_GB.utf8'); |
|
113 | + $locale_list = array($code.'.utf8', 'en.utf8', 'en_US.utf8', 'en_GB.utf8'); |
|
114 | 114 | $try_sort = false; |
115 | 115 | |
116 | - foreach($locale_list as $locale) { |
|
116 | + foreach ($locale_list as $locale) { |
|
117 | 117 | $my_local = setlocale(LC_COLLATE, $locale); |
118 | 118 | if ($my_local) { |
119 | 119 | $try_sort = true; |
@@ -151,7 +151,7 @@ discard block |
||
151 | 151 | $flatten = array(); |
152 | 152 | array_walk_recursive( |
153 | 153 | $array, |
154 | - function ($value) use (&$flatten) { |
|
154 | + function($value) use (&$flatten) { |
|
155 | 155 | $flatten[] = $value; |
156 | 156 | } |
157 | 157 | ); |
@@ -219,10 +219,10 @@ |
||
219 | 219 | // action links |
220 | 220 | echo '<div class="actions">'; |
221 | 221 | if (!api_is_anonymous()) { |
222 | - if (api_get_session_id() == 0) |
|
223 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
|
222 | + if (api_get_session_id() == 0) { |
|
223 | + echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
|
224 | 224 | Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
225 | - elseif (api_is_allowed_to_session_edit(false, true)) { |
|
225 | + } elseif (api_is_allowed_to_session_edit(false, true)) { |
|
226 | 226 | echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
227 | 227 | Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
228 | 228 | } |
@@ -29,7 +29,7 @@ discard block |
||
29 | 29 | return "<script> |
30 | 30 | function confirmation (name) |
31 | 31 | { |
32 | - if (confirm(\" " . get_lang("NoteConfirmDelete") . " \"+ name + \" ?\")) |
|
32 | + if (confirm(\" " . get_lang("NoteConfirmDelete")." \"+ name + \" ?\")) |
|
33 | 33 | {return true;} |
34 | 34 | else |
35 | 35 | {return false;} |
@@ -106,7 +106,7 @@ discard block |
||
106 | 106 | description AS note_comment, |
107 | 107 | session_id AS session_id |
108 | 108 | FROM $t_notebook |
109 | - WHERE c_id = $course_id AND notebook_id = '" . intval($notebook_id) . "' "; |
|
109 | + WHERE c_id = $course_id AND notebook_id = '".intval($notebook_id)."' "; |
|
110 | 110 | $result = Database::query($sql); |
111 | 111 | if (Database::num_rows($result) != 1) { |
112 | 112 | return array(); |
@@ -184,8 +184,8 @@ discard block |
||
184 | 184 | $sql = "DELETE FROM $t_notebook |
185 | 185 | WHERE |
186 | 186 | c_id = $course_id AND |
187 | - notebook_id='" . intval($notebook_id) . "' AND |
|
188 | - user_id = '" . api_get_user_id() . "'"; |
|
187 | + notebook_id='".intval($notebook_id)."' AND |
|
188 | + user_id = '" . api_get_user_id()."'"; |
|
189 | 189 | $result = Database::query($sql); |
190 | 190 | $affected_rows = Database::affected_rows($result); |
191 | 191 | if ($affected_rows != 1) { |
@@ -223,22 +223,22 @@ discard block |
||
223 | 223 | echo '<div class="actions">'; |
224 | 224 | if (!api_is_anonymous()) { |
225 | 225 | if (api_get_session_id() == 0) |
226 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
|
227 | - Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
|
226 | + echo '<a href="index.php?'.api_get_cidreq().'&action=addnote">'. |
|
227 | + Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32').'</a>'; |
|
228 | 228 | elseif (api_is_allowed_to_session_edit(false, true)) { |
229 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
|
230 | - Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
|
229 | + echo '<a href="index.php?'.api_get_cidreq().'&action=addnote">'. |
|
230 | + Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32').'</a>'; |
|
231 | 231 | } |
232 | 232 | } else { |
233 | - echo '<a href="javascript:void(0)">' . Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
|
233 | + echo '<a href="javascript:void(0)">'.Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32').'</a>'; |
|
234 | 234 | } |
235 | 235 | |
236 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=creation_date&direction=' . $link_sort_direction . '">' . |
|
237 | - Display::return_icon('notes_order_by_date_new.png', get_lang('OrderByCreationDate'), '', '32') . '</a>'; |
|
238 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=update_date&direction=' . $link_sort_direction . '">' . |
|
239 | - Display::return_icon('notes_order_by_date_mod.png', get_lang('OrderByModificationDate'), '', '32') . '</a>'; |
|
240 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=title&direction=' . $link_sort_direction . '">' . |
|
241 | - Display::return_icon('notes_order_by_title.png', get_lang('OrderByTitle'), '', '32') . '</a>'; |
|
236 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=creation_date&direction='.$link_sort_direction.'">'. |
|
237 | + Display::return_icon('notes_order_by_date_new.png', get_lang('OrderByCreationDate'), '', '32').'</a>'; |
|
238 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=update_date&direction='.$link_sort_direction.'">'. |
|
239 | + Display::return_icon('notes_order_by_date_mod.png', get_lang('OrderByModificationDate'), '', '32').'</a>'; |
|
240 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=title&direction='.$link_sort_direction.'">'. |
|
241 | + Display::return_icon('notes_order_by_title.png', get_lang('OrderByTitle'), '', '32').'</a>'; |
|
242 | 242 | echo '</div>'; |
243 | 243 | |
244 | 244 | if (!isset($_SESSION['notebook_view']) || !in_array($_SESSION['notebook_view'], array('creation_date', 'update_date', 'title'))) { |
@@ -248,9 +248,9 @@ discard block |
||
248 | 248 | // Database table definition |
249 | 249 | $t_notebook = Database :: get_course_table(TABLE_NOTEBOOK); |
250 | 250 | if ($_SESSION['notebook_view'] == 'creation_date' || $_SESSION['notebook_view'] == 'update_date') { |
251 | - $order_by = " ORDER BY " . $_SESSION['notebook_view'] . " $sort_direction "; |
|
251 | + $order_by = " ORDER BY ".$_SESSION['notebook_view']." $sort_direction "; |
|
252 | 252 | } else { |
253 | - $order_by = " ORDER BY " . $_SESSION['notebook_view'] . " $sort_direction "; |
|
253 | + $order_by = " ORDER BY ".$_SESSION['notebook_view']." $sort_direction "; |
|
254 | 254 | } |
255 | 255 | |
256 | 256 | //condition for the session |
@@ -263,7 +263,7 @@ discard block |
||
263 | 263 | $sql = "SELECT * FROM $t_notebook |
264 | 264 | WHERE |
265 | 265 | c_id = $course_id AND |
266 | - user_id = '" . api_get_user_id() . "' |
|
266 | + user_id = '".api_get_user_id()."' |
|
267 | 267 | $condition_session |
268 | 268 | $cond_extra $order_by |
269 | 269 | "; |
@@ -273,18 +273,18 @@ discard block |
||
273 | 273 | $session_img = api_get_session_image($row['session_id'], $_user['status']); |
274 | 274 | $updateValue = ''; |
275 | 275 | if ($row['update_date'] <> $row['creation_date']) { |
276 | - $updateValue = ', ' . get_lang('UpdateDate') . ': ' . Display::dateToStringAgoAndLongDate($row['update_date']); |
|
276 | + $updateValue = ', '.get_lang('UpdateDate').': '.Display::dateToStringAgoAndLongDate($row['update_date']); |
|
277 | 277 | } |
278 | 278 | |
279 | - $actions = '<a href="' . api_get_self() . '?action=editnote¬ebook_id=' . $row['notebook_id'] . '">' . |
|
280 | - Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL) . '</a>'; |
|
281 | - $actions .= '<a href="' . api_get_self() . '?action=deletenote¬ebook_id=' . $row['notebook_id'] . '" onclick="return confirmation(\'' . $row['title'] . '\');">' . |
|
282 | - Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL) . '</a>'; |
|
279 | + $actions = '<a href="'.api_get_self().'?action=editnote¬ebook_id='.$row['notebook_id'].'">'. |
|
280 | + Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL).'</a>'; |
|
281 | + $actions .= '<a href="'.api_get_self().'?action=deletenote¬ebook_id='.$row['notebook_id'].'" onclick="return confirmation(\''.$row['title'].'\');">'. |
|
282 | + Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>'; |
|
283 | 283 | |
284 | 284 | echo Display::panel( |
285 | 285 | $row['description'], |
286 | - $row['title'] . $session_img.' <div class="pull-right">'.$actions.'</div>', |
|
287 | - get_lang('CreationDate') . ': ' . Display::dateToStringAgoAndLongDate($row['creation_date']). $updateValue |
|
286 | + $row['title'].$session_img.' <div class="pull-right">'.$actions.'</div>', |
|
287 | + get_lang('CreationDate').': '.Display::dateToStringAgoAndLongDate($row['creation_date']).$updateValue |
|
288 | 288 | ); |
289 | 289 | } |
290 | 290 | } |
@@ -666,9 +666,10 @@ |
||
666 | 666 | $result = ''; |
667 | 667 | for ($i = $min; $i <= $max; $i ++) { |
668 | 668 | $result .= '<option value="'.$i.'"'; |
669 | - if (is_int($selected_num)) |
|
670 | - if ($selected_num == $i) { |
|
669 | + if (is_int($selected_num)) { |
|
670 | + if ($selected_num == $i) { |
|
671 | 671 | $result .= ' selected="selected"'; |
672 | + } |
|
672 | 673 | } |
673 | 674 | $result .= '>'.$i.'</option>'; |
674 | 675 | } |
@@ -32,7 +32,7 @@ discard block |
||
32 | 32 | } |
33 | 33 | |
34 | 34 | /** |
35 | - * @return array |
|
35 | + * @return string[] |
|
36 | 36 | */ |
37 | 37 | public static function toolList() |
38 | 38 | { |
@@ -290,7 +290,6 @@ discard block |
||
290 | 290 | * 'page_nr' = The page to display |
291 | 291 | * 'hide_navigation' = true to hide the navigation |
292 | 292 | * @param array $query_vars Additional variables to add in the query-string |
293 | - * @param array $form actions Additional variables to add in the query-string |
|
294 | 293 | * @param mixed An array with bool values to know which columns show. |
295 | 294 | * i.e: $visibility_options= array(true, false) we will only show the first column |
296 | 295 | * Can be also only a bool value. TRUE: show all columns, FALSE: show nothing |
@@ -331,7 +330,6 @@ discard block |
||
331 | 330 | * 'page_nr' = The page to display |
332 | 331 | * 'hide_navigation' = true to hide the navigation |
333 | 332 | * @param array $query_vars Additional variables to add in the query-string |
334 | - * @param array $form actions Additional variables to add in the query-string |
|
335 | 333 | * @param mixed An array with bool values to know which columns show. i.e: |
336 | 334 | * $visibility_options= array(true, false) we will only show the first column |
337 | 335 | * Can be also only a bool value. TRUE: show all columns, FALSE: show nothing |
@@ -388,10 +386,11 @@ discard block |
||
388 | 386 | * 'per_page' = number of items to show per page |
389 | 387 | * 'page_nr' = The page to display |
390 | 388 | * @param array $query_vars Additional variables to add in the query-string |
391 | - * @param array $column_show Array of binaries 1= show columns 0. hide a column |
|
389 | + * @param integer[] $column_show Array of binaries 1= show columns 0. hide a column |
|
392 | 390 | * @param array $column_order An array of integers that let us decide how the columns are going to be sort. |
393 | 391 | * i.e: $column_order=array('1''4','3','4'); The 2nd column will be order like the 4th column |
394 | 392 | * @param array $form_actions Set optional forms actions |
393 | + * @param string $table_name |
|
395 | 394 | * |
396 | 395 | * @author Julio Montoya |
397 | 396 | */ |
@@ -444,7 +443,7 @@ discard block |
||
444 | 443 | * @param bool $filter (true) or not (false) |
445 | 444 | * @param bool $returnValue |
446 | 445 | * |
447 | - * @return void |
|
446 | + * @return string|null |
|
448 | 447 | */ |
449 | 448 | public static function display_normal_message($message, $filter = true, $returnValue = false) |
450 | 449 | { |
@@ -474,7 +473,7 @@ discard block |
||
474 | 473 | /** |
475 | 474 | * Displays an confirmation message. Use this if something has been done successfully |
476 | 475 | * @param bool Filter (true) or not (false) |
477 | - * @return void |
|
476 | + * @return string|null |
|
478 | 477 | */ |
479 | 478 | public static function display_confirmation_message ($message, $filter = true, $returnValue = false) |
480 | 479 | { |
@@ -491,7 +490,7 @@ discard block |
||
491 | 490 | * @param string $message - include any additional html |
492 | 491 | * tags if you need them |
493 | 492 | * @param bool Filter (true) or not (false) |
494 | - * @return void |
|
493 | + * @return string|null |
|
495 | 494 | */ |
496 | 495 | public static function display_error_message ($message, $filter = true, $returnValue = false) |
497 | 496 | { |
@@ -681,6 +680,7 @@ discard block |
||
681 | 680 | * @param string The alt text (probably a language variable) |
682 | 681 | * @param array additional attributes (for instance height, width, onclick, ...) |
683 | 682 | * @param integer The wanted width of the icon (to be looked for in the corresponding img/icons/ folder) |
683 | + * @param string $image |
|
684 | 684 | * @return void |
685 | 685 | */ |
686 | 686 | public static function display_icon( |
@@ -886,6 +886,7 @@ discard block |
||
886 | 886 | /** |
887 | 887 | * Displays an HTML input tag |
888 | 888 | * |
889 | + * @param string $type |
|
889 | 890 | */ |
890 | 891 | public static function input($type, $name, $value, $attributes = array()) |
891 | 892 | { |
@@ -902,8 +903,8 @@ discard block |
||
902 | 903 | } |
903 | 904 | |
904 | 905 | /** |
905 | - * @param $name |
|
906 | - * @param $value |
|
906 | + * @param string $name |
|
907 | + * @param string $value |
|
907 | 908 | * @param array $attributes |
908 | 909 | * @return string |
909 | 910 | */ |
@@ -918,6 +919,7 @@ discard block |
||
918 | 919 | /** |
919 | 920 | * Displays an HTML select tag |
920 | 921 | * |
922 | + * @param string $blank_item_text |
|
921 | 923 | */ |
922 | 924 | public static function select( |
923 | 925 | $name, |
@@ -984,8 +986,8 @@ discard block |
||
984 | 986 | $( "#tabs" ).tabs(); |
985 | 987 | }); |
986 | 988 | </script> |
987 | - * @param array $headers list of the tab titles |
|
988 | - * @param array $items |
|
989 | + * @param string[] $headers list of the tab titles |
|
990 | + * @param string[] $items |
|
989 | 991 | * @param string $id id of the container of the tab in the example "tabs" |
990 | 992 | * @param array $attributes for the ul |
991 | 993 | * @param array $ul_attributes |
@@ -1039,7 +1041,7 @@ discard block |
||
1039 | 1041 | |
1040 | 1042 | /** |
1041 | 1043 | * @param $headers |
1042 | - * @param null $selected |
|
1044 | + * @param integer $selected |
|
1043 | 1045 | * |
1044 | 1046 | * @return string |
1045 | 1047 | */ |
@@ -1077,6 +1079,7 @@ discard block |
||
1077 | 1079 | * As you can see both function use the same "my_grid_name" this is very important otherwise nothing will work |
1078 | 1080 | * |
1079 | 1081 | * @param string the div id, this value must be the same with the first parameter of Display::grid_js() |
1082 | + * @param string $div_id |
|
1080 | 1083 | * @return string html |
1081 | 1084 | * |
1082 | 1085 | */ |
@@ -1632,6 +1635,8 @@ discard block |
||
1632 | 1635 | * @param string url that will be added (for jquery see hot_courses.tpl) |
1633 | 1636 | * @param string point info array see function CourseManager::get_course_ranking() |
1634 | 1637 | * @param bool add a div wrapper |
1638 | + * @param string $id |
|
1639 | + * @param string $url |
|
1635 | 1640 | * @todo use templates |
1636 | 1641 | **/ |
1637 | 1642 | public static function return_rating_system($id, $url, $point_info = array(), $add_div_wrapper = true) |
@@ -1727,6 +1732,9 @@ discard block |
||
1727 | 1732 | return '<'.$size.'>'.Security::remove_XSS($title).'</'.$size.'>'; |
1728 | 1733 | } |
1729 | 1734 | |
1735 | + /** |
|
1736 | + * @param string $title |
|
1737 | + */ |
|
1730 | 1738 | public static function page_subheader2($title, $second_title = null) |
1731 | 1739 | { |
1732 | 1740 | return self::page_header($title, $second_title, 'h4'); |
@@ -1957,6 +1965,7 @@ discard block |
||
1957 | 1965 | |
1958 | 1966 | /** |
1959 | 1967 | * @todo use twig |
1968 | + * @param string $title |
|
1960 | 1969 | */ |
1961 | 1970 | public static function group_button($title, $elements) |
1962 | 1971 | { |
@@ -2174,9 +2183,9 @@ discard block |
||
2174 | 2183 | |
2175 | 2184 | /** |
2176 | 2185 | * @param int $id |
2177 | - * @param array $content |
|
2186 | + * @param string[] $content |
|
2178 | 2187 | * @param int $col |
2179 | - * @param bool|true $right |
|
2188 | + * @param boolean $right |
|
2180 | 2189 | * @return string |
2181 | 2190 | */ |
2182 | 2191 | public static function toolbarAction($id, $content = array(), $col = 2, $right = true) |
@@ -2258,10 +2267,10 @@ discard block |
||
2258 | 2267 | /** |
2259 | 2268 | * @param string $title |
2260 | 2269 | * @param string $content |
2261 | - * @param null $id |
|
2270 | + * @param string $id |
|
2262 | 2271 | * @param array $params |
2263 | - * @param null $idAccordion |
|
2264 | - * @param null $idCollapse |
|
2272 | + * @param string $idAccordion |
|
2273 | + * @param string $idCollapse |
|
2265 | 2274 | * @param bool|true $open |
2266 | 2275 | * @param bool|false $fullClickable |
2267 | 2276 | * @return null|string |
@@ -49,12 +49,12 @@ discard block |
||
49 | 49 | ); |
50 | 50 | } |
51 | 51 | |
52 | - /** |
|
53 | - * Displays the page header |
|
54 | - * @param string The name of the page (will be showed in the page title) |
|
55 | - * @param string Optional help file name |
|
56 | - * @param string $page_header |
|
57 | - */ |
|
52 | + /** |
|
53 | + * Displays the page header |
|
54 | + * @param string The name of the page (will be showed in the page title) |
|
55 | + * @param string Optional help file name |
|
56 | + * @param string $page_header |
|
57 | + */ |
|
58 | 58 | public static function display_header($tool_name ='', $help = null, $page_header = null) |
59 | 59 | { |
60 | 60 | $origin = api_get_origin(); |
@@ -448,7 +448,7 @@ discard block |
||
448 | 448 | */ |
449 | 449 | public static function display_normal_message($message, $filter = true, $returnValue = false) |
450 | 450 | { |
451 | - $message = self::return_message($message, 'normal', $filter); |
|
451 | + $message = self::return_message($message, 'normal', $filter); |
|
452 | 452 | if ($returnValue) { |
453 | 453 | return $message; |
454 | 454 | } else { |
@@ -528,22 +528,22 @@ discard block |
||
528 | 528 | } |
529 | 529 | |
530 | 530 | if ($filter) { |
531 | - $message = api_htmlentities($message, ENT_QUOTES, api_is_xml_http_request() ? 'UTF-8' : api_get_system_encoding()); |
|
531 | + $message = api_htmlentities($message, ENT_QUOTES, api_is_xml_http_request() ? 'UTF-8' : api_get_system_encoding()); |
|
532 | 532 | } |
533 | 533 | |
534 | 534 | $class = ""; |
535 | 535 | switch($type) { |
536 | 536 | case 'warning': |
537 | 537 | $class .= 'alert alert-warning'; |
538 | - break; |
|
538 | + break; |
|
539 | 539 | case 'error': |
540 | 540 | $class .= 'alert alert-danger'; |
541 | - break; |
|
541 | + break; |
|
542 | 542 | case 'confirmation': |
543 | 543 | case 'confirm': |
544 | 544 | case 'success': |
545 | 545 | $class .= 'alert alert-success'; |
546 | - break; |
|
546 | + break; |
|
547 | 547 | case 'normal': |
548 | 548 | default: |
549 | 549 | $class .= 'alert alert-info'; |
@@ -686,7 +686,7 @@ discard block |
||
686 | 686 | * @param array additional attributes (for instance height, width, onclick, ...) |
687 | 687 | * @param integer The wanted width of the icon (to be looked for in the corresponding img/icons/ folder) |
688 | 688 | * @return void |
689 | - */ |
|
689 | + */ |
|
690 | 690 | public static function display_icon( |
691 | 691 | $image, |
692 | 692 | $alt_text = '', |
@@ -722,7 +722,7 @@ discard block |
||
722 | 722 | * @author Julio Montoya 2010 Function improved, adding image constants |
723 | 723 | * @author Yannick Warnier 2011 Added size handler |
724 | 724 | * @version Feb 2011 |
725 | - */ |
|
725 | + */ |
|
726 | 726 | public static function return_icon( |
727 | 727 | $image, |
728 | 728 | $alt_text = '', |
@@ -911,14 +911,14 @@ discard block |
||
911 | 911 | */ |
912 | 912 | public static function input($type, $name, $value, $attributes = array()) |
913 | 913 | { |
914 | - if (isset($type)) { |
|
915 | - $attributes['type']= $type; |
|
916 | - } |
|
917 | - if (isset($name)) { |
|
918 | - $attributes['name']= $name; |
|
919 | - } |
|
920 | - if (isset($value)) { |
|
921 | - $attributes['value']= $value; |
|
914 | + if (isset($type)) { |
|
915 | + $attributes['type']= $type; |
|
916 | + } |
|
917 | + if (isset($name)) { |
|
918 | + $attributes['name']= $name; |
|
919 | + } |
|
920 | + if (isset($value)) { |
|
921 | + $attributes['value']= $value; |
|
922 | 922 | } |
923 | 923 | return self::tag('input', '', $attributes); |
924 | 924 | } |
@@ -931,10 +931,10 @@ discard block |
||
931 | 931 | */ |
932 | 932 | public static function button($name, $value, $attributes = array()) |
933 | 933 | { |
934 | - if (!empty($name)) { |
|
934 | + if (!empty($name)) { |
|
935 | 935 | $attributes['name'] = $name; |
936 | - } |
|
937 | - return self::tag('button', $value, $attributes); |
|
936 | + } |
|
937 | + return self::tag('button', $value, $attributes); |
|
938 | 938 | } |
939 | 939 | |
940 | 940 | /** |
@@ -1001,7 +1001,7 @@ discard block |
||
1001 | 1001 | * in the $htmlHeadXtra variable before the display_header |
1002 | 1002 | * Add this script |
1003 | 1003 | * @example |
1004 | - * <script> |
|
1004 | + * <script> |
|
1005 | 1005 | $(function() { |
1006 | 1006 | $( "#tabs" ).tabs(); |
1007 | 1007 | }); |
@@ -1067,20 +1067,20 @@ discard block |
||
1067 | 1067 | */ |
1068 | 1068 | public static function tabsOnlyLink($headers, $selected = null) |
1069 | 1069 | { |
1070 | - $id = uniqid(); |
|
1071 | - $i = 1; |
|
1072 | - $lis = null; |
|
1073 | - foreach ($headers as $item) { |
|
1070 | + $id = uniqid(); |
|
1071 | + $i = 1; |
|
1072 | + $lis = null; |
|
1073 | + foreach ($headers as $item) { |
|
1074 | 1074 | $class = null; |
1075 | 1075 | if ($i == $selected) { |
1076 | 1076 | $class = 'active'; |
1077 | 1077 | } |
1078 | - $item = self::tag( |
|
1079 | - 'a', |
|
1080 | - $item['content'], |
|
1081 | - array('id' => $id.'-'.$i, 'href' => $item['url']) |
|
1082 | - ); |
|
1083 | - $lis .= self::tag('li', $item, array('class' => $class)); |
|
1078 | + $item = self::tag( |
|
1079 | + 'a', |
|
1080 | + $item['content'], |
|
1081 | + array('id' => $id.'-'.$i, 'href' => $item['url']) |
|
1082 | + ); |
|
1083 | + $lis .= self::tag('li', $item, array('class' => $class)); |
|
1084 | 1084 | $i++; |
1085 | 1085 | } |
1086 | 1086 | return self::tag('ul', $lis, array('class' => 'nav nav-tabs tabs-margin')); |
@@ -1292,8 +1292,8 @@ discard block |
||
1292 | 1292 | */ |
1293 | 1293 | public static function table($headers, $rows, $attributes = array()) |
1294 | 1294 | { |
1295 | - if (empty($attributes)) { |
|
1296 | - $attributes['class'] = 'data_table'; |
|
1295 | + if (empty($attributes)) { |
|
1296 | + $attributes['class'] = 'data_table'; |
|
1297 | 1297 | } |
1298 | 1298 | $table = new HTML_Table($attributes); |
1299 | 1299 | $row = 0; |
@@ -1301,17 +1301,17 @@ discard block |
||
1301 | 1301 | |
1302 | 1302 | // Course headers |
1303 | 1303 | if (!empty($headers)) { |
1304 | - foreach ($headers as $item) { |
|
1305 | - $table->setHeaderContents($row, $column, $item); |
|
1306 | - $column++; |
|
1307 | - } |
|
1308 | - $row = 1; |
|
1309 | - $column = 0; |
|
1304 | + foreach ($headers as $item) { |
|
1305 | + $table->setHeaderContents($row, $column, $item); |
|
1306 | + $column++; |
|
1307 | + } |
|
1308 | + $row = 1; |
|
1309 | + $column = 0; |
|
1310 | 1310 | } |
1311 | 1311 | |
1312 | 1312 | if (!empty($rows)) { |
1313 | - foreach($rows as $content) { |
|
1314 | - $table->setCellContents($row, $column, $content); |
|
1313 | + foreach($rows as $content) { |
|
1314 | + $table->setCellContents($row, $column, $content); |
|
1315 | 1315 | $row++; |
1316 | 1316 | } |
1317 | 1317 | } |
@@ -1417,7 +1417,7 @@ discard block |
||
1417 | 1417 | //$item_property['tool'] != TOOL_DROPBOX && |
1418 | 1418 | $item_property['tool'] != TOOL_NOTEBOOK && |
1419 | 1419 | $item_property['tool'] != TOOL_CHAT) |
1420 | - ) |
|
1420 | + ) |
|
1421 | 1421 | ) |
1422 | 1422 | // Take only what's visible or "invisible but where the user is a teacher" or where the visibility is unset. |
1423 | 1423 | && ($item_property['visibility'] == '1' |
@@ -1433,7 +1433,7 @@ discard block |
||
1433 | 1433 | ($item_property['to_user_id'] != $user_id) && |
1434 | 1434 | (!isset($item_property['to_group_id']) || !in_array($item_property['to_group_id'], $group_ids))) |
1435 | 1435 | ) { |
1436 | - continue; |
|
1436 | + continue; |
|
1437 | 1437 | } |
1438 | 1438 | |
1439 | 1439 | // If it's a survey, make sure the user's invited. Otherwise drop it. |
@@ -1651,21 +1651,21 @@ discard block |
||
1651 | 1651 | * |
1652 | 1652 | * @param string id of the rating ul element |
1653 | 1653 | * @param string url that will be added (for jquery see hot_courses.tpl) |
1654 | - * @param string point info array see function CourseManager::get_course_ranking() |
|
1655 | - * @param bool add a div wrapper |
|
1656 | - * @todo use templates |
|
1654 | + * @param string point info array see function CourseManager::get_course_ranking() |
|
1655 | + * @param bool add a div wrapper |
|
1656 | + * @todo use templates |
|
1657 | 1657 | **/ |
1658 | 1658 | public static function return_rating_system($id, $url, $point_info = array(), $add_div_wrapper = true) |
1659 | 1659 | { |
1660 | - $number_of_users_who_voted = isset($point_info['users_who_voted']) ? $point_info['users_who_voted'] : null; |
|
1661 | - $percentage = isset($point_info['point_average']) ? $point_info['point_average'] : 0; |
|
1660 | + $number_of_users_who_voted = isset($point_info['users_who_voted']) ? $point_info['users_who_voted'] : null; |
|
1661 | + $percentage = isset($point_info['point_average']) ? $point_info['point_average'] : 0; |
|
1662 | 1662 | |
1663 | - if (!empty($percentage)) { |
|
1663 | + if (!empty($percentage)) { |
|
1664 | 1664 | $percentage = $percentage*125/100; |
1665 | 1665 | } |
1666 | - $accesses = isset($point_info['accesses']) ? $point_info['accesses'] : 0; |
|
1666 | + $accesses = isset($point_info['accesses']) ? $point_info['accesses'] : 0; |
|
1667 | 1667 | |
1668 | - $star_label = sprintf(get_lang('XStarsOutOf5'), $point_info['point_average_star']); |
|
1668 | + $star_label = sprintf(get_lang('XStarsOutOf5'), $point_info['point_average_star']); |
|
1669 | 1669 | |
1670 | 1670 | $html = '<ul id="'.$id.'" class="star-rating"> |
1671 | 1671 | <li class="current-rating" style="width:'.$percentage.'px;"></li> |
@@ -1676,26 +1676,26 @@ discard block |
||
1676 | 1676 | <li><a href="javascript:void(0);" data-link="'.$url.'&star=5" title="'.$star_label.'" class="five-stars">5</a></li> |
1677 | 1677 | </ul>'; |
1678 | 1678 | |
1679 | - $labels = array(); |
|
1679 | + $labels = array(); |
|
1680 | 1680 | |
1681 | - $labels[]= $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes'); |
|
1682 | - $labels[]= $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits'); |
|
1683 | - /* if (!empty($number_of_users_who_voted)) { |
|
1681 | + $labels[]= $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes'); |
|
1682 | + $labels[]= $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits'); |
|
1683 | + /* if (!empty($number_of_users_who_voted)) { |
|
1684 | 1684 | $labels[]= get_lang('Average').' '.$point_info['point_average_star'].'/5'; |
1685 | 1685 | } */ |
1686 | 1686 | |
1687 | - $labels[]= $point_info['user_vote'] ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote'). ' [?] '; |
|
1687 | + $labels[]= $point_info['user_vote'] ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote'). ' [?] '; |
|
1688 | 1688 | |
1689 | - if (!$add_div_wrapper && api_is_anonymous()) { |
|
1690 | - $labels[]= Display::tag('span', get_lang('LoginToVote'), array('class' => 'error')); |
|
1691 | - } |
|
1689 | + if (!$add_div_wrapper && api_is_anonymous()) { |
|
1690 | + $labels[]= Display::tag('span', get_lang('LoginToVote'), array('class' => 'error')); |
|
1691 | + } |
|
1692 | 1692 | |
1693 | 1693 | $html .= Display::div(implode(' | ', $labels) , array('id' => 'vote_label_'.$id, 'class' => 'vote_label_info')); |
1694 | 1694 | $html .= ' '.Display::span(' ', array('id' => 'vote_label2_'.$id)); |
1695 | 1695 | |
1696 | 1696 | if ($add_div_wrapper) { |
1697 | - $html = Display::div($html, array('id' => 'rating_wrapper_'.$id)); |
|
1698 | - } |
|
1697 | + $html = Display::div($html, array('id' => 'rating_wrapper_'.$id)); |
|
1698 | + } |
|
1699 | 1699 | |
1700 | 1700 | return $html; |
1701 | 1701 | } |
@@ -1733,8 +1733,8 @@ discard block |
||
1733 | 1733 | return self::page_header($title, $second_title); |
1734 | 1734 | } |
1735 | 1735 | |
1736 | - public static function page_subheader_and_translate($title, $second_title = null) |
|
1737 | - { |
|
1736 | + public static function page_subheader_and_translate($title, $second_title = null) |
|
1737 | + { |
|
1738 | 1738 | $title = get_lang($title); |
1739 | 1739 | return self::page_subheader($title, $second_title); |
1740 | 1740 | } |
@@ -56,7 +56,7 @@ discard block |
||
56 | 56 | * @param string Optional help file name |
57 | 57 | * @param string $page_header |
58 | 58 | */ |
59 | - public static function display_header($tool_name ='', $help = null, $page_header = null) |
|
59 | + public static function display_header($tool_name = '', $help = null, $page_header = null) |
|
60 | 60 | { |
61 | 61 | $origin = api_get_origin(); |
62 | 62 | $showHeader = true; |
@@ -354,7 +354,7 @@ discard block |
||
354 | 354 | $grid_class = array(), |
355 | 355 | $elementCount = 0 |
356 | 356 | ) { |
357 | - $column = 0; |
|
357 | + $column = 0; |
|
358 | 358 | $default_items_per_page = isset($paging_options['per_page']) ? $paging_options['per_page'] : 20; |
359 | 359 | |
360 | 360 | $table = new SortableTableFromArray($content, $column, $default_items_per_page, $name); |
@@ -483,7 +483,7 @@ discard block |
||
483 | 483 | * @deprecated use Display::addFlash with Display::return_message |
484 | 484 | * @return void |
485 | 485 | */ |
486 | - public static function display_confirmation_message ($message, $filter = true, $returnValue = false) |
|
486 | + public static function display_confirmation_message($message, $filter = true, $returnValue = false) |
|
487 | 487 | { |
488 | 488 | $message = self::return_message($message, 'confirm', $filter); |
489 | 489 | if ($returnValue) { |
@@ -502,7 +502,7 @@ discard block |
||
502 | 502 | * |
503 | 503 | * @return void |
504 | 504 | */ |
505 | - public static function display_error_message ($message, $filter = true, $returnValue = false) |
|
505 | + public static function display_error_message($message, $filter = true, $returnValue = false) |
|
506 | 506 | { |
507 | 507 | $message = self::return_message($message, 'error', $filter); |
508 | 508 | if ($returnValue) { |
@@ -517,7 +517,7 @@ discard block |
||
517 | 517 | * @param string $type |
518 | 518 | * @param bool $filter |
519 | 519 | */ |
520 | - public static function return_message_and_translate($message, $type='normal', $filter = true) |
|
520 | + public static function return_message_and_translate($message, $type = 'normal', $filter = true) |
|
521 | 521 | { |
522 | 522 | $message = get_lang($message); |
523 | 523 | echo self::return_message($message, $type, $filter); |
@@ -584,14 +584,14 @@ discard block |
||
584 | 584 | } |
585 | 585 | // Encrypt email |
586 | 586 | $hmail = ''; |
587 | - for ($i = 0; $i < strlen($email); $i ++) { |
|
587 | + for ($i = 0; $i < strlen($email); $i++) { |
|
588 | 588 | $hmail .= '&#'.ord($email { |
589 | 589 | $i }).';'; |
590 | 590 | } |
591 | 591 | $hclickable_text = null; |
592 | 592 | // Encrypt clickable text if @ is present |
593 | 593 | if (strpos($clickable_text, '@')) { |
594 | - for ($i = 0; $i < strlen($clickable_text); $i ++) { |
|
594 | + for ($i = 0; $i < strlen($clickable_text); $i++) { |
|
595 | 595 | $hclickable_text .= '&#'.ord($clickable_text { |
596 | 596 | $i }).';'; |
597 | 597 | } |
@@ -623,7 +623,7 @@ discard block |
||
623 | 623 | } |
624 | 624 | // Encrypt email |
625 | 625 | $hmail = ''; |
626 | - for ($i = 0; $i < strlen($email); $i ++) { |
|
626 | + for ($i = 0; $i < strlen($email); $i++) { |
|
627 | 627 | $hmail .= '&#'.ord($email { |
628 | 628 | $i }).';'; |
629 | 629 | } |
@@ -643,7 +643,7 @@ discard block |
||
643 | 643 | public static function get_alphabet_options($selected_letter = '') |
644 | 644 | { |
645 | 645 | $result = ''; |
646 | - for ($i = 65; $i <= 90; $i ++) { |
|
646 | + for ($i = 65; $i <= 90; $i++) { |
|
647 | 647 | $letter = chr($i); |
648 | 648 | $result .= '<option value="'.$letter.'"'; |
649 | 649 | if ($selected_letter == $letter) { |
@@ -664,7 +664,7 @@ discard block |
||
664 | 664 | public static function get_numeric_options($min, $max, $selected_num = 0) |
665 | 665 | { |
666 | 666 | $result = ''; |
667 | - for ($i = $min; $i <= $max; $i ++) { |
|
667 | + for ($i = $min; $i <= $max; $i++) { |
|
668 | 668 | $result .= '<option value="'.$i.'"'; |
669 | 669 | if (is_int($selected_num)) |
670 | 670 | if ($selected_num == $i) { |
@@ -734,8 +734,8 @@ discard block |
||
734 | 734 | // important to use the public path (i.e. web/css/) rather than the |
735 | 735 | // internal path (/app/Resource/public/css/) because the path used |
736 | 736 | // in the end must be the public path |
737 | - $alternateCssPath = api_get_path(SYS_PUBLIC_PATH) . 'css/'; |
|
738 | - $alternateWebCssPath = api_get_path(WEB_PUBLIC_PATH) . 'css/'; |
|
737 | + $alternateCssPath = api_get_path(SYS_PUBLIC_PATH).'css/'; |
|
738 | + $alternateWebCssPath = api_get_path(WEB_PUBLIC_PATH).'css/'; |
|
739 | 739 | |
740 | 740 | $image = trim($image); |
741 | 741 | |
@@ -745,7 +745,7 @@ discard block |
||
745 | 745 | $size = ICON_SIZE_SMALL; |
746 | 746 | } |
747 | 747 | |
748 | - $size_extra = $size . '/'; |
|
748 | + $size_extra = $size.'/'; |
|
749 | 749 | |
750 | 750 | // Checking the img/ folder |
751 | 751 | $icon = $w_code_path.'img/'.$image; |
@@ -753,7 +753,7 @@ discard block |
||
753 | 753 | $theme = 'themes/chamilo/icons/'; |
754 | 754 | |
755 | 755 | if ($loadThemeIcon) { |
756 | - $theme = 'themes/' . api_get_visual_theme() . '/icons/'; |
|
756 | + $theme = 'themes/'.api_get_visual_theme().'/icons/'; |
|
757 | 757 | // Checking the theme icons folder example: app/Resources/public/css/themes/chamilo/icons/XXX |
758 | 758 | if (is_file($alternateCssPath.$theme.$size_extra.$image)) { |
759 | 759 | $icon = $alternateWebCssPath.$theme.$size_extra.$image; |
@@ -775,11 +775,11 @@ discard block |
||
775 | 775 | // ask for the SVG version directly |
776 | 776 | $svgIcons = api_get_setting('icons_mode_svg'); |
777 | 777 | if ($svgIcons == 'true' && $return_only_path == false) { |
778 | - $svgImage = substr($image, 0, -3) . 'svg'; |
|
779 | - if (is_file($code_path . $theme . 'svg/' . $svgImage)) { |
|
780 | - $icon = $w_code_path . $theme . 'svg/' . $svgImage; |
|
781 | - } elseif (is_file($code_path . 'img/icons/svg/' . $svgImage)) { |
|
782 | - $icon = $w_code_path . 'img/icons/svg/' . $svgImage; |
|
778 | + $svgImage = substr($image, 0, -3).'svg'; |
|
779 | + if (is_file($code_path.$theme.'svg/'.$svgImage)) { |
|
780 | + $icon = $w_code_path.$theme.'svg/'.$svgImage; |
|
781 | + } elseif (is_file($code_path.'img/icons/svg/'.$svgImage)) { |
|
782 | + $icon = $w_code_path.'img/icons/svg/'.$svgImage; |
|
783 | 783 | } |
784 | 784 | |
785 | 785 | if (empty($additional_attributes['height'])) { |
@@ -868,7 +868,7 @@ discard block |
||
868 | 868 | } |
869 | 869 | } |
870 | 870 | //some tags don't have this </XXX> |
871 | - if (in_array($tag, array('img','input','br'))) { |
|
871 | + if (in_array($tag, array('img', 'input', 'br'))) { |
|
872 | 872 | $return_value = '<'.$tag.' '.$attribute_list.' />'; |
873 | 873 | } else { |
874 | 874 | $return_value = '<'.$tag.' '.$attribute_list.' >'.$content.'</'.$tag.'>'; |
@@ -921,13 +921,13 @@ discard block |
||
921 | 921 | public static function input($type, $name, $value, $attributes = array()) |
922 | 922 | { |
923 | 923 | if (isset($type)) { |
924 | - $attributes['type']= $type; |
|
924 | + $attributes['type'] = $type; |
|
925 | 925 | } |
926 | 926 | if (isset($name)) { |
927 | - $attributes['name']= $name; |
|
927 | + $attributes['name'] = $name; |
|
928 | 928 | } |
929 | 929 | if (isset($value)) { |
930 | - $attributes['value']= $value; |
|
930 | + $attributes['value'] = $value; |
|
931 | 931 | } |
932 | 932 | return self::tag('input', '', $attributes); |
933 | 933 | } |
@@ -960,14 +960,14 @@ discard block |
||
960 | 960 | ) { |
961 | 961 | $html = ''; |
962 | 962 | $extra = ''; |
963 | - $default_id = 'id="' . $name . '" '; |
|
963 | + $default_id = 'id="'.$name.'" '; |
|
964 | 964 | foreach ($extra_attributes as $key => $parameter) { |
965 | 965 | if ($key == 'id') { |
966 | 966 | $default_id = ''; |
967 | 967 | } |
968 | - $extra .= $key . '="' . $parameter . '" '; |
|
968 | + $extra .= $key.'="'.$parameter.'" '; |
|
969 | 969 | } |
970 | - $html .= '<select name="' . $name . '" ' . $default_id . ' ' . $extra . '>'; |
|
970 | + $html .= '<select name="'.$name.'" '.$default_id.' '.$extra.'>'; |
|
971 | 971 | |
972 | 972 | if ($show_blank_item) { |
973 | 973 | if (empty($blank_item_text)) { |
@@ -975,14 +975,14 @@ discard block |
||
975 | 975 | } else { |
976 | 976 | $blank_item_text = Security::remove_XSS($blank_item_text); |
977 | 977 | } |
978 | - $html .= self::tag('option', '-- ' . $blank_item_text . ' --', array('value' => '-1')); |
|
978 | + $html .= self::tag('option', '-- '.$blank_item_text.' --', array('value' => '-1')); |
|
979 | 979 | } |
980 | 980 | if ($values) { |
981 | 981 | foreach ($values as $key => $value) { |
982 | 982 | if (is_array($value) && isset($value['name'])) { |
983 | 983 | $value = $value['name']; |
984 | 984 | } |
985 | - $html .= '<option value="' . $key . '"'; |
|
985 | + $html .= '<option value="'.$key.'"'; |
|
986 | 986 | |
987 | 987 | if (is_array($default)) { |
988 | 988 | foreach ($default as $item) { |
@@ -997,7 +997,7 @@ discard block |
||
997 | 997 | } |
998 | 998 | } |
999 | 999 | |
1000 | - $html .= '>' . $value . '</option>'; |
|
1000 | + $html .= '>'.$value.'</option>'; |
|
1001 | 1001 | } |
1002 | 1002 | } |
1003 | 1003 | $html .= '</select>'; |
@@ -1025,7 +1025,7 @@ discard block |
||
1025 | 1025 | */ |
1026 | 1026 | public static function tabs($headers, $items, $id = 'tabs', $attributes = array(), $ul_attributes = array()) |
1027 | 1027 | { |
1028 | - if (empty($headers) || count($headers) == 0 ) { |
|
1028 | + if (empty($headers) || count($headers) == 0) { |
|
1029 | 1029 | return ''; |
1030 | 1030 | } |
1031 | 1031 | |
@@ -1036,7 +1036,7 @@ discard block |
||
1036 | 1036 | if ($i == 1) { |
1037 | 1037 | $active = ' active'; |
1038 | 1038 | } |
1039 | - $item = self::tag('a', $item, array('href'=>'#'.$id.'-'.$i, 'role'=> 'tab', 'data-toggle' => 'tab', 'id' => $id . $i)); |
|
1039 | + $item = self::tag('a', $item, array('href'=>'#'.$id.'-'.$i, 'role'=> 'tab', 'data-toggle' => 'tab', 'id' => $id.$i)); |
|
1040 | 1040 | $ul_attributes['role'] = 'presentation'; |
1041 | 1041 | $ul_attributes['class'] = $active; |
1042 | 1042 | $lis .= self::tag('li', $item, $ul_attributes); |
@@ -1065,7 +1065,7 @@ discard block |
||
1065 | 1065 | |
1066 | 1066 | $main_div = self::tag('div', $ul.self::tag('div', $divs, ['class' => 'tab-content']), $attributes); |
1067 | 1067 | |
1068 | - return $main_div ; |
|
1068 | + return $main_div; |
|
1069 | 1069 | } |
1070 | 1070 | |
1071 | 1071 | /** |
@@ -1113,8 +1113,8 @@ discard block |
||
1113 | 1113 | */ |
1114 | 1114 | public static function grid_html($div_id) |
1115 | 1115 | { |
1116 | - $table = self::tag('table','', array('id' => $div_id)); |
|
1117 | - $table .= self::tag('div','', array('id' => $div_id.'_pager')); |
|
1116 | + $table = self::tag('table', '', array('id' => $div_id)); |
|
1117 | + $table .= self::tag('div', '', array('id' => $div_id.'_pager')); |
|
1118 | 1118 | return $table; |
1119 | 1119 | } |
1120 | 1120 | |
@@ -1175,7 +1175,7 @@ discard block |
||
1175 | 1175 | $obj->colNames = $column_names; |
1176 | 1176 | $obj->colModel = $column_model; |
1177 | 1177 | $obj->pager = '#'.$div_id.'_pager'; |
1178 | - $obj->datatype = 'json'; |
|
1178 | + $obj->datatype = 'json'; |
|
1179 | 1179 | $obj->viewrecords = 'true'; |
1180 | 1180 | |
1181 | 1181 | $all_value = 10000000; |
@@ -1236,10 +1236,10 @@ discard block |
||
1236 | 1236 | // Adding static data. |
1237 | 1237 | if (!empty($data)) { |
1238 | 1238 | $data_var = $div_id.'_data'; |
1239 | - $json.=' var '.$data_var.' = '.json_encode($data).';'; |
|
1239 | + $json .= ' var '.$data_var.' = '.json_encode($data).';'; |
|
1240 | 1240 | $obj->data = $data_var; |
1241 | 1241 | $obj->datatype = 'local'; |
1242 | - $json.="\n"; |
|
1242 | + $json .= "\n"; |
|
1243 | 1243 | } |
1244 | 1244 | |
1245 | 1245 | $obj->end = 'end'; |
@@ -1253,12 +1253,12 @@ discard block |
||
1253 | 1253 | } |
1254 | 1254 | |
1255 | 1255 | // Fixing true/false js values that doesn't need the "" |
1256 | - $json_encode = str_replace(':"true"',':true',$json_encode); |
|
1256 | + $json_encode = str_replace(':"true"', ':true', $json_encode); |
|
1257 | 1257 | // wrap_cell is not a valid jqgrid attributes is a hack to wrap a text |
1258 | 1258 | $json_encode = str_replace('"wrap_cell":true', 'cellattr : function(rowId, value, rowObject, colModel, arrData) { return \'class = "jqgrid_whitespace"\'; }', $json_encode); |
1259 | - $json_encode = str_replace(':"false"',':false',$json_encode); |
|
1259 | + $json_encode = str_replace(':"false"', ':false', $json_encode); |
|
1260 | 1260 | $json_encode = str_replace('"formatter":"action_formatter"', 'formatter:action_formatter', $json_encode); |
1261 | - $json_encode = str_replace(array('{"first":"first",','"end":"end"}'), '', $json_encode); |
|
1261 | + $json_encode = str_replace(array('{"first":"first",', '"end":"end"}'), '', $json_encode); |
|
1262 | 1262 | |
1263 | 1263 | // Creating the jqgrid element. |
1264 | 1264 | $json .= '$("#'.$div_id.'").jqGrid({'; |
@@ -1273,22 +1273,22 @@ discard block |
||
1273 | 1273 | $groups = ''; |
1274 | 1274 | foreach ($extra_params['groupHeaders'] as $group) { |
1275 | 1275 | //{ "startColumnName" : "courses", "numberOfColumns" : 1, "titleText" : "Order Info" }, |
1276 | - $groups .= '{ "startColumnName" : "' . $group['startColumnName'] . '", "numberOfColumns" : ' . $group['numberOfColumns'] . ', "titleText" : "' . $group['titleText'] . '" },'; |
|
1276 | + $groups .= '{ "startColumnName" : "'.$group['startColumnName'].'", "numberOfColumns" : '.$group['numberOfColumns'].', "titleText" : "'.$group['titleText'].'" },'; |
|
1277 | 1277 | |
1278 | 1278 | } |
1279 | 1279 | $json .= '$("#'.$div_id.'").jqGrid("setGroupHeaders", { |
1280 | 1280 | "useColSpanStyle" : false, |
1281 | 1281 | "groupHeaders" : [ |
1282 | - ' . $groups . ' |
|
1282 | + ' . $groups.' |
|
1283 | 1283 | ] |
1284 | 1284 | });'; |
1285 | 1285 | } |
1286 | 1286 | |
1287 | 1287 | $all_text = addslashes(get_lang('All')); |
1288 | 1288 | $json .= '$("'.$obj->pager.' option[value='.$all_value.']").text("'.$all_text.'");'; |
1289 | - $json.= "\n"; |
|
1289 | + $json .= "\n"; |
|
1290 | 1290 | // Adding edit/delete icons. |
1291 | - $json.= $formatter; |
|
1291 | + $json .= $formatter; |
|
1292 | 1292 | |
1293 | 1293 | return $json; |
1294 | 1294 | } |
@@ -1319,7 +1319,7 @@ discard block |
||
1319 | 1319 | } |
1320 | 1320 | |
1321 | 1321 | if (!empty($rows)) { |
1322 | - foreach($rows as $content) { |
|
1322 | + foreach ($rows as $content) { |
|
1323 | 1323 | $table->setCellContents($row, $column, $content); |
1324 | 1324 | $row++; |
1325 | 1325 | } |
@@ -1481,7 +1481,7 @@ discard block |
||
1481 | 1481 | |
1482 | 1482 | // Show all tool icons where there is something new. |
1483 | 1483 | $return = ' '; |
1484 | - foreach($notifications as $notification) { |
|
1484 | + foreach ($notifications as $notification) { |
|
1485 | 1485 | $lastDate = date('d/m/Y H:i', convert_sql_date($notification['lastedit_date'])); |
1486 | 1486 | $type = $notification['lastedit_type']; |
1487 | 1487 | $label = get_lang('TitleNotification').": ".get_lang($type)." ($lastDate)"; |
@@ -1524,9 +1524,9 @@ discard block |
||
1524 | 1524 | $main_user_table = Database :: get_main_table(TABLE_MAIN_USER); |
1525 | 1525 | $tbl_session = Database :: get_main_table(TABLE_MAIN_SESSION); |
1526 | 1526 | // Request for the name of the general coach |
1527 | - $sql ='SELECT tu.lastname, tu.firstname, ts.* |
|
1527 | + $sql = 'SELECT tu.lastname, tu.firstname, ts.* |
|
1528 | 1528 | FROM '.$tbl_session.' ts |
1529 | - LEFT JOIN '.$main_user_table .' tu |
|
1529 | + LEFT JOIN '.$main_user_table.' tu |
|
1530 | 1530 | ON ts.id_coach = tu.user_id |
1531 | 1531 | WHERE ts.id = '.intval($session_id); |
1532 | 1532 | $rs = Database::query($sql); |
@@ -1538,7 +1538,7 @@ discard block |
||
1538 | 1538 | $session['title'] = $session_info['name']; |
1539 | 1539 | $session['id_coach'] = $session_info['id_coach']; |
1540 | 1540 | $session['coach'] = ''; |
1541 | - $session['dates'] = ''; |
|
1541 | + $session['dates'] = ''; |
|
1542 | 1542 | |
1543 | 1543 | if (($session_info['access_end_date'] == '0000-00-00 00:00:00' && $session_info['access_start_date'] == '0000-00-00 00:00:00') || |
1544 | 1544 | ($session_info['access_end_date'] == '0000-00-00 00:00:00' && $session_info['access_start_date'] == '0000-00-00 00:00:00') || |
@@ -1610,7 +1610,7 @@ discard block |
||
1610 | 1610 | $session['dates'] = $dates['access']; |
1611 | 1611 | |
1612 | 1612 | |
1613 | - if (api_get_setting('show_session_coach') === 'true' ) { |
|
1613 | + if (api_get_setting('show_session_coach') === 'true') { |
|
1614 | 1614 | $session['coach'] = api_get_person_name( |
1615 | 1615 | $session_info['firstname'], |
1616 | 1616 | $session_info['lastname'] |
@@ -1664,9 +1664,9 @@ discard block |
||
1664 | 1664 | $percentage = isset($point_info['point_average']) ? $point_info['point_average'] : 0; |
1665 | 1665 | |
1666 | 1666 | if (!empty($percentage)) { |
1667 | - $percentage = $percentage*125/100; |
|
1667 | + $percentage = $percentage * 125 / 100; |
|
1668 | 1668 | } |
1669 | - $accesses = isset($point_info['accesses']) ? $point_info['accesses'] : 0; |
|
1669 | + $accesses = isset($point_info['accesses']) ? $point_info['accesses'] : 0; |
|
1670 | 1670 | |
1671 | 1671 | $star_label = sprintf(get_lang('XStarsOutOf5'), $point_info['point_average_star']); |
1672 | 1672 | |
@@ -1681,19 +1681,19 @@ discard block |
||
1681 | 1681 | |
1682 | 1682 | $labels = array(); |
1683 | 1683 | |
1684 | - $labels[]= $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes'); |
|
1685 | - $labels[]= $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits'); |
|
1684 | + $labels[] = $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes'); |
|
1685 | + $labels[] = $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits'); |
|
1686 | 1686 | /* if (!empty($number_of_users_who_voted)) { |
1687 | 1687 | $labels[]= get_lang('Average').' '.$point_info['point_average_star'].'/5'; |
1688 | 1688 | } */ |
1689 | 1689 | |
1690 | - $labels[]= $point_info['user_vote'] ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote'). ' [?] '; |
|
1690 | + $labels[] = $point_info['user_vote'] ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote').' [?] '; |
|
1691 | 1691 | |
1692 | 1692 | if (!$add_div_wrapper && api_is_anonymous()) { |
1693 | - $labels[]= Display::tag('span', get_lang('LoginToVote'), array('class' => 'error')); |
|
1693 | + $labels[] = Display::tag('span', get_lang('LoginToVote'), array('class' => 'error')); |
|
1694 | 1694 | } |
1695 | 1695 | |
1696 | - $html .= Display::div(implode(' | ', $labels) , array('id' => 'vote_label_'.$id, 'class' => 'vote_label_info')); |
|
1696 | + $html .= Display::div(implode(' | ', $labels), array('id' => 'vote_label_'.$id, 'class' => 'vote_label_info')); |
|
1697 | 1697 | $html .= ' '.Display::span(' ', array('id' => 'vote_label2_'.$id)); |
1698 | 1698 | |
1699 | 1699 | if ($add_div_wrapper) { |
@@ -1766,8 +1766,8 @@ discard block |
||
1766 | 1766 | if (!empty($list)) { |
1767 | 1767 | $html = '<dl class="dl-horizontal">'; |
1768 | 1768 | foreach ($list as $item) { |
1769 | - $html .= '<dt>' . $item['title'] . '</dt>'; |
|
1770 | - $html .= '<dd>' . $item['content'] . '</dd>'; |
|
1769 | + $html .= '<dt>'.$item['title'].'</dt>'; |
|
1770 | + $html .= '<dd>'.$item['content'].'</dd>'; |
|
1771 | 1771 | } |
1772 | 1772 | $html .= '</dl>'; |
1773 | 1773 | } |
@@ -1809,7 +1809,7 @@ discard block |
||
1809 | 1809 | * @param string $type |
1810 | 1810 | * @return null|string |
1811 | 1811 | */ |
1812 | - public static function badge($count, $type ="warning") |
|
1812 | + public static function badge($count, $type = "warning") |
|
1813 | 1813 | { |
1814 | 1814 | $class = ''; |
1815 | 1815 | |
@@ -1885,7 +1885,7 @@ discard block |
||
1885 | 1885 | if (!empty($content)) { |
1886 | 1886 | $html = '<span class="label '.$class.'">'; |
1887 | 1887 | $html .= $content; |
1888 | - $html .='</span>'; |
|
1888 | + $html .= '</span>'; |
|
1889 | 1889 | } |
1890 | 1890 | |
1891 | 1891 | return $html; |
@@ -1906,7 +1906,7 @@ discard block |
||
1906 | 1906 | $class = 'class ="active"'; |
1907 | 1907 | } |
1908 | 1908 | |
1909 | - if (basename($_SERVER['REQUEST_URI']) == basename($value['url']) ) { |
|
1909 | + if (basename($_SERVER['REQUEST_URI']) == basename($value['url'])) { |
|
1910 | 1910 | $class = 'class ="active"'; |
1911 | 1911 | } |
1912 | 1912 | $html .= "<li $class >"; |
@@ -2160,7 +2160,7 @@ discard block |
||
2160 | 2160 | } |
2161 | 2161 | $label = $nextValue + $localCounter + 1; |
2162 | 2162 | if ($isMedia) { |
2163 | - $label = ($fixedValue + 1) .' '.chr(97 + $localCounter); |
|
2163 | + $label = ($fixedValue + 1).' '.chr(97 + $localCounter); |
|
2164 | 2164 | $link_to_show = $link.$fixedValue.'#questionanchor'.$itemId; |
2165 | 2165 | } |
2166 | 2166 | $link = Display::url($label.' ', $link_to_show, $linkAttributes); |
@@ -2341,7 +2341,7 @@ discard block |
||
2341 | 2341 | $attributes['title'] = isset($attributes['title']) ? $attributes['title'] : $text; |
2342 | 2342 | |
2343 | 2343 | if (!$includeText) { |
2344 | - $text = '<span class="sr-only">' . $text . '</span>'; |
|
2344 | + $text = '<span class="sr-only">'.$text.'</span>'; |
|
2345 | 2345 | } |
2346 | 2346 | |
2347 | 2347 | return self::url("$icon $text", $url, $attributes); |
@@ -2356,9 +2356,9 @@ discard block |
||
2356 | 2356 | */ |
2357 | 2357 | public static function toolbarAction($id, $content = array(), $col = 2, $right = true) |
2358 | 2358 | { |
2359 | - $columns = 12/$col; |
|
2359 | + $columns = 12 / $col; |
|
2360 | 2360 | $html = ''; |
2361 | - $html .= '<div id="' . $id . '" class="actions">'; |
|
2361 | + $html .= '<div id="'.$id.'" class="actions">'; |
|
2362 | 2362 | $html .= '<div class="row">'; |
2363 | 2363 | if ($col > 4) { |
2364 | 2364 | $html = '<div class="alert alert-warning" role="alert"> |
@@ -2366,7 +2366,7 @@ discard block |
||
2366 | 2366 | </div>'; |
2367 | 2367 | } else { |
2368 | 2368 | for ($i = 0; $i < $col; $i++) { |
2369 | - $html .= '<div class="col-md-' . $columns . '">'; |
|
2369 | + $html .= '<div class="col-md-'.$columns.'">'; |
|
2370 | 2370 | if ($col == 2 && $i == 1) { |
2371 | 2371 | if ($right === true) { |
2372 | 2372 | $html .= '<div class="pull-right">'; |
@@ -2482,9 +2482,9 @@ discard block |
||
2482 | 2482 | $params['class'] = 'panel panel-default'; |
2483 | 2483 | $html = null; |
2484 | 2484 | if (!empty($title)) { |
2485 | - $html .= '<div class="panel-heading">'.$title.'</div>' . PHP_EOL; |
|
2485 | + $html .= '<div class="panel-heading">'.$title.'</div>'.PHP_EOL; |
|
2486 | 2486 | } |
2487 | - $html.= '<div class="panel-body">'.$content.'</div>' . PHP_EOL; |
|
2487 | + $html .= '<div class="panel-body">'.$content.'</div>'.PHP_EOL; |
|
2488 | 2488 | $html = Display::div($html, $params); |
2489 | 2489 | } |
2490 | 2490 | return $html; |
@@ -333,8 +333,12 @@ discard block |
||
333 | 333 | ) { |
334 | 334 | global $debug; |
335 | 335 | |
336 | - if ($debug) error_log('Called to update_event_exercice'); |
|
337 | - if ($debug) error_log('duration:' . $duration); |
|
336 | + if ($debug) { |
|
337 | + error_log('Called to update_event_exercice'); |
|
338 | + } |
|
339 | + if ($debug) { |
|
340 | + error_log('duration:' . $duration); |
|
341 | + } |
|
338 | 342 | |
339 | 343 | if ($exeid != '') { |
340 | 344 | /* |
@@ -386,8 +390,12 @@ discard block |
||
386 | 390 | WHERE exe_id = '".Database::escape_string($exeid)."'"; |
387 | 391 | $res = Database::query($sql); |
388 | 392 | |
389 | - if ($debug) error_log('update_event_exercice called'); |
|
390 | - if ($debug) error_log("$sql"); |
|
393 | + if ($debug) { |
|
394 | + error_log('update_event_exercice called'); |
|
395 | + } |
|
396 | + if ($debug) { |
|
397 | + error_log("$sql"); |
|
398 | + } |
|
391 | 399 | |
392 | 400 | //Deleting control time session track |
393 | 401 | //ExerciseLib::exercise_time_control_delete($exo_id); |
@@ -77,9 +77,7 @@ discard block |
||
77 | 77 | Database::query($sql); |
78 | 78 | |
79 | 79 | // Auto subscribe |
80 | - $user_status = $userInfo['status'] == SESSIONADMIN ? 'sessionadmin' : |
|
81 | - $userInfo['status'] == COURSEMANAGER ? 'teacher' : |
|
82 | - $userInfo['status'] == DRH ? 'DRH' : 'student'; |
|
80 | + $user_status = $userInfo['status'] == SESSIONADMIN ? 'sessionadmin' : $userInfo['status'] == COURSEMANAGER ? 'teacher' : $userInfo['status'] == DRH ? 'DRH' : 'student'; |
|
83 | 81 | $autoSubscribe = api_get_setting($user_status.'_autosubscribe'); |
84 | 82 | if ($autoSubscribe) { |
85 | 83 | $autoSubscribe = explode('|', $autoSubscribe); |
@@ -336,7 +334,7 @@ discard block |
||
336 | 334 | global $debug; |
337 | 335 | |
338 | 336 | if ($debug) error_log('Called to update_event_exercice'); |
339 | - if ($debug) error_log('duration:' . $duration); |
|
337 | + if ($debug) error_log('duration:'.$duration); |
|
340 | 338 | |
341 | 339 | if ($exeid != '') { |
342 | 340 | /* |
@@ -384,7 +382,7 @@ discard block |
||
384 | 382 | status = '".$status."', |
385 | 383 | questions_to_check = '".$remind_list."', |
386 | 384 | data_tracking = '".implode(',', $question_list)."', |
387 | - user_ip = '" . Database::escape_string(api_get_real_ip()) . "' |
|
385 | + user_ip = '" . Database::escape_string(api_get_real_ip())."' |
|
388 | 386 | WHERE exe_id = '".Database::escape_string($exeid)."'"; |
389 | 387 | $res = Database::query($sql); |
390 | 388 | |
@@ -1055,15 +1053,15 @@ discard block |
||
1055 | 1053 | |
1056 | 1054 | if (!empty($exe_list) && is_array($exe_list) && count($exe_list) > 0) { |
1057 | 1055 | $sql = "DELETE FROM $track_e_exercises |
1058 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1056 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1059 | 1057 | Database::query($sql); |
1060 | 1058 | |
1061 | 1059 | $sql = "DELETE FROM $track_attempts |
1062 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1060 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1063 | 1061 | Database::query($sql); |
1064 | 1062 | |
1065 | 1063 | $sql = "DELETE FROM $recording_table |
1066 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1064 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1067 | 1065 | Database::query($sql); |
1068 | 1066 | } |
1069 | 1067 | |
@@ -1105,7 +1103,7 @@ discard block |
||
1105 | 1103 | Event::addEvent( |
1106 | 1104 | LOG_EXERCISE_RESULT_DELETE, |
1107 | 1105 | LOG_EXERCISE_AND_USER_ID, |
1108 | - $exercise_id . '-' . $user_id, |
|
1106 | + $exercise_id.'-'.$user_id, |
|
1109 | 1107 | null, |
1110 | 1108 | null, |
1111 | 1109 | $course_id, |
@@ -1705,7 +1703,7 @@ discard block |
||
1705 | 1703 | Event::addEvent( |
1706 | 1704 | LOG_QUESTION_RESULT_DELETE, |
1707 | 1705 | LOG_EXERCISE_ATTEMPT_QUESTION_ID, |
1708 | - $exe_id . '-' . $question_id, |
|
1706 | + $exe_id.'-'.$question_id, |
|
1709 | 1707 | null, |
1710 | 1708 | null, |
1711 | 1709 | $courseId, |
@@ -1741,7 +1739,7 @@ discard block |
||
1741 | 1739 | Event::addEvent( |
1742 | 1740 | LOG_QUESTION_RESULT_DELETE, |
1743 | 1741 | LOG_EXERCISE_ATTEMPT_QUESTION_ID, |
1744 | - $exe_id . '-' . $question_id, |
|
1742 | + $exe_id.'-'.$question_id, |
|
1745 | 1743 | null, |
1746 | 1744 | null, |
1747 | 1745 | $courseId, |
@@ -239,7 +239,7 @@ discard block |
||
239 | 239 | } |
240 | 240 | |
241 | 241 | /** |
242 | - * @param doc_id id of document (id in mainDb.document table) |
|
242 | + * @param doc_id string of document (id in mainDb.document table) |
|
243 | 243 | * @author Sebastien Piraux <[email protected]> |
244 | 244 | * @desc Record information for upload event |
245 | 245 | * used in the works tool to record informations when |
@@ -313,6 +313,9 @@ discard block |
||
313 | 313 | * @param int session_id |
314 | 314 | * @param int learnpath_id (id of the learnpath) |
315 | 315 | * @param int learnpath_item_id (id of the learnpath_item) |
316 | + * @param integer $score |
|
317 | + * @param integer $weighting |
|
318 | + * @param integer $session_id |
|
316 | 319 | * |
317 | 320 | * @author Sebastien Piraux <[email protected]> |
318 | 321 | * @author Julio Montoya Armas <[email protected]> Reworked 2010 |
@@ -414,7 +417,9 @@ discard block |
||
414 | 417 | * @param integer Session ID (from the session table). Default value of null means "get from context". |
415 | 418 | * @param integer Learnpath ID (from c_lp table). Default value of null means "get from context". |
416 | 419 | * @param integer Learnpath item ID (from the c_lp_item table). Default value of null means "get from context". |
417 | - * @return boolean Result of the insert query |
|
420 | + * @param integer $exe_id |
|
421 | + * @param integer $position |
|
422 | + * @return false|string Result of the insert query |
|
418 | 423 | */ |
419 | 424 | public static function saveQuestionAttempt( |
420 | 425 | $score, |
@@ -588,6 +593,8 @@ discard block |
||
588 | 593 | * @param int Whether this answer is correct (1) or not (0) |
589 | 594 | * @param string Coordinates of this point (e.g. 123;324) |
590 | 595 | * @param bool update results? |
596 | + * @param integer $exe_id |
|
597 | + * @param integer $correct |
|
591 | 598 | * @return boolean Result of the insert query |
592 | 599 | * @uses Course code and user_id from global scope $_cid and $_user |
593 | 600 | */ |
@@ -800,11 +807,11 @@ discard block |
||
800 | 807 | /** |
801 | 808 | * Save the new message for one event and for one language |
802 | 809 | * |
803 | - * @param string $eventName |
|
810 | + * @param string $event_name |
|
804 | 811 | * @param array $users |
805 | 812 | * @param string $message |
806 | 813 | * @param string $subject |
807 | - * @param string $eventMessageLanguage |
|
814 | + * @param string $event_message_language |
|
808 | 815 | * @param int $activated |
809 | 816 | */ |
810 | 817 | public static function save_event_type_message($event_name, $users, $message, $subject, $event_message_language, $activated) |
@@ -938,10 +945,10 @@ discard block |
||
938 | 945 | } |
939 | 946 | |
940 | 947 | /** |
941 | - * @param $user_id |
|
948 | + * @param integer $user_id |
|
942 | 949 | * @param $exerciseId |
943 | - * @param $lp_id |
|
944 | - * @param $lp_item_id |
|
950 | + * @param integer $lp_id |
|
951 | + * @param integer $lp_item_id |
|
945 | 952 | * @return int |
946 | 953 | */ |
947 | 954 | public static function get_attempt_count_not_finished($user_id, $exerciseId, $lp_id, $lp_item_id) |
@@ -1118,6 +1125,7 @@ discard block |
||
1118 | 1125 | * @param int exercise id |
1119 | 1126 | * @param int $courseId |
1120 | 1127 | * @param int session id |
1128 | + * @param integer $user_id |
|
1121 | 1129 | * @return array with the results |
1122 | 1130 | * |
1123 | 1131 | */ |
@@ -1213,6 +1221,7 @@ discard block |
||
1213 | 1221 | * @param int exercise id |
1214 | 1222 | * @param int $courseId |
1215 | 1223 | * @param int session id |
1224 | + * @param integer $user_id |
|
1216 | 1225 | * @return array with the results |
1217 | 1226 | * |
1218 | 1227 | */ |
@@ -1301,6 +1310,8 @@ discard block |
||
1301 | 1310 | * @param int lp id |
1302 | 1311 | * @param int lp item id |
1303 | 1312 | * @param string order asc or desc |
1313 | + * @param integer $courseId |
|
1314 | + * @param string $order |
|
1304 | 1315 | * @return array with the results |
1305 | 1316 | * |
1306 | 1317 | */ |
@@ -1367,7 +1378,7 @@ discard block |
||
1367 | 1378 | * @param int exercise id |
1368 | 1379 | * @param int $courseId |
1369 | 1380 | * @param int session id |
1370 | - * @return array with the results |
|
1381 | + * @return string with the results |
|
1371 | 1382 | * |
1372 | 1383 | */ |
1373 | 1384 | public static function count_exercise_attempts_by_user($user_id, $exercise_id, $courseId, $session_id = 0) |
@@ -1538,7 +1549,7 @@ discard block |
||
1538 | 1549 | * @param int exercise id |
1539 | 1550 | * @param int course id |
1540 | 1551 | * @param int session id |
1541 | - * @return array with the results |
|
1552 | + * @return integer with the results |
|
1542 | 1553 | * |
1543 | 1554 | */ |
1544 | 1555 | public static function get_count_exercises_attempted_by_course($courseId, $session_id = 0) |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | |
177 | 177 | $sql = "SELECT MAX(description_type) as MAX |
178 | 178 | FROM $tbl_course_description |
179 | - WHERE c_id = $course_id AND session_id='" . $this->session_id . "'"; |
|
179 | + WHERE c_id = $course_id AND session_id='".$this->session_id."'"; |
|
180 | 180 | $rs = Database::query($sql); |
181 | 181 | $max = Database::fetch_array($rs); |
182 | 182 | $description_type = $max['MAX'] + 1; |
@@ -211,7 +211,7 @@ discard block |
||
211 | 211 | 'session_id' => $this->session_id |
212 | 212 | ]; |
213 | 213 | |
214 | - $last_id = Database::insert($table, $params); |
|
214 | + $last_id = Database::insert($table, $params); |
|
215 | 215 | |
216 | 216 | if ($last_id > 0) { |
217 | 217 | $sql = "UPDATE $table SET id = iid WHERE iid = $last_id"; |
@@ -319,8 +319,8 @@ discard block |
||
319 | 319 | $sql = "DELETE FROM $tbl_course_description |
320 | 320 | WHERE |
321 | 321 | c_id = $course_id AND |
322 | - id = '" . intval($this->id) . "' AND |
|
323 | - session_id = '" . intval($this->session_id) . "'"; |
|
322 | + id = '".intval($this->id)."' AND |
|
323 | + session_id = '" . intval($this->session_id)."'"; |
|
324 | 324 | $result = Database::query($sql); |
325 | 325 | $affected_rows = Database::affected_rows($result); |
326 | 326 | if ($this->id > 0) { |
@@ -349,7 +349,7 @@ discard block |
||
349 | 349 | $course_id = api_get_course_int_id(); |
350 | 350 | |
351 | 351 | $sql = "SELECT id FROM $tbl_course_description |
352 | - WHERE c_id = $course_id AND description_type = '" . intval($description_type) . "'"; |
|
352 | + WHERE c_id = $course_id AND description_type = '".intval($description_type)."'"; |
|
353 | 353 | $rs = Database::query($sql); |
354 | 354 | $row = Database::fetch_array($rs); |
355 | 355 | $description_id = $row['id']; |
@@ -359,7 +359,7 @@ discard block |
||
359 | 359 | |
360 | 360 | /** |
361 | 361 | * Get description titles by default |
362 | - * @return array |
|
362 | + * @return string[] |
|
363 | 363 | */ |
364 | 364 | public function get_default_description_title() |
365 | 365 | { |
@@ -378,7 +378,7 @@ discard block |
||
378 | 378 | |
379 | 379 | /** |
380 | 380 | * Get description titles editable by default |
381 | - * @return array |
|
381 | + * @return boolean[] |
|
382 | 382 | */ |
383 | 383 | public function get_default_description_title_editable() |
384 | 384 | { |
@@ -397,7 +397,7 @@ discard block |
||
397 | 397 | |
398 | 398 | /** |
399 | 399 | * Get description icons by default |
400 | - * @return array |
|
400 | + * @return string[] |
|
401 | 401 | */ |
402 | 402 | public function get_default_description_icon() |
403 | 403 | { |
@@ -417,7 +417,7 @@ discard block |
||
417 | 417 | |
418 | 418 | /** |
419 | 419 | * Get questions by default for help |
420 | - * @return array |
|
420 | + * @return string[] |
|
421 | 421 | */ |
422 | 422 | public function get_default_question() |
423 | 423 | { |
@@ -436,7 +436,7 @@ discard block |
||
436 | 436 | |
437 | 437 | /** |
438 | 438 | * Get informations by default for help |
439 | - * @return array |
|
439 | + * @return string[] |
|
440 | 440 | */ |
441 | 441 | public function get_default_information() |
442 | 442 | { |
@@ -574,7 +574,7 @@ discard block |
||
574 | 574 | /** |
575 | 575 | * get progress of a description |
576 | 576 | * |
577 | - * @return int |
|
577 | + * @return string |
|
578 | 578 | */ |
579 | 579 | public function get_progress() |
580 | 580 | { |
@@ -33,11 +33,11 @@ discard block |
||
33 | 33 | } |
34 | 34 | |
35 | 35 | /** |
36 | - * This function is for editing profile info from profile_id. |
|
37 | - * @param int $profileId |
|
38 | - * @param string $name |
|
39 | - * @param string $description |
|
40 | - */ |
|
36 | + * This function is for editing profile info from profile_id. |
|
37 | + * @param int $profileId |
|
38 | + * @param string $name |
|
39 | + * @param string $description |
|
40 | + */ |
|
41 | 41 | public function updateProfileInfo($profileId, $name, $description) |
42 | 42 | { |
43 | 43 | $profileId = intval($profileId); |
@@ -128,9 +128,9 @@ discard block |
||
128 | 128 | } |
129 | 129 | |
130 | 130 | /** |
131 | - * This function is for getting profile info from profile_id. |
|
132 | - * @param int $profileId |
|
133 | - */ |
|
131 | + * This function is for getting profile info from profile_id. |
|
132 | + * @param int $profileId |
|
133 | + */ |
|
134 | 134 | |
135 | 135 | public function getProfileInfo($profileId) |
136 | 136 | { |
@@ -326,6 +326,7 @@ discard block |
||
326 | 326 | |
327 | 327 | /** |
328 | 328 | * Gets an element |
329 | + * @param integer $gradebook_id |
|
329 | 330 | */ |
330 | 331 | public function get_skill_info($skill_id, $gradebook_id) |
331 | 332 | { |
@@ -544,7 +545,7 @@ discard block |
||
544 | 545 | * Gets an element |
545 | 546 | * @param int $id |
546 | 547 | * |
547 | - * @return array|mixed |
|
548 | + * @return integer |
|
548 | 549 | */ |
549 | 550 | public function get($id) |
550 | 551 | { |
@@ -885,7 +886,7 @@ discard block |
||
885 | 886 | /** |
886 | 887 | * Get user's skills |
887 | 888 | * |
888 | - * @param int $userId User's id |
|
889 | + * @param integer $user_id User's id |
|
889 | 890 | * @param bool $get_skill_data |
890 | 891 | */ |
891 | 892 | public function get_user_skills($user_id, $get_skill_data = false) |
@@ -1090,7 +1091,7 @@ discard block |
||
1090 | 1091 | * @param bool return a flat array or not |
1091 | 1092 | * @param int depth of the skills |
1092 | 1093 | * @param bool filter status |
1093 | - * @return json |
|
1094 | + * @return string |
|
1094 | 1095 | */ |
1095 | 1096 | public function get_skills_tree_json($user_id = null, $skill_id = null, $return_flat_array = false, $main_depth = 2, $filter_status = false) |
1096 | 1097 | { |
@@ -803,7 +803,7 @@ discard block |
||
803 | 803 | $attributes = array( |
804 | 804 | 'skill_id' => $skill_id, |
805 | 805 | 'parent_id' => $parent_id, |
806 | - 'relation_type' => (isset($params['relation_type'])?$params['relation_type']:0), |
|
806 | + 'relation_type' => (isset($params['relation_type']) ? $params['relation_type'] : 0), |
|
807 | 807 | //'level' => $params['level'], |
808 | 808 | ); |
809 | 809 | $skill_rel_skill->save($attributes); |
@@ -1012,7 +1012,7 @@ discard block |
||
1012 | 1012 | $skills[$skill_id] = $skill_info; |
1013 | 1013 | // Uncomment code below to hide the searched skill |
1014 | 1014 | $skills[$skill_id]['data']['parent_id'] = $skill_info['extra']['parent_id']; |
1015 | - $skills[$skill_id]['parent_id'] = 1; |
|
1015 | + $skills[$skill_id]['parent_id'] = 1; |
|
1016 | 1016 | } |
1017 | 1017 | } |
1018 | 1018 | } |
@@ -1077,7 +1077,7 @@ discard block |
||
1077 | 1077 | $skill['data']['skill_has_gradebook'] = true; |
1078 | 1078 | } |
1079 | 1079 | $refs[$skill['id']] = &$skill; |
1080 | - $flat_array[$skill['id']] = &$skill; |
|
1080 | + $flat_array[$skill['id']] = &$skill; |
|
1081 | 1081 | } |
1082 | 1082 | |
1083 | 1083 | // Checking family value |
@@ -1098,7 +1098,7 @@ discard block |
||
1098 | 1098 | $refs['root']['children'][0] = $skills[1]; |
1099 | 1099 | $skills[$skill_id]['data']['family_id'] = 1; |
1100 | 1100 | $refs['root']['children'][0]['children'][0] = $skills[$skill_id]; |
1101 | - $flat_array[$skill_id] = $skills[$skill_id]; |
|
1101 | + $flat_array[$skill_id] = $skills[$skill_id]; |
|
1102 | 1102 | } else { |
1103 | 1103 | // Moving node to the children index of their parents |
1104 | 1104 | |
@@ -1107,7 +1107,7 @@ discard block |
||
1107 | 1107 | $skill['data']['family_id'] = $new_family_array[$skill['id']]; |
1108 | 1108 | } |
1109 | 1109 | $refs[$skill['parent_id']]['children'][] = &$skill; |
1110 | - $flat_array[$my_skill_id] = $skill; |
|
1110 | + $flat_array[$my_skill_id] = $skill; |
|
1111 | 1111 | } |
1112 | 1112 | } |
1113 | 1113 | |
@@ -1219,7 +1219,7 @@ discard block |
||
1219 | 1219 | INNER JOIN {$this->table_skill_rel_user} su |
1220 | 1220 | ON (s.id = su.skill_id) |
1221 | 1221 | WHERE user_id = $user_id"; |
1222 | - $result = Database::query($sql); |
|
1222 | + $result = Database::query($sql); |
|
1223 | 1223 | if (Database::num_rows($result)) { |
1224 | 1224 | $result = Database::fetch_row($result); |
1225 | 1225 | return $result[0]; |
@@ -1261,7 +1261,7 @@ discard block |
||
1261 | 1261 | */ |
1262 | 1262 | public function get_user_list_skill_ranking_count() |
1263 | 1263 | { |
1264 | - $sql = "SELECT count(*) FROM ( |
|
1264 | + $sql = "SELECT count(*) FROM ( |
|
1265 | 1265 | SELECT count(distinct 1) |
1266 | 1266 | FROM {$this->table} s |
1267 | 1267 | INNER JOIN {$this->table_skill_rel_user} su |
@@ -1316,7 +1316,7 @@ discard block |
||
1316 | 1316 | ON c.code = g.course_code |
1317 | 1317 | WHERE sg.skill_id = $skill_id |
1318 | 1318 | AND (g.session_id IS NULL OR g.session_id = 0)"; |
1319 | - $result = Database::query($sql); |
|
1319 | + $result = Database::query($sql); |
|
1320 | 1320 | |
1321 | 1321 | return Database::store_result($result, 'ASSOC'); |
1322 | 1322 | } |
@@ -1505,7 +1505,7 @@ discard block |
||
1505 | 1505 | WHERE sg.skill_id = $skillId |
1506 | 1506 | AND g.session_id > 0"; |
1507 | 1507 | |
1508 | - $result = Database::query($sql); |
|
1508 | + $result = Database::query($sql); |
|
1509 | 1509 | |
1510 | 1510 | return Database::store_result($result, 'ASSOC'); |
1511 | 1511 | } |
@@ -8,11 +8,11 @@ discard block |
||
8 | 8 | * Code |
9 | 9 | */ |
10 | 10 | require_once 'xapian.php'; |
11 | -require_once dirname(__FILE__) . '/../IndexableChunk.class.php'; |
|
11 | +require_once dirname(__FILE__).'/../IndexableChunk.class.php'; |
|
12 | 12 | //TODO: think another way without including specific fields here |
13 | -require_once api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php'; |
|
13 | +require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php'; |
|
14 | 14 | |
15 | -define('XAPIAN_DB', api_get_path(SYS_UPLOAD_PATH) . 'plugins/xapian/searchdb/'); |
|
15 | +define('XAPIAN_DB', api_get_path(SYS_UPLOAD_PATH).'plugins/xapian/searchdb/'); |
|
16 | 16 | |
17 | 17 | /** |
18 | 18 | * Queries the database. |
@@ -95,7 +95,7 @@ discard block |
||
95 | 95 | |
96 | 96 | // process each specific field prefix |
97 | 97 | foreach ($specific_fields as $specific_field) { |
98 | - $results[$count]['sf-' . $specific_field['code']] = xapian_get_doc_terms($document, $specific_field['code']); |
|
98 | + $results[$count]['sf-'.$specific_field['code']] = xapian_get_doc_terms($document, $specific_field['code']); |
|
99 | 99 | } |
100 | 100 | |
101 | 101 | // rest of data |
@@ -261,6 +261,6 @@ discard block |
||
261 | 261 | } else { |
262 | 262 | $message_error = get_lang('SearchOtherXapianError'); |
263 | 263 | } |
264 | - $display_message = get_lang('Error') . ' : ' . $message_error; |
|
264 | + $display_message = get_lang('Error').' : '.$message_error; |
|
265 | 265 | Display::display_error_message($display_message); |
266 | 266 | } |
@@ -239,6 +239,7 @@ |
||
239 | 239 | /** |
240 | 240 | * @author Isaac flores paz <[email protected]> |
241 | 241 | * @param String The xapian error message |
242 | + * @param string $xapian_error_message |
|
242 | 243 | * @return String The chamilo error message |
243 | 244 | */ |
244 | 245 | function display_xapian_error($xapian_error_message) { |