Что не так с моими экстеншенами в symfony3?

Question

[sl1m_dogg]

Объявил два сервиса с зависимостями, прописую ./bin/console debug:container и вижу оба сервиса в контейнере, но когда запускаю контроллер, то пишет null given. Код:
ideone.com/VQ65jr

Answer 1

[sl1m_dogg]

<?php

namespace MailerBundle\DependencyInjection;

use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\DependencyInjection\Reference;
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
use Symfony\Component\DependencyInjection\ContainerBuilder;

class MailerExtension extends Extension
{

    /**
     * Loads a specific configuration.
     *
     * @param array $configs An array of configuration values
     * @param ContainerBuilder $container A ContainerBuilder instance
     *
     * @throws \InvalidArgumentException When provided tag is not defined in this extension
     */
    public function load(array $configs, ContainerBuilder $container)
    {
        $container->setDefinition('emailsender', new Definition('MailerBundle\Sender\EmailSender'),[
            new Reference('swiftmailer.mailer'),
            new Reference('swiftmailer.transport')]);

        $container->setDefinition('emailnotifier', new Definition('MailerBundle\Controller\EmailController'),[
                  new Reference('emailsender')]);
    }
}

Answer 2

[BoShurik]

Параметры классов передаются в конструкторе Definition, а не в методе setDefinition

public function load(array $configs, ContainerBuilder $container)
    {
        $container->setDefinition('emailsender', new Definition('MailerBundle\Sender\EmailSender', [
            new Reference('swiftmailer.mailer'),
            new Reference('swiftmailer.transport')]));

        $container->setDefinition('emailnotifier', new Definition('MailerBundle\Controller\EmailController', [
                  new Reference('emailsender')]));
    }

Используй PhpStorm, он от таких ошибок защищает

Comments

[sl1m_dogg]

ну они и так в контейнере есть, в вендорах

[sl1m_dogg]

ребят, очень срочно, если кто поможет кодом буду очень признателен, затянул с тестовым, а еще с авторизацией и тестами разбиратся

[sl1m_dogg]

можно кодом? что то шторм не понимать

[BoShurik]

sl1m_dogg: обновил ответ

[sl1m_dogg]

BoShurik: если не ошибаюсь, то у меня сейчас такой же код

[sl1m_dogg]

EmailController: <?php

namespace MailerBundle\Controller;

use MailerBundle\Entity\Notification;
use MailerBundle\Sender\EmailSender;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;

class EmailController extends Controller
{
/**
* var EmailSender
*/
private $sender;

/**
* EmailController constructor.
* @param EmailSender $sender
*/
public function __construct(EmailSender $sender)
{
$this->sender = $sender;
}

/**
* return \Symfony\Component\HttpFoundation\Response
*/
public function indexAction()
{
return $this->render('MailerBundle:Email:index.html.twig');
}

/**
* @param Request $request
* return \Symfony\Component\HttpFoundation\Response
*/
public function sendAction(Request $request)
{
$notification = new Notification();
$user = $this->get('security.token_storage')->getToken()->getUser();
$notification->setBody($request->request->get('subject'), $user->getUsername());
$notification->setSubject($request->request->get('body'));

try {
$this->sender->send($notification);
} catch (\Exception $ex) {
$this->render('MailerBundle:Email:error.html.twig', [
'message' => $ex->getMessage()
]);
}

$this->render('MailerBundle:Email:success.html.twig');
}
}

[BoShurik]

В твоем коде

new Definition('MailerBundle\Sender\EmailSender'),[

, а надо

new Definition('MailerBundle\Sender\EmailSender', [

[sl1m_dogg]

Приавильно понял: <?php

namespace MailerBundle\DependencyInjection;

use Symfony\Component\DependencyInjection\Definition;
use Symfony\Component\DependencyInjection\Reference;
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
use Symfony\Component\DependencyInjection\ContainerBuilder;

class MailerExtension extends Extension
{

/**
* Loads a specific configuration.
*
* @param array $configs An array of configuration values
* @param ContainerBuilder $container A ContainerBuilder instance
*
* @throws \InvalidArgumentException When provided tag is not defined in this extension
*/
public function load(array $configs, ContainerBuilder $container)
{
$container->setDefinition('emailsender', new Definition('MailerBundle\Sender\EmailSender',[
new Reference('swiftmailer.mailer'),
new Reference('swiftmailer.transport')]));

$container->setDefinition('emailnotifier', new Definition('MailerBundle\Controller\EmailController',[
new Reference('emailsender')]));
}
}

[BoShurik]

Да, я так в ответе и написал

[sl1m_dogg]

BoShurik: не помогать. Может дело в 3й симфе... Хотя, мля, я так не долбался в жизни, особенно из-за ДИ.

[BoShurik]

Нет, там ничего не поменялось в этом плане. Значит проблема где-то в другом.

[sl1m_dogg]

./bin/console debug:container | grep email оба сервиса в контейнере

[BoShurik]

Кеш почистил?

[sl1m_dogg]

Catchable Fatal Error: Argument 1 passed to MailerBundle\Controller\EmailController::__construct() must be an instance of MailerBundle\Sender\EmailSender, none given, called in C:\xampp1\htdocs\murkatest\var\cache\dev\classes.php on line 2084 and defined

[BoShurik]

Покажи роутинг

[sl1m_dogg]

site:
path: /
defaults: { _controller: MailerBundle:Site:index }

email:
path: /email
defaults: { _controller: MailerBundle:Email:index }

send_email:
path: /email/send
defaults: { _controller: MailerBundle:Email:send }

login:
path: /login
defaults: { _controller: MailerBundle:Site:login }

[BoShurik]

symfony.com/doc/current/cookbook/controller/servic...

[sl1m_dogg]

BoShurik: # app/config/routing.yml
hello:
path: /hello
defaults: { _controller: app.hello_controller:indexAction } ?

[sl1m_dogg]

BoShurik: а как в моем случаи, а то я не понял логику как писать роуты?

[sl1m_dogg]

BoShurik: site:
path: /
defaults: { _controller: MailerBundle:Site:index }

email:
path: /email
defaults: { _controller: mailer.email_controller:indexAction }

send_email:
path: /email/send
defaults: { _controller: MailerBundle:Email:send }

login:
path: /login
defaults: { _controller: MailerBundle:Site:login }

[BoShurik]

emailnotifier:indexAction и emailnotifier:sendAction

[sl1m_dogg]

something went wrong: You have requested a non-existent service "mailer.email_controller".

[sl1m_dogg]

BoShurik: щя вылью на гитхаб что бы не кидать код сюда

[sl1m_dogg]

BoShurik: не помогло кстати

[sl1m_dogg]

https://github.com/artemzakholodilo/murkatest

[BoShurik]

Ты пробовал прописать?
emailnotifier:indexAction
и
emailnotifier:sendAction
В коде у тебя mailer.emailnotifier

[sl1m_dogg]

BoShurik: теперь 500ую отдает

[BoShurik]

а что за ошибка?

[sl1m_dogg]

BoShurik: Ошибка сервера

500

[BoShurik]

А в логах что?

[sl1m_dogg]

BoShurik: пока не смотрел, не уверен что роуты правильно написаны, это нововведение в 3ей симфе?

[BoShurik]

В 3ей симфони нет нововведений. В ней убраны все депрекейтед вещи 2ой ветки

[BoShurik]

Денис: в предыдущем вопросе Нужно правильно написать EmailExtension(DI) SYmfony3, что не так с моим кодом? он ответил на этот вопрос :)

хочу показать что у меня есть яйца и умею работать с extension