<commit_before><?php
/**
 * This file is part of PHP Mess Detector.
 *
 * Copyright (c) <NAME> <<EMAIL>>.
 * All rights reserved.
 *
 * Licensed under BSD License
 * For full copyright and license information, please see the LICENSE file.
 * Redistributions of files must retain the above copyright notice.
 *
 * @author <NAME> <<EMAIL>>
 * @copyright <NAME>. All rights reserved.
 * @license https://opensource.org/licenses/bsd-license.php BSD License
 * @link http://phpmd.org/
 */

namespace PHPMD\Rule\CleanCode;

use PDepend\Source\AST\AbstractASTClassOrInterface;
use PDepend\Source\AST\ASTValue;
use PHPMD\AbstractNode;
use PHPMD\AbstractRule;
use PHPMD\Rule\FunctionAware;
use PHPMD\Rule\MethodAware;
use PHPMD\Utility\Strings;

/**
 * Check for a boolean flag in the method/function signature.
 *
 * Boolean flags are signs for single responsibility principle violations.
 */
class BooleanArgumentFlag extends AbstractRule implements MethodAware, FunctionAware
{
    /**
     * Temporary cache of configured exceptions.
     *
     * @return array<string, int>
     */
    protected $exceptions;

    /**
     * This method checks if a method/function has boolean flag arguments and warns about them.
     *
     * @param \PHPMD\AbstractNode $node
     * @return void
     */
    public function apply(AbstractNode $node)
    {
        $name = $node->getName();

        if ($name) {
            $ignorePattern = trim($this->getStringProperty('ignorepattern', ''));

            if ($ignorePattern !== '' && preg_match($ignorePattern, $node->getName())) {
                return;
            }
        }

        $parent = $node->getNode()->getParent();

        if ($parent &&
            ($parent instanceof AbstractASTClassOrInterface) &&
            ($name = $parent->getName())
        ) {
            $exceptions = $this->getExceptionsList();

            if (isset($exceptions[$name])) {
                return;
            }
        }

        foreach ($node->findChildrenOfType('FormalParameter') as $param) {
            $declarator = $param->getFirstChildOfType('VariableDeclarator');
            $value = $declarator->getValue();

            if (false === $this->isBooleanValue($value)) {
                continue;
            }

            $this->addViolation($param, array($node->getImage(), $declarator->getImage()));
        }
    }

    protected function isBooleanValue(ASTValue $value = null)
    {
        return $value && $value->isValueAvailable() && ($value->getValue() === true || $value->getValue() === false);
    }

    /**
     * Gets exceptions from property
     *
     * @return array<string, int>
     */
    protected function getExceptionsList()
    {
        if ($this->exceptions === null) {
            $this->exceptions = array_flip(
                Strings::splitToList(
                    $this->getStringProperty('exceptions', ''),
                    ','
                )
            );
        }

        return $this->exceptions;
    }
}
<commit_msg>Decrease BooleanArgumentFlag complexity<commit_after><?php
/**
 * This file is part of PHP Mess Detector.
 *
 * Copyright (c) <NAME> <<EMAIL>>.
 * All rights reserved.
 *
 * Licensed under BSD License
 * For full copyright and license information, please see the LICENSE file.
 * Redistributions of files must retain the above copyright notice.
 *
 * @author <NAME> <<EMAIL>>
 * @copyright <NAME>. All rights reserved.
 * @license https://opensource.org/licenses/bsd-license.php BSD License
 * @link http://phpmd.org/
 */

namespace PHPMD\Rule\CleanCode;

use PDepend\Source\AST\AbstractASTClassOrInterface;
use PDepend\Source\AST\ASTValue;
use PHPMD\AbstractNode;
use PHPMD\AbstractRule;
use PHPMD\Rule\FunctionAware;
use PHPMD\Rule\MethodAware;
use PHPMD\Utility\Strings;

/**
 * Check for a boolean flag in the method/function signature.
 *
 * Boolean flags are signs for single responsibility principle violations.
 */
class BooleanArgumentFlag extends AbstractRule implements MethodAware, FunctionAware
{
    /**
     * Temporary cache of configured exceptions.
     *
     * @return array<string, int>
     */
    protected $exceptions;

    /**
     * This method checks if a method/function has boolean flag arguments and warns about them.
     *
     * @param \PHPMD\AbstractNode $node
     * @return void
     */
    public function apply(AbstractNode $node)
    {
        $name = $node->getName();

        if ($name) {
            $ignorePattern = trim($this->getStringProperty('ignorepattern', ''));

            if ($ignorePattern !== '' && preg_match($ignorePattern, $node->getName())) {
                return;
            }
        }

        $parent = $node->getNode()->getParent();

        if ($parent &&
            ($parent instanceof AbstractASTClassOrInterface) &&
            ($name = $parent->getName())
        ) {
            $exceptions = $this->getExceptionsList();

            if (isset($exceptions[$name])) {
                return;
            }
        }

        $this->scanFormalParameters($node);
    }

    protected function isBooleanValue(ASTValue $value = null)
    {
        return $value && $value->isValueAvailable() && ($value->getValue() === true || $value->getValue() === false);
    }

    /**
     * Gets exceptions from property
     *
     * @return array<string, int>
     */
    protected function getExceptionsList()
    {
        if ($this->exceptions === null) {
            $this->exceptions = array_flip(
                Strings::splitToList(
                    $this->getStringProperty('exceptions', ''),
                    ','
                )
            );
        }

        return $this->exceptions;
    }

    private function scanFormalParameters(AbstractNode $node)
    {
        foreach ($node->findChildrenOfType('FormalParameter') as $param) {
            $declarator = $param->getFirstChildOfType('VariableDeclarator');
            $value = $declarator->getValue();

            if (false === $this->isBooleanValue($value)) {
                continue;
            }

            $this->addViolation($param, array($node->getImage(), $declarator->getImage()));
        }
    }
}

{'fraction_non_alphanumeric': 0.13086540022730964, 'fraction_numerical': 0.0, 'mean_word_length': 2.3225458468176914, 'pattern_counts': {'":': 0, '<': 23, '<?xml version=': 0, '>': 64, 'https://': 2, 'lorem ipsum': 0, 'www.': 0, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 0, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '7147845', 'n_tokens_mistral': 1666, 'n_tokens_neox': 1630, 'n_words': 405}