|
1 | 1 | <?php |
| 2 | + |
2 | 3 | /* For licensing terms, see /license.txt */ |
| 4 | + |
3 | 5 | /** |
4 | 6 | * This script gives information about a course. |
5 | 7 | * |
@@ -68,15 +70,15 @@ function get_course_usage($course, $session_id = 0) |
68 | 70 |
|
69 | 71 | $usage = []; |
70 | 72 | $conditionSession = ''; |
71 | | - if ($session_id !== '') { |
72 | | - $session_id = intval($session_id); |
| 73 | + if ('' !== $session_id) { |
| 74 | + $session_id = (int) $session_id; |
73 | 75 | $conditionSession = " AND session_id = '$session_id' "; |
74 | 76 | } |
75 | 77 |
|
76 | 78 | foreach ($tables as $tableInfo) { |
77 | 79 | $table = $tableInfo[0]; |
78 | 80 | $title = $tableInfo[1]; |
79 | | - $sql = "SELECT COUNT(*) count FROM $table |
| 81 | + $sql = "SELECT COUNT(*) count FROM $table |
80 | 82 | WHERE c_id = '$courseId' $conditionSession "; |
81 | 83 | $rs = Database::query($sql); |
82 | 84 | $row = Database::fetch_array($rs); |
@@ -128,12 +130,12 @@ function get_course_usage($course, $session_id = 0) |
128 | 130 | FROM $table_course_user cu, $table_user u"; |
129 | 131 | if (api_is_multiple_url_enabled()) { |
130 | 132 | $sql .= " INNER JOIN ".Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER)." url_rel_user |
131 | | - ON |
132 | | - u.user_id = url_rel_user.user_id AND |
| 133 | + ON |
| 134 | + u.id = url_rel_user.user_id AND |
133 | 135 | url_rel_user.access_url_id = ".api_get_current_access_url_id(); |
134 | 136 | } |
135 | 137 | $sql .= " WHERE |
136 | | - cu.user_id = u.user_id AND |
| 138 | + cu.user_id = u.id AND |
137 | 139 | cu.c_id = '".$courseId."' AND |
138 | 140 | cu.relation_type <> ".COURSE_RELATION_TYPE_RRHH; |
139 | 141 | $res = Database::query($sql); |
|
0 commit comments