flugger / laravel-responder

src/TransformBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -158,7 +158,7 @@ 
     /**
      * Include relations to the transform.
      *
-     * @param  string[]|string $relations
+     * @param  string[] $relations
      * @return self
      */
     public function with($relations): TransformBuilder
@@ -171,7 +171,7 @@ 
     /**
      * Exclude relations from the transform.
      *
-     * @param  string[]|string $relations
+     * @param  string[] $relations
      * @return self
      */
     public function without($relations): TransformBuilder
@@ -197,7 +197,7 @@ 
     /**
      * Set the serializer.
      *
-     * @param  \League\Fractal\Serializer\SerializerAbstract|string $serializer
+     * @param  Serializers\NullSerializer $serializer
      * @return self
      * @throws \Flugg\Responder\Exceptions\InvalidSerializerException
      */

Spacing +39 added lines, -39 removed lines

@@ -64,21 +64,21 @@ 
      *
      * @var array
      */
-    protected $with = [];
+    protected $with = [ ];
 
     /**
      * A list of excluded relations.
      *
      * @var array
      */
-    protected $without = [];
+    protected $without = [ ];
 
     /**
      * A list of sparse fieldsets.
      *
      * @var array
      */
-    protected $only = [];
+    protected $only = [ ];
 
     /**
      * Construct the builder class.
@@ -87,7 +87,7 @@ 
      * @param \Flugg\Responder\Contracts\TransformFactory            $transformFactory
      * @param \Flugg\Responder\Contracts\Pagination\PaginatorFactory $paginatorFactory
      */
-    public function __construct(ResourceFactory $resourceFactory, TransformFactory $transformFactory, PaginatorFactory $paginatorFactory)
+    public function __construct( ResourceFactory $resourceFactory, TransformFactory $transformFactory, PaginatorFactory $paginatorFactory )
     {
         $this->resourceFactory = $resourceFactory;
         $this->transformFactory = $transformFactory;
@@ -103,14 +103,14 @@ 
      * @param  string|null                                                    $resourceKey
      * @return self
      */
-    public function resource($data = null, $transformer = null, string $resourceKey = null): TransformBuilder
+    public function resource( $data = null, $transformer = null, string $resourceKey = null ): TransformBuilder
     {
-        $this->resource = $this->resourceFactory->make($data, $transformer, $resourceKey);
+        $this->resource = $this->resourceFactory->make( $data, $transformer, $resourceKey );
 
-        if ($data instanceof CursorPaginator) {
-            $this->cursor($this->paginatorFactory->makeCursor($data));
-        } elseif ($data instanceof LengthAwarePaginator) {
-            $this->paginator($this->paginatorFactory->make($data));
+        if ( $data instanceof CursorPaginator ) {
+            $this->cursor( $this->paginatorFactory->makeCursor( $data ) );
+        } elseif ( $data instanceof LengthAwarePaginator ) {
+            $this->paginator( $this->paginatorFactory->make( $data ) );
         }
 
         return $this;
@@ -122,9 +122,9 @@ 
      * @param  \League\Fractal\Pagination\Cursor $cursor
      * @return self
      */
-    public function cursor(Cursor $cursor): TransformBuilder
+    public function cursor( Cursor $cursor ): TransformBuilder
     {
-        $this->resource->setCursor($cursor);
+        $this->resource->setCursor( $cursor );
 
         return $this;
     }
@@ -135,9 +135,9 @@ 
      * @param  \League\Fractal\Pagination\IlluminatePaginatorAdapter $paginator
      * @return self
      */
-    public function paginator(IlluminatePaginatorAdapter $paginator): TransformBuilder
+    public function paginator( IlluminatePaginatorAdapter $paginator ): TransformBuilder
     {
-        $this->resource->setPaginator($paginator);
+        $this->resource->setPaginator( $paginator );
 
         return $this;
     }
@@ -148,9 +148,9 @@ 
      * @param  array $meta
      * @return self
      */
-    public function meta(array $meta): TransformBuilder
+    public function meta( array $meta ): TransformBuilder
     {
-        $this->resource->setMeta($meta);
+        $this->resource->setMeta( $meta );
 
         return $this;
     }
@@ -161,9 +161,9 @@ 
      * @param  string[]|string $relations
      * @return self
      */
-    public function with($relations): TransformBuilder
+    public function with( $relations ): TransformBuilder
     {
-        $this->with = array_merge($this->with, is_array($relations) ? $relations : func_get_args());
+        $this->with = array_merge( $this->with, is_array( $relations ) ? $relations : func_get_args() );
 
         return $this;
     }
@@ -174,9 +174,9 @@ 
      * @param  string[]|string $relations
      * @return self
      */
-    public function without($relations): TransformBuilder
+    public function without( $relations ): TransformBuilder
     {
-        $this->without = array_merge($this->without, is_array($relations) ? $relations : func_get_args());
+        $this->without = array_merge( $this->without, is_array( $relations ) ? $relations : func_get_args() );
 
         return $this;
     }
@@ -187,9 +187,9 @@ 
      * @param  string[]|string $fields
      * @return self
      */
-    public function only($fields): TransformBuilder
+    public function only( $fields ): TransformBuilder
     {
-        $this->only = array_merge($this->only, is_array($fields) ? $fields : func_get_args());
+        $this->only = array_merge( $this->only, is_array( $fields ) ? $fields : func_get_args() );
 
         return $this;
     }
@@ -201,13 +201,13 @@ 
      * @return self
      * @throws \Flugg\Responder\Exceptions\InvalidSerializerException
      */
-    public function serializer($serializer): TransformBuilder
+    public function serializer( $serializer ): TransformBuilder
     {
-        if (is_string($serializer)) {
+        if ( is_string( $serializer ) ) {
             $serializer = new $serializer;
         }
 
-        if (! $serializer instanceof SerializerAbstract) {
+        if ( ! $serializer instanceof SerializerAbstract ) {
             throw new InvalidSerializerException;
         }
 
@@ -225,11 +225,11 @@ 
     {
         $this->prepareRelations();
 
-        return $this->transformFactory->make($this->resource, $this->serializer, [
+        return $this->transformFactory->make( $this->resource, $this->serializer, [
             'includes' => $this->with,
             'excludes' => $this->without,
             'fields' => $this->only,
-        ]);
+        ] );
     }
 
     /**
@@ -239,10 +239,10 @@ 
      */
     protected function prepareRelations()
     {
-        $this->setDefaultIncludes($this->resource->getTransformer());
-        $this->eagerLoadIfApplicable($this->resource->getData());
+        $this->setDefaultIncludes( $this->resource->getTransformer() );
+        $this->eagerLoadIfApplicable( $this->resource->getData() );
 
-        $this->with = $this->trimEagerLoadFunctions($this->with);
+        $this->with = $this->trimEagerLoadFunctions( $this->with );
     }
 
     /**
@@ -251,10 +251,10 @@ 
      * @param \Flugg\Responder\Transformers\Transformer|callable $transformer
      * @return void
      */
-    protected function setDefaultIncludes($transformer)
+    protected function setDefaultIncludes( $transformer )
     {
-        if ($transformer instanceof Transformer) {
-            $this->with($transformer->extractDefaultRelations());
+        if ( $transformer instanceof Transformer ) {
+            $this->with( $transformer->extractDefaultRelations() );
         }
     }
 
@@ -264,10 +264,10 @@ 
      * @param  mixed $data
      * @return void
      */
-    protected function eagerLoadIfApplicable($data)
+    protected function eagerLoadIfApplicable( $data )
     {
-        if ($data instanceof Model || $data instanceof Collection) {
-            $data->load($this->with);
+        if ( $data instanceof Model || $data instanceof Collection ) {
+            $data->load( $this->with );
         }
     }
 
@@ -277,10 +277,10 @@ 
      * @param  array $relations
      * @return void
      */
-    protected function trimEagerLoadFunctions(array $relations)
+    protected function trimEagerLoadFunctions( array $relations )
     {
-        return collect($relations)->map(function ($value, $key) {
-            return is_numeric($key) ? $value : $key;
+        return collect( $relations )->map( function ( $value, $key ) {
+            return is_numeric( $key ) ? $value : $key;
         })->values()->all();
     }
 }
\ No newline at end of file

src/Transformers/TransformerResolver.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
     /**
      * Make a simple closure transformer.
      *
-     * @return callable
+     * @return \Closure
      */
     protected function makeClosureTransformer(): callable
     {

Spacing +21 added lines, -21 removed lines

@@ -29,14 +29,14 @@ 
      *
      * @var array
      */
-    protected $bindings = [];
+    protected $bindings = [ ];
 
     /**
      * Construct the resolver class.
      *
      * @param \Illuminate\Contracts\Container\Container $container
      */
-    public function __construct(Container $container)
+    public function __construct( Container $container )
     {
         $this->container = $container;
     }
@@ -48,11 +48,11 @@ 
      * @param  string|callback $transformer
      * @return void
      */
-    public function bind($transformable, $transformer)
+    public function bind( $transformable, $transformer )
     {
-        $this->bindings = array_merge($this->bindings, is_array($transformable) ? $transformable : [
+        $this->bindings = array_merge( $this->bindings, is_array( $transformable ) ? $transformable : [
             $transformable => $transformer,
-        ]);
+        ] );
     }
 
     /**
@@ -62,13 +62,13 @@ 
      * @return \Flugg\Responder\Transformers\Transformer|callable
      * @throws \Flugg\Responder\Exceptions\InvalidTransformerException
      */
-    public function resolve($transformer)
+    public function resolve( $transformer )
     {
-        if (is_string($transformer)) {
-            return $this->container->make($transformer);
+        if ( is_string( $transformer ) ) {
+            return $this->container->make( $transformer );
         }
 
-        if (! is_callable($transformer) && ! $transformer instanceof Transformer) {
+        if ( ! is_callable( $transformer ) && ! $transformer instanceof Transformer ) {
             throw new InvalidTransformerException;
         }
 
@@ -81,11 +81,11 @@ 
      * @param  mixed $data
      * @return \Flugg\Responder\Transformers\Transformer|callable|null
      */
-    public function resolveFromData($data)
+    public function resolveFromData( $data )
     {
-        if ($transformable = $this->resolveTransformable($data)) {
-            if ($transformer = $this->resolveTransformer()) {
-                return $this->resolve($transformer);
+        if ( $transformable = $this->resolveTransformable( $data ) ) {
+            if ( $transformer = $this->resolveTransformer() ) {
+                return $this->resolve( $transformer );
             }
         }
 
@@ -98,11 +98,11 @@ 
      * @param  mixed $data
      * @return \Flugg\Responder\Contracts\Transformable|null
      */
-    protected function resolveTransformable($data)
+    protected function resolveTransformable( $data )
     {
-        if (is_array($data) || $data instanceof Traversable) {
-            foreach ($data as $item) {
-                if ($item instanceof Transformable) {
+        if ( is_array( $data ) || $data instanceof Traversable ) {
+            foreach ( $data as $item ) {
+                if ( $item instanceof Transformable ) {
                     return $item;
                 }
             }
@@ -117,10 +117,10 @@ 
      * @param  \Flugg\Responder\Contracts\Transformable $transformable
      * @return \Flugg\Responder\Contracts\Transformable|null
      */
-    protected function resolveTransformer(Transformable $transformable)
+    protected function resolveTransformer( Transformable $transformable )
     {
-        if (key_exists($this->bindings, get_class($transformable))) {
-            return $this->bindings[get_class($transformable)];
+        if ( key_exists( $this->bindings, get_class( $transformable ) ) ) {
+            return $this->bindings[ get_class( $transformable ) ];
         }
 
         return $transformable->transformer();
@@ -133,7 +133,7 @@ 
      */
     protected function makeClosureTransformer(): callable
     {
-        return function ($data) {
+        return function ( $data ) {
             return $data instanceof Arrayable ? $data->toArray() : $data;
         };
     }

src/FractalTransformFactory.php

Spacing +7 added lines, -7 removed lines

@@ -28,7 +28,7 @@ 
      *
      * @param \League\Fractal\Manager $manager
      */
-    public function __construct(Manager $manager)
+    public function __construct( Manager $manager )
     {
         $this->manager = $manager;
     }
@@ -41,13 +41,13 @@ 
      * @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
     {
-        return $this->manager->setSerializer($serializer)
-            ->parseIncludes($options['includes'] ?? [])
-            ->parseExcludes($options['excludes'] ?? [])
-            ->parseFieldsets($options['fields'] ?? [])
-            ->createData($resource)
+        return $this->manager->setSerializer( $serializer )
+            ->parseIncludes( $options[ 'includes' ] ?? [ ] )
+            ->parseExcludes( $options[ 'excludes' ] ?? [ ] )
+            ->parseFieldsets( $options[ 'fields' ] ?? [ ] )
+            ->createData( $resource )
             ->toArray();
     }
 }
\ No newline at end of file

src/Testing/MakesApiRequests.php

Spacing +40 added lines, -40 removed lines

@@ -21,10 +21,10 @@ 
      * @param  int   $status
      * @return $this
      */
-    protected function seeSuccess($data = null, $status = 200)
+    protected function seeSuccess( $data = null, $status = 200 )
     {
-        $response = $this->seeSuccessResponse($data, $status);
-        $this->seeSuccessData($response->getData(true)['data']);
+        $response = $this->seeSuccessResponse( $data, $status );
+        $this->seeSuccessData( $response->getData( true )[ 'data' ] );
 
         return $this;
     }
@@ -36,10 +36,10 @@ 
      * @param  int   $status
      * @return $this
      */
-    protected function seeSuccessEquals($data = null, $status = 200)
+    protected function seeSuccessEquals( $data = null, $status = 200 )
     {
-        $response = $this->seeSuccessResponse($data, $status);
-        $this->seeJsonEquals($response->getData(true));
+        $response = $this->seeSuccessResponse( $data, $status );
+        $this->seeJsonEquals( $response->getData( true ) );
 
         return $this;
     }
@@ -50,11 +50,11 @@ 
      * @param  mixed $data
      * @return $this
      */
-    protected function seeSuccessStructure($data = null)
+    protected function seeSuccessStructure( $data = null )
     {
-        $this->seeJsonStructure([
+        $this->seeJsonStructure( [
             'data' => $data,
-        ]);
+        ] );
 
         return $this;
     }
@@ -66,14 +66,14 @@ 
      * @param  int   $status
      * @return \Illuminate\Http\JsonResponse
      */
-    protected function seeSuccessResponse($data = null, $status = 200): JsonResponse
+    protected function seeSuccessResponse( $data = null, $status = 200 ): JsonResponse
     {
-        $response = $this->app->make(Responder::class)->success($data, $status);
+        $response = $this->app->make( Responder::class )->success( $data, $status );
 
-        $this->seeStatusCode($response->getStatusCode())->seeJson([
+        $this->seeStatusCode( $response->getStatusCode() )->seeJson( [
             'success' => true,
             'status' => $response->getStatusCode(),
-        ])->seeJsonStructure(['data']);
+        ] )->seeJsonStructure( [ 'data' ] );
 
         return $response;
     }
@@ -84,13 +84,13 @@ 
      * @param  mixed $data
      * @return $this
      */
-    protected function seeSuccessData($data = null)
+    protected function seeSuccessData( $data = null )
     {
-        collect($data)->each(function ($value, $key) {
-            if (is_array($value)) {
-                $this->seeSuccessData($value);
+        collect( $data )->each( function ( $value, $key ) {
+            if ( is_array( $value ) ) {
+                $this->seeSuccessData( $value );
             } else {
-                $this->seeJson([$key => $value]);
+                $this->seeJson( [ $key => $value ] );
             }
         });
 
@@ -103,20 +103,20 @@ 
      * @param  string|array|null $attributes
      * @return array
      */
-    protected function getSuccessData($attributes = null)
+    protected function getSuccessData( $attributes = null )
     {
-        $rawData = $this->decodeResponseJson()['data'];
+        $rawData = $this->decodeResponseJson()[ 'data' ];
 
-        if (is_null($attributes)) {
+        if ( is_null( $attributes ) ) {
             return $rawData;
-        } elseif (is_string($attributes)) {
-            return array_get($rawData, $attributes);
+        } elseif ( is_string( $attributes ) ) {
+            return array_get( $rawData, $attributes );
         }
 
-        $data = [];
+        $data = [ ];
 
-        foreach ($attributes as $attribute) {
-            $data[] = array_get($rawData, $attribute);
+        foreach ( $attributes as $attribute ) {
+            $data[ ] = array_get( $rawData, $attribute );
         }
 
         return $data;
@@ -129,25 +129,25 @@ 
      * @param  int|null $status
      * @return $this
      */
-    protected function seeError(string $error, int $status = null)
+    protected function seeError( string $error, int $status = null )
     {
-        if (! is_null($status)) {
-            $this->seeStatusCode($status);
+        if ( ! is_null( $status ) ) {
+            $this->seeStatusCode( $status );
         }
 
-        if ($this->app->config->get('responder.status_code')) {
-            $this->seeJson([
+        if ( $this->app->config->get( 'responder.status_code' ) ) {
+            $this->seeJson( [
                 'status' => $status,
-            ]);
+            ] );
         }
 
-        return $this->seeJson([
+        return $this->seeJson( [
             'success' => false,
-        ])->seeJsonSubset([
+        ] )->seeJsonSubset( [
             'error' => [
                 'code' => $error,
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -156,7 +156,7 @@ 
      * @param  int $status
      * @return $this
      */
-    abstract protected function seeStatusCode($status);
+    abstract protected function seeStatusCode( $status );
 
     /**
      * Assert that the response contains JSON.
@@ -165,7 +165,7 @@ 
      * @param  bool       $negate
      * @return $this
      */
-    abstract public function seeJson(array $data = null, $negate = false);
+    abstract public function seeJson( array $data = null, $negate = false );
 
     /**
      * Assert that the JSON response has a given structure.
@@ -174,7 +174,7 @@ 
      * @param  array|null $responseData
      * @return $this
      */
-    abstract public function seeJsonStructure(array $structure = null, $responseData = null);
+    abstract public function seeJsonStructure( array $structure = null, $responseData = null );
 
     /**
      * Assert that the response is a superset of the given JSON.
@@ -182,7 +182,7 @@ 
      * @param  array $data
      * @return $this
      */
-    abstract protected function seeJsonSubset(array $data);
+    abstract protected function seeJsonSubset( array $data );
 
     /**
      * Assert that the response contains an exact JSON array.
@@ -190,7 +190,7 @@ 
      * @param  array $data
      * @return $this
      */
-    abstract public function seeJsonEquals(array $data);
+    abstract public function seeJsonEquals( array $data );
 
     /**
      * Validate and return the decoded response JSON.

src/ErrorMessageResolver.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @param \Illuminate\Contracts\Translation\Translator $translator
      */
-    public function __construct(Translator $translator)
+    public function __construct( Translator $translator )
     {
         $this->translator = $translator;
     }
@@ -37,12 +37,12 @@ 
      * @param  string $errorCode
      * @return string|null
      */
-    public function resolve(string $errorCode)
+    public function resolve( string $errorCode )
     {
-        if (! $this->translator->has($errorCode = "errors.$errorCode")) {
+        if ( ! $this->translator->has( $errorCode = "errors.$errorCode" ) ) {
             return null;
         }
 
-        return $this->translator->trans($errorCode);
+        return $this->translator->trans( $errorCode );
     }
 }
\ No newline at end of file

src/Transformer.php

Spacing +6 added lines, -6 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @param \Flugg\Responder\TransformBuilder $transformBuilder
      */
-    public function __construct(TransformBuilder $transformBuilder)
+    public function __construct( TransformBuilder $transformBuilder )
     {
         $this->transformBuilder = $transformBuilder;
     }
@@ -40,12 +40,12 @@ 
      * @param  string[]                                                       $without
      * @return array
      */
-    public function transform($data = null, $transformer = null, array $with = [], array $without = []): array
+    public function transform( $data = null, $transformer = null, array $with = [ ], array $without = [ ] ): array
     {
-        return $this->transformBuilder->resource($data, $transformer)
-            ->with($with)
-            ->without($without)
-            ->serializer(new NullSerializer)
+        return $this->transformBuilder->resource( $data, $transformer )
+            ->with( $with )
+            ->without( $without )
+            ->serializer( new NullSerializer )
             ->transform();
     }
 }
\ No newline at end of file

src/Exceptions/InvalidSerializerException.php

Spacing +1 added lines, -1 removed lines

@@ -19,6 +19,6 @@
      */
     public function __construct()
     {
-        parent::__construct('Given serializer must be an instance of [' . SerializerAbstract::class . '].');
+        parent::__construct( 'Given serializer must be an instance of [' . SerializerAbstract::class . '].' );
     }
 }
\ No newline at end of file

src/Exceptions/Http/ValidationFailedException.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      *
      * @param \Illuminate\Contracts\Validation\Validator $validator
      */
-    public function __construct(Validator $validator)
+    public function __construct( Validator $validator )
     {
         $this->validator = $validator;
 
@@ -54,6 +54,6 @@ 
      */
     public function data()
     {
-        return [$this->validator->getMessageBag()->toArray()];
+        return [ $this->validator->getMessageBag()->toArray() ];
     }
 }
\ No newline at end of file

src/Exceptions/InvalidTransformerException.php

Spacing +1 added lines, -1 removed lines

@@ -19,6 +19,6 @@
      */
     public function __construct()
     {
-        parent::__construct('Given transformer must be a callable or an instance of [' . Transformer::class . '].');
+        parent::__construct( 'Given transformer must be a callable or an instance of [' . Transformer::class . '].' );
     }
 }
\ No newline at end of file