Completed
Pull Request — master (#45)
by Phecho
04:16
created
app/Http/Controllers/Dashboard/GroupsController.php 1 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.
app/Http/Controllers/GroupsController.php 2 patches
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.
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -137,7 +137,6 @@  discard block
 block discarded – undo
137 137
     /**
138 138
      * Shows the edit project namespace view.
139 139
      *
140
-     * @param \Gitamin\Models\Owner $namespace
141 140
      *
142 141
      * @return \Illuminate\View\View
143 142
      */
@@ -153,7 +152,6 @@  discard block
 block discarded – undo
153 152
     /**
154 153
      * Updates a project namespace.
155 154
      *
156
-     * @param \Gitamin\Models\Owner $namespace
157 155
      *
158 156
      * @return \Illuminate\Http\RedirectResponse
159 157
      */
Please login to merge, or discard this patch.
app/Http/Controllers/Projects/IssuesController.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -13,8 +13,6 @@
 block discarded – undo
13 13
 
14 14
 use Gitamin\Commands\Issue\AddIssueCommand;
15 15
 use Gitamin\Http\Controllers\Controller;
16
-use Gitamin\Models\Issue;
17
-use Gitamin\Models\Project;
18 16
 use GrahamCampbell\Binput\Facades\Binput;
19 17
 use Illuminate\Support\Facades\Auth;
20 18
 use Illuminate\Support\Facades\Redirect;
Please login to merge, or discard this patch.
app/Commands/Owner/UpdateOwnerCommand.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
     /**
66 66
      * Create an add project owner command instance.
67 67
      *
68
-     * @param \Gitamin\Models\ProjectNamepsace $owner
68
+     * @param Owner $owner
69 69
      * @param string                           $name
70 70
      * @param string                           $path
71 71
      * @param int                              $user_id
Please login to merge, or discard this patch.
app/Commands/Project/AddProjectCommand.php 1 patch
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,6 @@
 block discarded – undo
78 78
      * @param string $path
79 79
      * @param int    $creator_id
80 80
      * @param int    $owner_id
81
-     * @param bool   $issues_enabled
82 81
      *
83 82
      * @return void
84 83
      */
Please login to merge, or discard this patch.
app/Http/Controllers/Api/OwnerController.php 1 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/Controllers/FeedController.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
     /**
72 72
      * Generates a feed of all issues.
73 73
      *
74
-     * @param \Gitamin\Models\Owner|null $owner
74
+     * @param Owner $owner
75 75
      * @param bool                       $isRss
76 76
      *
77 77
      * @return \Illuminate\Http\Response
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -76,16 +76,16 @@
 block discarded – undo
76 76
      *
77 77
      * @return \Illuminate\Http\Response
78 78
      */
79
-    private function feedAction(Owner &$owner, $isRss)
79
+    private function feedAction(Owner & $owner, $isRss)
80 80
     {
81 81
         if ($owner->exists) {
82
-            $owner->projects->map(function ($project) {
83
-                $project->issues()->visible()->orderBy('created_at', 'desc')->get()->map(function ($issue) use ($isRss) {
82
+            $owner->projects->map(function($project) {
83
+                $project->issues()->visible()->orderBy('created_at', 'desc')->get()->map(function($issue) use ($isRss) {
84 84
                     $this->feedAddItem($issue, $isRss);
85 85
                 });
86 86
             });
87 87
         } else {
88
-            Issue::visible()->orderBy('created_at', 'desc')->get()->map(function ($issue) use ($isRss) {
88
+            Issue::visible()->orderBy('created_at', 'desc')->get()->map(function($issue) use ($isRss) {
89 89
                 $this->feedAddItem($issue, $isRss);
90 90
             });
91 91
         }
Please login to merge, or discard this patch.
app/Handlers/Commands/Owner/RemoveOwnerCommandHandler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
         event(new OwnerWasRemovedEvent($owner));
31 31
 
32 32
         // Remove the owner id from all project.
33
-        $owner->projects->map(function ($project) {
33
+        $owner->projects->map(function($project) {
34 34
             $project->update(['owner_id' => 0]);
35 35
         });
36 36
 
Please login to merge, or discard this patch.
app/Http/Controllers/ProjectsController.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,6 @@
 block discarded – undo
87 87
     /**
88 88
      * Display the specified resource.
89 89
      *
90
-     * @param string $owner
91 90
      * @param string $project_path
92 91
      *
93 92
      * @return \Illuminate\Http\Response
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
         $tags = preg_split('/ ?, ?/', $tags);
75 75
 
76 76
         // For every tag, do we need to create it?
77
-        $projectTags = array_map(function ($taggable) use ($project) {
77
+        $projectTags = array_map(function($taggable) use ($project) {
78 78
             return Tag::firstOrCreate(['name' => $taggable])->id;
79 79
         }, $tags);
80 80
 
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
      */
95 95
     public function showAction($owner_path, $project_path)
96 96
     {
97
-        $project = Project::findByPath($owner_path,$project_path);
97
+        $project = Project::findByPath($owner_path, $project_path);
98 98
 
99 99
         return View::make('projects.show')
100 100
             ->withPageTitle($project->name)
Please login to merge, or discard this patch.