flugger / laravel-responder

src/Transformers/Concerns/HasRelationships.php

Spacing +20 added lines, -20 removed lines

@@ -21,14 +21,14 @@ 
      *
      * @var string[]
      */
-    protected $relations = ['*'];
+    protected $relations = [ '*' ];
 
     /**
      * A list of autoloaded default relations.
      *
      * @var array
      */
-    protected $load = [];
+    protected $load = [ ];
 
     /**
      * Indicates if all relations are allowed.
@@ -37,7 +37,7 @@ 
      */
     public function allowsAllRelations(): bool
     {
-        return $this->relations == ['*'];
+        return $this->relations == [ '*' ];
     }
 
     /**
@@ -47,7 +47,7 @@ 
      */
     public function getRelations(): array
     {
-        return array_filter($this->relations, function ($relation) {
+        return array_filter( $this->relations, function ( $relation ) {
             return $relation != '*';
         });
     }
@@ -59,7 +59,7 @@ 
      */
     public function getDefaultRelations(): array
     {
-        return array_keys($this->load);
+        return array_keys( $this->load );
     }
 
     /**
@@ -69,14 +69,14 @@ 
      */
     public function extractDefaultRelations(): array
     {
-        return collect($this->getDefaultRelationsWithEagerLoads())
-            ->merge(collect($this->load)->map(function ($transformer, $relation) {
-                return collect($this->resolveContainer()->make($transformer)->extractDefaultRelations())
+        return collect( $this->getDefaultRelationsWithEagerLoads() )
+            ->merge( collect( $this->load )->map( function ( $transformer, $relation ) {
+                return collect( $this->resolveContainer()->make( $transformer )->extractDefaultRelations() )
                     ->keys()
-                    ->map(function ($nestedRelation) use ($relation) {
+                    ->map( function ( $nestedRelation ) use ($relation) {
                         return "$relation.$nestedRelation";
                     });
-            }))
+            }) )
             ->all();
     }
 
@@ -87,12 +87,12 @@ 
      */
     protected function getDefaultRelationsWithEagerLoads(): array
     {
-        return collect($this->load)->keys()->mapWithKeys(function ($relation) {
-            if (method_exists($this, $method = 'load' . ucfirst($relation))) {
-                return [$relation => $this->makeEagerLoadCallback($method)];
+        return collect( $this->load )->keys()->mapWithKeys( function ( $relation ) {
+            if ( method_exists( $this, $method = 'load' . ucfirst( $relation ) ) ) {
+                return [ $relation => $this->makeEagerLoadCallback( $method ) ];
             }
 
-            return [$relation => function () { }];
+            return [ $relation => function () { } ];
         })->all();
     }
 
@@ -102,10 +102,10 @@ 
      * @param  string $method
      * @return \Closure
      */
-    protected function makeEagerLoadCallback(string $method): Closure
+    protected function makeEagerLoadCallback( string $method ): Closure
     {
-        return function ($query) use ($method) {
-            return $this->$method($query);
+        return function ( $query ) use ($method) {
+            return $this->$method( $query );
         };
     }
 
@@ -116,10 +116,10 @@ 
      * @param  string                             $identifier
      * @return mixed
      */
-    protected function resolveRelation(Model $model, string $identifier)
+    protected function resolveRelation( Model $model, string $identifier )
     {
-        if (method_exists($this, $method = 'filter' . ucfirst($identifier))) {
-            return $this->$method($model->$identifier);
+        if ( method_exists( $this, $method = 'filter' . ucfirst( $identifier ) ) ) {
+            return $this->$method( $model->$identifier );
         }
 
         return $model->$identifier;

src/Transformers/Concerns/MakesResources.php

Spacing +16 added lines, -16 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @var \League\Fractal\ResourceInterface[]
      */
-    protected $resources = [];
+    protected $resources = [ ];
 
     /**
      * Make a resource.
@@ -31,11 +31,11 @@ 
      * @param  string|null                                                    $resourceKey
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    protected function resource($data = null, $transformer = null, string $resourceKey = null): ResourceInterface
+    protected function resource( $data = null, $transformer = null, string $resourceKey = null ): ResourceInterface
     {
-        $resourceFactory = $this->resolveContainer()->make(ResourceFactory::class);
+        $resourceFactory = $this->resolveContainer()->make( ResourceFactory::class );
 
-        return $resourceFactory->make($data, $transformer, $resourceKey);
+        return $resourceFactory->make( $data, $transformer, $resourceKey );
     }
 
     /**
@@ -47,14 +47,14 @@ 
      * @return \League\Fractal\Resource\ResourceInterface
      * @throws \LogicException
      */
-    protected function includeResource(string $identifier, $data, array $parameters): ResourceInterface
+    protected function includeResource( string $identifier, $data, array $parameters ): ResourceInterface
     {
-        if (method_exists($this, $method = 'include' . ucfirst($identifier))) {
-            $resource = $this->$method($data, $parameters);
-        } elseif ($data instanceof Model) {
-            $resource = $this->includeResourceFromModel($data, $identifier);
+        if ( method_exists( $this, $method = 'include' . ucfirst( $identifier ) ) ) {
+            $resource = $this->$method( $data, $parameters );
+        } elseif ( $data instanceof Model ) {
+            $resource = $this->includeResourceFromModel( $data, $identifier );
         } else {
-            throw new LogicException('Relation [' . $identifier . '] not found in [' . get_class($this) . '].');
+            throw new LogicException( 'Relation [' . $identifier . '] not found in [' . get_class( $this ) . '].' );
         }
 
         return $resource;
@@ -67,15 +67,15 @@ 
      * @param string                              $identifier
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    protected function includeResourceFromModel(Model $model, string $identifier): ResourceInterface
+    protected function includeResourceFromModel( Model $model, string $identifier ): ResourceInterface
     {
-        $data = $this->resolveRelation($model, $identifier);
+        $data = $this->resolveRelation( $model, $identifier );
 
-        if (key_exists($identifier, $this->resources)) {
-            return $this->resources[$identifier]->setData($data);
+        if ( key_exists( $identifier, $this->resources ) ) {
+            return $this->resources[ $identifier ]->setData( $data );
         }
 
-        return $this->resources[$identifier] = $this->resource($data, null, $identifier);
+        return $this->resources[ $identifier ] = $this->resource( $data, null, $identifier );
     }
 
     /**
@@ -92,5 +92,5 @@ 
      * @param  string                              $identifier
      * @return mixed
      */
-    protected abstract function resolveRelation(Model $model, string $identifier);
+    protected abstract function resolveRelation( Model $model, string $identifier );
 }
\ No newline at end of file

src/Transformers/Concerns/OverridesFractal.php

Spacing +15 added lines, -15 removed lines

@@ -22,8 +22,8 @@ 
      */
     public function getAvailableIncludes()
     {
-        if ($this->allowsAllRelations()) {
-            return $this->resolveScopedIncludes($this->getCurrentScope());
+        if ( $this->allowsAllRelations() ) {
+            return $this->resolveScopedIncludes( $this->getCurrentScope() );
         }
 
         return $this->getRelations();
@@ -47,11 +47,11 @@ 
      * @param  mixed                 $data
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    protected function callIncludeMethod(Scope $scope, $identifier, $data)
+    protected function callIncludeMethod( Scope $scope, $identifier, $data )
     {
-        $parameters = $this->resolveScopedParameters($scope, $identifier);
+        $parameters = $this->resolveScopedParameters( $scope, $identifier );
 
-        return $this->includeResource($identifier, $data, $parameters);
+        return $this->includeResource( $identifier, $data, $parameters );
     }
 
     /**
@@ -60,16 +60,16 @@ 
      * @param  \League\Fractal\Scope $scope
      * @return array
      */
-    protected function resolveScopedIncludes(Scope $scope): array
+    protected function resolveScopedIncludes( Scope $scope ): array
     {
-        $level = count($scope->getParentScopes());
+        $level = count( $scope->getParentScopes() );
         $includes = $scope->getManager()->getRequestedIncludes();
 
-        return Collection::make($includes)->map(function ($include) {
-            return explode('.', $include);
-        })->filter(function ($include) use ($level) {
-            return count($include) > $level;
-        })->pluck($level)->unique()->all();
+        return Collection::make( $includes )->map( function ( $include ) {
+            return explode( '.', $include );
+        })->filter( function ( $include ) use ($level) {
+            return count( $include ) > $level;
+        })->pluck( $level )->unique()->all();
     }
 
     /**
@@ -79,9 +79,9 @@ 
      * @param  string                $identifier
      * @return array
      */
-    protected function resolveScopedParameters(Scope $scope, string $identifier): array
+    protected function resolveScopedParameters( Scope $scope, string $identifier ): array
     {
-        return iterator_to_array($scope->getManager()->getIncludeParams($scope->getIdentifier($identifier)));
+        return iterator_to_array( $scope->getManager()->getIncludeParams( $scope->getIdentifier( $identifier ) ) );
     }
 
     /**
@@ -120,5 +120,5 @@ 
      * @param  array  $parameters
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    protected abstract function includeResource(string $identifier, $data, array $parameters): ResourceInterface;
+    protected abstract function includeResource( string $identifier, $data, array $parameters ): ResourceInterface;
 }
\ No newline at end of file

src/Http/Responses/ResponseBuilder.php

Spacing +16 added lines, -16 removed lines

@@ -36,7 +36,7 @@ 
      *
      * @param \Flugg\Responder\Contracts\ResponseFactory $responseFactory
      */
-    public function __construct(ResponseFactory $responseFactory)
+    public function __construct( ResponseFactory $responseFactory )
     {
         $this->responseFactory = $responseFactory;
     }
@@ -47,12 +47,12 @@ 
      * @param  string[]|string $decorator
      * @return $this
      */
-    public function decorator($decorator)
+    public function decorator( $decorator )
     {
-        $decorators = is_array($decorator) ? $decorator : func_get_args();
+        $decorators = is_array( $decorator ) ? $decorator : func_get_args();
 
-        foreach ($decorators as $decorator) {
-            $this->responseFactory = new $decorator($this->responseFactory);
+        foreach ( $decorators as $decorator ) {
+            $this->responseFactory = new $decorator( $this->responseFactory );
         };
 
         return $this;
@@ -65,13 +65,13 @@ 
      * @param  array    $headers
      * @return \Illuminate\Http\JsonResponse
      */
-    public function respond(int $status = null, array $headers = []): JsonResponse
+    public function respond( int $status = null, array $headers = [ ] ): JsonResponse
     {
-        if (! is_null($status)) {
-            $this->setStatusCode($status);
+        if ( ! is_null( $status ) ) {
+            $this->setStatusCode( $status );
         }
 
-        return $this->responseFactory->make($this->getOutput(), $this->status, $headers);
+        return $this->responseFactory->make( $this->getOutput(), $this->status, $headers );
     }
 
     /**
@@ -81,7 +81,7 @@ 
      */
     public function toArray(): array
     {
-        return $this->respond()->getData(true);
+        return $this->respond()->getData( true );
     }
 
     /**
@@ -91,7 +91,7 @@ 
      */
     public function toCollection(): Collection
     {
-        return new Collection($this->toArray());
+        return new Collection( $this->toArray() );
     }
 
     /**
@@ -100,9 +100,9 @@ 
      * @param  int $options
      * @return string
      */
-    public function toJson($options = 0): string
+    public function toJson( $options = 0 ): string
     {
-        return json_encode($this->toArray(), $options);
+        return json_encode( $this->toArray(), $options );
     }
 
     /**
@@ -111,9 +111,9 @@ 
      * @param  int $status
      * @return void
      */
-    protected function setStatusCode(int $status)
+    protected function setStatusCode( int $status )
     {
-        $this->validateStatusCode($this->status = $status);
+        $this->validateStatusCode( $this->status = $status );
     }
 
     /**
@@ -129,5 +129,5 @@ 
      * @param  int $status
      * @return void
      */
-    abstract protected function validateStatusCode(int $status);
+    abstract protected function validateStatusCode( int $status );
 }

src/Http/Responses/ErrorResponseBuilder.php

Spacing +9 added lines, -9 removed lines

@@ -56,11 +56,11 @@ 
      * @param \Flugg\Responder\Contracts\ResponseFactory $responseFactory
      * @param \Flugg\Responder\Contracts\ErrorFactory    $errorFactory
      */
-    public function __construct(ResponseFactory $responseFactory, ErrorFactory $errorFactory)
+    public function __construct( ResponseFactory $responseFactory, ErrorFactory $errorFactory )
     {
         $this->errorFactory = $errorFactory;
 
-        parent::__construct($responseFactory);
+        parent::__construct( $responseFactory );
     }
 
     /**
@@ -70,7 +70,7 @@ 
      * @param  string|null $message
      * @return $this
      */
-    public function error(string $errorCode = null, string $message = null)
+    public function error( string $errorCode = null, string $message = null )
     {
         $this->errorCode = $errorCode;
         $this->message = $message;
@@ -84,9 +84,9 @@ 
      * @param  array $data
      * @return $this
      */
-    public function data(array $data)
+    public function data( array $data )
     {
-        $this->data = array_merge((array) $this->data, $data);
+        $this->data = array_merge( (array) $this->data, $data );
 
         return $this;
     }
@@ -98,7 +98,7 @@ 
      */
     protected function getOutput(): array
     {
-        return $this->errorFactory->make($this->errorCode, $this->message, $this->data);
+        return $this->errorFactory->make( $this->errorCode, $this->message, $this->data );
     }
 
     /**
@@ -108,10 +108,10 @@ 
      * @return void
      * @throws \InvalidArgumentException
      */
-    protected function validateStatusCode(int $status)
+    protected function validateStatusCode( int $status )
     {
-        if ($status < 400 || $status >= 600) {
-            throw new InvalidArgumentException("{$status} is not a valid error HTTP status code.");
+        if ( $status < 400 || $status >= 600 ) {
+            throw new InvalidArgumentException( "{$status} is not a valid error HTTP status code." );
         }
     }
 }

src/ErrorMessageResolver.php

Spacing +10 added lines, -10 removed lines

@@ -26,14 +26,14 @@ 
      *
      * @var array
      */
-    protected $messages = [];
+    protected $messages = [ ];
 
     /**
      * Construct the resolver class.
      *
      * @param \Illuminate\Contracts\Translation\Translator $translator
      */
-    public function __construct(Translator $translator)
+    public function __construct( Translator $translator )
     {
         $this->translator = $translator;
     }
@@ -45,11 +45,11 @@ 
      * @param  string $message
      * @return void
      */
-    public function register(string $errorCode, string $message)
+    public function register( string $errorCode, string $message )
     {
-        $this->messages = array_merge($this->messages, is_array($errorCode) ? $errorCode : [
+        $this->messages = array_merge( $this->messages, is_array( $errorCode ) ? $errorCode : [
             $errorCode => $message,
-        ]);
+        ] );
     }
 
     /**
@@ -58,14 +58,14 @@ 
      * @param  string $errorCode
      * @return string|null
      */
-    public function resolve(string $errorCode)
+    public function resolve( string $errorCode )
     {
-        if (key_exists($errorCode, $this->messages)) {
-            return $this->messages[$errorCode];
+        if ( key_exists( $errorCode, $this->messages ) ) {
+            return $this->messages[ $errorCode ];
         }
 
-        if ($this->translator->has($errorCode = "errors.$errorCode")) {
-            return $this->translator->trans($errorCode);
+        if ( $this->translator->has( $errorCode = "errors.$errorCode" ) ) {
+            return $this->translator->trans( $errorCode );
         }
 
         return null;

src/FractalTransformFactory.php

Spacing +36 added lines, -36 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @param \League\Fractal\Manager $manager
      */
-    public function __construct(Manager $manager)
+    public function __construct( Manager $manager )
     {
         $this->manager = $manager;
     }
@@ -42,15 +42,15 @@ 
      * @param  array                                         $options
      * @return array
      */
-    public function make(ResourceInterface $resource, SerializerAbstract $serializer, array $options = []): array
+    public function make( ResourceInterface $resource, SerializerAbstract $serializer, array $options = [ ] ): array
     {
-        $options = $this->parseOptions($options, $resource);
+        $options = $this->parseOptions( $options, $resource );
 
-        return $this->manager->setSerializer($serializer)
-            ->parseIncludes($options['includes'])
-            ->parseExcludes($options['excludes'])
-            ->parseFieldsets($options['fieldsets'])
-            ->createData($resource)
+        return $this->manager->setSerializer( $serializer )
+            ->parseIncludes( $options[ 'includes' ] )
+            ->parseExcludes( $options[ 'excludes' ] )
+            ->parseFieldsets( $options[ 'fieldsets' ] )
+            ->createData( $resource )
             ->toArray();
     }
 
@@ -61,20 +61,20 @@ 
      * @param  \League\Fractal\Resource\ResourceInterface $resource
      * @return array
      */
-    protected function parseOptions(array $options, ResourceInterface $resource): array
+    protected function parseOptions( array $options, ResourceInterface $resource ): array
     {
-        $options = array_merge([
-            'includes' => [],
-            'excludes' => [],
-            'fieldsets' => [],
-        ], $options);
-
-        if (! empty($options['fieldsets'])) {
-            if (is_null($resourceKey = $resource->getResourceKey())) {
-                throw new LogicException('Filtering fields using sparse fieldsets require resource key to be set.');
+        $options = array_merge( [
+            'includes' => [ ],
+            'excludes' => [ ],
+            'fieldsets' => [ ],
+        ], $options );
+
+        if ( ! empty($options[ 'fieldsets' ]) ) {
+            if ( is_null( $resourceKey = $resource->getResourceKey() ) ) {
+                throw new LogicException( 'Filtering fields using sparse fieldsets require resource key to be set.' );
             }
 
-            $options['fieldsets'] = $this->parseFieldsets($options['fieldsets'], $resourceKey, $options['includes']);
+            $options[ 'fieldsets' ] = $this->parseFieldsets( $options[ 'fieldsets' ], $resourceKey, $options[ 'includes' ] );
         }
 
         return $options;
@@ -88,19 +88,19 @@ 
      * @param  array  $includes
      * @return array
      */
-    protected function parseFieldsets(array $fieldsets, string $resourceKey, array $includes): array
+    protected function parseFieldsets( array $fieldsets, string $resourceKey, array $includes ): array
     {
-        $includes = array_map(function ($include) use ($resourceKey) {
+        $includes = array_map( function ( $include ) use ($resourceKey) {
             return "$resourceKey.$include";
-        }, $includes);
+        }, $includes );
 
-        foreach ($fieldsets as $key => $fields) {
-            if (is_numeric($key)) {
-                unset($fieldsets[$key]);
+        foreach ( $fieldsets as $key => $fields ) {
+            if ( is_numeric( $key ) ) {
+                unset($fieldsets[ $key ]);
                 $key = $resourceKey;
             }
 
-            $fieldsets[$key] = $this->parseFieldset($key, (array) $fields, $includes);
+            $fieldsets[ $key ] = $this->parseFieldset( $key, (array) $fields, $includes );
         }
 
         return $fieldsets;
@@ -114,13 +114,13 @@ 
      * @param  array  $includes
      * @return string
      */
-    protected function parseFieldset(string $key, array $fields, array $includes): string
+    protected function parseFieldset( string $key, array $fields, array $includes ): string
     {
-        $childIncludes = array_reduce($includes, function ($segments, $include) use ($key) {
-            return array_merge($segments, $this->resolveChildIncludes($key, $include));
-        }, []);
+        $childIncludes = array_reduce( $includes, function ( $segments, $include ) use ($key) {
+            return array_merge( $segments, $this->resolveChildIncludes( $key, $include ) );
+        }, [ ] );
 
-        return implode(',', array_merge($fields, array_unique($childIncludes)));
+        return implode( ',', array_merge( $fields, array_unique( $childIncludes ) ) );
     }
 
     /**
@@ -130,14 +130,14 @@ 
      * @param  string $include
      * @return array
      */
-    protected function resolveChildIncludes($key, string $include): array
+    protected function resolveChildIncludes( $key, string $include ): array
     {
-        if (count($segments = explode('.', $include)) <= 1) {
-            return [];
+        if ( count( $segments = explode( '.', $include ) ) <= 1 ) {
+            return [ ];
         }
 
-        $relation = $key === array_shift($segments) ? [$segments[0]] : [];
+        $relation = $key === array_shift( $segments ) ? [ $segments[ 0 ] ] : [ ];
 
-        return array_merge($relation, $this->resolveChildIncludes($key, implode('.', $segments)));
+        return array_merge( $relation, $this->resolveChildIncludes( $key, implode( '.', $segments ) ) );
     }
 }
\ No newline at end of file

src/Resources/ResourceFactory.php

Spacing +20 added lines, -20 removed lines

@@ -40,7 +40,7 @@ 
      * @param \Flugg\Responder\Resources\DataNormalizer                   $normalizer
      * @param \Flugg\Responder\Contracts\Transformers\TransformerResolver $transformerResolver
      */
-    public function __construct(DataNormalizer $normalizer, TransformerResolver $transformerResolver)
+    public function __construct( DataNormalizer $normalizer, TransformerResolver $transformerResolver )
     {
         $this->normalizer = $normalizer;
         $this->transformerResolver = $transformerResolver;
@@ -54,17 +54,17 @@ 
      * @param  string|null                                                    $resourceKey
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    public function make($data = null, $transformer = null, string $resourceKey = null): ResourceInterface
+    public function make( $data = null, $transformer = null, string $resourceKey = null ): ResourceInterface
     {
-        if ($data instanceof ResourceInterface) {
-            return $data->setTransformer($this->resolveTransformer($data->getData(), $transformer ?: $data->getTransformer()));
-        } elseif (is_null($data = $this->normalizer->normalize($data))) {
-            return $this->instatiateResource($data);
+        if ( $data instanceof ResourceInterface ) {
+            return $data->setTransformer( $this->resolveTransformer( $data->getData(), $transformer ?: $data->getTransformer() ) );
+        } elseif ( is_null( $data = $this->normalizer->normalize( $data ) ) ) {
+            return $this->instatiateResource( $data );
         }
 
-        $transformer = $this->resolveTransformer($data, $transformer);
+        $transformer = $this->resolveTransformer( $data, $transformer );
 
-        return $this->instatiateResource($data, $transformer, $resourceKey);
+        return $this->instatiateResource( $data, $transformer, $resourceKey );
     }
 
     /**
@@ -74,13 +74,13 @@ 
      * @param  \Flugg\Responder\Transformers\Transformer|string|callable|null $transformer
      * @return \Flugg\Responder\Transformers\Transformer|callable
      */
-    protected function resolveTransformer($data, $transformer)
+    protected function resolveTransformer( $data, $transformer )
     {
-        if (isset($transformer)) {
-            return $this->transformerResolver->resolve($transformer);
+        if ( isset($transformer) ) {
+            return $this->transformerResolver->resolve( $transformer );
         }
 
-        return $this->transformerResolver->resolveFromData($data);
+        return $this->transformerResolver->resolveFromData( $data );
     }
 
     /**
@@ -91,15 +91,15 @@ 
      * @param  string|null                                             $resourceKey
      * @return \League\Fractal\Resource\ResourceInterface
      */
-    protected function instatiateResource($data, $transformer = null, string $resourceKey = null): ResourceInterface
+    protected function instatiateResource( $data, $transformer = null, string $resourceKey = null ): ResourceInterface
     {
-        if (is_null($data)) {
+        if ( is_null( $data ) ) {
             return new NullResource;
-        } elseif ($this->shouldCreateCollection($data)) {
-            return new CollectionResource($data, $transformer, $resourceKey);
+        } elseif ( $this->shouldCreateCollection( $data ) ) {
+            return new CollectionResource( $data, $transformer, $resourceKey );
         }
 
-        return new ItemResource($data, $transformer, $resourceKey);
+        return new ItemResource( $data, $transformer, $resourceKey );
     }
 
     /**
@@ -108,10 +108,10 @@ 
      * @param  mixed $data
      * @return bool
      */
-    protected function shouldCreateCollection($data): bool
+    protected function shouldCreateCollection( $data ): bool
     {
-        if (is_array($data)) {
-            return ! is_scalar(Arr::first($data));
+        if ( is_array( $data ) ) {
+            return ! is_scalar( Arr::first( $data ) );
         }
 
         return $data instanceof Traversable;