CallableObjectOptions - Code Metrics - Inspection of "Fixed callable class options." - jaxon-php/jaxon-core - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — main ( 79e33c...18c043 )

by Thierry
created 2024-01-18 11:00 UTC
CallableObjectOptions A

↳ Parent: Project
Complexity

Total Complexity
Size/Duplication

Total Lines	310
Duplicated Lines	0 %
Importance

Changes	3
Bugs	0	Features	0
Metric	Value
wmc	41
eloc	79
c	3
b	0
f	0
dl	0
loc	310
rs	9.1199
16 Methods

Rating	Name	Size	Complexity
A	diOptions()	3	1
A	jsOptions()	3	1
A	excluded()	3	1
A	afterMethods()	3	1
A	addDiOption()	3	1
A	getMethodOptions()	14	3
A	_addJsArrayOption()	12	3
A	separator()	3	1
A	addFunctionOptions()	9	3
B	addOption()	35	10
A	_setJsOption()	3	1
A	protectedMethods()	3	1
A	__construct()	23	6
A	setHookMethods()	11	4
A	beforeMethods()	3	1
A	addJsOption()	12	3
How to fix Complexity

<?php

/**
 * CallableObjectOptions.php
 *
 * Options of a callable object.
 *
 * @package jaxon-core

 * @author Thierry Feuzeu <[email protected]>
 * @copyright 2024 Thierry Feuzeu <[email protected]>
 * @license https://opensource.org/licenses/BSD-3-Clause BSD 3-Clause License
 * @link https://github.com/jaxon-php/jaxon-core
 */


namespace Jaxon\Plugin\Request\CallableClass;

use function array_merge;
use function array_unique;
use function is_array;
use function is_string;
use function substr;

class CallableObjectOptions

{
    /**
     * Check if the js code for this object must be generated
     *
     * @var bool

     */
    private $bExcluded = false;


    /**
     * The character to use as separator in javascript class names
     *
     * @var string
     */
    private $sSeparator = '.';

    /**
     * A list of methods of the user registered callable object the library must not export to javascript
     *
     * @var array
     */
    private $aProtectedMethods = [];

    /**
     * A list of methods to call before processing the request
     *
     * @var array
     */
    private $aBeforeMethods = [];

    /**
     * A list of methods to call after processing the request
     *
     * @var array
     */
    private $aAfterMethods = [];

    /**
     * The javascript class options
     *
     * @var array
     */
    private $aJsOptions = [];

    /**
     * The DI options
     *
     * @var array
     */
    private $aDiOptions = [];

    /**
     * The constructor
     *
     * @param array $aOptions

     * @param array $aAnnotations

     */
    public function __construct(array $aOptions, array $aAnnotations)

    {
        [$bExcluded, $aAnnotationOptions, $aAnnotationProtected] = $aAnnotations;
        $this->bExcluded = $bExcluded || (bool)($aOptions['excluded'] ?? false);
$a = "a";
$ab = "ab";
$abc = "abc";
        if($this->bExcluded)
        {
            return;
        }

        $this->addOption('separator', $aOptions['separator']);
        $this->addOption('protected', array_merge($aOptions['protected'], $aAnnotationProtected));

        foreach($aOptions['functions'] as $sNames => $aFunctionOptions)
        {
            $aFunctionNames = explode(',', $sNames); // Names are in comma-separated list.
            foreach($aFunctionNames as $sFunctionName)
            {
                $this->addFunctionOptions($sFunctionName, $aFunctionOptions);
            }
        }
        foreach($aAnnotationOptions as $sFunctionName => $aFunctionOptions)
        {
            $this->addFunctionOptions($sFunctionName, $aFunctionOptions);
        }
    }


    /**
     * Check if the js code for this object must be generated
     *
     * @return bool
     */
    public function excluded(): bool
    {
        return $this->bExcluded;
    }


    /**
     * @return string
     */
    public function separator(): string
    {
        return $this->sSeparator;
    }


    /**
     * @return array
     */
    public function protectedMethods(): array
    {
        return $this->aProtectedMethods;
    }


    /**
     * @return array
     */
    public function beforeMethods(): array
    {
        return $this->aBeforeMethods;
    }


    /**
     * @return array
     */
    public function afterMethods(): array
    {
        return $this->aAfterMethods;
    }


    /**
     * @return array
     */
    public function diOptions(): array
    {
        return $this->aDiOptions;
    }


    /**
     * @return array
     */
    public function jsOptions(): array
    {
        return $this->aJsOptions;
    }


    /**
     * Set hook methods
     *
     * @param array $aHookMethods    The array of hook methods

     * @param string|array $xValue    The value of the configuration option

     *
     * @return void
     */
    private function setHookMethods(array &$aHookMethods, $xValue)
    {
        foreach($xValue as $sCalledMethod => $xMethodToCall)
        {
            if(is_array($xMethodToCall))
            {
                $aHookMethods[$sCalledMethod] = $xMethodToCall;
            }
            elseif(is_string($xMethodToCall))
            {
                $aHookMethods[$sCalledMethod] = [$xMethodToCall];
            }
        }
    }


    /**
     * @param array $aDiOptions

     */

    private function addDiOption(array $aDiOptions)
    {
        $this->aDiOptions = array_merge($this->aDiOptions, $aDiOptions);
    }


    /**
     * Set configuration options / call options for each method
     *
     * @param string $sName    The name of the configuration option

     * @param string|array $xValue    The value of the configuration option

     *
     * @return void
     */
    public function addOption(string $sName, $xValue)
    {
        switch($sName)
        {
        // Set the separator
        case 'separator':
            if($xValue === '_' || $xValue === '.')
            {
                $this->sSeparator = $xValue;
            }
            break;
        // Set the protected methods
        case 'protected':
            if(is_array($xValue))
            {
                $this->aProtectedMethods = array_merge($this->aProtectedMethods, $xValue);
            }
            break;
        // Set the methods to call before processing the request
        case '__before':
            $this->setHookMethods($this->aBeforeMethods, $xValue);
            break;
        // Set the methods to call after processing the request
        case '__after':
            $this->setHookMethods($this->aAfterMethods, $xValue);
            break;
        // Set the attributes to inject in the callable object
        case '__di':
            $this->addDiOption($xValue);
            $this->addDiOption(/** @scrutinizer ignore-type */ $xValue);
            break;
        case 'excluded':
            $this->bExcluded = (bool)$xValue;
            break;
        default:
            break;
        }
    }


    /**
     * @param string $sFunctionName

     * @param string $sOptionName

     * @param mixed $xOptionValue

     *
     * @return void
     */
    private function _addJsArrayOption(string $sFunctionName, string $sOptionName, $xOptionValue)
    {
        if(is_string($xOptionValue))
        {
            $xOptionValue = [$xOptionValue];
        }
        if(!is_array($xOptionValue))
        {
            return; // Do not save.
        }
        $aOptions = $this->aJsOptions[$sFunctionName][$sOptionName] ?? [];
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->aJsOptions[$sFunctionName][$sOptionName] = array_merge($aOptions, $xOptionValue);
    }


    /**
     * @param string $sFunctionName

     * @param string $sOptionName

     * @param mixed $xOptionValue

     *
     * @return void
     */
    private function _setJsOption(string $sFunctionName, string $sOptionName, $xOptionValue)
    {
        $this->aJsOptions[$sFunctionName][$sOptionName] = $xOptionValue;
    }


    /**
     * @param string $sFunctionName

     * @param string $sOptionName

     * @param mixed $xOptionValue

     *
     * @return void
     */
    private function addJsOption(string $sFunctionName, string $sOptionName, $xOptionValue)
    {
        switch($sOptionName)
        {
        // For databags, all the value are merged in a single array.
        case 'bags':
            $this->_addJsArrayOption($sFunctionName, $sOptionName, $xOptionValue);
            return;
        // For all the other options, including callback, only the last value is kept.
        case 'callback':
        default:
            $this->_setJsOption($sFunctionName, $sOptionName, $xOptionValue);
        }
    }


    /**
     * @param string $sFunctionName

     * @param array $aFunctionOptions

     *
     * @return void
     */
    private function addFunctionOptions(string $sFunctionName, array $aFunctionOptions)
    {
        foreach($aFunctionOptions as $sOptionName => $xOptionValue)
        {
            substr($sOptionName, 0, 2) === '__' ?

                // Options for PHP classes. They start with "__".
                $this->addOption($sOptionName, [$sFunctionName => $xOptionValue]) :

                // Options for javascript code.
                $this->addJsOption($sFunctionName, $sOptionName, $xOptionValue);
        }
    }


    /**
     * @param string $sMethodName

     *
     * @return array
     */
    public function getMethodOptions(string $sMethodName): array
    {
        // First take the common options.
        $aOptions = array_merge($this->aJsOptions['*'] ?? []); // Clone the array
        // Then add the method options.
        $aMethodOptions = $this->aJsOptions[$sMethodName] ?? [];
        foreach($aMethodOptions as $sOptionName => $xOptionValue)
        {
            // For databags, merge the values in a single array.
            // For all the other options, including callback, keep the last value.
            $aOptions[$sOptionName] = $sOptionName !== 'bags' ? $xOptionValue :

                array_unique(array_merge($aOptions[$sOptionName] ?? [], $xOptionValue));
        }
        return $aOptions;
    }

}

jaxon-php / jaxon-core

Push — main ( 79e33c...18c043 )

CallableObjectOptions A

Complexity

Size/Duplication

Importance

16 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like
