Hackathonners / swap

app/Judite/Models/Exchange.php

Spacing +2 added lines, -2 removed lines

@@ -65,11 +65,11 @@
      */
     public function setExchangeEnrollments(Enrollment $from, Enrollment $to) : Exchange
     {
-        if (! $from->availableForExchange() || is_null($to->shift)) {
+        if ( ! $from->availableForExchange() || is_null($to->shift)) {
             throw new EnrollmentCannotBeExchangedException();
         }
 
-        if (! $from->course->is($to->course)) {
+        if ( ! $from->course->is($to->course)) {
             throw new ExchangeEnrollmentsOnDifferentCoursesException();
         }
 

app/Http/Controllers/EnrollmentExchangeController.php

Spacing +5 added lines, -5 removed lines

@@ -31,10 +31,10 @@ 
     public function create($id)
     {
         try {
-            $data = DB::transaction(function () use ($id) {
+            $data = DB::transaction(function() use ($id) {
                 $enrollment = student()->enrollments()->findOrFail($id);
 
-                if (! $enrollment->availableForExchange()) {
+                if ( ! $enrollment->availableForExchange()) {
                     throw new \LogicException('The enrollment is not available for exchange.');
                 }
 
@@ -45,7 +45,7 @@ 
                 return compact('enrollment', 'matchingEnrollments');
             });
 
-            $data['matchingEnrollments'] = $data['matchingEnrollments']->map(function ($item) {
+            $data['matchingEnrollments'] = $data['matchingEnrollments']->map(function($item) {
                 return [
                     'id' => $item->id,
                     '_toString' => $item->present()->inlineToString(),
@@ -71,7 +71,7 @@ 
     public function store($id, CreateRequest $request)
     {
         try {
-            $exchange = DB::transaction(function () use ($id, $request) {
+            $exchange = DB::transaction(function() use ($id, $request) {
                 $this->validate($request, [
                     'to_enrollment_id' => 'exists:enrollments,id',
                 ]);
@@ -119,7 +119,7 @@ 
      */
     public function destroy($id)
     {
-        DB::transaction(function () use ($id) {
+        DB::transaction(function() use ($id) {
             student()->requestedExchanges()->findOrFail($id)->delete();
         });
 

app/Http/Controllers/DashboardController.php

Spacing +4 added lines, -4 removed lines

@@ -35,13 +35,13 @@ 
      */
     protected function adminDashboard()
     {
-        $courses = DB::transaction(function () {
+        $courses = DB::transaction(function() {
             return Course::withCount('enrollments')
                 ->orderedList()
                 ->get();
         });
 
-        $courses = $courses->groupBy(function ($course) {
+        $courses = $courses->groupBy(function($course) {
             return $course->present()->getOrdinalYear();
         });
 
@@ -55,7 +55,7 @@ 
      */
     protected function studentDashboard()
     {
-        $data = DB::transaction(function () {
+        $data = DB::transaction(function() {
             $data['enrollments'] = student()->enrollments()
                 ->withCount('exchangesAsSource')
                 ->orderByCourse()
@@ -69,7 +69,7 @@ 
         // Group all enrollments by the year of their associated course, so
         // the enrollments listing is organized by year. This will allow
         // a better experience, since it matches the official order.
-        $data['enrollments'] = $data['enrollments']->groupBy(function ($enrollment) {
+        $data['enrollments'] = $data['enrollments']->groupBy(function($enrollment) {
             return $enrollment->course->present()->getOrdinalYear();
         });
 

app/Http/Controllers/ExchangeController.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function confirm($id)
     {
-        $exchange = DB::transaction(function () use ($id) {
+        $exchange = DB::transaction(function() use ($id) {
             $exchange = student()->proposedExchanges()->findOrFail($id);
 
             return $exchange->perform();
@@ -48,7 +48,7 @@ 
      */
     public function decline($id)
     {
-        $exchange = DB::transaction(function () use ($id) {
+        $exchange = DB::transaction(function() use ($id) {
             $exchange = student()->proposedExchanges()->findOrFail($id);
             $exchange->delete();
 

app/Http/Middleware/AuthorizeStudent.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function handle($request, Closure $next)
     {
-        if (! Auth::user()->isStudent()) {
+        if ( ! Auth::user()->isStudent()) {
             throw new AuthorizationException('Unauthorized.');
         }
 

app/Http/Middleware/AuthorizeAdministrator.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function handle($request, Closure $next)
     {
-        if (! Auth::user()->isAdmin()) {
+        if ( ! Auth::user()->isAdmin()) {
             throw new AuthorizationException('Unauthorized.');
         }
 

app/Support/helpers.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-if (! function_exists('student')) {
+if ( ! function_exists('student')) {
     /**
      * Get the authenticated student.
      *

app/Http/Controllers/EnrollmentController.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
     public function store($courseId)
     {
         try {
-            $course = DB::transaction(function () use ($courseId) {
+            $course = DB::transaction(function() use ($courseId) {
                 $course = Course::findOrFail($courseId);
                 Auth::user()->student->enroll($course);
 
@@ -52,7 +52,7 @@ 
      */
     public function destroy($courseId)
     {
-        $course = DB::transaction(function () use ($courseId) {
+        $course = DB::transaction(function() use ($courseId) {
             $course = Course::findOrFail($courseId);
             student()->unenroll($course);