Kunstmaan / KunstmaanBundlesCMS

src/Kunstmaan/LeadGenerationBundle/Controller/PopupsAdminListController.php

<?php

namespace Kunstmaan\LeadGenerationBundle\Controller;

use Kunstmaan\AdminBundle\Entity\EntityInterface;
use Kunstmaan\AdminListBundle\AdminList\Configurator\AdminListConfiguratorInterface;
use Kunstmaan\AdminListBundle\AdminList\ItemAction\SimpleItemAction;
use Kunstmaan\AdminListBundle\Controller\AdminListController;
use Kunstmaan\LeadGenerationBundle\AdminList\PopupAdminListConfigurator;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\HttpFoundation\Request;

class PopupsAdminListController extends AdminListController
{
    /**
     * @var AdminListConfiguratorInterface
     */
    private $configurator;

    /**
     * @return AdminListConfiguratorInterface
     */
    public function getAdminListConfigurator($listAction = false)
    {
        if (!isset($this->configurator)) {
            $this->configurator = new PopupAdminListConfigurator($this->getEntityManager());

            if ($listAction) {
                $create_route = function (EntityInterface $item) {
                    return array(
                        'path' => 'kunstmaanleadgenerationbundle_admin_rule_abstractrule_detail',
                        'params' => array('popup' => $item->getId()),
                    );
                };
                $this->configurator->addItemAction(new SimpleItemAction($create_route, 'th-list', 'Manage rules'));
            }
        }

        return $this->configurator;
    }

    /**
     * The index action
     *
     * @Route("/", name="kunstmaanleadgenerationbundle_admin_popup_abstractpopup")
     */
    public function indexAction(Request $request)
    {
        return parent::doIndexAction($this->getAdminListConfigurator(true), $request);
    }

    /**
     * The delete action
     *
     * @param int $id
     *
     * @Route("/{id}/delete", requirements={"id" = "\d+"}, name="kunstmaanleadgenerationbundle_admin_popup_abstractpopup_delete", methods={"GET", "POST"})
     *
     * @return array

Should the return type not be \Symfony\Component\HttpFoundation\Response?

     */
    public function deleteAction(Request $request, $id)
    {
        return parent::doDeleteAction($this->getAdminListConfigurator(), $id, $request);

$this->getAdminListConfigurator() of type object<Kunstmaan\AdminListBundle\AdminList\Configurator\AdminListConfiguratorInterface> is not a sub-type of object<Kunstmaan\AdminListBundle\AdminList\Configurator\AbstractAdminListConfigurator>. It seems like you assume a concrete implementation of the interface Kunstmaan\AdminListBundle\AdminList\Configurator\AdminListConfiguratorInterface to be always present.

It seems like you call parent on a different method (doDeleteAction() instead of deleteAction()). Are you sure this is correct? If so, you might want to change this to $this->doDeleteAction().

    }

    /**
     * The edit action
     *
     * @param int $id
     *
     * @Route("/{id}/edit", requirements={"id" = "\d+"}, name="kunstmaanleadgenerationbundle_admin_popup_abstractpopup_edit", methods={"GET", "POST"})
     *
     * @return array

Should the return type not be \Symfony\Component\HttpFoundation\Response?

     */
    public function editAction(Request $request, $id)
    {
        return parent::doEditAction($this->getAdminListConfigurator(), $id, $request);

It seems like you call parent on a different method (doEditAction() instead of editAction()). Are you sure this is correct? If so, you might want to change this to $this->doEditAction().

    }

    /**
     * The add action
     *
     * @Route("/add",  name="kunstmaanleadgenerationbundle_admin_popup_abstractpopup_add", methods={"GET", "POST"})
     *
     * @return array

Should the return type not be \Symfony\Component\HttpFoundation\Response?

     */
    public function addAction(Request $request)
    {
        $type = $request->get('type');

        return parent::doAddAction($this->getAdminListConfigurator(), $type, $request);

$this->getAdminListConfigurator() of type object<Kunstmaan\AdminListBundle\AdminList\Configurator\AdminListConfiguratorInterface> is not a sub-type of object<Kunstmaan\AdminListBundle\AdminList\Configurator\AbstractAdminListConfigurator>. It seems like you assume a concrete implementation of the interface Kunstmaan\AdminListBundle\AdminList\Configurator\AdminListConfiguratorInterface to be always present.

    }
}