gitaminhq / Gitamin

app/Presenters/IssuePresenter.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 namespace Gitamin\Presenters;
 
-use Gitamin\Facades\Setting;
 use Gitamin\Presenters\Traits\TimestampsTrait;
 use GrahamCampbell\Markdown\Facades\Markdown;
 use Jenssegers\Date\Date;

app/Composers/TimezoneLocaleComposer.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     {
         $enabledLangs = Config::get('langs');
 
-        $langs = array_map(function ($lang) use ($enabledLangs) {
+        $langs = array_map(function($lang) use ($enabledLangs) {
             $locale = basename($lang);
 
             return [$locale => $enabledLangs[$locale]];

app/Handlers/Commands/Project/UpdateProjectCommandHandler.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
             'team_id'     => $command->team_id,
         ];
 
-        return array_filter($params, function ($val) {
+        return array_filter($params, function($val) {
             return $val !== null;
         });
     }

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

@@ -36,7 +36,7 @@ 
     /**
      * Create a new send issue email notification handler.
      *
-     * @param \Illuminate\Contracts\Mail\Mailer $mailer
+     * @param MailQueue $mailer
      * @param \Gitamin\Models\Subscriber        $subscriber
      *
      * @return void
@@ -50,7 +50,7 @@ 
     /**
      * Handle the event.
      *
-     * @param \Gitamin\Events\Issue\IssueHasAddedEvent $event
+     * @param IssueWasAddedEvent $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);
 

Doc Comments +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
      *
      * @param \Symfony\Component\HttpFoundation\Request $request
      *
-     * @return \Illuminate\Http\JsonResponse
+     * @return \Illuminate\Http\Response
      */
     public function getTeams(Request $request)
     {
@@ -44,7 +44,7 @@ 
      *
      * @param \Gitamin\Models\ProjectTeam $team
      *
-     * @return \Illuminate\Http\JsonResponse
+     * @return \Illuminate\Http\Response
      */
     public function getTeam(ProjectTeam $team)
     {
@@ -54,7 +54,7 @@ 
     /**
      * Create a new project team.
      *
-     * @return \Illuminate\Http\JsonResponse
+     * @return \Illuminate\Http\Response
      */
     public function postTeams()
     {
@@ -76,7 +76,7 @@ 
      *
      * @param \Gitamin\Models\ProjectTeam $team
      *
-     * @return \Illuminate\Http\JsonResponse
+     * @return \Illuminate\Http\Response
      */
     public function putTeam(ProjectTeam $team)
     {
@@ -99,7 +99,7 @@ 
      *
      * @param \Gitamin\Models\ProjectTeam $team
      *
-     * @return \Illuminate\Http\JsonResponse
+     * @return \Illuminate\Http\Response
      */
     public function deleteTeam(ProjectTeam $team)
     {

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/SubscribeRoutes.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
             'middleware' => ['app.hasSetting', 'localize'],
             'setting'    => 'app_name',
             'as'         => 'subscribe.',
-        ], function ($router) {
-            $router->group(['middleware' => 'app.subscribers'], function ($router) {
+        ], function($router) {
+            $router->group(['middleware' => 'app.subscribers'], function($router) {
                 $router->get('subscribe', [
                     'as'   => 'subscribe',
                     'uses' => 'SubscribeController@showSubscribe',

app/Models/Invite.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         parent::boot();
 
-        self::creating(function ($invite) {
+        self::creating(function($invite) {
             if (!$invite->code) {
                 $invite->code = self::generateInviteCode();
             }