gitaminhq / Gitamin

app/Handlers/Events/Issue/SendIssueEmailNotificationHandler.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
                 $this->mailer->queue([
                     'html' => 'emails.issues.new-html',
                     'text' => 'emails.issues.new-text',
-                ], $mail, function (Message $message) use ($mail) {
+                ], $mail, function(Message $message) use ($mail) {
                     $message->to($mail['email'])->subject($mail['subject']);
                 });
             }

Doc Comments +2 added lines, -2 removed lines

@@ -50,9 +50,9 @@
     /**
      * Handle the event.
      *
-     * @param \Gitamin\Events\Issue\IssueHasAddedEvent $event
+     * @param IssueWasAddedEvent $event
      *
-     * @return void
+     * @return false|null
      */
     public function handle(IssueWasAddedEvent $event)
     {

app/Handlers/Events/User/SendInviteUserEmailHandler.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         $this->mailer->queue([
             'html' => 'emails.users.invite-html',
             'text' => 'emails.users.invite-text',
-        ], $mail, function (Message $message) use ($mail) {
+        ], $mail, function(Message $message) use ($mail) {
             $message->to($mail['email'])->subject($mail['subject']);
         });
     }

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     /**
      * Handle the event.
      *
-     * @param \Gitamin\Events\UserWasInvitedEvent $event
+     * @param UserWasInvitedEvent $event
      *
      * @return void
      */

app/Http/Controllers/Api/ProjectController.php

Spacing +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
             $tags = preg_split('/ ?, ?/', Binput::get('tags'));
 
             // For every tag, do we need to create it?
-            $projectTags = array_map(function ($taggable) use ($project) {
+            $projectTags = array_map(function($taggable) use ($project) {
                 return Tag::firstOrCreate([
                     'name' => $taggable,
                 ])->id;
@@ -124,7 +124,7 @@ 
             $tags = preg_split('/ ?, ?/', Binput::get('tags'));
 
             // For every tag, do we need to create it?
-            $projectTags = array_map(function ($taggable) use ($project) {
+            $projectTags = array_map(function($taggable) use ($project) {
                 return Tag::firstOrCreate(['name' => $taggable])->id;
             }, $tags);
 

app/Http/Controllers/Dashboard/DashboardController.php

Spacing +4 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
         $allIssues = Issue::whereBetween('created_at', [
             $this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
             $this->startDate->format('Y-m-d').' 23:59:59',
-        ])->orderBy('created_at', 'desc')->get()->groupBy(function (Issue $issue) {
+        ])->orderBy('created_at', 'desc')->get()->groupBy(function(Issue $issue) {
             return (new Date($issue->created_at))
                 ->setTimezone($this->dateTimeZone)->toDateString();
         });
@@ -102,7 +102,7 @@ 
         }
 
         // Sort the array so it takes into account the added days
-        $allIssues = $allIssues->sortBy(function ($value, $key) {
+        $allIssues = $allIssues->sortBy(function($value, $key) {
             return strtotime($key);
         }, SORT_REGULAR, false);
 
@@ -119,7 +119,7 @@ 
         $allSubscribers = Subscriber::whereBetween('created_at', [
             $this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
             $this->startDate->format('Y-m-d').' 23:59:59',
-        ])->orderBy('created_at', 'desc')->get()->groupBy(function (Subscriber $issue) {
+        ])->orderBy('created_at', 'desc')->get()->groupBy(function(Subscriber $issue) {
             return (new Date($issue->created_at))
                 ->setTimezone($this->dateTimeZone)->toDateString();
         });
@@ -134,7 +134,7 @@ 
         }
 
         // Sort the array so it takes into account the added days
-        $allSubscribers = $allSubscribers->sortBy(function ($value, $key) {
+        $allSubscribers = $allSubscribers->sortBy(function($value, $key) {
             return strtotime($key);
         }, SORT_REGULAR, false);
 

app/Http/Controllers/ExploreController.php

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
         $allIssues = Issue::where('visible', '>=', $issueVisiblity)->whereBetween('created_at', [
             $startDate->copy()->subDays($daysToShow)->format('Y-m-d').' 00:00:00',
             $startDate->format('Y-m-d').' 23:59:59',
-        ])->orderBy('scheduled_at', 'desc')->orderBy('created_at', 'desc')->get()->groupBy(function (Issue $issue) use ($dateTimeZone) {
+        ])->orderBy('scheduled_at', 'desc')->orderBy('created_at', 'desc')->get()->groupBy(function(Issue $issue) use ($dateTimeZone) {
             // If it's scheduled, get the scheduled at date.
             if ($issue->is_scheduled) {
                 return (new Date($issue->scheduled_at))
@@ -82,7 +82,7 @@ 
         }
 
         // Sort the array so it takes into account the added days
-        $allIssues = $allIssues->sortBy(function ($value, $key) {
+        $allIssues = $allIssues->sortBy(function($value, $key) {
             return strtotime($key);
         }, SORT_REGULAR, true)->all();
 

app/Http/Controllers/RssController.php

Spacing +3 added lines, -3 removed lines

@@ -38,13 +38,13 @@
         $feed->setDateFormat('datetime');
 
         if ($group->exists) {
-            $group->projects->map(function ($project) use ($feed) {
-                $project->issues()->visible()->orderBy('created_at', 'desc')->get()->map(function ($issue) use ($feed) {
+            $group->projects->map(function($project) use ($feed) {
+                $project->issues()->visible()->orderBy('created_at', 'desc')->get()->map(function($issue) use ($feed) {
                     $this->feedAddItem($feed, $issue);
                 });
             });
         } else {
-            Issue::visible()->orderBy('created_at', 'desc')->get()->map(function ($issue) use ($feed) {
+            Issue::visible()->orderBy('created_at', 'desc')->get()->map(function($issue) use ($feed) {
                 $this->feedAddItem($feed, $issue);
             });
         }

app/Http/Routes/ApiRoutes.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
             'namespace'  => 'Api',
             'prefix'     => 'api/v1',
             'middleware' => ['accept:application/json', 'timezone', 'auth.api.optional'],
-        ], function ($router) {
+        ], function($router) {
             // General
             $router->get('ping', 'GeneralController@ping');
 
@@ -44,7 +44,7 @@ 
             $router->get('issues/{issue}', 'IssueController@getIssue');
 
             // Authorization Required
-            $router->group(['middleware' => 'auth.api'], function ($router) {
+            $router->group(['middleware' => 'auth.api'], function($router) {
                 $router->get('subscribers', 'SubscriberController@getSubscribers');
 
                 $router->post('projects', 'ProjectController@postProjects');

app/Http/Routes/DashboardRoutes.php

Indentation +4 added lines, -4 removed lines

@@ -91,7 +91,7 @@ 
                 'as'     => 'activities.',
                 'prefix' => 'activities',
                 ], function ($router) {
-                   $router->get('/', [
+                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'ActivityController@showActivities',
                 ]);
@@ -102,7 +102,7 @@ 
                 'as'     => 'milestones.',
                 'prefix' => 'milestones',
                 ], function ($router) {
-                   $router->get('/', [
+                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'MilestoneController@showMilestones',
                 ]);
@@ -113,7 +113,7 @@ 
                 'as'     => 'merge_requests.',
                 'prefix' => 'merge_requests',
                 ], function ($router) {
-                   $router->get('/', [
+                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'MergeRequestController@showMergeRequests',
                 ]);
@@ -124,7 +124,7 @@ 
                 'as'     => 'snippets.',
                 'prefix' => 'snippets',
                 ], function ($router) {
-                   $router->get('/', [
+                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'SnippetController@showSnippets',
                 ]);

Spacing +14 added lines, -14 removed lines

@@ -30,7 +30,7 @@ 
             'prefix'     => 'dashboard',
             'namespace'  => 'Dashboard',
             'as'         => 'dashboard.',
-        ], function ($router) {
+        ], function($router) {
             // Dashboard
             $router->get('/', [
                 'as'   => 'index',
@@ -41,7 +41,7 @@ 
             $router->group([
                 'as'     => 'projects.',
                 'prefix' => 'projects',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'ProjectController@showProjects',
@@ -67,7 +67,7 @@ 
             $router->group([
                 'as'     => 'teams.',
                 'prefix' => 'teams',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'TeamController@showProjectTeams',
@@ -90,7 +90,7 @@ 
             $router->group([
                 'as'     => 'activities.',
                 'prefix' => 'activities',
-                ], function ($router) {
+                ], function($router) {
                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'ActivityController@showActivities',
@@ -101,7 +101,7 @@ 
             $router->group([
                 'as'     => 'milestones.',
                 'prefix' => 'milestones',
-                ], function ($router) {
+                ], function($router) {
                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'MilestoneController@showMilestones',
@@ -112,7 +112,7 @@ 
             $router->group([
                 'as'     => 'merge_requests.',
                 'prefix' => 'merge_requests',
-                ], function ($router) {
+                ], function($router) {
                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'MergeRequestController@showMergeRequests',
@@ -123,7 +123,7 @@ 
             $router->group([
                 'as'     => 'snippets.',
                 'prefix' => 'snippets',
-                ], function ($router) {
+                ], function($router) {
                    $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'SnippetController@showSnippets',
@@ -134,7 +134,7 @@ 
             $router->group([
                 'as'     => 'issues.',
                 'prefix' => 'issues',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'IssueController@showIssues',
@@ -159,7 +159,7 @@ 
             $router->group([
                 'as'     => 'subscribers.',
                 'prefix' => 'subscribers',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'SubscriberController@showSubscribers',
@@ -176,13 +176,13 @@ 
             $router->group([
                 'as'     => 'group.',
                 'prefix' => 'group',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('/', [
                     'as'   => 'index',
                     'uses' => 'GroupController@showGroupView',
                 ]);
 
-                $router->group(['middleware' => 'admin'], function ($router) {
+                $router->group(['middleware' => 'admin'], function($router) {
                     $router->get('add', [
                         'as'   => 'add',
                         'uses' => 'GroupController@showAddGroupMemberView',
@@ -203,7 +203,7 @@ 
             $router->group([
                 'as'     => 'settings.',
                 'prefix' => 'settings',
-            ], function ($router) {
+            ], function($router) {
                 $router->get('general', [
                     'as'   => 'general',
                     'uses' => 'SettingsController@showGeneralView',
@@ -228,7 +228,7 @@ 
             });
 
             // User Settings
-            $router->group(['prefix' => 'user'], function ($router) {
+            $router->group(['prefix' => 'user'], function($router) {
                 $router->get('/', [
                     'as'   => 'user',
                     'uses' => 'UserController@showUser',
@@ -241,7 +241,7 @@ 
              * Internal API.
              * This should only be used for making requests within the dashboard.
              */
-            $router->group(['prefix' => 'api'], function ($router) {
+            $router->group(['prefix' => 'api'], function($router) {
                 $router->post('projects/teams/order', 'ApiController@postUpdateProjectTeamOrder');
                 $router->post('projects/order', 'ApiController@postUpdateProjectOrder');
                 $router->post('projects/{project}', 'ApiController@postUpdateProject');

app/Http/Routes/InstallRoutes.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      */
     public function map(Registrar $router)
     {
-        $router->group(['middleware' => ['app.isInstalled', 'localize']], function ($router) {
+        $router->group(['middleware' => ['app.isInstalled', 'localize']], function($router) {
             $router->controller('install', 'InstallController');
         });
     }