flugger / laravel-responder

src/Console/MakeTransformer.php

Spacing +25 added lines, -25 removed lines

@@ -43,7 +43,7 @@ 
      */
     protected function getStub()
     {
-        if ($this->option('plain')) {
+        if ( $this->option( 'plain' ) ) {
             return __DIR__ . '/../../resources/stubs/transformer.plain.stub';
         }
 
@@ -56,7 +56,7 @@ 
      * @param  string $rootNamespace
      * @return string
      */
-    protected function getDefaultNamespace($rootNamespace)
+    protected function getDefaultNamespace( $rootNamespace )
     {
         return $rootNamespace . '\Transformers';
     }
@@ -67,19 +67,19 @@ 
      * @param  string $name
      * @return string
      */
-    protected function buildClass($name)
+    protected function buildClass( $name )
     {
-        $replace = [];
+        $replace = [ ];
 
-        if (! $this->option('model') && ! $this->option('plain')) {
-            $this->input->setOption('model', $this->resolveModelFromClassName());
+        if ( ! $this->option( 'model' ) && ! $this->option( 'plain' ) ) {
+            $this->input->setOption( 'model', $this->resolveModelFromClassName() );
         }
 
-        if ($this->option('model')) {
-            $replace = $this->buildModelReplacements($replace);
+        if ( $this->option( 'model' ) ) {
+            $replace = $this->buildModelReplacements( $replace );
         }
 
-        return str_replace(array_keys($replace), array_values($replace), parent::buildClass($name));
+        return str_replace( array_keys( $replace ), array_values( $replace ), parent::buildClass( $name ) );
     }
 
     /**
@@ -89,7 +89,7 @@ 
      */
     protected function resolveModelFromClassName()
     {
-        return 'App\\' . str_replace('Transformer', '', array_last(explode('/', $this->getNameInput())));
+        return 'App\\' . str_replace( 'Transformer', '', array_last( explode( '/', $this->getNameInput() ) ) );
     }
 
     /**
@@ -98,19 +98,19 @@ 
      * @param  array $replace
      * @return array
      */
-    protected function buildModelReplacements(array $replace)
+    protected function buildModelReplacements( array $replace )
     {
-        if (! class_exists($modelClass = $this->parseModel($this->option('model')))) {
-            if ($this->confirm("A {$modelClass} model does not exist. Do you want to generate it?", true)) {
-                $this->call('make:model', ['name' => $modelClass]);
+        if ( ! class_exists( $modelClass = $this->parseModel( $this->option( 'model' ) ) ) ) {
+            if ( $this->confirm( "A {$modelClass} model does not exist. Do you want to generate it?", true ) ) {
+                $this->call( 'make:model', [ 'name' => $modelClass ] );
             }
         }
 
-        return array_merge($replace, [
+        return array_merge( $replace, [
             'DummyFullModelClass' => $modelClass,
-            'DummyModelClass' => class_basename($modelClass),
-            'DummyModelVariable' => lcfirst(class_basename($modelClass)),
-        ]);
+            'DummyModelClass' => class_basename( $modelClass ),
+            'DummyModelVariable' => lcfirst( class_basename( $modelClass ) ),
+        ] );
     }
 
     /**
@@ -119,15 +119,15 @@ 
      * @param  string $model
      * @return string
      */
-    protected function parseModel($model)
+    protected function parseModel( $model )
     {
-        if (preg_match('([^A-Za-z0-9_/\\\\])', $model)) {
-            throw new InvalidArgumentException('Model name contains invalid characters.');
+        if ( preg_match( '([^A-Za-z0-9_/\\\\])', $model ) ) {
+            throw new InvalidArgumentException( 'Model name contains invalid characters.' );
         }
 
-        $model = trim(str_replace('/', '\\', $model), '\\');
+        $model = trim( str_replace( '/', '\\', $model ), '\\' );
 
-        if (! Str::startsWith($model, $rootNamespace = $this->laravel->getNamespace())) {
+        if ( ! Str::startsWith( $model, $rootNamespace = $this->laravel->getNamespace() ) ) {
             $model = $rootNamespace . $model;
         }
 
@@ -142,8 +142,8 @@ 
     protected function getOptions()
     {
         return [
-            ['model', 'm', InputOption::VALUE_OPTIONAL, 'Generate a model transformer.'],
-            ['plain', 'p', InputOption::VALUE_NONE, 'Generate a plain transformer.'],
+            [ 'model', 'm', InputOption::VALUE_OPTIONAL, 'Generate a model transformer.' ],
+            [ 'plain', 'p', InputOption::VALUE_NONE, 'Generate a plain transformer.' ],
         ];
     }
 }
\ No newline at end of file

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;