rtens\domin\delivery\web\renderers\link\LinkPrinter

Complexity

Total Complexity

13

Size/Duplication

Total Lines	100
Duplicated Lines	13 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	7
Bugs	0	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	createLinkElements()	0	3	1
B	createDropDown()	0	32	4
B	createLinks()	0	22	4
A	makeUrl()	13	13	3

<?php
namespace rtens\domin\delivery\web\renderers\link;

use rtens\domin\ActionRegistry;
use rtens\domin\delivery\web\Element;
use rtens\domin\delivery\web\resources\ExecutionResource;
use rtens\domin\delivery\web\WebCommentParser;

class LinkPrinter {

    /** @var LinkRegistry */
    private $links;

    /** @var ActionRegistry */
    private $actions;

    /** @var WebCommentParser */
    private $parser;

    public function __construct(LinkRegistry $links, ActionRegistry $actions, WebCommentParser $parser) {
        $this->links = $links;
        $this->actions = $actions;
        $this->parser = $parser;
    }

    /**
     * @param mixed $object
     * @return array
     */
    public function createLinkElements($object) {
        return $this->createLinks($object, 'btn btn-xs btn-primary');
    }

    /**
     * @param mixed $object
     * @param string|null $caption
     * @return array|\rtens\domin\delivery\web\Element[]
     */
    public function createDropDown($object, $caption = null) {
        $links = $this->createLinks($object);
        if (empty($links)) {
            return [];
        }

        if (!$caption) {

The expression $caption of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            if (is_object($object)) {
                $caption = (new \ReflectionClass($object))->getShortName();
            } else {
                $caption = 'Actions';
            }
        }

        return [
            new Element('div', ['class' => 'dropdown'], [
                new Element('button', [
                    'class' => 'btn btn-xs btn-primary dropdown-toggle',
                    'type' => 'button',
                    'data-toggle' => 'dropdown',
                    'aria-haspopup' => 'true',
                    'aria-expanded' => 'false'
                ], [
                    $caption,
                    new Element('span', ['class' => 'caret'])
                ]),
                new Element('ul', ['class' => 'dropdown-menu'], array_map(function (Element $element) {
                    return new Element('li', [], [$element]);
                }, $links))
            ])
        ];
    }

    private function createLinks($object, $classes = '') {
        return array_map(function (Link $link) use ($object, $classes) {
            $action = $this->actions->getAction($link->actionId());

            $parameters = $link->parameters($object);
            if ($link->force()) {
                $parameters[ExecutionResource::FORCE_ARG] = true;
            }

            $attributes = ['class' => $classes, 'href' => $this->makeUrl($link->actionId(), $parameters)];
            if ($link->confirm() !== null) {
                $attributes['onclick'] = "return confirm('{$link->confirm()}');";
            }
            $description = $action->description();
            if (!is_null($description)) {
                $attributes['title'] = str_replace('"', "'", strip_tags($this->parser->shorten($description)));
            }
            return new Element('a', $attributes, [
                $action->caption()
            ]);
        }, $this->links->getLinks($object));
    }

    private function makeUrl($actionId, array $parameters) {

This method seems to be duplicated in your project.

        $url = $actionId;

        if ($parameters) {

The expression $parameters of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $keyValues = [];
            foreach ($parameters as $key => $value) {
                $keyValues[] = urlencode($key) . '=' . urlencode($value);
            }
            $url .= '?' . implode('&', $keyValues);
        }

        return $url;
    }
}