Acme\App\Core\Component\Blog\Application\Query\DQL\PostListQuery

Complexity

Total Complexity

2

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	execute()	0	15	1

<?php

declare(strict_types=1);

/*
 * This file is part of the Explicit Architecture POC,
 * which is created on top of the Symfony Demo application.
 *
 * (c) Herberto Graça <herberto.graca@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Acme\App\Core\Component\Blog\Application\Query\DQL;

use Acme\App\Core\Component\Blog\Application\Query\PostListQueryInterface;
use Acme\App\Core\Component\Blog\Domain\Post\Post;
use Acme\App\Core\Port\Persistence\DQL\DqlQueryBuilderInterface;
use Acme\App\Core\Port\Persistence\QueryServiceInterface;
use Acme\App\Core\Port\Persistence\QueryServiceRouterInterface;
use Acme\App\Core\Port\Persistence\ResultCollectionInterface;
use Acme\App\Core\SharedKernel\Component\User\Domain\User\UserId;

final class PostListQuery implements PostListQueryInterface
{
    /**
     * @var DqlQueryBuilderInterface
     */
    private $dqlQueryBuilder;

    /**
     * @var QueryServiceInterface
     */
    private $queryService;

    public function __construct(
        DqlQueryBuilderInterface $dqlQueryBuilder,
        QueryServiceRouterInterface $queryService
    ) {
        $this->dqlQueryBuilder = $dqlQueryBuilder;
        $this->queryService = $queryService;

It seems like $queryService of type object<Acme\App\Core\Port\Persistence\QueryServiceRouterInterface> is incompatible with the declared type object<Acme\App\Core\Port\Persistence\QueryServiceInterface> of property $queryService.

    }

    /**
     * Since this class only has one public method, it makes sense that it is designed as a callable, using the
     * magic method name `__invoke()`, instead of having a single public method called `execute()` which adds nothing
     * to code readability.
     * However, by using `__invoke()` we lose code completion, so in the end I prefer to use this `execute()` method.
     */
    public function execute(UserId $userId): ResultCollectionInterface
    {
        $dqlQuery = $this->dqlQueryBuilder->create(Post::class, 'Post')
            ->select(
                'Post.id',
                'Post.title',
                'Post.publishedAt'
            )
            ->where('Post.authorId = :authorId')
            ->orderBy('Post.publishedAt', 'DESC')
            ->setParameter('authorId', $userId)
            ->build();

        return $this->queryService->query($dqlQuery);

The method query() does not seem to exist on object<Acme\App\Core\Port\Persistence\QueryServiceInterface>.

    }
}