Skip to content

Commit

Permalink
Merge bbda970 into 72724e8
Browse files Browse the repository at this point in the history
  • Loading branch information
antonkomarev committed Jun 30, 2019
2 parents 72724e8 + bbda970 commit ec18c3f
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 15 deletions.
2 changes: 2 additions & 0 deletions .scrutinizer.yml
Expand Up @@ -6,3 +6,5 @@ coding_style:
spaces:
before_parentheses:
closure_definition: true
around_operators:
concatenation: true
5 changes: 2 additions & 3 deletions src/Http/Controllers/Api/Jobs/Collect/Response.php
Expand Up @@ -15,7 +15,6 @@

use Illuminate\Contracts\Support\Responsable as ResponsableContract;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;

final class Response implements ResponsableContract
{
Expand All @@ -34,10 +33,10 @@ public function __construct(iterable $jobs)
*/
public function toResponse($request)
{
return $this->toJson($request);
return $this->toJson();
}

private function toJson(Request $request): JsonResponse
private function toJson(): JsonResponse
{
$jobs = [];
foreach ($this->jobs as $job) {
Expand Down
5 changes: 2 additions & 3 deletions src/Http/Controllers/Api/Jobs/Get/Response.php
Expand Up @@ -16,7 +16,6 @@
use Cog\Contracts\Paket\Job\Entities\Job as JobContract;
use Illuminate\Contracts\Support\Responsable as ResponsableContract;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;

final class Response implements ResponsableContract
{
Expand All @@ -35,10 +34,10 @@ public function __construct(JobContract $job)
*/
public function toResponse($request)
{
return $this->toJson($request);
return $this->toJson();
}

private function toJson(Request $request): JsonResponse
private function toJson(): JsonResponse
{
return response()->json($this->job->toArray());
}
Expand Down
5 changes: 2 additions & 3 deletions src/Http/Controllers/Api/Jobs/Post/Response.php
Expand Up @@ -16,7 +16,6 @@
use Cog\Contracts\Paket\Job\Entities\Job as JobContract;
use Illuminate\Contracts\Support\Responsable as ResponsableContract;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;

final class Response implements ResponsableContract
{
Expand All @@ -35,10 +34,10 @@ public function __construct(JobContract $job)
*/
public function toResponse($request)
{
return $this->toJson($request);
return $this->toJson();
}

private function toJson(Request $request): JsonResponse
private function toJson(): JsonResponse
{
return response()->json($this->job->toArray(), 201);
}
Expand Down
5 changes: 2 additions & 3 deletions src/Http/Controllers/Api/Requirements/Collect/Response.php
Expand Up @@ -15,7 +15,6 @@

use Illuminate\Contracts\Support\Responsable as ResponsableContract;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;

final class Response implements ResponsableContract
{
Expand All @@ -34,10 +33,10 @@ public function __construct(iterable $requirements)
*/
public function toResponse($request)
{
return $this->toJson($request);
return $this->toJson();
}

private function toJson(Request $request): JsonResponse
private function toJson(): JsonResponse
{
return response()->json($this->requirements);
}
Expand Down
5 changes: 2 additions & 3 deletions src/Http/Controllers/App/Response.php
Expand Up @@ -14,7 +14,6 @@
namespace Cog\Laravel\Paket\Http\Controllers\App;

use Illuminate\Contracts\Support\Responsable as ResponsableContract;
use Illuminate\Http\Request;
use Illuminate\Http\Response as IlluminateResponse;
use Illuminate\Support\Facades\Config;

Expand All @@ -28,10 +27,10 @@ final class Response implements ResponsableContract
*/
public function toResponse($request)
{
return $this->toHtml($request);
return $this->toHtml();
}

private function toHtml(Request $request): IlluminateResponse
private function toHtml(): IlluminateResponse
{
return response()->view('paket::app', [
'cssFile' => 'app.css',
Expand Down

0 comments on commit ec18c3f

Please sign in to comment.