chore: use native php enum type (#357)

Author: rgomezcasas

src/Mooc/Videos/Application/Create/CreateVideoCommandHandler.php

@@ -18,7 +18,7 @@ public function __construct(private VideoCreator $creator) {}
     public function __invoke(CreateVideoCommand $command): void
     {
         $id = new VideoId($command->id());
-        $type = new VideoType($command->type());
+        $type = VideoType::from($command->type());
         $title = new VideoTitle($command->title());
         $url = new VideoUrl($command->url());
         $courseId = new CourseId($command->courseId());

src/Mooc/Videos/Application/Find/VideoResponseConverter.php

@@ -12,7 +12,7 @@ public function __invoke(Video $video): VideoResponse
     {
         return new VideoResponse(
             $video->id()->value(),
-            $video->type()->value(),
+            $video->type()->value,
             $video->title()->value(),
             $video->url()->value(),
             $video->courseId()->value()

src/Mooc/Videos/Domain/Video.php

@@ -30,7 +30,7 @@ public static function create(
         $video->record(
             new VideoCreatedDomainEvent(
                 $id->value(),
-                $type->value(),
+                $type->value,
                 $title->value(),
                 $url->value(),
                 $courseId->value()

src/Mooc/Videos/Domain/VideoType.php

@@ -4,20 +4,8 @@
 
 namespace CodelyTv\Mooc\Videos\Domain;
 
-use CodelyTv\Shared\Domain\ValueObject\Enum;
-use InvalidArgumentException;
-
-/**
- * @method static VideoType screencast()
- * @method static VideoType interview()
- */
-final class VideoType extends Enum
+enum VideoType: string
 {
-    public const SCREENCAST = 'screencast';
-    public const INTERVIEW = 'interview';
-
-    protected function throwExceptionForInvalidValue($value): never
-    {
-        throw new InvalidArgumentException(sprintf('The <%s> value is not a valid video type', $value));
-    }
+    case SCREENCAST = 'screencast';
+    case INTERVIEW = 'interview';
 }

src/Shared/Domain/Criteria/Filter.php

@@ -16,7 +16,7 @@ public static function fromValues(array $values): self
     {
         return new self(
             new FilterField($values['field']),
-            new FilterOperator($values['operator']),
+            FilterOperator::from($values['operator']),
             new FilterValue($values['value'])
         );
     }
@@ -38,6 +38,6 @@ public function value(): FilterValue
 
     public function serialize(): string
     {
-        return sprintf('%s.%s.%s', $this->field->value(), $this->operator->value(), $this->value->value());
+        return sprintf('%s.%s.%s', $this->field->value(), $this->operator->value, $this->value->value());
     }
 }

src/Shared/Domain/Criteria/FilterOperator.php

@@ -4,31 +4,17 @@
 
 namespace CodelyTv\Shared\Domain\Criteria;
 
-use CodelyTv\Shared\Domain\ValueObject\Enum;
-use InvalidArgumentException;
-
-/**
- * @method static FilterOperator gt()
- * @method static FilterOperator lt()
- * @method static FilterOperator like()
- */
-final class FilterOperator extends Enum
+enum FilterOperator: string
 {
-    public const EQUAL = '=';
-    public const NOT_EQUAL = '!=';
-    public const GT = '>';
-    public const LT = '<';
-    public const CONTAINS = 'CONTAINS';
-    public const NOT_CONTAINS = 'NOT_CONTAINS';
-    private static array $containing = [self::CONTAINS, self::NOT_CONTAINS];
+    case EQUAL = '=';
+    case NOT_EQUAL = '!=';
+    case GT = '>';
+    case LT = '<';
+    case CONTAINS = 'CONTAINS';
+    case NOT_CONTAINS = 'NOT_CONTAINS';
 
     public function isContaining(): bool
     {
-        return in_array($this->value(), self::$containing, true);
-    }
-
-    protected function throwExceptionForInvalidValue($value): never
-    {
-        throw new InvalidArgumentException(sprintf('The filter <%s> is invalid', $value));
+        return in_array($this->value, [self::CONTAINS->value, self::NOT_CONTAINS->value], true);
     }
 }

src/Shared/Domain/Criteria/Order.php

@@ -10,17 +10,17 @@ public function __construct(private OrderBy $orderBy, private OrderType $orderTy
 
     public static function createDesc(OrderBy $orderBy): self
     {
-        return new self($orderBy, OrderType::desc());
+        return new self($orderBy, OrderType::DESC);
     }
 
     public static function fromValues(?string $orderBy, ?string $order): self
     {
-        return $orderBy === null ? self::none() : new self(new OrderBy($orderBy), new OrderType($order));
+        return $orderBy === null ? self::none() : new self(new OrderBy($orderBy), OrderType::from($order));
     }
 
     public static function none(): self
     {
-        return new self(new OrderBy(''), OrderType::none());
+        return new self(new OrderBy(''), OrderType::NONE);
     }
 
     public function orderBy(): OrderBy
@@ -40,6 +40,6 @@ public function isNone(): bool
 
     public function serialize(): string
     {
-        return sprintf('%s.%s', $this->orderBy->value(), $this->orderType->value());
+        return sprintf('%s.%s', $this->orderBy->value(), $this->orderType->value);
     }
 }

src/Shared/Domain/Criteria/OrderType.php

@@ -4,27 +4,14 @@
 
 namespace CodelyTv\Shared\Domain\Criteria;
 
-use CodelyTv\Shared\Domain\ValueObject\Enum;
-use InvalidArgumentException;
-
-/**
- * @method static OrderType asc()
- * @method static OrderType desc()
- * @method static OrderType none()
- */
-final class OrderType extends Enum
+enum OrderType: string
 {
-    public const ASC = 'asc';
-    public const DESC = 'desc';
-    public const NONE = 'none';
+    case ASC = 'asc';
+    case DESC = 'desc';
+    case NONE = 'none';
 
     public function isNone(): bool
     {
-        return $this->equals(self::none());
-    }
-
-    protected function throwExceptionForInvalidValue($value): never
-    {
-        throw new InvalidArgumentException($value);
+        return $this->value === self::NONE->value;
     }
 }

src/Shared/Domain/ValueObject/Enum.php

@@ -60,7 +60,7 @@ private function buildComparison(): callable
                 ? $this->hydrate($field, $filter->value()->value())
                 : $filter->value()->value();
 
-            return new Comparison($field, $filter->operator()->value(), $value);
+            return new Comparison($field, $filter->operator()->value, $value);
         };
     }