@@ -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;} |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | description AS note_comment, |
108 | 108 | session_id AS session_id |
109 | 109 | FROM $t_notebook |
110 | - WHERE c_id = $course_id AND notebook_id = '" . intval($notebook_id) . "' "; |
|
110 | + WHERE c_id = $course_id AND notebook_id = '".intval($notebook_id)."' "; |
|
111 | 111 | $result = Database::query($sql); |
112 | 112 | if (Database::num_rows($result) != 1) { |
113 | 113 | return array(); |
@@ -181,8 +181,8 @@ discard block |
||
181 | 181 | $sql = "DELETE FROM $t_notebook |
182 | 182 | WHERE |
183 | 183 | c_id = $course_id AND |
184 | - notebook_id='" . intval($notebook_id) . "' AND |
|
185 | - user_id = '" . api_get_user_id() . "'"; |
|
184 | + notebook_id='".intval($notebook_id)."' AND |
|
185 | + user_id = '" . api_get_user_id()."'"; |
|
186 | 186 | $result = Database::query($sql); |
187 | 187 | $affected_rows = Database::affected_rows($result); |
188 | 188 | if ($affected_rows != 1) { |
@@ -220,22 +220,22 @@ discard block |
||
220 | 220 | echo '<div class="actions">'; |
221 | 221 | if (!api_is_anonymous()) { |
222 | 222 | if (api_get_session_id() == 0) |
223 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=addnote">' . |
|
224 | - Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
|
223 | + echo '<a href="index.php?'.api_get_cidreq().'&action=addnote">'. |
|
224 | + Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32').'</a>'; |
|
225 | 225 | elseif (api_is_allowed_to_session_edit(false, true)) { |
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 | } |
229 | 229 | } else { |
230 | - echo '<a href="javascript:void(0)">' . Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32') . '</a>'; |
|
230 | + echo '<a href="javascript:void(0)">'.Display::return_icon('new_note.png', get_lang('NoteAddNew'), '', '32').'</a>'; |
|
231 | 231 | } |
232 | 232 | |
233 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=creation_date&direction=' . $link_sort_direction . '">' . |
|
234 | - Display::return_icon('notes_order_by_date_new.png', get_lang('OrderByCreationDate'), '', '32') . '</a>'; |
|
235 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=update_date&direction=' . $link_sort_direction . '">' . |
|
236 | - Display::return_icon('notes_order_by_date_mod.png', get_lang('OrderByModificationDate'), '', '32') . '</a>'; |
|
237 | - echo '<a href="index.php?' . api_get_cidreq() . '&action=changeview&view=title&direction=' . $link_sort_direction . '">' . |
|
238 | - Display::return_icon('notes_order_by_title.png', get_lang('OrderByTitle'), '', '32') . '</a>'; |
|
233 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=creation_date&direction='.$link_sort_direction.'">'. |
|
234 | + Display::return_icon('notes_order_by_date_new.png', get_lang('OrderByCreationDate'), '', '32').'</a>'; |
|
235 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=update_date&direction='.$link_sort_direction.'">'. |
|
236 | + Display::return_icon('notes_order_by_date_mod.png', get_lang('OrderByModificationDate'), '', '32').'</a>'; |
|
237 | + echo '<a href="index.php?'.api_get_cidreq().'&action=changeview&view=title&direction='.$link_sort_direction.'">'. |
|
238 | + Display::return_icon('notes_order_by_title.png', get_lang('OrderByTitle'), '', '32').'</a>'; |
|
239 | 239 | echo '</div>'; |
240 | 240 | |
241 | 241 | if (!isset($_SESSION['notebook_view']) || !in_array($_SESSION['notebook_view'], array('creation_date', 'update_date', 'title'))) { |
@@ -246,9 +246,9 @@ discard block |
||
246 | 246 | $t_notebook = Database :: get_course_table(TABLE_NOTEBOOK); |
247 | 247 | $order_by = ""; |
248 | 248 | if ($_SESSION['notebook_view'] == 'creation_date' || $_SESSION['notebook_view'] == 'update_date') { |
249 | - $order_by = " ORDER BY " . $_SESSION['notebook_view'] . " $sort_direction "; |
|
249 | + $order_by = " ORDER BY ".$_SESSION['notebook_view']." $sort_direction "; |
|
250 | 250 | } else { |
251 | - $order_by = " ORDER BY " . $_SESSION['notebook_view'] . " $sort_direction "; |
|
251 | + $order_by = " ORDER BY ".$_SESSION['notebook_view']." $sort_direction "; |
|
252 | 252 | } |
253 | 253 | |
254 | 254 | //condition for the session |
@@ -261,7 +261,7 @@ discard block |
||
261 | 261 | $sql = "SELECT * FROM $t_notebook |
262 | 262 | WHERE |
263 | 263 | c_id = $course_id AND |
264 | - user_id = '" . api_get_user_id() . "' |
|
264 | + user_id = '".api_get_user_id()."' |
|
265 | 265 | $condition_session |
266 | 266 | $cond_extra $order_by |
267 | 267 | "; |
@@ -274,18 +274,18 @@ discard block |
||
274 | 274 | |
275 | 275 | $updateValue = ''; |
276 | 276 | if ($row['update_date'] <> $row['creation_date']) { |
277 | - $updateValue = ', ' . get_lang('UpdateDate') . ': ' . date_to_str_ago($update_date) . ' <span class="dropbox_date">' . $update_date . '</span>'; |
|
277 | + $updateValue = ', '.get_lang('UpdateDate').': '.date_to_str_ago($update_date).' <span class="dropbox_date">'.$update_date.'</span>'; |
|
278 | 278 | } |
279 | 279 | |
280 | - $actions = '<a href="' . api_get_self() . '?action=editnote¬ebook_id=' . $row['notebook_id'] . '">' . |
|
281 | - Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL) . '</a>'; |
|
282 | - $actions .= '<a href="' . api_get_self() . '?action=deletenote¬ebook_id=' . $row['notebook_id'] . '" onclick="return confirmation(\'' . $row['title'] . '\');">' . |
|
283 | - Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL) . '</a>'; |
|
280 | + $actions = '<a href="'.api_get_self().'?action=editnote¬ebook_id='.$row['notebook_id'].'">'. |
|
281 | + Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL).'</a>'; |
|
282 | + $actions .= '<a href="'.api_get_self().'?action=deletenote¬ebook_id='.$row['notebook_id'].'" onclick="return confirmation(\''.$row['title'].'\');">'. |
|
283 | + Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>'; |
|
284 | 284 | |
285 | 285 | echo Display::panel( |
286 | 286 | $row['description'], |
287 | - $row['title'] . $session_img.' <div class="pull-right">'.$actions.'</div>', |
|
288 | - get_lang('CreationDate') . ': ' . date_to_str_ago($creation_date) . ' <span class="dropbox_date">' . $creation_date . $updateValue."</span>" |
|
287 | + $row['title'].$session_img.' <div class="pull-right">'.$actions.'</div>', |
|
288 | + get_lang('CreationDate').': '.date_to_str_ago($creation_date).' <span class="dropbox_date">'.$creation_date.$updateValue."</span>" |
|
289 | 289 | ); |
290 | 290 | } |
291 | 291 | } |
@@ -44,7 +44,7 @@ discard block |
||
44 | 44 | Database::insert($legal_table, $params); |
45 | 45 | |
46 | 46 | return true; |
47 | - } elseif($last['type'] != $type && $language==$last['language_id']) { |
|
47 | + } elseif ($last['type'] != $type && $language == $last['language_id']) { |
|
48 | 48 | //update |
49 | 49 | $id = $last['legal_id']; |
50 | 50 | $params = [ |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | public static function get_last_condition_version($language) |
78 | 78 | { |
79 | 79 | $legal_conditions_table = Database::get_main_table(TABLE_MAIN_LEGAL); |
80 | - $language= Database::escape_string($language); |
|
80 | + $language = Database::escape_string($language); |
|
81 | 81 | $sql = "SELECT version FROM $legal_conditions_table |
82 | 82 | WHERE language_id = '".$language."' |
83 | 83 | ORDER BY legal_id DESC LIMIT 1 "; |
@@ -97,10 +97,10 @@ discard block |
||
97 | 97 | * @param int $language language id |
98 | 98 | * @return array all the info of a Term and condition |
99 | 99 | */ |
100 | - public static function get_last_condition ($language) |
|
100 | + public static function get_last_condition($language) |
|
101 | 101 | { |
102 | 102 | $legal_conditions_table = Database::get_main_table(TABLE_MAIN_LEGAL); |
103 | - $language= Database::escape_string($language); |
|
103 | + $language = Database::escape_string($language); |
|
104 | 104 | $sql = "SELECT * FROM $legal_conditions_table |
105 | 105 | WHERE language_id = '".$language."' |
106 | 106 | ORDER BY version DESC |
@@ -124,9 +124,9 @@ discard block |
||
124 | 124 | ORDER BY version DESC |
125 | 125 | LIMIT 1 "; |
126 | 126 | $result = Database::query($sql); |
127 | - if (Database::num_rows($result)>0){ |
|
127 | + if (Database::num_rows($result) > 0) { |
|
128 | 128 | $version = Database::fetch_array($result); |
129 | - $version = explode(':',$version[0]); |
|
129 | + $version = explode(':', $version[0]); |
|
130 | 130 | |
131 | 131 | return $version[0]; |
132 | 132 | } else { |
@@ -153,7 +153,7 @@ discard block |
||
153 | 153 | break; |
154 | 154 | // Page link |
155 | 155 | case 1: |
156 | - $preview ='<fieldset> |
|
156 | + $preview = '<fieldset> |
|
157 | 157 | <legend>'.get_lang('TermsAndConditions').'</legend>'; |
158 | 158 | $preview .= '<div id="legal-accept-wrapper" class="form-item"> |
159 | 159 | <label class="option" for="legal-accept"> |
@@ -185,25 +185,25 @@ discard block |
||
185 | 185 | $number_of_items = intval($number_of_items); |
186 | 186 | $column = intval($column); |
187 | 187 | |
188 | - $sql = "SELECT version, original_name as language, content, changes, type, FROM_UNIXTIME(date) |
|
188 | + $sql = "SELECT version, original_name as language, content, changes, type, FROM_UNIXTIME(date) |
|
189 | 189 | FROM $legal_conditions_table inner join $lang_table l on(language_id = l.id) "; |
190 | 190 | $sql .= "ORDER BY language, version ASC "; |
191 | 191 | $sql .= "LIMIT $from, $number_of_items "; |
192 | 192 | |
193 | 193 | $result = Database::query($sql); |
194 | - $legals = array (); |
|
195 | - $versions = array (); |
|
194 | + $legals = array(); |
|
195 | + $versions = array(); |
|
196 | 196 | while ($legal = Database::fetch_array($result)) { |
197 | 197 | // max 2000 chars |
198 | 198 | //echo strlen($legal[1]); echo '<br>'; |
199 | - $versions[]=$legal[0]; |
|
200 | - $languages[]=$legal[1]; |
|
201 | - if (strlen($legal[2])>2000) |
|
202 | - $legal[2]= substr($legal[2],0,2000).' ... '; |
|
203 | - if ($legal[4]==0) |
|
204 | - $legal[4]= get_lang('HTMLText'); |
|
205 | - elseif($legal[4]==1) |
|
206 | - $legal[4]=get_lang('PageLink'); |
|
199 | + $versions[] = $legal[0]; |
|
200 | + $languages[] = $legal[1]; |
|
201 | + if (strlen($legal[2]) > 2000) |
|
202 | + $legal[2] = substr($legal[2], 0, 2000).' ... '; |
|
203 | + if ($legal[4] == 0) |
|
204 | + $legal[4] = get_lang('HTMLText'); |
|
205 | + elseif ($legal[4] == 1) |
|
206 | + $legal[4] = get_lang('PageLink'); |
|
207 | 207 | $legals[] = $legal; |
208 | 208 | } |
209 | 209 | return $legals; |
@@ -220,7 +220,7 @@ discard block |
||
220 | 220 | FROM $legal_conditions_table |
221 | 221 | ORDER BY legal_id DESC "; |
222 | 222 | $result = Database::query($sql); |
223 | - $url = Database::fetch_array($result,'ASSOC'); |
|
223 | + $url = Database::fetch_array($result, 'ASSOC'); |
|
224 | 224 | $result = $url['count_result']; |
225 | 225 | |
226 | 226 | return $result; |
@@ -241,6 +241,6 @@ discard block |
||
241 | 241 | WHERE legal_id="'.$legal_id.'" AND language_id="'.$language_id.'"'; |
242 | 242 | $rs = Database::query($sql); |
243 | 243 | |
244 | - return Database::result($rs,0,'type'); |
|
244 | + return Database::result($rs, 0, 'type'); |
|
245 | 245 | } |
246 | 246 | } |
@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | $this->fields = $settings; |
61 | 61 | |
62 | 62 | global $language_files; |
63 | - $language_files[] = 'plugin_' . $this->get_name(); |
|
63 | + $language_files[] = 'plugin_'.$this->get_name(); |
|
64 | 64 | } |
65 | 65 | |
66 | 66 | /** |
@@ -266,7 +266,7 @@ discard block |
||
266 | 266 | { |
267 | 267 | $settings = $this->get_settings(); |
268 | 268 | foreach ($settings as $setting) { |
269 | - if ($setting['variable'] == ($this->get_name() . '_' . $name)) { |
|
269 | + if ($setting['variable'] == ($this->get_name().'_'.$name)) { |
|
270 | 270 | return $setting['selected_value']; |
271 | 271 | } |
272 | 272 | } |
@@ -321,7 +321,7 @@ discard block |
||
321 | 321 | |
322 | 322 | $interfaceLanguageId = api_get_language_id($language_interface); |
323 | 323 | $interfaceLanguageInfo = api_get_language_info($interfaceLanguageId); |
324 | - $languageParentId = (!empty($interfaceLanguageInfo['parent_id'])?intval($interfaceLanguageInfo['parent_id']):0); |
|
324 | + $languageParentId = (!empty($interfaceLanguageInfo['parent_id']) ? intval($interfaceLanguageInfo['parent_id']) : 0); |
|
325 | 325 | |
326 | 326 | //1. Loading english if exists |
327 | 327 | $english_path = $root.$plugin_name."/lang/english.php"; |
@@ -399,7 +399,7 @@ discard block |
||
399 | 399 | if (!empty($this->course_settings)) { |
400 | 400 | foreach ($this->course_settings as $setting) { |
401 | 401 | $variable = $setting['name']; |
402 | - $value =''; |
|
402 | + $value = ''; |
|
403 | 403 | if (isset($setting['init_value'])) { |
404 | 404 | $value = ($setting['init_value']); |
405 | 405 | } |
@@ -611,7 +611,7 @@ discard block |
||
611 | 611 | |
612 | 612 | //Avoid Tab Name Spaces |
613 | 613 | $tabNameNoSpaces = preg_replace('/\s+/', '', $tabName); |
614 | - $subkeytext = "Tabs" . $tabNameNoSpaces; |
|
614 | + $subkeytext = "Tabs".$tabNameNoSpaces; |
|
615 | 615 | |
616 | 616 | //Check if it is already added |
617 | 617 | $checkCondition = array( |
@@ -627,7 +627,7 @@ discard block |
||
627 | 627 | return false; |
628 | 628 | } |
629 | 629 | //End Check |
630 | - $subkey = 'custom_tab_' . $tabNum; |
|
630 | + $subkey = 'custom_tab_'.$tabNum; |
|
631 | 631 | $attributes = array( |
632 | 632 | 'variable' => 'show_tabs', |
633 | 633 | 'subkey' => $subkey, |
@@ -645,7 +645,7 @@ discard block |
||
645 | 645 | |
646 | 646 | //Save the id |
647 | 647 | $settings = $this->get_settings(); |
648 | - $setData = array ( |
|
648 | + $setData = array( |
|
649 | 649 | 'comment' => $subkey |
650 | 650 | ); |
651 | 651 | $whereCondition = array( |
@@ -686,7 +686,7 @@ discard block |
||
686 | 686 | $i = 1; |
687 | 687 | foreach ($tabs as $row) { |
688 | 688 | $attributes = array( |
689 | - 'subkey' => 'custom_tab_' . $i |
|
689 | + 'subkey' => 'custom_tab_'.$i |
|
690 | 690 | ); |
691 | 691 | $this->updateTab($row['subkey'], $attributes); |
692 | 692 | $i++; |
@@ -78,9 +78,7 @@ discard block |
||
78 | 78 | Database::query($sql); |
79 | 79 | |
80 | 80 | // Auto subscribe |
81 | - $user_status = $userInfo['status'] == SESSIONADMIN ? 'sessionadmin' : |
|
82 | - $userInfo['status'] == COURSEMANAGER ? 'teacher' : |
|
83 | - $userInfo['status'] == DRH ? 'DRH' : 'student'; |
|
81 | + $user_status = $userInfo['status'] == SESSIONADMIN ? 'sessionadmin' : $userInfo['status'] == COURSEMANAGER ? 'teacher' : $userInfo['status'] == DRH ? 'DRH' : 'student'; |
|
84 | 82 | $autoSubscribe = api_get_setting($user_status.'_autosubscribe'); |
85 | 83 | if ($autoSubscribe) { |
86 | 84 | $autoSubscribe = explode('|', $autoSubscribe); |
@@ -334,7 +332,7 @@ discard block |
||
334 | 332 | global $debug; |
335 | 333 | |
336 | 334 | if ($debug) error_log('Called to update_event_exercice'); |
337 | - if ($debug) error_log('duration:' . $duration); |
|
335 | + if ($debug) error_log('duration:'.$duration); |
|
338 | 336 | |
339 | 337 | if ($exeid != '') { |
340 | 338 | /* |
@@ -382,7 +380,7 @@ discard block |
||
382 | 380 | status = '".$status."', |
383 | 381 | questions_to_check = '".$remind_list."', |
384 | 382 | data_tracking = '".implode(',', $question_list)."', |
385 | - user_ip = '" . Database::escape_string(api_get_real_ip()) . "' |
|
383 | + user_ip = '" . Database::escape_string(api_get_real_ip())."' |
|
386 | 384 | WHERE exe_id = '".Database::escape_string($exeid)."'"; |
387 | 385 | $res = Database::query($sql); |
388 | 386 | |
@@ -1036,15 +1034,15 @@ discard block |
||
1036 | 1034 | |
1037 | 1035 | if (!empty($exe_list) && is_array($exe_list) && count($exe_list) > 0) { |
1038 | 1036 | $sql = "DELETE FROM $track_e_exercises |
1039 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1037 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1040 | 1038 | Database::query($sql); |
1041 | 1039 | |
1042 | 1040 | $sql = "DELETE FROM $track_attempts |
1043 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1041 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1044 | 1042 | Database::query($sql); |
1045 | 1043 | |
1046 | 1044 | $sql = "DELETE FROM $recording_table |
1047 | - WHERE exe_id IN (" . implode(',', $exe_list) . ")"; |
|
1045 | + WHERE exe_id IN (".implode(',', $exe_list).")"; |
|
1048 | 1046 | Database::query($sql); |
1049 | 1047 | } |
1050 | 1048 | |
@@ -1086,7 +1084,7 @@ discard block |
||
1086 | 1084 | Event::addEvent( |
1087 | 1085 | LOG_EXERCISE_RESULT_DELETE, |
1088 | 1086 | LOG_EXERCISE_AND_USER_ID, |
1089 | - $exercise_id . '-' . $user_id, |
|
1087 | + $exercise_id.'-'.$user_id, |
|
1090 | 1088 | null, |
1091 | 1089 | null, |
1092 | 1090 | $course_id, |
@@ -1458,7 +1456,7 @@ discard block |
||
1458 | 1456 | $res = Database::query($sql); |
1459 | 1457 | $list = array(); |
1460 | 1458 | while ($row = Database::fetch_array($res, 'ASSOC')) { |
1461 | - $list[$row['exe_id']] = $row; /* |
|
1459 | + $list[$row['exe_id']] = $row; /* |
|
1462 | 1460 | $sql = "SELECT * FROM $table_track_attempt WHERE exe_id = {$row['exe_id']}"; |
1463 | 1461 | $res_question = Database::query($sql); |
1464 | 1462 | while($row_q = Database::fetch_array($res_question,'ASSOC')) { |
@@ -1684,7 +1682,7 @@ discard block |
||
1684 | 1682 | Event::addEvent( |
1685 | 1683 | LOG_QUESTION_RESULT_DELETE, |
1686 | 1684 | LOG_EXERCISE_ATTEMPT_QUESTION_ID, |
1687 | - $exe_id . '-' . $question_id, |
|
1685 | + $exe_id.'-'.$question_id, |
|
1688 | 1686 | null, |
1689 | 1687 | null, |
1690 | 1688 | $courseId, |
@@ -1720,7 +1718,7 @@ discard block |
||
1720 | 1718 | Event::addEvent( |
1721 | 1719 | LOG_QUESTION_RESULT_DELETE, |
1722 | 1720 | LOG_EXERCISE_ATTEMPT_QUESTION_ID, |
1723 | - $exe_id . '-' . $question_id, |
|
1721 | + $exe_id.'-'.$question_id, |
|
1724 | 1722 | null, |
1725 | 1723 | null, |
1726 | 1724 | $courseId, |
@@ -44,17 +44,17 @@ |
||
44 | 44 | ' ORDER BY title, code '. |
45 | 45 | ' LIMIT 11'; |
46 | 46 | $rs = Database::query($sql); |
47 | - $i=0; |
|
47 | + $i = 0; |
|
48 | 48 | while ($course = Database :: fetch_array($rs)) { |
49 | 49 | $i++; |
50 | - if ($i<=10) { |
|
50 | + if ($i <= 10) { |
|
51 | 51 | $return .= '<a href="javascript: void(0);" onclick="javascript: add_user_to_url(\''.addslashes($course['code']).'\',\''.addslashes($course['title']).' ('.addslashes($course['code']).')'.'\')">'.$course['title'].' ('.$course['code'].')</a><br />'; |
52 | 52 | } else { |
53 | 53 | $return .= '...<br />'; |
54 | 54 | } |
55 | 55 | } |
56 | 56 | } |
57 | - $xajax_response -> addAssign('ajax_list_courses','innerHTML',api_utf8_encode($return)); |
|
57 | + $xajax_response -> addAssign('ajax_list_courses', 'innerHTML', api_utf8_encode($return)); |
|
58 | 58 | return $xajax_response; |
59 | 59 | } |
60 | 60 | } |
@@ -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']; |
@@ -734,7 +734,7 @@ discard block |
||
734 | 734 | { |
735 | 735 | $skill_rel_skill = new SkillRelSkill(); |
736 | 736 | $skills = $skill_rel_skill->get_direct_parents($skill_id, true); |
737 | - foreach($skills as &$skill) { |
|
737 | + foreach ($skills as &$skill) { |
|
738 | 738 | $skill['data'] = self::get($skill['skill_id']); |
739 | 739 | $skill_info2 = $skill_rel_skill->get_skill_info($skill['skill_id']); |
740 | 740 | $skill['data']['parent_id'] = $skill_info2['parent_id']; |
@@ -772,7 +772,7 @@ discard block |
||
772 | 772 | $attributes = array( |
773 | 773 | 'skill_id' => $skill_id, |
774 | 774 | 'parent_id' => $parent_id, |
775 | - 'relation_type' => (isset($params['relation_type'])?$params['relation_type']:0), |
|
775 | + 'relation_type' => (isset($params['relation_type']) ? $params['relation_type'] : 0), |
|
776 | 776 | //'level' => $params['level'], |
777 | 777 | ); |
778 | 778 | $skill_rel_skill->save($attributes); |
@@ -969,8 +969,8 @@ discard block |
||
969 | 969 | // 2nd node |
970 | 970 | $skills[$skill_id] = $skill_info; |
971 | 971 | // Uncomment code below to hide the searched skill |
972 | - $skills[$skill_id]['data']['parent_id'] = $skill_info['extra']['parent_id']; |
|
973 | - $skills[$skill_id]['parent_id'] = 1; |
|
972 | + $skills[$skill_id]['data']['parent_id'] = $skill_info['extra']['parent_id']; |
|
973 | + $skills[$skill_id]['parent_id'] = 1; |
|
974 | 974 | } |
975 | 975 | } |
976 | 976 | } |
@@ -1033,7 +1033,7 @@ discard block |
||
1033 | 1033 | $skill['data']['skill_has_gradebook'] = true; |
1034 | 1034 | } |
1035 | 1035 | $refs[$skill['id']] = &$skill; |
1036 | - $flat_array[$skill['id']] = &$skill; |
|
1036 | + $flat_array[$skill['id']] = &$skill; |
|
1037 | 1037 | } |
1038 | 1038 | |
1039 | 1039 | // Checking family value |
@@ -1054,7 +1054,7 @@ discard block |
||
1054 | 1054 | $refs['root']['children'][0] = $skills[1]; |
1055 | 1055 | $skills[$skill_id]['data']['family_id'] = 1; |
1056 | 1056 | $refs['root']['children'][0]['children'][0] = $skills[$skill_id]; |
1057 | - $flat_array[$skill_id] = $skills[$skill_id]; |
|
1057 | + $flat_array[$skill_id] = $skills[$skill_id]; |
|
1058 | 1058 | } else { |
1059 | 1059 | // Moving node to the children index of their parents |
1060 | 1060 | |
@@ -1063,7 +1063,7 @@ discard block |
||
1063 | 1063 | $skill['data']['family_id'] = $new_family_array[$skill['id']]; |
1064 | 1064 | } |
1065 | 1065 | $refs[$skill['parent_id']]['children'][] = &$skill; |
1066 | - $flat_array[$my_skill_id] = $skill; |
|
1066 | + $flat_array[$my_skill_id] = $skill; |
|
1067 | 1067 | } |
1068 | 1068 | } |
1069 | 1069 | |
@@ -1161,7 +1161,7 @@ discard block |
||
1161 | 1161 | INNER JOIN {$this->table_skill_rel_user} su |
1162 | 1162 | ON (s.id = su.skill_id) |
1163 | 1163 | WHERE user_id = $user_id"; |
1164 | - $result = Database::query($sql); |
|
1164 | + $result = Database::query($sql); |
|
1165 | 1165 | if (Database::num_rows($result)) { |
1166 | 1166 | $result = Database::fetch_row($result); |
1167 | 1167 | return $result[0]; |
@@ -1203,7 +1203,7 @@ discard block |
||
1203 | 1203 | */ |
1204 | 1204 | public function get_user_list_skill_ranking_count() |
1205 | 1205 | { |
1206 | - $sql = "SELECT count(*) FROM ( |
|
1206 | + $sql = "SELECT count(*) FROM ( |
|
1207 | 1207 | SELECT count(distinct 1) |
1208 | 1208 | FROM {$this->table} s |
1209 | 1209 | INNER JOIN {$this->table_skill_rel_user} su |
@@ -1256,7 +1256,7 @@ discard block |
||
1256 | 1256 | ON c.code = g.course_code |
1257 | 1257 | WHERE sg.skill_id = $skill_id |
1258 | 1258 | AND (g.session_id IS NULL OR g.session_id = 0)"; |
1259 | - $result = Database::query($sql); |
|
1259 | + $result = Database::query($sql); |
|
1260 | 1260 | |
1261 | 1261 | return Database::store_result($result, 'ASSOC'); |
1262 | 1262 | } |
@@ -1443,7 +1443,7 @@ discard block |
||
1443 | 1443 | WHERE sg.skill_id = $skillId |
1444 | 1444 | AND g.session_id > 0"; |
1445 | 1445 | |
1446 | - $result = Database::query($sql); |
|
1446 | + $result = Database::query($sql); |
|
1447 | 1447 | |
1448 | 1448 | return Database::store_result($result, 'ASSOC'); |
1449 | 1449 | } |
@@ -6,7 +6,7 @@ discard block |
||
6 | 6 | */ |
7 | 7 | |
8 | 8 | require_once 'xapian.php'; |
9 | -require_once dirname(__FILE__) . '/../IndexableChunk.class.php'; |
|
9 | +require_once dirname(__FILE__).'/../IndexableChunk.class.php'; |
|
10 | 10 | |
11 | 11 | /** |
12 | 12 | * Abstract helper class |
@@ -130,7 +130,7 @@ discard block |
||
130 | 130 | if (!empty($chunk->terms)) { |
131 | 131 | foreach ($chunk->terms as $term) { |
132 | 132 | /* FIXME: think of getting weight */ |
133 | - $doc->add_term($term['flag'] . $term['name'], 1); |
|
133 | + $doc->add_term($term['flag'].$term['name'], 1); |
|
134 | 134 | } |
135 | 135 | } |
136 | 136 | |
@@ -215,7 +215,7 @@ discard block |
||
215 | 215 | $doc->clear_terms(); |
216 | 216 | foreach ($terms as $term) { |
217 | 217 | //add directly |
218 | - $doc->add_term($prefix . $term, 1); |
|
218 | + $doc->add_term($prefix.$term, 1); |
|
219 | 219 | } |
220 | 220 | $this->db->replace_document($did, $doc); |
221 | 221 | $this->db->flush(); |
@@ -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 | } |