Victoire\Bundle\WidgetMapBundle\Builder\WidgetMapBuilder

Complexity

Total Complexity

34

Size/Duplication

Total Lines	182
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	12
Bugs	1	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
C	build()	0	48	10
C	getAvailablePosition()	0	28	7
A	orderizeWidgetMap()	0	19	4
B	removeOverwritedWidgetMaps()	0	21	6
A	removeDeletedWidgetMaps()	0	10	4
A	findRootWidgetMap()	0	12	3

<?php

namespace Victoire\Bundle\WidgetMapBundle\Builder;

use Doctrine\ORM\EntityManager;
use Victoire\Bundle\CoreBundle\Entity\View;
use Victoire\Bundle\WidgetMapBundle\Entity\WidgetMap;

/**
 * View WidgetMap builder.
 *
 * ref: victoire_widget_map.builder
 */
class WidgetMapBuilder
{
    /**
     * This method build widgetmaps relativly to given view and it's templates
     *
     * @param View          $view
     * @param EntityManager $em
     * @param bool          $updatePage
     *
     * @return array
     */
    public function build(View $view, EntityManager $em = null, $updatePage = true)

The parameter $em is not used and could be removed.

    {
        $widgetMaps = [];
        // populate a $widgetmaps array with widgetmaps of given view + widgetmaps of it's templates
        if ($view->getWidgetMaps()) {
            $widgetMaps = $view->getWidgetMaps()->toArray();

The method toArray cannot be called on $view->getWidgetMaps() (of type array).

        }
        $template = clone $view;
        $builtWidgetMap = [];

        while (null !== $template = $template->getTemplate()) {
            if ($template->getWidgetMaps()) {
                foreach ($template->getWidgetMaps()->toArray() as $item) {
                    $widgetMaps[] = $item;
                }
            }
        }

        $slots = $this->removeOverwritedWidgetMaps($widgetMaps);

        $this->removeDeletedWidgetMaps($slots);

        foreach ($slots as $slot => $widgetMaps) {
            $mainWidgetMap = null;

$mainWidgetMap is not used, you could remove the assignment.

            $builtWidgetMap[$slot] = [];

            $rootWidgetMap = $this->findRootWidgetMap($widgetMaps);

Are you sure the assignment to $rootWidgetMap is correct as $this->findRootWidgetMap($widgetMaps) (which targets Victoire\Bundle\WidgetMapBundle\Builder\WidgetMapBuilder::findRootWidgetMap()) seems to always return null.

            if ($rootWidgetMap) {
                $builtWidgetMap[$slot][] = $rootWidgetMap;
                $builtWidgetMap = $this->orderizeWidgetMap($rootWidgetMap, $builtWidgetMap, $slot, $widgetMaps, $view);
            }
        }
        $_builtWidgetMap = $builtWidgetMap;
        $builtWidgetMap = [];
        foreach ($_builtWidgetMap as $slot => $__builtWidgetMap) {
            foreach ($__builtWidgetMap as $key => $item) {
                // store each widgetmap in an array with the widget id as key to make the "View::getWidgetMapByWidgetAndView"
                // method more efficient
                $builtWidgetMap[$slot][$item->getWidget()->getId()] = $item;
            }
        }
        if ($updatePage) {
            $view->setBuiltWidgetMap($builtWidgetMap);

$builtWidgetMap is of type array, but the function expects a string.

        }

        return $builtWidgetMap;
    }

    /**
     * This method takes the builtWidgetMap for view and creates an array that indicate, for
     * each widgetmap, if the position "after" and "before" are available
     * @param  View  $view
     * @return array
     */
    public function getAvailablePosition(View $view)
    {
        $widgetMaps = $view->getBuiltWidgetMap();

        $availablePositions = [];
        foreach ($widgetMaps as $slot => $widgetMap) {
            foreach ($widgetMap as $_widgetMap) {
                $availablePositions[$slot][$_widgetMap->getId()]['id'] = $_widgetMap->getId();
                $availablePositions[$slot][$_widgetMap->getId()][WidgetMap::POSITION_BEFORE] = true;
                $availablePositions[$slot][$_widgetMap->getId()][WidgetMap::POSITION_AFTER] = true;
                if ($_widgetMap->getReplaced()) {
                    $availablePositions[$slot][$_widgetMap->getId()]['replaced'] = $_widgetMap->getReplaced()->getId();
                }
            }
            /** @var WidgetMap $_widgetMap */
            foreach ($widgetMap as $_widgetMap) {
                if ($_widgetMap->getParent()) {
                    if ($substitute = $_widgetMap->getParent()->getSubstituteForView($view)) {
                        $availablePositions[$slot][$substitute->getId()][$_widgetMap->getPosition()] = false;
                    } else {
                        $availablePositions[$slot][$_widgetMap->getParent()->getId()][$_widgetMap->getPosition()] = false;
                    }
                }
            }
        }

        return $availablePositions;
    }

    /*
     * Get the children of given WidgetMap and place them recursively in the "builtWidgetMap" array at the right place
     * depending of the children parents and positions
     * @param WidgetMap $currentWidgetMap
     */
    protected function orderizeWidgetMap($currentWidgetMap, $builtWidgetMap, $slot, $widgetMaps, $view)
    {
        $children = $currentWidgetMap->getChildren($view);
        foreach ($children as $child) {
            // check if the founded child belongs to the view
            if (in_array($child, $widgetMaps)
            ) {
                // Find the position of the "currentWidgetMap" inside the builtWidgetMap,
                // add "1" to this position if wanted position is "after", 0 is it's before.
                $offset = array_search($currentWidgetMap, $builtWidgetMap[$slot]) + ($child->getPosition() == WidgetMap::POSITION_AFTER ? 1 : 0);
                // insert the child in builtWidgetMap at offset position
                array_splice($builtWidgetMap[$slot], $offset, 0, [$child]);
                // call myself with child
                $builtWidgetMap = $this->orderizeWidgetMap($child, $builtWidgetMap, $slot, $widgetMaps, $view);
            }
        }

        return $builtWidgetMap;
    }

    /**
     * Create a $slot array that'll contain, for each slot, a widgetmap id as key and a widgetmap as value
     * Do not keeps widgetMaps that are overwrited
     *
     * @param $widgetMaps
     * @return array
     */
    protected function removeOverwritedWidgetMaps($widgetMaps)
    {
        $slots = [];
        /** @var WidgetMap $widgetMapItem */
        foreach ($widgetMaps as $widgetMapItem) {
            $id = $widgetMapItem->getId();
            // If the widgetmap replace one (has a "replaced"), use the replaced id as key
            if ($widgetMapItem->getReplaced()) {
                $id = $widgetMapItem->getReplaced()->getId();
            }
            // If "id" is not present in slot array or
            if (empty($slots[$widgetMapItem->getSlot()][$id])
                // or if the id exists AND the inserted widgetmap is overwrite|delete, then erase the initial widget by it
                || !empty($slots[$widgetMapItem->getSlot()][$id])
                && $widgetMapItem->getAction() !== WidgetMap::ACTION_CREATE) {
                $slots[$widgetMapItem->getSlot()][$id] = $widgetMapItem;
            }
        }

        return $slots;
    }

    /**
     * If "delete" widgetmaps are found, remove it because they're not rendered
     * @param $slots
     */
    protected function removeDeletedWidgetMaps(&$slots)
    {
        foreach ($slots as $slot => $widgetMaps) {
            foreach ($widgetMaps as $key => $widgetMap) {
                if ($widgetMap->getAction() == WidgetMap::ACTION_DELETE) {
                    unset($slots[$slot][$key]);
                }
            }
        }
    }

    /**
     * Find the "root" widgetmap (the one that has no parent)
     * @param WidgetMap[] $widgetMaps
     */
    private function findRootWidgetMap($widgetMaps)
    {
        $rootWidgetMap = null;
        foreach ($widgetMaps as $_widgetMap) {
            if (!$_widgetMap->getParent()) {
                $rootWidgetMap = $_widgetMap;
                break;
            }
        }

        return $rootWidgetMap;
    }

}