Merge branch 'refs/heads/5.x' into 5.x-lambda-has-many-through

flat3 · Sep 19, 2024 · 29d3379 · 29d3379
2 parents 9d92e69 + 5785ee1
commit 29d3379
Show file tree

Hide file tree

Showing 161 changed files with 363 additions and 173 deletions.
diff --git a/src/Attributes/LodataEnum.php b/src/Attributes/LodataEnum.php
@@ -12,11 +12,20 @@
 #[Attribute(Attribute::TARGET_CLASS | Attribute::IS_REPEATABLE)]
 class LodataEnum extends LodataProperty
 {
-    protected string $enum;
-
-    public function __construct(string $name, string $enum, ?string $source = null)
-    {
-        parent::__construct($name, $source);
+    /** @var string */
+    protected $enum;
+
+    public function __construct(
+        string $name,
+        string $enum,
+        ?string $description = null,
+        ?string $source = null,
+        ?bool $nullable = true,
+        ?bool $immutable = false
+    ) {
+        parent::__construct($name, $description, $source);
+        $this->nullable = $nullable;
+        $this->immutable = $immutable;
         $this->enum = $enum;
     }
 
@@ -33,4 +42,4 @@ public function getType(): Type
 
         return Lodata::getEnumerationType($this->enum);
     }
-}
+}
diff --git a/src/ComplexType.php b/src/ComplexType.php
@@ -6,6 +6,8 @@
 
 use ArrayAccess;
 use Flat3\Lodata\Annotation\Core\V1\Computed;
+use Flat3\Lodata\Annotation\Core\V1\ComputedDefaultValue;
+use Flat3\Lodata\Annotation\Core\V1\Immutable;
 use Flat3\Lodata\Controller\Transaction;
 use Flat3\Lodata\Exception\Protocol\NotAcceptableException;
 use Flat3\Lodata\Helper\Constants;
@@ -243,9 +245,11 @@ public function getOpenAPISchema(): array
         return [
             'type' => Constants::oapiObject,
             'title' => $this->getName(),
-            'properties' => (object) $this->getDeclaredProperties()->map(function (DeclaredProperty $property) {
-                return $property->getOpenAPISchema();
-            })
+            'properties' => (object) $this->getProperties()
+                ->sliceByClass([DeclaredProperty::class, GeneratedProperty::class])
+                ->map(function (Property $property) {
+                    return $property->getOpenAPISchema();
+                })
         ];
     }
 
@@ -259,7 +263,8 @@ public function getOpenAPICreateSchema(): array
             'type' => Constants::oapiObject,
             'title' => __('lodata:::name (Create schema)', ['name' => $this->getName()]),
             'properties' => (object) $this->getDeclaredProperties()->filter(function (DeclaredProperty $property) {
-                return $property->getAnnotations()->sliceByClass([Computed::class])->isEmpty();
+                return $property->getAnnotations()->sliceByClass([Computed::class])->isEmpty()
+                    || $property->getAnnotations()->sliceByClass([ComputedDefaultValue::class])->hasEntries();
             })->map(function (DeclaredProperty $property) {
                 return $property->getOpenAPISchema();
             })
@@ -276,7 +281,7 @@ public function getOpenAPIUpdateSchema(): array
             'type' => Constants::oapiObject,
             'title' => __('lodata:::name (Update schema)', ['name' => $this->getName()]),
             'properties' => (object) $this->getDeclaredProperties()->filter(function (DeclaredProperty $property) {
-                return $property->getAnnotations()->sliceByClass([Computed::class])->isEmpty();
+                return $property->getAnnotations()->sliceByClass([Computed::class, Immutable::class])->isEmpty();
             })->map(function (DeclaredProperty $property) {
                 return $property->getOpenAPISchema();
             })
@@ -322,7 +327,7 @@ public function assertPropertyValues(PropertyValues $propertyValues): PropertyVa
     public function assertUpdateProperties(PropertyValues $propertyValues): PropertyValues
     {
         return $this->assertPropertyValues($propertyValues)->filter(function (PropertyValue $propertyValue) {
-            return !$propertyValue->getProperty()->isImmutable();
+            return !($propertyValue->getProperty()->isImmutable() || $propertyValue->getProperty()->isComputed());
         });
     }
 
@@ -353,7 +358,7 @@ public function assertCreateProperties(
                 continue;
             }
 
-            if ($declaredProperty->isNullable() || $declaredProperty->isComputed()) {
+            if ($declaredProperty->isNullable() || $declaredProperty->isComputed() || $declaredProperty->isComputedDefault()) {
                 continue;
             }
 
@@ -371,6 +376,8 @@ public function assertCreateProperties(
             $declaredProperty->assertAllowsValue(null);
         }
 
-        return $propertyValues;
+        return $propertyValues->filter(function (PropertyValue $propertyValue) {
+            return !$propertyValue->getProperty()->isComputed();
+        });
     }
 }
diff --git a/src/Controller/Request.php b/src/Controller/Request.php
@@ -63,6 +63,7 @@ public function __construct(IlluminateRequest $request)
     {
         $this->method = $request->getRealMethod();
         $this->headers = $request->headers;
+        $this->cookies = $request->cookies;
         $this->query = $request->query;
         $this->content = $request->content;
         $this->server = $request->server;

diff --git a/src/Controller/Transaction.php b/src/Controller/Transaction.php
@@ -642,7 +642,7 @@ public function getAcceptedContentTypes(): MediaTypes
     {
         $formatQueryOption = $this->getFormat()->getValue();
 
-        if (Str::startsWith($formatQueryOption, ['json', 'xml'])) {
+        if ($formatQueryOption && Str::startsWith($formatQueryOption, ['json', 'xml'])) {
             if (!in_array($formatQueryOption, ['json', 'xml'])) {
                 throw new BadRequestException(
                     'invalid_short_format',

diff --git a/src/Drivers/CSVEntityType.php b/src/Drivers/CSVEntityType.php
@@ -4,7 +4,7 @@
 
 namespace Flat3\Lodata\Drivers;
 
-use Flat3\Lodata\Annotation\Core\V1\ComputedDefaultValue;
+use Flat3\Lodata\Annotation\Core\V1\Computed;
 use Flat3\Lodata\DeclaredProperty;
 use Flat3\Lodata\EntityType;
 use Flat3\Lodata\Type;
@@ -18,6 +18,6 @@ class CSVEntityType extends EntityType
     public function __construct($identifier)
     {
         parent::__construct($identifier);
-        $this->setKey((new DeclaredProperty('offset', Type::int64()))->addAnnotation(new ComputedDefaultValue));
+        $this->setKey((new DeclaredProperty('offset', Type::int64()))->addAnnotation(new Computed));
     }
-}
+}
diff --git a/src/Drivers/EloquentEntitySet.php b/src/Drivers/EloquentEntitySet.php
@@ -7,7 +7,7 @@
 use Doctrine\DBAL\Schema\Column;
 use Exception;
 use Flat3\Lodata\Annotation\Capabilities\V1\DeepInsertSupport;
-use Flat3\Lodata\Annotation\Core\V1\ComputedDefaultValue;
+use Flat3\Lodata\Annotation\Core\V1\Computed;
 use Flat3\Lodata\Annotation\Core\V1\Description;
 use Flat3\Lodata\Attributes\LodataIdentifier;
 use Flat3\Lodata\Attributes\LodataProperty;
@@ -188,14 +188,12 @@ public function read(PropertyValue $key): Entity
      */
     protected function setModelAttributes(Model $model, PropertyValues $propertyValues): Model
     {
-        foreach ($propertyValues->getDeclaredPropertyValues() as $propertyValue) {
-            $model->setAttribute(
-                $this->getPropertySourceName($propertyValue->getProperty()),
-                $propertyValue->getPrimitive()->toMixed()
+        return $propertyValues->getDeclaredPropertyValues()->reduce(function (Model $model, PropertyValue $value) {
+            return $model->setAttribute(
+                $this->getPropertySourceName($value->getProperty()),
+                $value->getPrimitive()->toMixed(),
             );
-        }
-
-        return $model;
+        }, $model);
     }
 
     /**
@@ -230,7 +228,7 @@ public function create(PropertyValues $propertyValues): Entity
             foreach ($navigationProperty->getConstraints() as $constraint) {
                 $referencedProperty = $constraint->getReferencedProperty();
                 $model->setAttribute(
-                    $referencedProperty->getName(),
+                    $this->getPropertySourceName($referencedProperty),
                     $this->navigationSource->getParent()->getEntityId()->getPrimitive()->toMixed()
                 );
             }
@@ -685,7 +683,7 @@ public function columnToDeclaredProperty(Column $column): ?DeclaredProperty
 
         $defaultValue = $model->getAttributeValue($column->getName());
         if ($defaultValue) {
-            $property->addAnnotation(new ComputedDefaultValue);
+            $property->addAnnotation(new Computed);
             $property->setDefaultValue($defaultValue);
         }
 

diff --git a/src/EntityType.php b/src/EntityType.php
@@ -5,6 +5,7 @@
 namespace Flat3\Lodata;
 
 use Flat3\Lodata\Annotation\Core\V1\Computed;
+use Flat3\Lodata\Annotation\Core\V1\Immutable;
 use Flat3\Lodata\Controller\Transaction;
 use Flat3\Lodata\Exception\Internal\PathNotHandledException;
 use Flat3\Lodata\Facades\Lodata;
@@ -98,7 +99,8 @@ public function getOpenAPIUpdateSchema(): array
             'type' => Constants::oapiObject,
             'title' => __('lodata:::name (Update schema)', ['name' => $this->getName()]),
             'properties' => (object) $this->getDeclaredProperties()->filter(function (DeclaredProperty $property) {
-                return $property->getAnnotations()->sliceByClass([Computed::class])->isEmpty() && $property !== $this->getKey();
+                return $property->getAnnotations()->sliceByClass([Computed::class, Immutable::class])->isEmpty()
+                    && $property !== $this->getKey();
             })->map(function (DeclaredProperty $property) {
                 return $property->getOpenAPISchema();
             })

diff --git a/src/GeneratedProperty.php b/src/GeneratedProperty.php
@@ -13,6 +13,11 @@
  */
 abstract class GeneratedProperty extends Property
 {
+    public function __construct($name, ?Type $type = null)
+    {
+        parent::__construct($name, $type !== null ? $type : Type::untyped());
+    }
+
     /**
      * Generate the property value for this property on the provided entity
      * @param  ComplexValue  $value  Entity this property is generated on

diff --git a/src/Helper/ObjectArray.php b/src/Helper/ObjectArray.php
@@ -302,6 +302,20 @@ public function map(callable $callback)
         return array_map($callback, $this->array);
     }
 
+    /**
+     * Reduce the objects in the array
+     * @param  callable(mixed $initial, mixed $value, mixed $key): mixed  $callback
+     * @param $initial
+     * @return mixed|null
+     */
+    public function reduce(callable $callback, $initial = null)
+    {
+        foreach ($this->array as $key => $value) {
+            $initial = $callback($initial, $value, $key);
+        }
+        return $initial;
+    }
+
     /**
      * Sort the objects in the array
      * @param  callable  $callback

diff --git a/src/PathSegment/OpenAPI.php b/src/PathSegment/OpenAPI.php
@@ -1105,7 +1105,7 @@ public static function applyProperty(?Property $property = null, array $schema =
         $schema['nullable'] = $property->isNullable();
 
         if ($property->hasStaticDefaultValue()) {
-            $schema['default'] = $property->computeDefaultValue();
+            $schema['default'] = $property->computeDefaultValue()->toJson();
         }
 
         if ($property->hasMaxLength()) {

diff --git a/src/Primitive.php b/src/Primitive.php
@@ -18,6 +18,7 @@
 use Flat3\Lodata\Interfaces\ResponseInterface;
 use Flat3\Lodata\Interfaces\SerializeInterface;
 use Flat3\Lodata\Traits\HasIdentifier;
+use Stringable;
 
 /**
  * Primitive
@@ -58,6 +59,16 @@ public function __construct($value = null)
      */
     abstract public function set($value);
 
+    /**
+     * Test the provided value is formatted correctly for this type
+     * @param $value
+     * @return bool
+     */
+    public function allows($value): bool
+    {
+        return true;
+    }
+
     /**
      * Get the internal representation of the value
      * @return mixed
@@ -181,7 +192,7 @@ public function matches($value): bool
             $value = $value->get();
         }
 
-        if (is_scalar($value) || is_null($value) || $value instanceof \Stringable) {
+        if (is_scalar($value) || is_null($value) || $value instanceof Stringable) {
             return (string) $this->get() === (string) $value;
         }
 

diff --git a/src/PrimitiveType.php b/src/PrimitiveType.php
@@ -99,4 +99,14 @@ public function getOpenAPISchema(): array
     {
         return $this->instance()->getOpenAPISchema();
     }
-}
+
+    /**
+     * Test the provided value is correctly formatted for this type
+     * @param $value
+     * @return bool
+     */
+    public function allowsValue($value): bool
+    {
+        return $this->instance()->allows($value);
+    }
+}
diff --git a/src/Property.php b/src/Property.php
@@ -370,7 +370,11 @@ public function isSearchable(): bool
      */
     public function assertAllowsValue($value)
     {
-        if (!$this->isNullable() && $value === null) {
+        if ($value === null) {
+            if ($this->isNullable()) {
+                return;
+            }
+
             throw new BadRequestException(
                 'property_not_nullable',
                 sprintf("The property '%s' cannot be set to null", $this->getName())
@@ -383,6 +387,13 @@ public function assertAllowsValue($value)
                 sprintf("The value property '%s' exceeds the maximum length", $this->getName())
             );
         }
+
+        if (!$this->type->allowsValue($value)) {
+            throw new BadRequestException(
+                'property_value_invalid',
+                sprintf("The value property '%s' is not valid", $this->getName()),
+            );
+        }
     }
 
     /**
@@ -409,8 +420,16 @@ public function getOpenAPISchema(): array
     public function isComputed(): bool
     {
         return $this instanceof ComputedProperty ||
-            $this->hasAnnotation(new Computed, Boolean::true()) ||
-            $this->hasAnnotation(new ComputedDefaultValue, Boolean::true());
+            $this->hasAnnotation(new Computed, Boolean::true());
+    }
+
+    /**
+     * Determine whether this property is computed on create operations
+     * @return bool
+     */
+    public function isComputedDefault(): bool
+    {
+        return $this->hasAnnotation(new ComputedDefaultValue, Boolean::true());
     }
 
     /**

diff --git a/src/Type.php b/src/Type.php
@@ -229,4 +229,14 @@ public function is(string $class): bool
     {
         return is_a($this->factory, $class, true);
     }
+
+    /**
+     * Test the provided value is correctly formatted for this type
+     * @param $value
+     * @return bool
+     */
+    public function allowsValue($value): bool
+    {
+        return true;
+    }
 }
diff --git a/src/Type/Guid.php b/src/Type/Guid.php
@@ -9,6 +9,7 @@
 use Flat3\Lodata\PathSegment\OpenAPI;
 use Flat3\Lodata\Primitive;
 use Flat3\Lodata\Property;
+use Stringable;
 
 /**
  * Guid
@@ -27,7 +28,7 @@ public function matches($value): bool
             $value = $value->get();
         }
 
-        if (is_scalar($value) || is_null($value) || $value instanceof \Stringable) {
+        if (is_scalar($value) || is_null($value) || $value instanceof Stringable) {
             return (string) $this->get() === strtoupper((string) $value);
         }
 
@@ -81,4 +82,9 @@ public function getOpenAPISchema(?Property $property = null): array
             'pattern' => '^'.Lexer::guid.'$',
         ]);
     }
+
+    public function allows($value): bool
+    {
+        return Lexer::patternCheck(Lexer::guid, (string) $value);
+    }
 }