smtech / canvas-ics-sync

classes/UserAPIToken.php

Spacing +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
 				UserAPIToken_Exception::TOKEN_REQUIRED
 			);
 		}
-		if($this->consumerKey && $this->id && $this->sql) {
+		if ($this->consumerKey && $this->id && $this->sql) {
 			$_token = $this->sql->real_escape_string($token);
 			if (!$this->sql->query("UPDATE `" . self::USER_TOKENS_TABLE . "` set `token` = '$_token' WHERE `consumer_key` = '{$this->consumerKey}' AND `id` = '{$this->id}'")) {
 				throw new UserAPIToken_Exception(

classes/CanvasAPIviaLTI_Installer.php

Spacing +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
 						}
 						
 						$htaccessFile = __DIR__ . '/.htaccess';
-						if(!file_put_contents($htaccessFile, "AuthType Basic\nAuthName \"{$secrets->app->name} Admin\"\nAuthUserFile $htpasswdFile\nRequire valid-user\n")) {
+						if (!file_put_contents($htaccessFile, "AuthType Basic\nAuthName \"{$secrets->app->name} Admin\"\nAuthUserFile $htpasswdFile\nRequire valid-user\n")) {
 							throw new CanvasAPIviaLTI_Installer_Exception(
 								"Failed to create $htaccessFile",
 								CanvasAPIviaLTI_Installer_Exception::HTACCESS_FILE
@@ -154,7 +154,7 @@ 
 		} elseif (file_exists($ltiSchema)) {
 			$queries = explode(";", file_get_contents($ltiSchema));
 			$created = true;
-			foreach($queries as $query) {
+			foreach ($queries as $query) {
 				if (!empty(trim($query))) {
 					if (!$sql->query($query)) {
 						throw new CanvasAPIviaLTI_Installer_Exception(

sync.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 			`synced` ASC
 ");
 
-while($schedule = $schedulesResponse->fetch_assoc()) {
+while ($schedule = $schedulesResponse->fetch_assoc()) {
 	$calendarResponse = $sql->query("
 		SELECT *
 			FROM `calendars`

app.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 $smarty->assign('category', '');
 $smarty->assign('formAction', $metadata['APP_URL'] . '/import.php');
 $smarty->assign('formHidden', array(
-	'canvas_url' => $_SESSION['canvasInstanceUrl']. '/courses/' . $_SESSION['toolProvider']->user->getResourceLink()->settings['custom_canvas_course_id']
+	'canvas_url' => $_SESSION['canvasInstanceUrl'] . '/courses/' . $_SESSION['toolProvider']->user->getResourceLink()->settings['custom_canvas_course_id']
 ));
 $smarty->display('course.tpl');
 

admin/install.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 	
 /* ...otherwise, let's start with the SECRETS_FILE */
 } else {
-	if(!file_exists(SECRETS_FILE)) {
+	if (!file_exists(SECRETS_FILE)) {
 		if (isset($_REQUEST['step']) && $_REQUEST['step'] == CanvasAPIviaLTI_Installer::SECRETS_ENTERED_STEP) {
 			CanvasAPIviaLTI_Installer::createSecretsFile(CanvasAPIviaLTI_Installer::SECRETS_ENTERED_STEP);
 		} else {

admin/consumers.php

Braces +3 added lines, -2 removed lines

@@ -47,10 +47,11 @@
 /* look up consumer to edit, if requested */
 } elseif (isset($_REQUEST['consumer_key'])) {
 	$consumer = new LTI_Tool_Consumer($_REQUEST['consumer_key'], LTI_Data_Connector::getDataConnector($sql));
-	if (isset($_REQUEST['action']))
-		switch ($_REQUEST['action']) {
+	if (isset($_REQUEST['action'])) {
+			switch ($_REQUEST['action']) {
 			case 'delete': {
 				$consumer->delete();
+	}
 				break;
 			}
 			case 'select': {

purge.php

Spacing +2 added lines, -2 removed lines

@@ -15,14 +15,14 @@
 		)
 	);
 	do {
-		foreach($events as $event) {
+		foreach ($events as $event) {
 			$api->delete("calendar_events/{$event['id']}",
 				array(
 					'cancel_reason' => $metadata['APP_NAME'] . " course_url={$_REQUEST['course_url']}"
 				)
 			);
 		}
-	} while($events = $eventsApi->nextPage());
+	} while ($events = $eventsApi->nextPage());
 	
 	$smarty->assign('content', 'Calendar purged.');
 } else {

common-app.inc.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 $FIELD_MAP = array(
 	'calendar_event[title]' => 'SUMMARY',
 	'calendar_event[description]' => 'DESCRIPTION',
-	'calendar_event[start_at]' => array (
+	'calendar_event[start_at]' => array(
 		0 => 'X-CURRENT-DTSTART',
 		1 => 'DTSTART'
 	),

import.php

Spacing +4 added lines, -4 removed lines

@@ -6,7 +6,7 @@ 
 	$_REQUEST['canvas_url'] = $argv[2];
 	$_REQUEST['schedule'] = $argv[3];
 
-	define ('IGNORE_LTI', true);
+	define('IGNORE_LTI', true);
 }
 
 require_once 'common.inc.php';
@@ -99,7 +99,7 @@ 
 
 	if ($canvasContext = getCanvasContext($_REQUEST['canvas_url'])) {
 		/* check ICS feed to be sure it exists */
-		if(urlExists($_REQUEST['cal'])) {
+		if (urlExists($_REQUEST['cal'])) {
 			/* look up the canvas object -- mostly to make sure that it exists! */
 			if ($canvasObject = $api->get($canvasContext['verification_url'])) {
 
@@ -185,10 +185,10 @@ 
 				// TODO:0 the best window for syncing would be the term of the course in question, right? issue:12
 				// TODO:0 Arbitrarily selecting events in for a year on either side of today's date, probably a better system? issue:12
 				foreach ($ics->selectComponents(
-					date('Y')-1, // startYear
+					date('Y') - 1, // startYear
 					date('m'), // startMonth
 					date('d'), // startDay
-					date('Y')+1, // endYEar
+					date('Y') + 1, // endYEar
 					date('m'), // endMonth
 					date('d'), // endDay
 					'vevent', // cType

Braces +6 added lines, -2 removed lines

@@ -305,7 +305,9 @@ 
 							), false) . '</pre>',
 							NotificationMessage::ERROR
 						);
-						if (php_sapi_name() != 'cli') $smarty->display('page.tpl');
+						if (php_sapi_name() != 'cli') {
+						    $smarty->display('page.tpl');
+						}
 						exit;
 					}
 					$sql->query("
@@ -421,7 +423,9 @@ 
 			'The Canvas URL you submitted could not be parsed.<pre>' . $_REQUEST['canvas_url'] . '</pre>',
 			NotificationMessage::ERROR
 		);
-		if (php_sapi_name() != 'cli') $smarty->display('page.tpl');
+		if (php_sapi_name() != 'cli') {
+		    $smarty->display('page.tpl');
+		}
 		exit;
 	}
 }