smtech / canvas-ics-sync

classes/CanvasAPIviaLTI_Installer.php

Doc Comments +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 	/**
 	 * Generate a SECRETS_FILE from user input.
 	 *
-	 * @param scalar $step optional Where are we in the SECRETS_FILE creation workflow? (defaults to SECRETS_NEEDED_STEP -- the beginning)
+	 * @param integer $step optional Where are we in the SECRETS_FILE creation workflow? (defaults to SECRETS_NEEDED_STEP -- the beginning)
 	 *
 	 * @throws CanvasAPIviaLTI_Installer_Exception If form submission does not contain all required MySQL credentals (host, username, password and database)
 	 * @throws CanvasAPIviaLTI_Installer_Exception If SECRETS_FILE cannot be created
@@ -261,7 +261,7 @@ 
 	/**
 	 * Obtain a Canvas API token, if needed.
 	 *
-	 * @param scalar $step optional Where are we in the API token negotiation workflow? (defaults to API_DECISION_NEEDED_STEP -- the beginning)
+	 * @param integer $step optional Where are we in the API token negotiation workflow? (defaults to API_DECISION_NEEDED_STEP -- the beginning)
 	 * @param boolean $skip optional Skip this step (defaults to FALSE)
 	 *
 	 * @throws CanvasAPIviaLTI_Installer_Exception If $step is not a pre-defined *_STEP constant

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(

classes/UserAPIToken.php

Doc Comments +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
 	}
 	
 	/**
-	 * @return string|boolean The API access token for this user, or FALSE if no token has been acquired
+	 * @return string|false The API access token for this user, or FALSE if no token has been acquired
 	 **/
 	public function getToken() {
 		if ($this->token) {
@@ -133,7 +133,7 @@ 
 	}
 	
 	/**
-	 * @return string|boolean The URL of the API for which the user's API token is valid, or FALSE if no token has been acquired
+	 * @return string|false The URL of the API for which the user's API token is valid, or FALSE if no token has been acquired
 	 **/
 	function getAPIUrl() {
 		if ($this->apiUrl) {

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(

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 {

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