FlexiPeeHP\Stitky

Complexity

Total Complexity

5

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	unsetLabel()	0	11	2
A	unsetLabels()	0	4	1
A	getLabels()	0	3	1
A	setLabel()	0	4	1

<?php
/**
 * FlexiPeeHP - Document Default Address support
 *
 * @author     Vítězslav Dvořák <vitex@arachne.cz>
 * @copyright  (C) 2018 Spoje.Net
 */

namespace FlexiPeeHP;

/**
 * Add for Objects use with "firma" data column
 * @author Vítězslav Dvořák <info@vitexsoftware.cz>
 */
trait Stitky
{

    /**
     * Get all labels for current record
     * 
     * @return array
     */
    public function getLabels()
    {
        return Stitek::listToArray($this->getDataValue('stitky'));

It seems like getDataValue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Set one of availble Labels for current record
     * 
     * @param string $label
     * 
     * @return boolean Operation success
     */
    public function setLabel($label)
    {
        $this->insertToFlexiBee(['id' => $this->getRecordID(), 'stitky' => $label]);

It seems like insertToFlexiBee() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like getRecordID() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->lastResultCode == 201;
    }

    /**
     * UnSet Label for Current Object record
     *
     * @param string     $label
     * @param FlexiBeeRW $this
     *
     * @return boolean   success result ?
     */
    public function unsetLabel($label)
    {
        $result = true;
        $labels = self::getLabels($this);

The call to FlexiPeeHP\Stitky::getLabels() has too many arguments starting with $this.

The method FlexiPeeHP\Stitky::getLabels() is not static, but was called statically.

        if (array_key_exists($label, $labels)) {
            unset($labels[$label]);
            $this->insertToFlexiBee(['id' => $this->getMyKey(), 'stitky@removeAll' => 'true',

It seems like getMyKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                'stitky' => $labels]);
            $result = ($this->lastResponseCode == 201);
        }
        return $result;
    }

    /**
     * UnSet all Labels for Current Object record
     *
     * @return boolean   success result ?
     */
    public function unsetLabels()
    {
        $this->insertToFlexiBee(['id' => $this->getMyKey(), 'stitky@removeAll' => 'true']);
        return $this->lastResponseCode == 201;
    }
}