TraitLines::getLineItem() - Code Metrics - Inspection of "2 new tests on local files" - danielgp/eFactura - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — main ( 577f79...66b7bb )

by Daniel

created 2024-01-27 17:23 UTC

TraitLines::getLineItem() B

↳ Parent: TraitLines

Complexity

Conditions	10
Paths	10

Size

Total Lines	29
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	22
c	0
b	0
f	0
nc	10
nop	1
dl	0
loc	29
rs	7.6666

How to fix Complexity

<?php

/**
 *
 * The MIT License (MIT)
 *
 * Copyright (c) 2024 Daniel Popiniuc
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in all
 * copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 *  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 * SOFTWARE.
 *
 */

namespace danielgp\efactura;

trait TraitLines
{

    use TraitBasic,
        TraitTax;

    private function getDocumentLines(\SimpleXMLElement $arrayDataIn, string $strTag): array
    {
        $arrayLines = [];
        $intLineNo  = 0;
        foreach ($arrayDataIn->children('cac', true) as $strNodeName => $child) {
            if ($strNodeName === ($strTag . 'Line')) {
                $intLineNo++;
                $intLineStr              = ($intLineNo < 10 ? '0' : '') . $intLineNo;
                $arrayLines[$intLineStr] = $this->getLine($child);
                $arrayLines[$intLineStr] = $this->getLine(/** @scrutinizer ignore-type */ $child);
            }
        }
        return $arrayLines;
    }

    private function getLine(\SimpleXMLElement $child): array
    {
        $arrayOutput = [];
        foreach ($this->arraySettings['CustomOrder']['Lines@Read'] as $strElement => $strType) {
            switch ($strType) {
                case 'Item':
                    $arrayOutput['Item'] = $this->getLineItem($child->children('cac', true)->Item);
                    break;
                case 'Multiple':
                    $intLineNo           = 0;
                    foreach ($child->children('cac', true)->$strElement as $value2) {
                        $intLineNo++;
                        $intLineStr                            = ($intLineNo < 10 ? '0' : '') . $intLineNo;
                        $arrayOutput[$strElement][$intLineStr] = $this->getElements($value2);
                    }
                    break;
                case 'Single':
                    if (count($child->children('cbc', true)->$strElement) !== 0) {
                        $arrayOutput[$strElement] = $child->children('cbc', true)->$strElement->__toString();
                    }
                    if (count($child->children('cac', true)->$strElement) !== 0) {
                        $arrayOutput[$strElement] = $this->getElements($child->children('cac', true)->$strElement);
                    }
                    break;
                case 'SingleTag':
                    if (count($child->children('cbc', true)->$strElement) !== 0) {
                        $arrayOutput[$strElement] = $this->getElementSingle($child->children('cbc', true)->$strElement);
                    }
                    break;
            }
        }
        return $arrayOutput;
    }

    private function getLineItem(\SimpleXMLElement $child3): array
    {
        $arrayOutput = [];
        foreach ($this->arraySettings['CustomOrder']['Lines_Item@Read'] as $key => $value) {
            switch ($value) {
                case 'MultipleAggregate':
                    $intLineNo = 0;
                    foreach ($child3->children('cac', true)->$key as $value2) {
                        $intLineNo++;
                        $intLineStr                     = ($intLineNo < 10 ? '0' : '') . $intLineNo;
                        $arrayOutput[$key][$intLineStr] = $this->getElements($value2);
                    }
                    break;
                case 'SingleAggregate':
                    if (count($child3->children('cac', true)->$key) !== 0) {
                        $arrayOutput[$key] = $this->getElements($child3->children('cac', true)->$key);
                    }
                    break;
                case 'SingleBasic':
                    if (count($child3->children('cbc', true)->$key) !== 0) {
                        $arrayOutput[$key] = $child3->children('cbc', true)->$key->__toString();
                    }
                    break;
                case 'TaxCategory':
                    $arrayOutput[$key] = $this->getTaxCategory($child3->children('cac', true)->$key);
                    break;
            }
        }
        return $arrayOutput;
    }
}


danielgp / eFactura

Push — main ( 577f79...66b7bb )

TraitLines::getLineItem() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like