Kunstmaan\MediaBundle\AdminList\ItemAction\MediaDeleteItemAction

Complexity

Total Complexity

5

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getUrlFor()	0	7	1
A	getLabelFor()	0	4	1
A	getIconFor()	0	4	1
A	getTemplate()	0	4	1

<?php

namespace Kunstmaan\MediaBundle\AdminList\ItemAction;

use Kunstmaan\AdminListBundle\AdminList\ItemAction\ItemActionInterface;

class MediaDeleteItemAction implements ItemActionInterface
{
    /**
     * @var string
     */
    private $redirectUrl;

    /**
     * @param string $redirectUrl
     */
    public function __construct($redirectUrl)
    {
        $this->redirectUrl = $redirectUrl;
    }

    /**
     * @param mixed $item
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,string|array>.

     */
    public function getUrlFor($item)
    {
        return array(
          'path' => 'KunstmaanMediaBundle_media_delete',
          'params' => array('mediaId' => $item->getId(), 'redirectUrl' => $this->redirectUrl),
        );
    }

    /**
     * @param mixed $item
     *
     * @return string
     */
    public function getLabelFor($item)
    {
        return 'Delete';
    }

    /**
     * @param mixed $item
     *
     * @return string
     */
    public function getIconFor($item)
    {
        return 'remove-sign';
    }

    /**
     * @return string

Should the return type not be string|null?

     */
    public function getTemplate()
    {
        return null;
    }
}