CalculationBase::removeFunction() - Code Metrics - Inspection of "Add Ability to Add Custom Functions to Calculation" - PHPOffice/PhpSpreadsheet - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#4390)

by Owen

created 2025-03-07 19:48 UTC

CalculationBase::removeFunction() A

↳ Parent: CalculationBase

Complexity

Conditions	3
Paths	3

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	6
c	1
b	0
f	0
dl	0
loc	12
rs	10
cc	3
nc	3
nop	1

<?php

namespace PhpOffice\PhpSpreadsheet\Calculation;

class CalculationBase
{
    /**
     * Get a list of all implemented functions as an array of function objects.
     *
     * return array<string, array<string, mixed>>
     */
    public static function getFunctions(): array
    {
        return FunctionArray::$phpSpreadsheetFunctions;
    }

    /**
     * Get address of list of all implemented functions as an array of function objects.
     *
     * @return array<string, array<string, mixed>>
     */
    protected static function &getFunctionsAddress(): array
    {
        return FunctionArray::$phpSpreadsheetFunctions;
    }

    /**
     * @param array<string, array<string, mixed>> $value
     */
    public static function addFunction(string $key, array $value): bool
    {
        $key = strtoupper($key);
        if (array_key_exists($key, FunctionArray::$phpSpreadsheetFunctions)) {
            return false;
        }
        $value['custom'] = true;
        FunctionArray::$phpSpreadsheetFunctions[$key] = $value;

        return true;
    }

    public static function removeFunction(string $key): bool
    {
        $key = strtoupper($key);
        if (array_key_exists($key, FunctionArray::$phpSpreadsheetFunctions)) {
            if (FunctionArray::$phpSpreadsheetFunctions[$key]['custom'] ?? false) {
                unset(FunctionArray::$phpSpreadsheetFunctions[$key]);

                return true;
            }
        }

        return false;
    }
}


1			<?php
2
3			namespace PhpOffice\PhpSpreadsheet\Calculation;
4
5			class CalculationBase
6			{
7			/**
8			* Get a list of all implemented functions as an array of function objects.
9			*
10			* return array<string, array<string, mixed>>
11			*/
12			public static function getFunctions(): array
13			{
14			return FunctionArray::$phpSpreadsheetFunctions;
15			}
16
17			/**
18			* Get address of list of all implemented functions as an array of function objects.
19			*
20			* @return array<string, array<string, mixed>>
21			*/
22			protected static function &getFunctionsAddress(): array
23			{
24			return FunctionArray::$phpSpreadsheetFunctions;
25			}
26
27			/**
28			* @param array<string, array<string, mixed>> $value
29			*/
30			public static function addFunction(string $key, array $value): bool
31			{
32			$key = strtoupper($key);
33			if (array_key_exists($key, FunctionArray::$phpSpreadsheetFunctions)) {
34			return false;
35			}
36			$value['custom'] = true;
37			FunctionArray::$phpSpreadsheetFunctions[$key] = $value;
38
39			return true;
40			}
41
42			public static function removeFunction(string $key): bool
43			{
44			$key = strtoupper($key);
45			if (array_key_exists($key, FunctionArray::$phpSpreadsheetFunctions)) {
46			if (FunctionArray::$phpSpreadsheetFunctions[$key]['custom'] ?? false) {
47			unset(FunctionArray::$phpSpreadsheetFunctions[$key]);
48
49			return true;
50			}
51			}
52
53			return false;
54			}
55			}
56

PHPOffice / PhpSpreadsheet

Pull Request — master (#4390)

CalculationBase::removeFunction() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like