smtech / canvashack

control-panel.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
 
 if (isset($_REQUEST['hack'])) {
 	while (list($id, $setting) = each($_REQUEST['hack'])) {
-		try{
+		try {
 
 			$hack = CanvasHack::getCanvasHackById($sql, $id);
 			if ($setting === 'enable') {
@@ -24,13 +24,13 @@ 
 
 $hacksContents = scandir(realpath(__DIR__ . '/hacks'), SCANDIR_SORT_ASCENDING);
 $hacks = array();
-foreach($hacksContents as $item) {
+foreach ($hacksContents as $item) {
 	if (is_dir($path = realpath(__DIR__ . "/hacks/$item")) && file_exists($manifest = "$path/manifest.xml")) {
 		try {
 			$hacks[$item] = new CanvasHack($sql, $path);
 		} catch (CanvasHack_Exception $e) {
 			$smarty->addMessage(
-				'CanvasHack Manifest Error ['. $e->getCode() . ']',
+				'CanvasHack Manifest Error [' . $e->getCode() . ']',
 				$e->getMessage(),
 				NotificationMessage::ERROR
 			);

common.inc.php

Spacing +5 added lines, -5 removed lines

@@ -13,9 +13,9 @@ 
 
 /* prepare the toolbox */
 if (empty($_SESSION[Toolbox::class])) {
-    $_SESSION[Toolbox::class] =& Toolbox::fromConfiguration(CONFIG_FILE);
+    $_SESSION[Toolbox::class] = & Toolbox::fromConfiguration(CONFIG_FILE);
 }
-$toolbox =& $_SESSION[Toolbox::class];
+$toolbox = & $_SESSION[Toolbox::class];
 $toolbox->smarty_prependTemplateDir(__DIR__ . '/templates', basename(__DIR__));
 $toolbox->smarty_assign([
     'category' => DataUtilities::titleCase(preg_replace('/[\-_]+/', ' ', basename(__DIR__)))
@@ -24,9 +24,9 @@ 
 /*
  * FIXME convience variables until plugins are all updated
  */
- $api =& $toolbox->getAPI();
- $sql =& $toolbox->getMySQL();
- $smarty =& $toolbox->getSmarty();
+ $api = & $toolbox->getAPI();
+ $sql = & $toolbox->getMySQL();
+ $smarty = & $toolbox->getSmarty();
 
 /* set the Tool Consumer's instance URL, if present */
 if (empty($_SESSION[CANVAS_INSTANCE_URL]) &&

src/CanvasHack.php

Spacing +2 added lines, -4 removed lines

@@ -134,13 +134,11 @@
         $_name = $this->sql->real_escape_string($this->name);
         $_abstract = (
             isset($this->abstract) ?
-                $this->sql->real_escape_string($this->abstract) :
-                $this->sql->real_escape_string($this->description)
+                $this->sql->real_escape_string($this->abstract) : $this->sql->real_escape_string($this->description)
         );
         $_description = (
             isset($this->description) ?
-                $this->sql->real_escape_string($this->description) :
-                $this->sql->real_escape_string($this->abstract)
+                $this->sql->real_escape_string($this->description) : $this->sql->real_escape_string($this->abstract)
         );
         $_path = $this->sql->real_escape_string($this->path);
 

index.php

Spacing +3 added lines, -5 removed lines

@@ -13,8 +13,7 @@ 
 
 /* store any requested actions for future handling */
 $action = (empty($_REQUEST['action']) ?
-    ACTION_UNSPECIFIED :
-    strtolower($_REQUEST['action'])
+    ACTION_UNSPECIFIED : strtolower($_REQUEST['action'])
 );
 
 /* action requests only come from outside the LTI! */
@@ -37,8 +36,7 @@ 
 /* if not authenticated, default to showing credentials */
 } else {
     $action = (empty($action) ?
-        ACTION_CONFIG :
-        $action
+        ACTION_CONFIG : $action
     );
 }
 
@@ -47,7 +45,7 @@ 
     /* reset cached install data from config file */
     case ACTION_INSTALL:
         $_SESSION['toolbox'] = Toolbox::fromConfiguration(CONFIG_FILE, true);
-        $toolbox =& $_SESSION['toolbox'];
+        $toolbox = & $_SESSION['toolbox'];
 
         /* test to see if we can connect to the API */
         try {