ernestwisniewski/kbin

File MagazineRepository.php has 450 lines of code (exceeds 250 allowed). Consider refactoring.

Invalid

<?php

// SPDX-FileCopyrightText: 2023 /kbin contributors <https://kbin.pub/>
//
// SPDX-License-Identifier: AGPL-3.0-only

MagazineRepository has 25 functions (exceeds 20 allowed). Consider refactoring.

Open

class MagazineRepository extends ServiceEntityRepository
{
    public const PER_PAGE = 48;

    public const SORT_HOT = 'hot';

Method findTrashed has 51 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

    public function findTrashed(Magazine $magazine, int $page = 1, int $perPage = self::PER_PAGE): PagerfantaInterface
    {
        // @todo union adapter
        $conn = $this->_em->getConnection();

Method findPaginated has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function findPaginated(MagazinePageView $criteria): PagerfantaInterface
    {
        $qb = $this->createQueryBuilder('m')
            ->andWhere('m.visibility = :visibility')
            ->setParameter('visibility', VisibilityInterface::VISIBILITY_VISIBLE);

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function findForDeletionPaginated(int $page): PagerfantaInterface
    {
        $query = $this->createQueryBuilder('m')
            ->where('m.apId IS NULL')
            ->andWhere('m.visibility = :visibility')

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function findModlog(Magazine $magazine, ?int $page = 1, int $perPage = self::PER_PAGE): PagerfantaInterface
    {
        $criteria = Criteria::create()->orderBy(['createdAt' => 'DESC']);

        $logs = new Pagerfanta(new SelectableAdapter($magazine->logs, $criteria));

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function findModerators(
        Magazine $magazine,
        ?int $page = 1,
        int $perPage = self::PER_PAGE
    ): PagerfantaInterface {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public function findAudience(Magazine $magazine): array
    {
        $dql =
            'SELECT COUNT(u.id), u.apInboxUrl FROM '.User::class.' u WHERE u IN ('.
            'SELECT IDENTITY(ms.user) FROM '.MagazineSubscription::class.' ms WHERE ms.magazine = :magazine)'.

• Read up
  Read up