Zemit\Mvc\Model\Position::_setPosition()

Complexity

Conditions	13
Paths	2

Size

Total Lines	44
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	182

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	Position::setPositionBehavior()	0	4	1
A	Position::reorder()	0	16	5
A	Position::getPositionBehavior()	0	3	1

<?php

/**
 * This file is part of the Zemit Framework.
 *
 * (c) Zemit Team <contact@zemit.com>
 *
 * For the full copyright and license information, please view the LICENSE.txt
 * file that was distributed with this source code.
 */

namespace Zemit\Mvc\Model;

use Exception;
use Zemit\Mvc\Model;
use Zemit\Mvc\Model\Behavior\Position as PositionBehavior;

trait Position
{
    use Model\AbstractTrait\AbstractBehavior;
    use Attribute;
    use Options;
    
    public Model\Behavior\Position $positionBehavior;
    
    /**
     * Initializing Position
     */
    public function initializePosition(?array $options = null): void
    {
        $options ??= $this->getOptionsManager()->get('position') ?? [];
        $this->setPositionBehavior(new PositionBehavior($options));
    }
    
    /**
     * Set Position Behavior
     */
    public function setPositionBehavior(PositionBehavior $positionBehavior): void
    {
        $this->positionBehavior = $positionBehavior;
        $this->addBehavior($this->positionBehavior);
    }
    
    /**
     * Get Position Behavior
     */
    public function getPositionBehavior(): PositionBehavior
    {
        return $this->positionBehavior;
    }
    
    /**
     * Re-ordering an entity
     * - Update position+1 done using afterSave event
     *
     * @return mixed
     * @throws Exception
     */
    public function reorder(?int $position = null, ?string $positionField = null)
    {
        $positionField ??= $this->getPositionBehavior()->getField();
        
        if ($this->fireEventCancel('beforeReorder') === false) {

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

            return false;
        }
        
        $this->assign([$positionField => $position], [$positionField]);

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

        $saved = $this->save() && (!$this->hasSnapshotData() || $this->hasUpdated($positionField));

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

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

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

        
        if ($saved) {
            $this->fireEvent('afterReorder');

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

        }
        
        return $saved;
    }
}