App\Blog\Post\PostService

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getPostTags()	0	5	1
A	savePost()	0	16	3
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace App\Blog\Post;

use App\Blog\Entity\Post;
use App\Blog\Entity\Tag;
use App\Blog\Tag\TagRepository;
use App\Entity\User;

final class PostService
{
    private PostRepository $repository;
    private TagRepository $tagRepository;

    public function __construct(PostRepository $repository, TagRepository $tagRepository)
    {
        $this->repository = $repository;
        $this->tagRepository = $tagRepository;
    }

    public function savePost(User $user, Post $model, PostForm $form): void
    {
        $model->setTitle($form->getTitle());
        $model->setContent($form->getContent());

It seems like $form->getContent() can also be of type null; however, parameter $content of App\Blog\Entity\Post::setContent() does only seem to accept string, maybe add an additional type check?

        $model->resetTags();

        foreach ($form->getTags() as $tag) {
            $model->addTag($this->tagRepository->getOrCreate($tag));
        }

        if ($model->isNewRecord()) {
            $model->setPublic(true);
            $model->setUser($user);
        }

        $this->repository->save($model);
    }

    public function getPostTags(Post $post): array
    {
        return array_map(
            static fn (Tag $tag) => $tag->getLabel(),
            $post->getTags()
        );
    }
}