@@ -14,7 +14,7 @@ |
||
14 | 14 | if (empty($_SESSION[Toolbox::class])) { |
15 | 15 | $_SESSION[Toolbox::class] = Toolbox::fromConfiguration(CONFIG_FILE); |
16 | 16 | } |
17 | -$toolbox =& $_SESSION[Toolbox::class]; |
|
17 | +$toolbox = & $_SESSION[Toolbox::class]; |
|
18 | 18 | |
19 | 19 | /* set the Tool Consumer's instance URL, if present */ |
20 | 20 | if (empty($_SESSION[CANVAS_INSTANCE_URL]) && |
@@ -30,7 +30,7 @@ |
||
30 | 30 | ); |
31 | 31 | $list[] = "<a target=\"_parent\" href=\"{$_SESSION[CANVAS_INSTANCE_URL]}/courses/{$course['id']}\">{$course['name']}</a>"; |
32 | 32 | } |
33 | - $toolbox->smarty_addMessage($courses->count(). ' courses published', implode(', ', $list), NotificationMessage::GOOD); |
|
33 | + $toolbox->smarty_addMessage($courses->count() . ' courses published', implode(', ', $list), NotificationMessage::GOOD); |
|
34 | 34 | } catch (Exception $e) { |
35 | 35 | $toolbox->exceptionErrorMessage($e); |
36 | 36 | } |
@@ -61,8 +61,7 @@ |
||
61 | 61 | $sections = $toolbox->api_get("courses/{$course['id']}/sections"); |
62 | 62 | foreach ($sections as $section) { |
63 | 63 | $sis_course_id = (isset($parentCourses[$section['sis_section_id']]) ? |
64 | - $parentCourses[$section['sis_section_id']] : |
|
65 | - false |
|
64 | + $parentCourses[$section['sis_section_id']] : false |
|
66 | 65 | ); |
67 | 66 | if ($sis_course_id === false) { |
68 | 67 | $parentCourse = $course; |
@@ -6,7 +6,7 @@ |
||
6 | 6 | |
7 | 7 | $cache = new \Battis\HierarchicalSimpleCache($sql, basename(__DIR__)); |
8 | 8 | $cache->pushKey(basename(__FILE__, '.php')); |
9 | -$cache->setLifetime(60*60); |
|
9 | +$cache->setLifetime(60 * 60); |
|
10 | 10 | |
11 | 11 | define('STEP_INSTRUCTIONS', 1); |
12 | 12 | define('STEP_LISTING', 2); |
@@ -5,7 +5,7 @@ |
||
5 | 5 | use Battis\BootstrapSmarty\NotificationMessage; |
6 | 6 | |
7 | 7 | $cache = new \Battis\HierarchicalSimpleCache($sql, basename(__DIR__) . '/' . basename(__FILE__, '.php')); |
8 | -$cache->setLifetime(60*60); |
|
8 | +$cache->setLifetime(60 * 60); |
|
9 | 9 | |
10 | 10 | define('ENROLL', true); |
11 | 11 | // DELETE is the enrollment ID to be deleted (i.e. not a boolean value) |
@@ -50,7 +50,7 @@ |
||
50 | 50 | ); |
51 | 51 | } |
52 | 52 | } |
53 | - $cache->setCache('enrollments', $data, 15*60); |
|
53 | + $cache->setCache('enrollments', $data, 15 * 60); |
|
54 | 54 | } |
55 | 55 | $toolbox->smarty_assign('account', $_REQUEST['account']); |
56 | 56 | $toolbox->smarty_assign('term', $_REQUEST['term']); |
@@ -148,8 +148,7 @@ |
||
148 | 148 | $enrollment = $toolbox->api_post( |
149 | 149 | ( |
150 | 150 | $courseEnrollment ? |
151 | - "/courses/{$_REQUEST['course']}/enrollments" : |
|
152 | - "/sections/{$_REQUEST['section']}/enrollments" |
|
151 | + "/courses/{$_REQUEST['course']}/enrollments" : "/sections/{$_REQUEST['section']}/enrollments" |
|
153 | 152 | ), |
154 | 153 | array( |
155 | 154 | 'enrollment[user_id]' => $user['id'], |
@@ -136,7 +136,7 @@ |
||
136 | 136 | print_r($this->getAPI()->last_request, true) . |
137 | 137 | '</pre><p>Last Headers</p><pre>' . |
138 | 138 | print_r($this->getAPI()->last_headers, true) . |
139 | - '</pre><p>Error Message</p><pre>' . $e->getMessage() .'</pre>', |
|
139 | + '</pre><p>Error Message</p><pre>' . $e->getMessage() . '</pre>', |
|
140 | 140 | NotificationMessage::ERROR |
141 | 141 | ); |
142 | 142 | } |
@@ -8,8 +8,7 @@ discard block |
||
8 | 8 | |
9 | 9 | /* store any requested actions for future handling */ |
10 | 10 | $action = (empty($_REQUEST['action']) ? |
11 | - ACTION_UNSPECIFIED : |
|
12 | - strtolower($_REQUEST['action']) |
|
11 | + ACTION_UNSPECIFIED : strtolower($_REQUEST['action']) |
|
13 | 12 | ); |
14 | 13 | |
15 | 14 | /* action requests only come from outside the LTI! */ |
@@ -25,7 +24,7 @@ discard block |
||
25 | 24 | @session_unset(); |
26 | 25 | @session_start(); |
27 | 26 | @session_regenerate_id(true); |
28 | - $_SESSION[Toolbox::class] =& $toolbox; |
|
27 | + $_SESSION[Toolbox::class] = & $toolbox; |
|
29 | 28 | session_write_close(); |
30 | 29 | $toolbox->lti_authenticate(); |
31 | 30 | exit; |
@@ -39,8 +38,7 @@ discard block |
||
39 | 38 | /* if not authenticated, default to showing credentials */ |
40 | 39 | } else { |
41 | 40 | $action = (empty($action) ? |
42 | - ACTION_CONFIG : |
|
43 | - $action |
|
41 | + ACTION_CONFIG : $action |
|
44 | 42 | ); |
45 | 43 | } |
46 | 44 | |
@@ -49,7 +47,7 @@ discard block |
||
49 | 47 | /* reset cached install data from config file */ |
50 | 48 | case ACTION_INSTALL: |
51 | 49 | $_SESSION['toolbox'] = Toolbox::fromConfiguration(CONFIG_FILE, true); |
52 | - $toolbox =& $_SESSION['toolbox']; |
|
50 | + $toolbox = & $_SESSION['toolbox']; |
|
53 | 51 | |
54 | 52 | /* test to see if we can connect to the API */ |
55 | 53 | try { |