Aoe\FeatureFlag\System\Db\FeatureFlagData

Complexity

Total Complexity

4

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getContentElementsPIDs()	0	18	1
A	getContentElements()	0	39	1
A	updateContentElements()	0	14	2

<?php
namespace Aoe\FeatureFlag\System\Db;

/***************************************************************
 *  Copyright notice
 *
 *  (c) 2021 AOE GmbH <dev@aoe.com>
 *
 *  All rights reserved
 *
 *  This script is part of the TYPO3 project. The TYPO3 project is
 *  free software; you can redistribute it and/or modify
 *  it under the terms of the GNU General Public License as published by
 *  the Free Software Foundation; either version 3 of the License, or
 *  (at your option) any later version.
 *
 *  The GNU General Public License can be found at
 *  http://www.gnu.org/copyleft/gpl.html.
 *
 *  This script is distributed in the hope that it will be useful,
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 *  GNU General Public License for more details.
 *
 *  This copyright notice MUST APPEAR in all copies of the script!
 ***************************************************************/

use TYPO3\CMS\Core\Database\Connection;
use TYPO3\CMS\Core\Database\ConnectionPool;
use TYPO3\CMS\Core\Database\Query\QueryBuilder;
use TYPO3\CMS\Core\Utility\GeneralUtility;

class FeatureFlagData
{
    /**
     * @var string
     */
    const TABLE_MAPPING = 'tx_featureflag_domain_model_mapping';

    /**
     * @var string
     */
    const TABLE_FLAGS = 'tx_featureflag_domain_model_featureflag';

    /**
     * @param string $table
     * @param integer $behavior
     * @param integer $enabled
     *
     * @return array
     */
    public function getContentElements($table, $behavior, $enabled)
    {
        /** @var QueryBuilder $queryBuilder */
        $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
            ->getQueryBuilderForTable($table);

        $queryBuilder->getRestrictions()->removeAll();

        $queryBuilder
            ->select($table . '.uid')
            ->from($table)
            ->from(self::TABLE_MAPPING)
            ->from(self::TABLE_FLAGS)
            ->where(
                $queryBuilder->expr()->andX(
                    $queryBuilder->expr()->eq(
                        self::TABLE_MAPPING . '.feature_flag',
                        self::TABLE_FLAGS . '.uid'
                    ),
                    $queryBuilder->expr()->eq(
                        $table . '.uid',
                        self::TABLE_MAPPING . '.foreign_table_uid'
                    ),
                    $queryBuilder->expr()->eq(
                        self::TABLE_FLAGS . '.enabled',
                        $queryBuilder->createNamedParameter($enabled, Connection::PARAM_INT)
                    ),
                    $queryBuilder->expr()->eq(
                        self::TABLE_MAPPING . '.foreign_table_name',
                        $queryBuilder->createNamedParameter($table, Connection::PARAM_STR)
                    ),
                    $queryBuilder->expr()->eq(
                        self::TABLE_MAPPING . '.behavior',
                        $queryBuilder->createNamedParameter($behavior, Connection::PARAM_INT)
                    )
                )
            );

        return $queryBuilder->execute()->fetchAllAssociative();
    }

    /**
     * @param string $table
     * @param array $uids
     * @param bool $isVisible
     */
    public function updateContentElements($table, array $uids, $isVisible)
    {
        /** @var QueryBuilder $queryBuilder */
        $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
            ->getQueryBuilderForTable($table);

        $queryBuilder->getRestrictions()->removeAll();

        $query = $queryBuilder
            ->update($table)
            ->set('hidden', $isVisible === true ? 0 : 1)
            ->add('where', $queryBuilder->expr()->in('uid', $uids));

        $query->execute();
    }

    /**
     * @param string $table
     * @param integer $uid
     *
     * @return string
     */
    public function getContentElementsPIDs($table, $uid)
    {
        /** @var QueryBuilder $queryBuilder */
        $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
            ->getQueryBuilderForTable($table);

        $queryBuilder->getRestrictions()->removeAll();

        $query = $queryBuilder
            ->select($table . '.pid')
            ->from($table)
            ->where(
                $queryBuilder->expr()->eq(
                    $table . '.uid', $queryBuilder->createNamedParameter($uid, Connection::PARAM_INT)
                )
            );

        return $query->execute()->fetchColumn(0);

The function Doctrine\DBAL\ForwardCompatibility\Result::fetchColumn() has been deprecated: Use fetchOne() instead.

The expression return $query->execute()->fetchColumn(0) could also return false which is incompatible with the documented return type string. Did you maybe forget to handle an error condition?

    }
}