@@ -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 { |