DNADesign\Elemental\Services\ReorderElements

Complexity

Total Complexity

7

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	reorder()	0	54	5
A	__construct()	0	9	2

<?php

namespace DNADesign\Elemental\Services;

use DNADesign\Elemental\Models\BaseElement;
use InvalidArgumentException;
use SilverStripe\Core\Convert;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\Queries\SQLUpdate;

class ReorderElements
{
    /**
     * @var BaseElement $block
     */
    private $element;

    public function __construct(BaseElement $element)
    {
        if (!is_subclass_of($element, BaseElement::class)) {
            throw new InvalidArgumentException(sprintf(
                'Invalid ' . BaseElement::class . ' passed to ' . __CLASS__
            ));
        }

        $this->element = $element;
    }

    /**
     * @param
     */
    public function reorder($elementToBeAfterID = 0)
    {
        $element = $this->element;
        $parentId = $element->ParentID;

The property ParentID does not exist on DNADesign\Elemental\Models\BaseElement. Since you implemented __get, consider adding a @property annotation.

        $currentPosition = (int) $element->Sort;
        $sortAfterPosition = 0;

        if ($elementToBeAfterID) {
            /** @var BaseElement $afterBlock */
            $afterElement = BaseElement::get()->byID($elementToBeAfterID);

            if (!$afterElement) {

$afterElement is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

                throw new InvalidArgumentException(sprintf(
                    '%s#%s not found',
                    BaseElement::class,
                    $elementToBeAfterID
                ));
            }

            if ($afterElement->ParentID !== $parentId) {
                throw new InvalidArgumentException(
                    'Trying to sort element to be placed after an element from a different elemental area'
                );
            }

            $sortAfterPosition = (int) $afterElement->Sort;
        }

        // We are updating records with SQL queries to avoid the ORM triggering the creation of new versions
        // for each element that is affected by this reordering.
        $tableName = Convert::raw2sql(DataObject::getSchema()->tableName(BaseElement::class));

        if ($sortAfterPosition < $currentPosition) {
            $operator = '+';
            $filter = "\"$tableName\".\"Sort\" > $sortAfterPosition AND \"$tableName\".\"Sort\" < $currentPosition";
            $newBlockPosition = $sortAfterPosition + 1;
        } else {
            $operator = '-';
            $filter = "\"$tableName\".\"Sort\" <= $sortAfterPosition AND \"$tableName\".\"Sort\" > $currentPosition";
            $newBlockPosition = $sortAfterPosition;
        }

        $query = SQLUpdate::create()
            ->setTable("\"$tableName\"")
            ->assignSQL('"Sort"', "\"$tableName\".\"Sort\" $operator 1")
            ->addWhere([$filter, "\"$tableName\".\"ParentID\"" => $parentId]);

        $query->execute();

        // Now use the ORM to write a new version of the record that we are directly reordering
        $element->Sort = $newBlockPosition;
        $element->write();

        return $element;
    }
}