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

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

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': {

import.php

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;
 	}
 }