Symfony2. Как правильно обратиться к mailer?

Question

[1alexandr]

Здравствуйте, уважаемые знатоки!
Подскажите мне пожалуйста, что я делаю неправильно (я нуб, не ругайтесь). Вот кусок кода:

public function showAction(Request $request)
    {
        $dataMail = [
            'name' => '',
            'subject' => '',
            'email' => '',
            'message' => '',
        ];

        $form = $this->formFactory->create('send_mail_form', $dataMail);
        
        if ($request->getMethod() == 'POST') {
            $form->bind($request);
            $dataMail = $form->getData();

            $message = \Swift_Message::newInstance()
                ->setSubject($dataMail['subject'])
                ->setFrom($dataMail['email'])
                ->setTo('[e-mail]')
                ->setBody($this->renderView('AllergoMainBundle:Service:body_mail.html.twig', [
                    'name' => $dataMail['name'],
                    'message' => $dataMail['message'],
                ]));
            $this->get('mailer')->send($message);
        }

        return [
            'form' => $form->createView(),
        ];
    }

Вылазит такая ошибка, как я понял из-за $this->get('mailer'):

Error: Call to a member function get() on a non-object

Мне кажется я понял как решить проблему, нужно найти наймспейс мэйлера, я не могу его найти

Answer 1

[Alex]

Всё должно работать, если класс наследуется от симфонийского контроллера

Comments

[1alexandr]

Класс наследуется

[Alex]

Самое интересное, что получается, что $this это не объект, что выглядит очень странно. Поэтому скорее всего, как обычно, ошибка в другом месте

Answer 2

[Сергей Протько]

В дополнение к ответу shoomyst, если вы не хотите наследоваться от симфоневского контроллера, реализуйте интерфейс ContainerAwareInterface для вашего контроллера и забирайте из контейнера самостоятельно. Так же можно подключить трейт.

class MyController implements ContainerAwareInterface
{
    use \Symfony\Component\DependencyInjection\ContainerAwareTrait;

    function myAction(Request $request) {
        // ...
        $this->container->get('mailer')->send($message);
    }
}

#edited
Либо, как советует OnYourLips, если уж делаете контроллеры как сервисы, то делайте их правильно, с указанием всех зависимостей в конструкторе. Правда я считаю такой подход лишним, лучше просто держать тонкие контроллеры которые передают просто данные на обработку в сервис, и наша бизнес логика не зависит от HttpFoundation и минимально привязана к фреймворку.

Comments

[Сергей Протько]

OnYourLips: плохой совет если речь идет о сервисах. В контроллерах норм, ибо не думаю что автор вопроса будет заботиться о таких вещах как ленивая инициализация сервисов и т.д. А контроллеры как сервисы это вообще отдельная история. Не вижу в этом смысла так как переиспользовать контроллер все равно не выйдет, и если делать их тонкими в этом нет особо смысла.

[Сергей Протько]

OnYourLips: Вообще есть две веские причины инджектить весь контейнер - до Symfony 2.3 как раз таки изза отсутствия ленивой инициализации сервисов, а до сих пор как средство разрешения циклических зависимостей. Последнее для контроллеров редкость, да и вообще лучше стараться избегать подобного. Но все же...

Answer 3

[Quber]

Вы забыли в начале файла, где название класса указать наследуемость от симфонического класса extends Controller
Делается так

class YourController extends Controller
{
    public function showAction(Request $request) {
        // your actions
    }
}

Это позволит вызывать Вам mailer. Но как отметили выше, есть еще один подход, связанный с ContainerAwareInterface. Но я считаю это лишнее. Сделайте как у меня и всё.

Comments

[1alexandr]

мой контроллер наследуется от controller, значит ошибка в другом

[Иван]

1alexandr: а родительский класс Controller точно тот, что нужен?)

[1alexandr]

Иван: Да, точно, возможно это все произошло из-за того что я прописал контроллер в сервисе

Answer 4

[Иван]

Не нужен вам неймспейс мейлера, если вызываете его как сервис через get('mailer').
Убедитесь, что наследуете контроллер от Symfony\Bundle\FrameworkBundle\Controller\Controller;

Comments

[1alexandr]

наследуется именно он