Completed
Pull Request — master (#52)
by Phecho
03:17
created
app/Commands/User/InviteGroupMemberCommand.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
     /**
33 33
      * Create a new invite group member command instance.
34 34
      *
35
-     * @param array $email
35
+     * @param array $emails
36 36
      *
37 37
      * @return void
38 38
      */
Please login to merge, or discard this patch.
app/Handlers/Events/Issue/SendIssueEmailNotificationHandler.php 2 patches
Doc Comments   +3 added lines, -3 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,9 +50,9 @@  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
-     * @return void
55
+     * @return false|null
56 56
      */
57 57
     public function handle(IssueWasAddedEvent $event)
58 58
     {
Please login to merge, or discard this patch.
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.
app/Handlers/Events/Subscriber/SendSubscriberVerificationEmailHandler.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     /**
28 28
      * Create a new send subscriber verification email handler.
29 29
      *
30
-     * @param \Illuminate\Contracts\Mail\Mailer $mailer
30
+     * @param MailQueue $mailer
31 31
      *
32 32
      * @return void
33 33
      */
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
     /**
40 40
      * Handle the event.
41 41
      *
42
-     * @param \Gitamin\Events\SubscriberHasSubscribedEvent $event
42
+     * @param SubscriberHasSubscribedEvent $event
43 43
      *
44 44
      * @return void
45 45
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
         $this->mailer->queue([
55 55
             'html' => 'emails.users.invite-html',
56 56
             'text' => 'emails.users.invite-text',
57
-        ], $mail, function (Message $message) use ($mail) {
57
+        ], $mail, function(Message $message) use ($mail) {
58 58
             $message->to($mail['email'])->subject($mail['subject']);
59 59
         });
60 60
     }
Please login to merge, or discard this patch.
app/Handlers/Events/User/SendInviteUserEmailHandler.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
     /**
28 28
      * Create a new send invite user email handler.
29 29
      *
30
-     * @param \Illuminate\Contracts\Mail\Mailer $mailer
30
+     * @param MailQueue $mailer
31 31
      *
32 32
      * @return void
33 33
      */
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
     /**
40 40
      * Handle the event.
41 41
      *
42
-     * @param \Gitamin\Events\UserWasInvitedEvent $event
42
+     * @param UserWasInvitedEvent $event
43 43
      *
44 44
      * @return void
45 45
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
         $this->mailer->queue([
55 55
             'html' => 'emails.users.invite-html',
56 56
             'text' => 'emails.users.invite-text',
57
-        ], $mail, function (Message $message) use ($mail) {
57
+        ], $mail, function(Message $message) use ($mail) {
58 58
             $message->to($mail['email'])->subject($mail['subject']);
59 59
         });
60 60
     }
Please login to merge, or discard this patch.
app/Http/Controllers/Api/AbstractApiController.php 1 patch
Doc Comments   +4 added lines, -5 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
      *
111 111
      * @param mixed
112 112
      *
113
-     * @return \Illuminate\Http\JsonResponse
113
+     * @return \Illuminate\Http\Response
114 114
      */
115 115
     public function item($item)
116 116
     {
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
      *
123 123
      * @param \Illuminate\Support\Collection $collection
124 124
      *
125
-     * @return \Illuminate\Http\JsonResponse
125
+     * @return \Illuminate\Http\Response
126 126
      */
127 127
     public function collection(Collection $collection)
128 128
     {
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
      * @param \Illuminate\Pagination\Paginator $paginator
136 136
      * @param \Illuminate\Http\Request         $request
137 137
      *
138
-     * @return \Illuminate\Http\JsonResponse
138
+     * @return \Illuminate\Http\Response
139 139
      */
140 140
     protected function paginator(Paginator $paginator, Request $request)
141 141
     {
@@ -173,9 +173,8 @@  discard block
 block discarded – undo
173 173
     /**
174 174
      * Respond with a no content response.
175 175
      *
176
-     * @param string $message
177 176
      *
178
-     * @return \Illuminate\Http\JsonResponse
177
+     * @return \Illuminate\Http\Response
179 178
      */
180 179
     protected function noContent()
181 180
     {
Please login to merge, or discard this patch.
app/Http/Controllers/Api/IssueController.php 1 patch
Doc Comments   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
      * @param \Symfony\Component\HttpFoundation\Request $request
33 33
      * @param \Illuminate\Contracts\Auth\Guard          $auth
34 34
      *
35
-     * @return \Illuminate\Http\JsonResponse
35
+     * @return \Illuminate\Http\Response
36 36
      */
37 37
     public function getIssues(Request $request, Guard $auth)
38 38
     {
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
      *
49 49
      * @param \Gitamin\Models\Issue $issue
50 50
      *
51
-     * @return \Illuminate\Http\JsonResponse
51
+     * @return \Illuminate\Http\Response
52 52
      */
53 53
     public function getIssue(Issue $issue)
54 54
     {
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
      *
61 61
      * @param \Illuminate\Contracts\Auth\Guard $auth
62 62
      *
63
-     * @return \Illuminate\Http\JsonResponse
63
+     * @return \Illuminate\Http\Response
64 64
      */
65 65
     public function postIssues(Guard $auth)
66 66
     {
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
     /**
86 86
      * Update an existing issue.
87 87
      *
88
-     * @param \Gitamin\Models\Inicdent $issue
88
+     * @param Issue $issue
89 89
      *
90
-     * @return \Illuminate\Http\JsonResponse
90
+     * @return \Illuminate\Http\Response
91 91
      */
92 92
     public function putIssue(Issue $issue)
93 93
     {
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
      *
116 116
      * @param \Gitamin\Models\Issue $issue
117 117
      *
118
-     * @return \Illuminate\Http\JsonResponse
118
+     * @return \Illuminate\Http\Response
119 119
      */
120 120
     public function deleteIssue(Issue $issue)
121 121
     {
Please login to merge, or discard this patch.
app/Http/Controllers/Api/SubscriberController.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
      *
30 30
      * @param \Symfony\Component\HttpFoundation\Request $request
31 31
      *
32
-     * @return \Illuminate\Http\JsonResponse
32
+     * @return \Illuminate\Http\Response
33 33
      */
34 34
     public function getSubscribers(Request $request)
35 35
     {
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
     /**
42 42
      * Create a new subscriber.
43 43
      *
44
-     * @return \Illuminate\Http\JsonResponse
44
+     * @return \Illuminate\Http\Response
45 45
      */
46 46
     public function postSubscribers()
47 47
     {
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
      *
60 60
      * @param \Gitamin\Models\Subscriber $subscriber
61 61
      *
62
-     * @return \Illuminate\Http\JsonResponse
62
+     * @return \Illuminate\Http\Response
63 63
      */
64 64
     public function deleteSubscriber(Subscriber $subscriber)
65 65
     {
Please login to merge, or discard this patch.
app/Http/Controllers/Dashboard/ProjectController.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@
 block discarded – undo
146 146
     /**
147 147
      * Updates a project.
148 148
      *
149
-     * @param \Gitamin\Models\Project $projects
149
+     * @param \Gitamin\Models\Project $project
150 150
      *
151 151
      * @return \Illuminate\Http\RedirectResponse
152 152
      */
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
         $tags = preg_split('/ ?, ?/', $tags);
208 208
 
209 209
         // For every tag, do we need to create it?
210
-        $projectTags = array_map(function ($taggable) use ($project) {
210
+        $projectTags = array_map(function($taggable) use ($project) {
211 211
             return Tag::firstOrCreate(['name' => $taggable])->id;
212 212
         }, $tags);
213 213
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
         $tags = preg_split('/ ?, ?/', $tags);
256 256
 
257 257
         // For every tag, do we need to create it?
258
-        $projectTags = array_map(function ($taggable) use ($project) {
258
+        $projectTags = array_map(function($taggable) use ($project) {
259 259
             return Tag::firstOrCreate(['name' => $taggable])->id;
260 260
         }, $tags);
261 261
 
Please login to merge, or discard this patch.
app/Http/Controllers/RepositoryController.php 3 patches
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -44,6 +44,10 @@
 block discarded – undo
44 44
         return $this->showTree($team.'/'.$project, '');
45 45
     }
46 46
 
47
+    /**
48
+     * @param string $repo
49
+     * @param string $path
50
+     */
47 51
     public function showTree($repo, $path)
48 52
     {
49 53
         $repository = $this->getRepositoryFromName([Setting::get('git_repositories_path')], $repo);
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@
 block discarded – undo
12 12
 namespace Gitamin\Http\Controllers;
13 13
 
14 14
 use Gitamin\Facades\Setting;
15
-use Gitamin\Models\Project;
16 15
 use Gitter\Client;
17 16
 use Illuminate\Routing\Controller;
18 17
 use Illuminate\Support\Facades\View;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -153,7 +153,7 @@
 block discarded – undo
153 153
         }
154 154
 
155 155
         $allRepositories = array_unique($allRepositories, SORT_REGULAR);
156
-        uksort($allRepositories, function ($k1, $k2) {
156
+        uksort($allRepositories, function($k1, $k2) {
157 157
             return strtolower($k2) < strtolower($k1);
158 158
         });
159 159
 
Please login to merge, or discard this patch.