FlexiPeeHP\SkladovyPohybPolozka

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	addSerialNumber()	0	14	4

<?php
/**
 * FlexiPeeHP - Objekt položky skladového pohybu.
 *
 * @author     Vítězslav Dvořák <vitex@arachne.cz>
 * @copyright  (C) 2015-2018 Spoje.Net
 */

namespace FlexiPeeHP;

/**
 * Položka skladového pohybu
 *
 * @link https://demo.flexibee.eu/c/demo/skladovy-pohyb-polozka/properties
 */
class SkladovyPohybPolozka extends RW
{

    use Stitky;
    /**
     * Evidence užitá objektem.
     *
     * @var string
     */
    public $evidence = 'skladovy-pohyb-polozka';

    /**
     * Add Items Serial Number
     * 
     * @param string  $number serial number
     * @param boolean $isMain main number
     * 
     * @return boolean Success
     */
    public function addSerialNumber($number, $isMain = false)
    {
        $numberBranch['kod'] = $number;

$numberBranch was never initialized. Although not strictly required by PHP, it is generally a good practice to add $numberBranch = array(); before regardless.

        $crrentSerialNumbers = $this->getDataValue('vyrobniCislaPrijata');
        if ($isMain) {
            if (!empty($crrentSerialNumbers)) {
                foreach (array_keys($crrentSerialNumbers) as $serialNumberID) {
                    unset($this->data['vyrobniCislaPrijata'][$serialNumberID]['vyrobnicislohlav']);
                }
            }
            $numberBranch['vyrobnicislohlav'] = 1;
        }
        $this->setDataValue('mnozMj', count($crrentSerialNumbers) + 1);
        return $this->addArrayToBranch($numberBranch, 'vyrobniCislaPrijata');
    }
}