Почему не передается Container в конструктор?

Question

[Roman Gor]

Симфа кидает ошибку:

ContextErrorException: Catchable Fatal Error: Argument 1 passed to Uasector\UserBundle\UserProvider\AbstractSocialNetworkProvider::__construct() must be an instance of Symfony\Component\DependencyInjection\Container, none given, called in /Users/roman/Development/Sites/uasector.dev/app/cache/dev/appDevDebugProjectContainer.php on line 5846 and defined in /Users/roman/Development/Sites/uasector.dev/src/Uasector/UserBundle/UserProvider/AbstractSocialNetworkProvider.php line 33

AbstractSocialNetworkProvider.php line 33:

public function __construct(Container $container, $kernelWebDir, $uploadDir, $defaultAvatarPath)

Как я понимаю, в конструктор не передается передается объект Container. Почему такое может быть?

Вот, сервис:

services:
    uasector_user.uasector_abstract_user_data_service:
        class: %uasector.user.abstract_social_network_provider.class%
        arguments: ['@service_container', '%kernel.root_dir%', '%uasector_user.image.upload_directory%', '%uasector_user.image.default_avatar_path%']

https://github.com/RomanGorbatko/sy2 - сам проект.

Answer 1

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

Простите за невежество, но зачем вы весь контейнер в сервис пихаете? Это не хорошо. Единственное оправдание этому - циклические зависимости, что как бы плохо но иногда бывает.

Comments

[Roman Gor]

Я бы мог вам ответить, если бы на 100% был уверен в таком решении. Но увы. Я учусь.

Вы, наверное, имеете ввиду, что я зря передаю service_container агрументом? Достаточно было бы вызвать service_container в конструкторе. Я и так делал. Релузьтат, примерно, тот же.

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

RomanGorbatko: нет, я пытаюсь сказать что вы вообще не должны работать с контейнером в сервисе. Все нужные сервисы должны быть переданы в ваш юзер провайдер и все.

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

RomanGorbatko: и да, вы кэш чистили? Пробовали ContainerAwareInterface?

[Roman Gor]

Сергей Протько: Кеш чищу регулярно. Вы имеете ввиду заменить в конструкторе Container на ContainerAwareInterface $container? Пробовал и так. Тоже самое.

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

RomanGorbatko: нет, я рекомедую вам вообще забыть про контейнер. А Интерфейс ContainerAware должен имплементить ваш сервис если что.

[Roman Gor]

Сергей Протько: Сергей, если есть возможность - зайдите в скайп. Я Вас добавил.

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

RomanGorbatko: нет желания, читайте документацию.

Answer 2

[Alex]

В arguments не нужны кавычки, как для сервисов, так и для параметров

Comments

[Roman Gor]

Я был бы рад если бы это решило проблему, но увы.

[Alex]

Ну тогда надо смотреть создание наследников, как они вызывают родительский конструктор

[Alex]

PS. для абстрактного класса (сервиса) вообще не надо указывать аргументы. Его ж нельзя создать

[Roman Gor]

shoomyst:

uasector_user.uasector_abstract_user_data_service:
        class: %uasector.user.abstract_social_network_provider.class%
#        arguments: [@service_container, '%kernel.root_dir%', '%uasector_user.image.upload_directory%', '%uasector_user.image.default_avatar_path%']

И грохнуть конструктор?

[Alex]

RomanGorbatko: вообще убрать этот сервис из конфига, а аргументы перенести в конкретный provider, fb например

[Roman Gor]

shoomyst: Что-то начинает складываться... Я могу с вами связаться в каком-то более быстром чате? Скайп?

[Alex]

Не, у меня сейчас не так много свободного времени, чтобы обучать кого-то. Иногда тут на пару вопросов отвечаю

[Roman Gor]

shoomyst: Как в провайдере получить параметры типа kernel.root_dir, uasector_user.image.upload_directory ?

[Alex]

RomanGorbatko: вы же их передайте в конструктор

[Roman Gor]

shoomyst: По вашему совету грохнул сервис. "а аргументы перенести в конкретный provider, fb например" - теперь вопрос как в том же fb получить эти параметры.

[Alex]

services:
uasector.user.facebook_provider:
class: %uasector.user.facebook_provider.class%
arguments: ['@service_container', '%kernel.root_dir%', '%uasector_user.image.upload_directory%', '%uasector_user.image.default_avatar_path%']

[Roman Gor]

shoomyst: Точно та же ошибка, только уже другой провайдер :)

ContextErrorException: Catchable Fatal Error: Argument 1 passed to Uasector\UserBundle\UserProvider\FacebookProvider::__construct() must be an instance of Uasector\UserBundle\UserProvider\Container, none given, called in /Users/roman/Development/Sites/uasector.dev/app/cache/dev/appDevDebugProjectContainer.php on line 6116 and defined in /Users/roman/Development/Sites/uasector.dev/src/Uasector/UserBundle/UserProvider/FacebookProvider.php line 17

Answer 3

[Борис Беньковский]

У вас как-бы "abstract class AbstractSocialNetworkProvider". Дело в том, что нельзя пытаться вызвать конструктор абстрактного класса. И странно, что симфони ругается на типы... покажите

Users/roman/Development/Sites/uasector.dev/app/cache/dev/appDevDebugProjectContainer.php on line 5846

мне интересно, что там.

Comments

[Roman Gor]

return $this->services['uasector_user.uasector_abstract_user_data_service'] = new \Uasector\UserBundle\UserProvider\AbstractSocialNetworkProvider($this, '/Users/roman/Development/Sites/uasector.dev/app', '/upload/users/avatars/', '/../src/Uasector/UserBundle/Resources/public/images/default_avatar.png');