@@ -22,6 +22,9 @@ discard block |
||
| 22 | 22 | $this->convertToLines($result); |
| 23 | 23 | } |
| 24 | 24 | |
| 25 | + /** |
|
| 26 | + * @param string $result |
|
| 27 | + */ |
|
| 25 | 28 | private function addSuggestionFix($result) |
| 26 | 29 | { |
| 27 | 30 | $arr = explode('<br />', nl2br($result)); |
@@ -47,6 +50,9 @@ discard block |
||
| 47 | 50 | } |
| 48 | 51 | } |
| 49 | 52 | |
| 53 | + /** |
|
| 54 | + * @param string $result |
|
| 55 | + */ |
|
| 50 | 56 | private function convertToLines($result) |
| 51 | 57 | { |
| 52 | 58 | $arr = explode('<br />', nl2br($result)); |
@@ -95,7 +95,7 @@ discard block |
||
| 95 | 95 | * |
| 96 | 96 | * @param \Illuminate\Http\Request $request |
| 97 | 97 | * |
| 98 | - * @return \Illuminate\Http\Response |
|
| 98 | + * @return \Illuminate\Http\RedirectResponse |
|
| 99 | 99 | */ |
| 100 | 100 | public function store(IssueRequest $request) |
| 101 | 101 | { |
@@ -112,7 +112,6 @@ discard block |
||
| 112 | 112 | /** |
| 113 | 113 | * Display the specified resource. |
| 114 | 114 | * |
| 115 | - * @param int $id |
|
| 116 | 115 | * |
| 117 | 116 | * @return \Illuminate\Http\Response |
| 118 | 117 | */ |
@@ -141,7 +140,6 @@ discard block |
||
| 141 | 140 | /** |
| 142 | 141 | * Show the form for editing the specified resource. |
| 143 | 142 | * |
| 144 | - * @param int $id |
|
| 145 | 143 | * |
| 146 | 144 | * @return \Illuminate\Http\Response |
| 147 | 145 | */ |
@@ -173,9 +171,8 @@ discard block |
||
| 173 | 171 | * Update the specified resource in storage. |
| 174 | 172 | * |
| 175 | 173 | * @param \Illuminate\Http\Request $request |
| 176 | - * @param int $id |
|
| 177 | 174 | * |
| 178 | - * @return \Illuminate\Http\Response |
|
| 175 | + * @return \Illuminate\Http\RedirectResponse |
|
| 179 | 176 | */ |
| 180 | 177 | public function update(IssueRequest $request, $slug) |
| 181 | 178 | { |
@@ -212,9 +209,8 @@ discard block |
||
| 212 | 209 | /** |
| 213 | 210 | * Remove the specified resource from storage. |
| 214 | 211 | * |
| 215 | - * @param int $id |
|
| 216 | 212 | * |
| 217 | - * @return \Illuminate\Http\Response |
|
| 213 | + * @return \Illuminate\Http\RedirectResponse |
|
| 218 | 214 | */ |
| 219 | 215 | public function destroy($slug) |
| 220 | 216 | { |
@@ -62,9 +62,9 @@ discard block |
||
| 62 | 62 | /** |
| 63 | 63 | * Store a newly created resource in storage. |
| 64 | 64 | * |
| 65 | - * @param GitScrum\Http\Requests\SprintRequest $request |
|
| 65 | + * @param SprintRequest $request |
|
| 66 | 66 | * |
| 67 | - * @return \Illuminate\Http\Response |
|
| 67 | + * @return \Illuminate\Http\RedirectResponse |
|
| 68 | 68 | */ |
| 69 | 69 | public function store(SprintRequest $request) |
| 70 | 70 | { |
@@ -111,7 +111,6 @@ discard block |
||
| 111 | 111 | /** |
| 112 | 112 | * Show the form for editing the specified resource. |
| 113 | 113 | * |
| 114 | - * @param int $id |
|
| 115 | 114 | * |
| 116 | 115 | * @return \Illuminate\Http\Response |
| 117 | 116 | */ |
@@ -128,9 +127,8 @@ discard block |
||
| 128 | 127 | * Update the specified resource in storage. |
| 129 | 128 | * |
| 130 | 129 | * @param \Illuminate\Http\Request $request |
| 131 | - * @param int $id |
|
| 132 | 130 | * |
| 133 | - * @return \Illuminate\Http\Response |
|
| 131 | + * @return \Illuminate\Http\RedirectResponse |
|
| 134 | 132 | */ |
| 135 | 133 | public function update(SprintRequest $request, $slug) |
| 136 | 134 | { |
@@ -144,9 +142,8 @@ discard block |
||
| 144 | 142 | /** |
| 145 | 143 | * Remove the specified resource from storage. |
| 146 | 144 | * |
| 147 | - * @param int $id |
|
| 148 | 145 | * |
| 149 | - * @return \Illuminate\Http\Response |
|
| 146 | + * @return \Illuminate\Http\RedirectResponse |
|
| 150 | 147 | */ |
| 151 | 148 | public function destroy(Request $request) |
| 152 | 149 | { |
@@ -70,7 +70,6 @@ |
||
| 70 | 70 | /** |
| 71 | 71 | * Display the specified resource. |
| 72 | 72 | * |
| 73 | - * @param int $id |
|
| 74 | 73 | * |
| 75 | 74 | * @return \Illuminate\Http\Response |
| 76 | 75 | */ |
@@ -87,9 +87,8 @@ |
||
| 87 | 87 | * Update the specified resource in storage. |
| 88 | 88 | * |
| 89 | 89 | * @param \Illuminate\Http\Request $request |
| 90 | - * @param int $id |
|
| 91 | 90 | * |
| 92 | - * @return \Illuminate\Http\Response |
|
| 91 | + * @return \Illuminate\Http\RedirectResponse|null |
|
| 93 | 92 | */ |
| 94 | 93 | public function update(Request $request, $slug) |
| 95 | 94 | { |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | * |
| 53 | 53 | * @param \Illuminate\Http\Request $request |
| 54 | 54 | * |
| 55 | - * @return \Illuminate\Http\Response |
|
| 55 | + * @return \Illuminate\Http\RedirectResponse |
|
| 56 | 56 | */ |
| 57 | 57 | public function store(Request $request) |
| 58 | 58 | { |
@@ -65,7 +65,6 @@ discard block |
||
| 65 | 65 | /** |
| 66 | 66 | * Display the specified resource. |
| 67 | 67 | * |
| 68 | - * @param int $id |
|
| 69 | 68 | * |
| 70 | 69 | * @return \Illuminate\Http\Response |
| 71 | 70 | */ |
@@ -82,7 +81,6 @@ discard block |
||
| 82 | 81 | /** |
| 83 | 82 | * Show the form for editing the specified resource. |
| 84 | 83 | * |
| 85 | - * @param int $id |
|
| 86 | 84 | * |
| 87 | 85 | * @return \Illuminate\Http\Response |
| 88 | 86 | */ |
@@ -105,9 +103,8 @@ discard block |
||
| 105 | 103 | * Update the specified resource in storage. |
| 106 | 104 | * |
| 107 | 105 | * @param \Illuminate\Http\Request $request |
| 108 | - * @param int $id |
|
| 109 | 106 | * |
| 110 | - * @return \Illuminate\Http\Response |
|
| 107 | + * @return \Illuminate\Http\RedirectResponse |
|
| 111 | 108 | */ |
| 112 | 109 | public function update(Request $request, $slug) |
| 113 | 110 | { |
@@ -74,6 +74,9 @@ |
||
| 74 | 74 | return Carbon::createFromFormat('Y-m-d H:i:s', $this->attributes['created_at'])->diffForHumans(); |
| 75 | 75 | } |
| 76 | 76 | |
| 77 | + /** |
|
| 78 | + * @param string $alias |
|
| 79 | + */ |
|
| 77 | 80 | public function track($alias, $model) |
| 78 | 81 | { |
| 79 | 82 | if (!isset($model->config_status_id)) { |
@@ -68,9 +68,9 @@ discard block |
||
| 68 | 68 | |
| 69 | 69 | public function issuesHasUsers() |
| 70 | 70 | { |
| 71 | - $users = $this->issues->map(function ($issue) { |
|
| 71 | + $users = $this->issues->map(function($issue) { |
|
| 72 | 72 | return $issue->users; |
| 73 | - })->reject(function ($value) { |
|
| 73 | + })->reject(function($value) { |
|
| 74 | 74 | return $value == null; |
| 75 | 75 | })->flatten(1)->unique('id')->splice(0, 3); |
| 76 | 76 | |
@@ -106,11 +106,11 @@ discard block |
||
| 106 | 106 | |
| 107 | 107 | public function pullrequests() |
| 108 | 108 | { |
| 109 | - $prs = $this->branches->map(function ($branch) { |
|
| 109 | + $prs = $this->branches->map(function($branch) { |
|
| 110 | 110 | if ($branch->pullrequests->count()) { |
| 111 | 111 | return $branch->pullrequests; |
| 112 | 112 | } |
| 113 | - })->reject(function ($value) { |
|
| 113 | + })->reject(function($value) { |
|
| 114 | 114 | return $value == null; |
| 115 | 115 | }); |
| 116 | 116 | |
@@ -135,9 +135,9 @@ discard block |
||
| 135 | 135 | |
| 136 | 136 | public function totalAdditions() |
| 137 | 137 | { |
| 138 | - $additions = $this->branches->map(function ($branch) { |
|
| 138 | + $additions = $this->branches->map(function($branch) { |
|
| 139 | 139 | return $branch->commits; |
| 140 | - })->flatten(1)->map(function ($commit) { |
|
| 140 | + })->flatten(1)->map(function($commit) { |
|
| 141 | 141 | return $commit->files; |
| 142 | 142 | })->flatten(1)->sum('additions'); |
| 143 | 143 | |
@@ -146,7 +146,7 @@ discard block |
||
| 146 | 146 | |
| 147 | 147 | public function totalPullRequests() |
| 148 | 148 | { |
| 149 | - $prs = $this->branches->map(function ($branch) { |
|
| 149 | + $prs = $this->branches->map(function($branch) { |
|
| 150 | 150 | return $branch->pullrequests()->count(); |
| 151 | 151 | }); |
| 152 | 152 | |
@@ -239,9 +239,9 @@ discard block |
||
| 239 | 239 | public function activities() |
| 240 | 240 | { |
| 241 | 241 | $activities = $this->issues() |
| 242 | - ->with('statuses')->get()->map(function ($issue) { |
|
| 242 | + ->with('statuses')->get()->map(function($issue) { |
|
| 243 | 243 | return $issue->statuses; |
| 244 | - })->flatten(1)->map(function ($statuses) { |
|
| 244 | + })->flatten(1)->map(function($statuses) { |
|
| 245 | 245 | return $statuses; |
| 246 | 246 | })->sortByDesc('created_at'); |
| 247 | 247 | |
@@ -252,9 +252,9 @@ discard block |
||
| 252 | 252 | |
| 253 | 253 | public function issueTypes() |
| 254 | 254 | { |
| 255 | - $types = $this->issues->map(function ($issue) { |
|
| 255 | + $types = $this->issues->map(function($issue) { |
|
| 256 | 256 | return $issue->type; |
| 257 | - })->groupBy('slug')->map(function ($type) { |
|
| 257 | + })->groupBy('slug')->map(function($type) { |
|
| 258 | 258 | return [ |
| 259 | 259 | 'sprint' => $this->slug, |
| 260 | 260 | 'slug' => $type->first()->slug, |
@@ -268,7 +268,7 @@ discard block |
||
| 268 | 268 | |
| 269 | 269 | public function issueStatus() |
| 270 | 270 | { |
| 271 | - $status = $this->issues->map(function ($issue) { |
|
| 271 | + $status = $this->issues->map(function($issue) { |
|
| 272 | 272 | return $issue->status; |
| 273 | 273 | })->groupBy('slug')->all(); |
| 274 | 274 | |
@@ -74,7 +74,7 @@ discard block |
||
| 74 | 74 | |
| 75 | 75 | public function totalLines() |
| 76 | 76 | { |
| 77 | - $lines = $this->files->map(function ($file) { |
|
| 77 | + $lines = $this->files->map(function($file) { |
|
| 78 | 78 | return count(preg_split('/\R/', $file->raw)); |
| 79 | 79 | }); |
| 80 | 80 | |
@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | |
| 84 | 84 | public function totalAdditions() |
| 85 | 85 | { |
| 86 | - $additions = $this->files->map(function ($file) { |
|
| 86 | + $additions = $this->files->map(function($file) { |
|
| 87 | 87 | return $file->additions; |
| 88 | 88 | }); |
| 89 | 89 | |
@@ -92,7 +92,7 @@ discard block |
||
| 92 | 92 | |
| 93 | 93 | public function totalChanges() |
| 94 | 94 | { |
| 95 | - $changes = $this->files->map(function ($file) { |
|
| 95 | + $changes = $this->files->map(function($file) { |
|
| 96 | 96 | return $file->changes; |
| 97 | 97 | }); |
| 98 | 98 | |
@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | |
| 102 | 102 | public function totalDeletions() |
| 103 | 103 | { |
| 104 | - $deletions = $this->files->map(function ($file) { |
|
| 104 | + $deletions = $this->files->map(function($file) { |
|
| 105 | 105 | return $file->deletions; |
| 106 | 106 | }); |
| 107 | 107 | |
@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | |
| 111 | 111 | public function totalPHPCS($type = 'ERROR') |
| 112 | 112 | { |
| 113 | - $errors = $this->files->map(function ($file) use ($type) { |
|
| 113 | + $errors = $this->files->map(function($file) use ($type) { |
|
| 114 | 114 | return $file->filePhpcs()->where('type', '=', $type)->groupBy('type')->count(); |
| 115 | 115 | }); |
| 116 | 116 | |