kamilabs / api-core-bundle

RequestProcessor/Step/Common/PersistStep.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     {
         $entity = $this->getArtifact('entity');
 
-        if(true !== $this->getArtifact('validation')) {
+        if (true !== $this->getArtifact('validation')) {
             return new ArtifactCollection();
         }
 

Tests/RequestProcessor/Step/Filter/ValidateFiltersTest.php

Spacing +1 added lines, -1 removed lines

@@ -23,6 +23,6 @@
 
         $response = $step->execute($request);
         $this->assertInstanceOf(ArtifactCollection::class, $response);
-        $this->assertEquals([0 => ['type' => 'eq','property' => 'title','value' => 'test']], $response->get('filters')->getValue());
+        $this->assertEquals([0 => ['type' => 'eq', 'property' => 'title', 'value' => 'test']], $response->get('filters')->getValue());
     }
 }

Stenographer/Stenographer.php

Braces +3 added lines, -1 removed lines

@@ -216,7 +216,9 @@
                 $annotation instanceof OneToMany ||
                 $annotation instanceof ManyToOne ||
                 $annotation instanceof ManyToMany
-            ) $targetEntity = $annotation->targetEntity;
+            ) {
+                $targetEntity = $annotation->targetEntity;
+            }
         }
 
         return str_replace('_', '-', Inflector::tableize(substr($targetEntity, strripos($targetEntity, '\\') + 1)));

Spacing +7 added lines, -9 removed lines

@@ -177,7 +177,7 @@ 
     private function getReturnedParams($route, $anonymousClass, $canAccessClass)
     {
         $reflection = $this->getEntityReflection($route->getDefault('_entity'));
-        $availableParams = $reflection->getName() . "\n";
+        $availableParams = $reflection->getName()."\n";
 
         foreach ($reflection->getProperties() as $property) {
             $anonymous = $this->reader->getPropertyAnnotation($property, $anonymousClass);
@@ -189,10 +189,9 @@ 
                 $availableAccess = $access ? $access->roles : $availableAccess;
                 $availableAccess = implode(', ', $availableAccess);
 
-                $availableParams .= Inflector::tableize($property->getName()) . ':';
-                $availableParams .= $relation ? '<a href="#operations-tag-'.$this->getRelatedPropertyModelName($property).'">Related model</a>' :
-                    ($column ? $column->type : 'Unknown');
-                $availableParams .= ', Access: ' . $availableAccess . "\n";
+                $availableParams .= Inflector::tableize($property->getName()).':';
+                $availableParams .= $relation ? '<a href="#operations-tag-'.$this->getRelatedPropertyModelName($property).'">Related model</a>' : ($column ? $column->type : 'Unknown');
+                $availableParams .= ', Access: '.$availableAccess."\n";
             }
         }
 
@@ -246,8 +245,7 @@ 
 
                 $param['description'] = $relation ?
                     '<a href="#operations-tag-'.$this->getRelatedPropertyModelName($property).'">'.
-                    'Related model description</a><pre>Access: ' . $availableAccess . '</pre>' :
-                    '<pre>Access: ' . $availableAccess . '</pre>';
+                    'Related model description</a><pre>Access: '.$availableAccess.'</pre>' : '<pre>Access: '.$availableAccess.'</pre>';
 
                 $availableParams[] = $param;
             }
@@ -259,8 +257,8 @@ 
     private function getRelatedPropertyModelName($property)
     {
         $annotations = $this->reader->getPropertyAnnotations($property);
-        foreach ($annotations as $annotation){
-            if(
+        foreach ($annotations as $annotation) {
+            if (
                 $annotation instanceof OneToOne ||
                 $annotation instanceof OneToMany ||
                 $annotation instanceof ManyToOne ||

RequestProcessor/Step/Common/SortStep.php

Spacing +9 added lines, -12 removed lines

@@ -50,23 +50,20 @@
         $reflection = $this->getArtifact('reflection');
         $query = 'e.%s';
 
-        if(strripos($sort, '.')){
+        if (strripos($sort, '.')) {
             $sort = explode('.', $sort);
-            if($reflection->hasProperty(Inflector::camelize($sort[0]))){
+            if ($reflection->hasProperty(Inflector::camelize($sort[0]))) {
                 $property = $reflection->getProperty(Inflector::camelize($sort[0]));
-                $ann = $this->reader->getPropertyAnnotation($property, OneToOne::class)?:
-                    $this->reader->getPropertyAnnotation($property, OneToMany::class)?:
-                        $this->reader->getPropertyAnnotation($property, ManyToMany::class)?:
-                            $this->reader->getPropertyAnnotation($property, ManyToOne::class)?:null;
-                if($ann){
+                $ann = $this->reader->getPropertyAnnotation($property, OneToOne::class) ?: $this->reader->getPropertyAnnotation($property, OneToMany::class) ?: $this->reader->getPropertyAnnotation($property, ManyToMany::class) ?: $this->reader->getPropertyAnnotation($property, ManyToOne::class) ?: null;
+                if ($ann) {
                     $reflection = new \ReflectionClass($ann->targetEntity);
-                    $query = $sort[0] . '.%s';
+                    $query = $sort[0].'.%s';
                     $sort = Inflector::camelize(($sort[1]));
-                } else{
-                    throw new BadRequestHttpException('Related property ' . Inflector::camelize($sort[0]) . ' isn\'t related model in sort field');
+                } else {
+                    throw new BadRequestHttpException('Related property '.Inflector::camelize($sort[0]).' isn\'t related model in sort field');
                 }
-            } else{
-                throw new BadRequestHttpException('Related property ' . Inflector::camelize($sort[0]) . ' not found in sort field');
+            } else {
+                throw new BadRequestHttpException('Related property '.Inflector::camelize($sort[0]).' not found in sort field');
             }
         }
 

Tests/DependencyInjection/ConfigurationTest.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
 class ConfigurationTest extends TestCase
 {
-    public function testCanByConstructed ()
+    public function testCanByConstructed()
     {
         $configuration = new Configuration();
         $treeBuilder = $configuration->getConfigTreeBuilder();

RequestProcessor/Step/Filter/FilterStep.php

Spacing +6 added lines, -9 removed lines

@@ -68,7 +68,7 @@ 
             /** @var \ReflectionClass $reflection */
             $reflection = $this->getArtifact('reflection');
 
-            if(strpos($config['property'], '.')) {
+            if (strpos($config['property'], '.')) {
                 $this->applyFilterToRelation($config, $reflection, $queryBuilder, '_'.$index);
             } else {
                 $this->applyRegularFilter($config, $reflection, $queryBuilder, '_'.$index);
@@ -119,14 +119,14 @@ 
                 $this->checkAccessForRelatedProperty($reflection, $propertyRelatedModel);
                 $config['property'] = Inflector::camelize($propertyRelatedModel);
                 $shortcut = Inflector::camelize($relatedModelName);
-                if($queryBuilder->getDQLPart('join')){
+                if ($queryBuilder->getDQLPart('join')) {
                     $joins = $queryBuilder->getDQLPart('join')['e'];
-                    foreach ($joins as $join){
-                        if($join->getAlias() != $relatedModelName) {
+                    foreach ($joins as $join) {
+                        if ($join->getAlias() != $relatedModelName) {
                             $queryBuilder->leftJoin(sprintf('e.%s', $relatedModelName), $relatedModelName);
                         }
                     }
-                } else{
+                } else {
                     $queryBuilder->leftJoin(sprintf('e.%s', $relatedModelName), $relatedModelName);
                 }
                 call_user_func([$this, sprintf('apply%sFilter', $config['type'])], $config, $queryBuilder, $shortcut, $index);
@@ -150,10 +150,7 @@ 
     protected function getRelationAnnotation(\ReflectionProperty $property): Annotation
     {
         /** @var Annotation $annotation */
-        $annotation = $this->reader->getPropertyAnnotation($property, OneToOne::class) ?:
-            $this->reader->getPropertyAnnotation($property, OneToMany::class) ?:
-                $this->reader->getPropertyAnnotation($property, ManyToMany::class) ?:
-                    $this->reader->getPropertyAnnotation($property, ManyToOne::class);
+        $annotation = $this->reader->getPropertyAnnotation($property, OneToOne::class) ?: $this->reader->getPropertyAnnotation($property, OneToMany::class) ?: $this->reader->getPropertyAnnotation($property, ManyToMany::class) ?: $this->reader->getPropertyAnnotation($property, ManyToOne::class);
         if (!$annotation) {
             throw new BadRequestHttpException(sprintf('Property %s is not configured for filtering', $property->getName()));
         }