LeKoala\Admini\Forms\AdminiCmsActionsExtension

Complexity

Total Complexity

14

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	onAfterUpdateCMSActions()	0	20	3
B	updateStyle()	0	38	11

<?php

namespace LeKoala\Admini\Forms;

use LeKoala\Admini\LeftAndMain;
use SilverStripe\Core\Extension;
use LeKoala\Admini\MaterialIcons;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\FormAction;
use SilverStripe\Control\Controller;

/**
 * Improve how actions are rendered if you are using the cms-actions module
 */
class AdminiCmsActionsExtension extends Extension
{
    /**
     * @param FieldList||FormAction[] $actions

The doc comment FieldList||FormAction[] at position 2 could not be parsed: Unknown type name '|' at position 2 in FieldList||FormAction[].

     * @return void
     */
    public function onAfterUpdateCMSActions(FieldList $actions)
    {
        $controller = Controller::curr();

        // Avoid side effects on regular admin
        if (!$controller instanceof LeftAndMain) {
            return;
        }

        /** @var \SilverStripe\Forms\Tab $dropUpContainer */
        $dropUpContainer =  $actions->fieldByName('ActionMenus.MoreOptions');

Are you sure the assignment to $dropUpContainer is correct as $actions->fieldByName('ActionMenus.MoreOptions') targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

        // Replace with our own logic
        if ($dropUpContainer) {

$dropUpContainer is of type SilverStripe\Forms\Tab, thus it always evaluated to true.

            $group = new AdminiDropUpField($dropUpContainer->getChildren());
            $actions->insertBefore('RightGroup', $group);
            $actions->removeByName('ActionMenus');
        }

        $this->updateStyle($actions);
    }

    protected function updateStyle($actions)
    {
        /** @var FormAction $action */
        foreach ($actions as $action) {
            if ($action->hasMethod("getChildren")) {
                foreach ($action->getChildren() as $child) {
                    $this->updateStyle($child);
                }
                continue;
            }
            if ($action->hasExtraClass('btn-outline-danger')) {
                $action->removeExtraClass('btn-outline-danger');
                $action->addExtraClass('btn-danger');
                if ($action->hasExtraClass('font-icon-trash-bin')) {
                    $action->removeExtraClass('font-icon-trash-bin');
                    $action->setIcon(MaterialIcons::DELETE);
                }
            }
            if ($action->hasExtraClass('btn-outline-primary')) {
                $action->removeExtraClass('btn-outline-primary');
                $action->addExtraClass('btn-outline-success');
                if ($action->hasExtraClass('font-icon-level-up')) {
                    $action->removeExtraClass('font-icon-level-up');
                    $action->setIcon(MaterialIcons::KEYBOARD_RETURN);
                }
                if ($action->hasExtraClass('font-icon-angle-double-left')) {
                    $action->removeExtraClass('font-icon-angle-double-left');
                    $action->setIcon(MaterialIcons::NAVIGATE_BEFORE);
                }
                if ($action->hasExtraClass('font-icon-angle-double-right')) {
                    $action->removeExtraClass('font-icon-angle-double-right');
                    $action->addExtraClass('btn-flex btn-icon-end');
                    $action->setIcon(MaterialIcons::NAVIGATE_NEXT);
                }
            }
            if ($action->hasExtraClass('btn-primary')) {
                $action->removeExtraClass('btn-primary');
                $action->addExtraClass('btn-success');
            }
        }
    }
}