src/DataFixtures/LoadPostData.php

@@ -3,7 +3,6 @@ declare(strict_types=1);
 
 namespace App\DataFixtures;
 
-use App\Enum\Blog\PostTypeEnum;
 use Doctrine\Bundle\FixturesBundle\Fixture;
 use Doctrine\Common\DataFixtures\OrderedFixtureInterface;
 use Doctrine\Persistence\ObjectManager;
@@ -29,31 +28,31 @@ class LoadPostData extends Fixture implements OrderedFixtureInterface
         /** @var User $prWlUser */
         $prWlUser = $this->getReference('test_user_'.LoadUserData::USER_PRWL_ID);
 
-        $longPost = (new Post(self::POST_ID_LONG, $mainUser, new \DateTime(), PostTypeEnum::Post))
+        $longPost = (new Post(self::POST_ID_LONG, $mainUser, new \DateTime(), Post::TYPE_POST))
             ->setText('Test post with many comments')
             ->setPrivate(false)
             ->setDeleted(false)
         ;
 
-        $shortPost = (new Post(self::POST_ID_SHORT, $mainUser, new \DateTime(), PostTypeEnum::Post))
+        $shortPost = (new Post(self::POST_ID_SHORT, $mainUser, new \DateTime(), Post::TYPE_POST))
             ->setText('Test short post')
             ->setPrivate(false)
             ->setDeleted(false)
         ;
 
-        $privateUserPost = (new Post(self::POST_ID_PR_USER, $privateUser, new \DateTime(), PostTypeEnum::Post))
+        $privateUserPost = (new Post(self::POST_ID_PR_USER, $privateUser, new \DateTime(), Post::TYPE_POST))
             ->setText('Post from private user. Should not be visible in the public feed.')
             ->setPrivate(false)
             ->setDeleted(false)
         ;
 
-        $wlUserPost = (new Post(self::POST_ID_WL_USER, $wlUser, new \DateTime(), PostTypeEnum::Post))
+        $wlUserPost = (new Post(self::POST_ID_WL_USER, $wlUser, new \DateTime(), Post::TYPE_POST))
             ->setText('Post from whitelist-only user. Should only be visible for whitelisted users.')
             ->setPrivate(false)
             ->setDeleted(false)
         ;
 
-        $privateWlUserPost = (new Post(self::POST_ID_PR_WL_USER, $prWlUser, new \DateTime(), PostTypeEnum::Post))
+        $privateWlUserPost = (new Post(self::POST_ID_PR_WL_USER, $prWlUser, new \DateTime(), Post::TYPE_POST))
             ->setText('Post from private AND whitelist-only user. Should not be visible in the public feed.')
             ->setPrivate(false)
             ->setDeleted(false)

src/Entity/Blog/Post.php

@@ -4,9 +4,9 @@ declare(strict_types=1);
 namespace App\Entity\Blog;
 
 use App\Entity\User;
-use App\Enum\Blog\PostTypeEnum;
 use App\Repository\Blog\TagRepository;
-use Doctrine\Common\Collections\{ArrayCollection, Collection};
+use Doctrine\Common\Collections\ArrayCollection;
+use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity(repositoryClass: TagRepository::class)]
@@ -14,6 +14,9 @@ use Doctrine\ORM\Mapping as ORM;
 #[ORM\Table(name: 'tags', schema: 'posts')]
 class Post
 {
+    public const TYPE_POST = 'post';
+    public const TYPE_FEED = 'feed';
+
     #[ORM\Id]
     #[ORM\Column(name: 'id', type: 'text')]
     private string $id;
@@ -27,8 +30,9 @@ class Post
     #[ORM\Column(name: 'updated_at', type: 'datetime', nullable: true)]
     private ?\DateTime $updatedAt;
 
-    #[ORM\Column(name: 'type', type: 'string', length: 6, enumType: PostTypeEnum::class)]
+    // TODO: Native Enum
-    private PostTypeEnum $type;
+    #[ORM\Column(name: 'type', type: 'string', length: 6)]
+    private string $type = self::TYPE_POST;
 
     #[ORM\Column(name: 'private', type: 'boolean', nullable: true)]
     private bool $private;
@@ -56,7 +60,7 @@ class Post
     private Collection $comments;
 
 
-    public function __construct(string $id, User $author, \DateTime $createdAt, PostTypeEnum $type)
+    public function __construct(string $id, User $author, \DateTime $createdAt, string $type)
     {
         $this->id = $id;
         $this->author = $author;
@@ -101,7 +105,7 @@ class Post
         return $this->updatedAt;
     }
 
-    public function getType(): PostTypeEnum
+    public function getType(): string
     {
         return $this->type;
     }

src/Entity/User.php

@@ -5,7 +5,8 @@ declare(strict_types=1);
 namespace App\Entity;
 
 use App\Repository\UserRepository;
-use Doctrine\Common\Collections\{ArrayCollection, Collection};
+use Doctrine\Common\Collections\ArrayCollection;
+use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 
 #[ORM\Entity(repositoryClass: UserRepository::class)]
@@ -15,6 +16,10 @@ use Doctrine\ORM\Mapping as ORM;
 #[ORM\Index(columns: ['is_removed'], name: 'idx_user_removed')]
 class User
 {
+    public const AVATAR_SIZE_SMALL = '24';
+    public const AVATAR_SIZE_MEDIUM = '40';
+    public const AVATAR_SIZE_LARGE = '80';
+
     #[ORM\Id]
     #[ORM\Column(name: 'id', type: 'integer')]
     private ?int $id = null;

src/Enum/AvatarSizeEnum.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace App\Enum;
-
-enum AvatarSizeEnum: int
-{
-    case Small = 24;
-    case Medium = 40;
-    case Large = 80;
-}

src/Enum/Blog/PostTypeEnum.php

@@ -1,10 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace App\Enum\Blog;
-
-enum PostTypeEnum: string
-{
-    case Post = 'post';
-    case Feed = 'feed';
-}

src/Factory/Blog/PostFactory.php

@@ -3,7 +3,6 @@ declare(strict_types=1);
 
 namespace App\Factory\Blog;
 
-use App\Enum\Blog\PostTypeEnum;
 use App\Factory\AbstractFactory;
 use App\Factory\UserFactory;
 use Doctrine\ORM\EntityManagerInterface;
@@ -118,7 +117,7 @@ class PostFactory extends AbstractFactory
                 $postData->getId(),
                 $author,
                 new \DateTime($postData->getCreated()),
-                PostTypeEnum::tryFrom($postData->getType()) ?? PostTypeEnum::Post,
+                $postData->getType() ?: Post::TYPE_POST
             );
             $this->postRepository->save($post);
         }

src/Twig/PointUrlExtension.php

@@ -3,9 +3,10 @@ declare(strict_types=1);
 
 namespace App\Twig;
 
-use App\Enum\AvatarSizeEnum;
+use App\Entity\User;
 use Twig\Extension\AbstractExtension;
-use Twig\{TwigFilter, TwigFunction};
+use Twig\TwigFilter;
+use Twig\TwigFunction;
 
 class PointUrlExtension extends AbstractExtension
 {
@@ -15,7 +16,7 @@ class PointUrlExtension extends AbstractExtension
     ) {
     }
 
-    public function getFunctions(): array
+    public function getFunctions()
     {
         return [
             new TwigFunction('point_avatar', [$this, 'avatarFunction']),
@@ -28,7 +29,7 @@ class PointUrlExtension extends AbstractExtension
         ];
     }
 
-    public function getFilters(): array
+    public function getFilters()
     {
         return [
             new TwigFilter('point_avatar', [$this, 'avatarFunction']),
@@ -43,20 +44,20 @@ class PointUrlExtension extends AbstractExtension
 
     public function avatarSmallFunction(string $login): string
     {
-        return $this->avatarFunction($login, AvatarSizeEnum::Small);
+        return $this->avatarFunction($login, User::AVATAR_SIZE_SMALL);
     }
 
     public function avatarMediumFunction(string $login): string
     {
-        return $this->avatarFunction($login, AvatarSizeEnum::Medium);
+        return $this->avatarFunction($login, User::AVATAR_SIZE_MEDIUM);
     }
 
     public function avatarLargeFunction(string $login): string
     {
-        return $this->avatarFunction($login, AvatarSizeEnum::Large);
+        return $this->avatarFunction($login, User::AVATAR_SIZE_LARGE);
     }
 
-    public function avatarFunction(string $login, AvatarSizeEnum $size): string
+    public function avatarFunction(string $login, $size): string
     {
         return $this->getAvatarUrlByLogin($login, $size);
     }
@@ -76,8 +77,12 @@ class PointUrlExtension extends AbstractExtension
         return sprintf('%s://%s/%s', $this->pointScheme, $this->pointDomain, $postId);
     }
 
-    private function getAvatarUrlByLogin(string $login, AvatarSizeEnum $size): string
+    private function getAvatarUrlByLogin(string $login, string $size): string
     {
-        return sprintf('%s://%s/avatar/%s/%s', $this->pointScheme, $this->pointDomain, \urlencode($login), $size->value);
+        if (!in_array($size, [User::AVATAR_SIZE_SMALL, User::AVATAR_SIZE_MEDIUM, User::AVATAR_SIZE_LARGE], true)) {
+            throw new \InvalidArgumentException('Avatar size must be one of restricted variants. See User::AVATAR_SIZE_* constants.');
+        }
+
+        return sprintf('%s://%s/avatar/%s/%s', $this->pointScheme, $this->pointDomain, urlencode($login), $size);
     }
 }