smtech / advisor-dashboard

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) . '"' ?>],

course/common.inc.php

Indentation +8 added lines, -8 removed lines

@@ -11,14 +11,14 @@
 /* get and cache ID of first student in the advisory group */
 $firstStudent = $toolbox->cache_get('first-student');
 if (empty($firstStudent)) {
-	$enrollments = $toolbox->api_get(
-		'courses/' . $_SESSION[ToolProvider::class]['canvas']['course_id'] . '/enrollments',
-		[
-			'role[]' => 'StudentEnrollment'
-		]
-	);
-	$firstStudent = $enrollments[0]['user']['id'];
-	$toolbox->cache_set('first-student', $firstStudent);
+    $enrollments = $toolbox->api_get(
+        'courses/' . $_SESSION[ToolProvider::class]['canvas']['course_id'] . '/enrollments',
+        [
+            'role[]' => 'StudentEnrollment'
+        ]
+    );
+    $firstStudent = $enrollments[0]['user']['id'];
+    $toolbox->cache_set('first-student', $firstStudent);
 }
 
 /* generate faculty journal URL for use by `smtech/canvashack-plugin-faculty-journal` */

course/relative-grades.php

Indentation +38 added lines, -38 removed lines

@@ -7,14 +7,14 @@ 
 
 $accounts = $toolbox->getAccountList();
 function isAcademic($account) {
-	global $accounts;
-	if ($account == 132) { // FIXME really, hard-coded values? Really?
-		return true;
-	} elseif ($account == 1 || !is_integer($account)) {
-		return false;
-	} else {
-		return isAcademic($accounts[$account]['parent_account_id']);
-	}
+    global $accounts;
+    if ($account == 132) { // FIXME really, hard-coded values? Really?
+        return true;
+    } elseif ($account == 1 || !is_integer($account)) {
+        return false;
+    } else {
+        return isAcademic($accounts[$account]['parent_account_id']);
+    }
 }
 
 $toolbox->cache_pushKey(basename(__FILE__, '.php'));
@@ -24,12 +24,12 @@ 
 
 $advisees = $toolbox->cache_get('advisees');
 if ($advisees === false) {
-	$advisees = $toolbox->api_get(
-		'courses/' . $_SESSION[COURSE_ID] . '/enrollments', [
-			'role[]' => 'StudentEnrollment' // FIXME this shouldn't require the faux-array
-		]
-	);
-	$toolbox->cache_set('advisees', $advisees);
+    $advisees = $toolbox->api_get(
+        'courses/' . $_SESSION[COURSE_ID] . '/enrollments', [
+            'role[]' => 'StudentEnrollment' // FIXME this shouldn't require the faux-array
+        ]
+    );
+    $toolbox->cache_set('advisees', $advisees);
 }
 
 $advisee = (isset($_REQUEST['advisee']) ? $_REQUEST['advisee'] : $advisees[0]['user']['id']);
@@ -38,40 +38,40 @@ 
 
 $courses = $toolbox->cache_get('courses');
 if ($courses === false) {
-	$allCourses = $toolbox->api_get("users/$advisee/courses");
-
-	$courses = [];
-	foreach ($allCourses as $course) {
-		if (
-			!empty($course['account_id']) &&
-			isAcademic($course['account_id'])
-		) {
-
-			$courses[$course['id']] = $course;
-		}
-	}
-	$toolbox->cache_set('courses', $courses);
+    $allCourses = $toolbox->api_get("users/$advisee/courses");
+
+    $courses = [];
+    foreach ($allCourses as $course) {
+        if (
+            !empty($course['account_id']) &&
+            isAcademic($course['account_id'])
+        ) {
+
+            $courses[$course['id']] = $course;
+        }
+    }
+    $toolbox->cache_set('courses', $courses);
 }
 
 $analytics = $toolbox->cache_get('analytics');
 if ($analytics === false) {
-	$analytics = [];
-	foreach ($courses as $course) {
-		$analytics[$course['id']] = $toolbox->api_get("courses/{$course['id']}/analytics/users/$advisee/assignments");
-	}
-	$toolbox->cache_set('analytics', $analytics);
+    $analytics = [];
+    foreach ($courses as $course) {
+        $analytics[$course['id']] = $toolbox->api_get("courses/{$course['id']}/analytics/users/$advisee/assignments");
+    }
+    $toolbox->cache_set('analytics', $analytics);
 }
 
 $toolbox->cache_popKey();
 $toolbox->cache_popKey();
 
 $toolbox->smarty_assign([
-	'advisee' => $advisee,
-	'advisees' => $advisees,
-	'terms' => $terms,
-	'courses' => $courses,
-	'analytics' => $analytics,
-	'canvasInstanceUrl' => $_SESSION[CANVAS_INSTANCE_URL]
+    'advisee' => $advisee,
+    'advisees' => $advisees,
+    'terms' => $terms,
+    'courses' => $courses,
+    'analytics' => $analytics,
+    'canvasInstanceUrl' => $_SESSION[CANVAS_INSTANCE_URL]
 ]);
 
 /*

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');