smtech / advisor-dashboard

src/Toolbox.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * Configure course and account navigation placements
      *
-     * @return Generator
+     * @return \smtech\LTI\Configuration\Generator
      */
     public function getGenerator()
     {

course/observers.php

Indentation +31 added lines, -31 removed lines

@@ -3,43 +3,43 @@
 require_once 'common.inc.php';
 
 $toolbox->cache_pushKey(basename(__FILE__, '.php')); {
-	$observers = $toolbox->cache_get('observers');
-	if ($observers === false) {
-		$observers = [];
-		$enrollments = $toolbox->api_get(
-			'courses/' . $_SESSION[COURSE_ID] . '/enrollments', [
-				'role[]' => 'ObserverEnrollment' // FIXME this shouldn't requrie the faux-array
-			]
-		);
-		foreach ($enrollments as $enrollment) {
-			$observers[] = $toolbox->api_get("users/{$enrollment['user']['id']}/profile");
-		}
-		$toolbox->cache_set('observers', $observers);
-	}
+    $observers = $toolbox->cache_get('observers');
+    if ($observers === false) {
+        $observers = [];
+        $enrollments = $toolbox->api_get(
+            'courses/' . $_SESSION[COURSE_ID] . '/enrollments', [
+                'role[]' => 'ObserverEnrollment' // FIXME this shouldn't requrie the faux-array
+            ]
+        );
+        foreach ($enrollments as $enrollment) {
+            $observers[] = $toolbox->api_get("users/{$enrollment['user']['id']}/profile");
+        }
+        $toolbox->cache_set('observers', $observers);
+    }
 
-	$observees = $toolbox->cache_get('observees');
-	if ($observees === false) {
-		$observees = [];
-		foreach ($observers as $observer) {
-			$response = $toolbox->api_get("users/{$observer['id']}/observees");
-			$observees[$observer['id']] = $response[0];
-		}
-		$toolbox->cache_set('observees', $observees);
-	}
+    $observees = $toolbox->cache_get('observees');
+    if ($observees === false) {
+        $observees = [];
+        foreach ($observers as $observer) {
+            $response = $toolbox->api_get("users/{$observer['id']}/observees");
+            $observees[$observer['id']] = $response[0];
+        }
+        $toolbox->cache_set('observees', $observees);
+    }
 
-	$passwords = [];
-	foreach ($observers as $observer) {
-		$response = $toolbox->mysql_query("
+    $passwords = [];
+    foreach ($observers as $observer) {
+        $response = $toolbox->mysql_query("
 			SELECT * FROM `observers` WHERE `id` = '{$observer['id']}' LIMIT 1
 		");
-		$password = $response->fetch_assoc();
-		$passwords[$observer['id']] = $password['password'];
-	}
+        $password = $response->fetch_assoc();
+        $passwords[$observer['id']] = $password['password'];
+    }
 } $toolbox->cache_popKey();
 
 $toolbox->smarty_assign([
-	'observers' => $observers,
-	'passwords' => $passwords,
-	'observees' => $observees
+    'observers' => $observers,
+    'passwords' => $passwords,
+    'observees' => $observees
 ]);
 $toolbox->smarty_display('observers.tpl');

js/relative-grades.js.php

Indentation +28 added lines, -28 removed lines

@@ -21,10 +21,10 @@ 
 
 $points = 0;
 function normalize($numerator, $denominator = false) {
-	global $points;
-	$denominator = ($denominator !== false ? $denominator : $points);
-	$points = $denominator;
-	return min(100, $numerator / $denominator * 100);
+    global $points;
+    $denominator = ($denominator !== false ? $denominator : $points);
+    $points = $denominator;
+    return min(100, $numerator / $denominator * 100);
 }
 
 header('Content-Type: application/javascript');
@@ -34,7 +34,7 @@ 
 
 $analytics = $toolbox->cache_get('analytics');
 if ($analytics === false) {
-	exit;
+    exit;
 }
 
 ?>
@@ -47,29 +47,29 @@ 
 
 	<?php
 
-		$labels = array();
-		$max_scores = array();
-		$min_scores = array();
-		$medians = array();
-		$first_quartiles = array();
-		$third_quartiles = array();
-		$scores = array();
-		foreach ($analytic as $data) {
-			if ($data['points_possible'] > 0 && $data['max_score'] > 0) {
-				$labels[] = ''; // htmlentities($data['title']);
-				$max_scores[] = normalize($data['max_score'], $data['points_possible']);
-				$min_scores[] = normalize($data['min_score']);
-				$medians[] = normalize($data['median']);
-				$first_quartiles[] = normalize($data['first_quartile']);
-				$third_quartiles[] = normalize($data['third_quartile']);
-				if (empty($data['submission'])) {
-					$scores[] = '""'; /* some assignments may not have grades */
-				} else {
-					$scores[] = normalize($data['submission']['score']);
-				}
-			}
-		}
-	?>
+        $labels = array();
+        $max_scores = array();
+        $min_scores = array();
+        $medians = array();
+        $first_quartiles = array();
+        $third_quartiles = array();
+        $scores = array();
+        foreach ($analytic as $data) {
+            if ($data['points_possible'] > 0 && $data['max_score'] > 0) {
+                $labels[] = ''; // htmlentities($data['title']);
+                $max_scores[] = normalize($data['max_score'], $data['points_possible']);
+                $min_scores[] = normalize($data['min_score']);
+                $medians[] = normalize($data['median']);
+                $first_quartiles[] = normalize($data['first_quartile']);
+                $third_quartiles[] = normalize($data['third_quartile']);
+                if (empty($data['submission'])) {
+                    $scores[] = '""'; /* some assignments may not have grades */
+                } else {
+                    $scores[] = normalize($data['submission']['score']);
+                }
+            }
+        }
+    ?>
 
 	var data = {
 		labels: [<?= '"' . implode('", "', $labels) . '"' ?>],

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! */
@@ -42,8 +41,7 @@ 
 /* if not authenticated, default to showing credentials */
 } else {
     $action = (empty($action) ?
-        ACTION_CONFIG :
-        $action
+        ACTION_CONFIG : $action
     );
 }
 
@@ -53,7 +51,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 {