@@ -13,8 +13,7 @@ discard block |
||
| 13 | 13 | |
| 14 | 14 | /* store any requested actions for future handling */ |
| 15 | 15 | $action = (empty($_REQUEST['action']) ? |
| 16 | - ACTION_UNSPECIFIED : |
|
| 17 | - strtolower($_REQUEST['action']) |
|
| 16 | + ACTION_UNSPECIFIED : strtolower($_REQUEST['action']) |
|
| 18 | 17 | ); |
| 19 | 18 | |
| 20 | 19 | /* action requests only come from outside the LTI! */ |
@@ -49,8 +48,7 @@ discard block |
||
| 49 | 48 | /* if not authenticated, default to showing credentials */ |
| 50 | 49 | } else { |
| 51 | 50 | $action = (empty($action) ? |
| 52 | - ACTION_CONFIG : |
|
| 53 | - $action |
|
| 51 | + ACTION_CONFIG : $action |
|
| 54 | 52 | ); |
| 55 | 53 | } |
| 56 | 54 | |
@@ -59,7 +57,7 @@ discard block |
||
| 59 | 57 | /* reset cached install data from config file */ |
| 60 | 58 | case ACTION_INSTALL: |
| 61 | 59 | $_SESSION['toolbox'] = Toolbox::fromConfiguration(CONFIG_FILE, true); |
| 62 | - $toolbox =& $_SESSION['toolbox']; |
|
| 60 | + $toolbox = & $_SESSION['toolbox']; |
|
| 63 | 61 | |
| 64 | 62 | /* test to see if we can connect to the API */ |
| 65 | 63 | try { |
@@ -13,9 +13,9 @@ |
||
| 13 | 13 | |
| 14 | 14 | /* prepare the toolbox */ |
| 15 | 15 | if (empty($_SESSION[Toolbox::class])) { |
| 16 | - $_SESSION[Toolbox::class] =& Toolbox::fromConfiguration(CONFIG_FILE); |
|
| 16 | + $_SESSION[Toolbox::class] = & Toolbox::fromConfiguration(CONFIG_FILE); |
|
| 17 | 17 | } |
| 18 | -$toolbox =& $_SESSION[Toolbox::class]; |
|
| 18 | +$toolbox = & $_SESSION[Toolbox::class]; |
|
| 19 | 19 | $toolbox->smarty_prependTemplateDir(__DIR__ . '/templates', basename(__DIR__)); |
| 20 | 20 | $toolbox->smarty_assign([ |
| 21 | 21 | 'category' => DataUtilities::titleCase(preg_replace('/[\-_]+/', ' ', basename(__DIR__))) |
@@ -186,10 +186,10 @@ |
||
| 186 | 186 | // TODO:0 the best window for syncing would be the term of the course in question, right? issue:12 |
| 187 | 187 | // TODO:0 Arbitrarily selecting events in for a year on either side of today's date, probably a better system? issue:12 |
| 188 | 188 | foreach ($ics->selectComponents( |
| 189 | - date('Y')-1, // startYear |
|
| 189 | + date('Y') - 1, // startYear |
|
| 190 | 190 | date('m'), // startMonth |
| 191 | 191 | date('d'), // startDay |
| 192 | - date('Y')+1, // endYEar |
|
| 192 | + date('Y') + 1, // endYEar |
|
| 193 | 193 | date('m'), // endMonth |
| 194 | 194 | date('d'), // endDay |
| 195 | 195 | 'vevent', // cType |
@@ -424,7 +424,9 @@ |
||
| 424 | 424 | 'The Canvas URL you submitted could not be parsed.<pre>' . $_REQUEST['canvas_url'] . '</pre>', |
| 425 | 425 | NotificationMessage::ERROR |
| 426 | 426 | ); |
| 427 | - if (php_sapi_name() != 'cli') $smarty->display('page.tpl'); |
|
| 427 | + if (php_sapi_name() != 'cli') { |
|
| 428 | + $smarty->display('page.tpl'); |
|
| 429 | + } |
|
| 428 | 430 | exit; |
| 429 | 431 | } |
| 430 | 432 | } |
@@ -257,7 +257,7 @@ discard block |
||
| 257 | 257 | $this->logThrow(new Exception("Cannot sync calendars without a valid Canvas context"), $log); |
| 258 | 258 | } |
| 259 | 259 | |
| 260 | - if (!DataUtilities::URLexists($this>getFeedUrl())) { |
|
| 260 | + if (!DataUtilities::URLexists($this > getFeedUrl())) { |
|
| 261 | 261 | $this->logThrow(new Exception("Cannot sync calendars with a valid calendar feed"), $log); |
| 262 | 262 | } |
| 263 | 263 | |
@@ -285,10 +285,10 @@ discard block |
||
| 285 | 285 | * today's date, probably a better system? issue:12 |
| 286 | 286 | */ |
| 287 | 287 | foreach ($ics->selectComponents( |
| 288 | - date('Y')-1, // startYear |
|
| 288 | + date('Y') - 1, // startYear |
|
| 289 | 289 | date('m'), // startMonth |
| 290 | 290 | date('d'), // startDay |
| 291 | - date('Y')+1, // endYEar |
|
| 291 | + date('Y') + 1, // endYEar |
|
| 292 | 292 | date('m'), // endMonth |
| 293 | 293 | date('d'), // endDay |
| 294 | 294 | 'vevent', // cType |
@@ -33,5 +33,5 @@ |
||
| 33 | 33 | } |
| 34 | 34 | |
| 35 | 35 | $smarty->assign('ics', $ics); |
| 36 | -$smarty->assign('veventProperties', array('unique' => array('CLASS','CREATED','SUMMARY','DESCRIPTION','DTSTART','X-CURRENT-DTSTART','DTEND','X-CURRENT-DTEND','DURATION','GEO','LAST-MOD','LOCATION','ORGANIZER','PRIORITY','DTSTAMP','SEQ','STATUS','TRANSP','UID','URL','RECURID'),'multiple'=>array('ATTACH','ATTENDEE','CATEGORIES','COMMENT','CONTACT','EXDATE','EXRULE','RSTATUS','RELATED','RESOURCES','RDATE','RRULE','X-PROP'))); |
|
| 36 | +$smarty->assign('veventProperties', array('unique' => array('CLASS', 'CREATED', 'SUMMARY', 'DESCRIPTION', 'DTSTART', 'X-CURRENT-DTSTART', 'DTEND', 'X-CURRENT-DTEND', 'DURATION', 'GEO', 'LAST-MOD', 'LOCATION', 'ORGANIZER', 'PRIORITY', 'DTSTAMP', 'SEQ', 'STATUS', 'TRANSP', 'UID', 'URL', 'RECURID'), 'multiple'=>array('ATTACH', 'ATTENDEE', 'CATEGORIES', 'COMMENT', 'CONTACT', 'EXDATE', 'EXRULE', 'RSTATUS', 'RELATED', 'RESOURCES', 'RDATE', 'RRULE', 'X-PROP'))); |
|
| 37 | 37 | $smarty->display('visualize.tpl'); |