Completed
Pull Request — master (#65)
by Phecho
04:59
created
app/Http/Middleware/Timezone.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,6 @@
 block discarded – undo
20 20
      *
21 21
      * @param \Illuminate\Http\Request $request
22 22
      * @param \Closure                 $next
23
-     * @param string                   $type
24 23
      *
25 24
      * @return mixed
26 25
      */
Please login to merge, or discard this patch.
app/Presenters/IssuePresenter.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,6 @@
 block discarded – undo
11 11
 
12 12
 namespace Gitamin\Presenters;
13 13
 
14
-use Gitamin\Facades\Setting;
15 14
 use Gitamin\Presenters\Traits\TimestampsTrait;
16 15
 use GrahamCampbell\Markdown\Facades\Markdown;
17 16
 use Jenssegers\Date\Date;
Please login to merge, or discard this patch.
app/Composers/TimezoneLocaleComposer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
     {
30 30
         $enabledLangs = Config::get('langs');
31 31
 
32
-        $langs = array_map(function ($lang) use ($enabledLangs) {
32
+        $langs = array_map(function($lang) use ($enabledLangs) {
33 33
             $locale = basename($lang);
34 34
 
35 35
             return [$locale => $enabledLangs[$locale]];
Please login to merge, or discard this patch.
app/Handlers/Commands/Project/UpdateProjectCommandHandler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
             'team_id'     => $command->team_id,
55 55
         ];
56 56
 
57
-        return array_filter($params, function ($val) {
57
+        return array_filter($params, function($val) {
58 58
             return $val !== null;
59 59
         });
60 60
     }
Please login to merge, or discard this patch.
app/Handlers/Events/Issue/SendIssueEmailNotificationHandler.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@
 block discarded – undo
81 81
                 $this->mailer->queue([
82 82
                     'html' => 'emails.issues.new-html',
83 83
                     'text' => 'emails.issues.new-text',
84
-                ], $mail, function (Message $message) use ($mail) {
84
+                ], $mail, function(Message $message) use ($mail) {
85 85
                     $message->to($mail['email'])->subject($mail['subject']);
86 86
                 });
87 87
             }
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
     /**
37 37
      * Create a new send issue email notification handler.
38 38
      *
39
-     * @param \Illuminate\Contracts\Mail\Mailer $mailer
39
+     * @param MailQueue $mailer
40 40
      * @param \Gitamin\Models\Subscriber        $subscriber
41 41
      *
42 42
      * @return void
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
     /**
51 51
      * Handle the event.
52 52
      *
53
-     * @param \Gitamin\Events\Issue\IssueHasAddedEvent $event
53
+     * @param IssueWasAddedEvent $event
54 54
      *
55 55
      * @return void
56 56
      */
Please login to merge, or discard this patch.
app/Http/Controllers/Api/ProjectController.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
             $tags = preg_split('/ ?, ?/', Binput::get('tags'));
85 85
 
86 86
             // For every tag, do we need to create it?
87
-            $projectTags = array_map(function ($taggable) use ($project) {
87
+            $projectTags = array_map(function($taggable) use ($project) {
88 88
                 return Tag::firstOrCreate([
89 89
                     'name' => $taggable,
90 90
                 ])->id;
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
             $tags = preg_split('/ ?, ?/', Binput::get('tags'));
125 125
 
126 126
             // For every tag, do we need to create it?
127
-            $projectTags = array_map(function ($taggable) use ($project) {
127
+            $projectTags = array_map(function($taggable) use ($project) {
128 128
                 return Tag::firstOrCreate(['name' => $taggable])->id;
129 129
             }, $tags);
130 130
 
Please login to merge, or discard this patch.
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
      *
31 31
      * @param \Symfony\Component\HttpFoundation\Request $request
32 32
      *
33
-     * @return \Illuminate\Http\JsonResponse
33
+     * @return \Illuminate\Http\Response
34 34
      */
35 35
     public function getTeams(Request $request)
36 36
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
      *
45 45
      * @param \Gitamin\Models\ProjectTeam $team
46 46
      *
47
-     * @return \Illuminate\Http\JsonResponse
47
+     * @return \Illuminate\Http\Response
48 48
      */
49 49
     public function getTeam(ProjectTeam $team)
50 50
     {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
     /**
55 55
      * Create a new project team.
56 56
      *
57
-     * @return \Illuminate\Http\JsonResponse
57
+     * @return \Illuminate\Http\Response
58 58
      */
59 59
     public function postTeams()
60 60
     {
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
      *
77 77
      * @param \Gitamin\Models\ProjectTeam $team
78 78
      *
79
-     * @return \Illuminate\Http\JsonResponse
79
+     * @return \Illuminate\Http\Response
80 80
      */
81 81
     public function putTeam(ProjectTeam $team)
82 82
     {
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
      *
100 100
      * @param \Gitamin\Models\ProjectTeam $team
101 101
      *
102
-     * @return \Illuminate\Http\JsonResponse
102
+     * @return \Illuminate\Http\Response
103 103
      */
104 104
     public function deleteTeam(ProjectTeam $team)
105 105
     {
Please login to merge, or discard this patch.
app/Http/Routes/ApiRoutes.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
             'namespace'  => 'Api',
30 30
             'prefix'     => 'api/v1',
31 31
             'middleware' => ['accept:application/json', 'timezone', 'auth.api.optional'],
32
-        ], function ($router) {
32
+        ], function($router) {
33 33
             // General
34 34
             $router->get('ping', 'GeneralController@ping');
35 35
 
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
             $router->get('issues/{issue}', 'IssueController@getIssue');
45 45
 
46 46
             // Authorization Required
47
-            $router->group(['middleware' => 'auth.api'], function ($router) {
47
+            $router->group(['middleware' => 'auth.api'], function($router) {
48 48
                 $router->get('subscribers', 'SubscriberController@getSubscribers');
49 49
 
50 50
                 $router->post('projects', 'ProjectController@postProjects');
Please login to merge, or discard this patch.
app/Http/Routes/DashboardRoutes.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
                 'as'     => 'activities.',
92 92
                 'prefix' => 'activities',
93 93
                 ], function ($router) {
94
-                   $router->get('/', [
94
+                    $router->get('/', [
95 95
                     'as'   => 'index',
96 96
                     'uses' => 'ActivityController@showActivities',
97 97
                 ]);
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
                 'as'     => 'milestones.',
103 103
                 'prefix' => 'milestones',
104 104
                 ], function ($router) {
105
-                   $router->get('/', [
105
+                    $router->get('/', [
106 106
                     'as'   => 'index',
107 107
                     'uses' => 'MilestoneController@showMilestones',
108 108
                 ]);
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
                 'as'     => 'merge_requests.',
114 114
                 'prefix' => 'merge_requests',
115 115
                 ], function ($router) {
116
-                   $router->get('/', [
116
+                    $router->get('/', [
117 117
                     'as'   => 'index',
118 118
                     'uses' => 'MergeRequestController@showMergeRequests',
119 119
                 ]);
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
                 'as'     => 'snippets.',
125 125
                 'prefix' => 'snippets',
126 126
                 ], function ($router) {
127
-                   $router->get('/', [
127
+                    $router->get('/', [
128 128
                     'as'   => 'index',
129 129
                     'uses' => 'SnippetController@showSnippets',
130 130
                 ]);
Please login to merge, or discard this patch.
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
             'middleware' => ['app.hasSetting'],
31 31
             'setting'    => 'app_name',
32 32
             'as'         => 'dashboard.',
33
-        ], function ($router) {
33
+        ], function($router) {
34 34
             $router->get('dashboard', [
35 35
                 'as'   => 'index',
36 36
                 'uses' => 'DashboardController@showDashboard',
@@ -44,13 +44,13 @@  discard block
 block discarded – undo
44 44
             'prefix'     => 'dashboard',
45 45
             'namespace'  => 'Dashboard',
46 46
             'as'         => 'dashboard.',
47
-        ], function ($router) {
47
+        ], function($router) {
48 48
             
49 49
             // Projects
50 50
             $router->group([
51 51
                 'as'     => 'projects.',
52 52
                 'prefix' => 'projects',
53
-            ], function ($router) {
53
+            ], function($router) {
54 54
                 $router->get('/', [
55 55
                     'as'   => 'index',
56 56
                     'uses' => 'ProjectController@index',
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
             $router->group([
81 81
                 'as'     => 'teams.',
82 82
                 'prefix' => 'teams',
83
-            ], function ($router) {
83
+            ], function($router) {
84 84
                 $router->get('/', [
85 85
                     'as'   => 'index',
86 86
                     'uses' => 'TeamController@showProjectTeams',
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
             $router->group([
104 104
                 'as'     => 'activities.',
105 105
                 'prefix' => 'activities',
106
-                ], function ($router) {
106
+                ], function($router) {
107 107
                    $router->get('/', [
108 108
                     'as'   => 'index',
109 109
                     'uses' => 'ActivityController@showActivities',
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
             $router->group([
115 115
                 'as'     => 'milestones.',
116 116
                 'prefix' => 'milestones',
117
-                ], function ($router) {
117
+                ], function($router) {
118 118
                    $router->get('/', [
119 119
                     'as'   => 'index',
120 120
                     'uses' => 'MilestoneController@showMilestones',
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
             $router->group([
126 126
                 'as'     => 'merge_requests.',
127 127
                 'prefix' => 'merge_requests',
128
-                ], function ($router) {
128
+                ], function($router) {
129 129
                    $router->get('/', [
130 130
                     'as'   => 'index',
131 131
                     'uses' => 'MergeRequestController@showMergeRequests',
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
             $router->group([
137 137
                 'as'     => 'snippets.',
138 138
                 'prefix' => 'snippets',
139
-                ], function ($router) {
139
+                ], function($router) {
140 140
                    $router->get('/', [
141 141
                     'as'   => 'index',
142 142
                     'uses' => 'SnippetController@showSnippets',
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
             $router->group([
148 148
                 'as'     => 'issues.',
149 149
                 'prefix' => 'issues',
150
-            ], function ($router) {
150
+            ], function($router) {
151 151
                 $router->get('/', [
152 152
                     'as'   => 'index',
153 153
                     'uses' => 'IssueController@showIssues',
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
             $router->group([
173 173
                 'as'     => 'subscribers.',
174 174
                 'prefix' => 'subscribers',
175
-            ], function ($router) {
175
+            ], function($router) {
176 176
                 $router->get('/', [
177 177
                     'as'   => 'index',
178 178
                     'uses' => 'SubscriberController@showSubscribers',
@@ -189,13 +189,13 @@  discard block
 block discarded – undo
189 189
             $router->group([
190 190
                 'as'     => 'group.',
191 191
                 'prefix' => 'group',
192
-            ], function ($router) {
192
+            ], function($router) {
193 193
                 $router->get('/', [
194 194
                     'as'   => 'index',
195 195
                     'uses' => 'GroupController@showGroupView',
196 196
                 ]);
197 197
 
198
-                $router->group(['middleware' => 'admin'], function ($router) {
198
+                $router->group(['middleware' => 'admin'], function($router) {
199 199
                     $router->get('add', [
200 200
                         'as'   => 'add',
201 201
                         'uses' => 'GroupController@showAddGroupMemberView',
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
             $router->group([
217 217
                 'as'     => 'settings.',
218 218
                 'prefix' => 'settings',
219
-            ], function ($router) {
219
+            ], function($router) {
220 220
                 $router->get('general', [
221 221
                     'as'   => 'general',
222 222
                     'uses' => 'SettingsController@showGeneralView',
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
             });
242 242
 
243 243
             // User Settings
244
-            $router->group(['prefix' => 'user'], function ($router) {
244
+            $router->group(['prefix' => 'user'], function($router) {
245 245
                 $router->get('/', [
246 246
                     'as'   => 'user',
247 247
                     'uses' => 'UserController@showUser',
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
              * Internal API.
255 255
              * This should only be used for making requests within the dashboard.
256 256
              */
257
-            $router->group(['prefix' => 'api'], function ($router) {
257
+            $router->group(['prefix' => 'api'], function($router) {
258 258
                 $router->post('projects/teams/order', 'ApiController@postUpdateProjectTeamOrder');
259 259
                 $router->post('projects/order', 'ApiController@postUpdateProjectOrder');
260 260
                 $router->post('projects/{project}', 'ApiController@postUpdateProject');
Please login to merge, or discard this patch.
app/Http/Routes/SignupRoutes.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
             'middleware' => ['app.hasSetting', 'guest'],
30 30
             'setting'    => 'app_name',
31 31
             'as'         => 'signup.',
32
-        ], function ($router) {
32
+        ], function($router) {
33 33
             $router->get('signup/invite/{code}', [
34 34
                 'as'   => 'invite',
35 35
                 'uses' => 'SignupController@getSignup',
Please login to merge, or discard this patch.