utiliser un bundle dans un service

**merou19** · 10/02/2019, 17h50

Bonjour à tous,

J'ai besoin de votre aide parce que j'ai des messages d'erreurs que je n'arrive pas à résoudre.

Dans mon projet, j'ai un service qui s'appele 'cptservice' et qui marche bien.
Mais dedans, j'ai des valeurs codées en dur, et je souhaite les externaliser dans un fichier INI.

Pour manipuler les fichier INI, j'ai trouvé sur packagist le bundle 'hassankhan/config', que j'ai installé sans problème.

Donc je veux simplement utilser ce bundle dans mon service :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
use Noodlehaus\Config;
 
class projetCompteur
{
    private $entitymanager;
    protected $request_stack;
 
    public function __construct(EntityManager $entitymanager, RequestStack $request_stack)
    {
        $this->entitymanager = $entitymanager;
        $this->request = $request_stack->getCurrentRequest();
    }
 
    /**
     * @param string $nom
     * @param string $sens
     * @param smallint $valeur
     */
 
    public function modifierCompteur($nom, $sens, $valeur)
    {
        $conf = new Config('list.ini');
        $data = $conf->all();
        var_dump($data);

Je ne mets pas tout le code pour ne pas faire de gros pavés, mais dans la suite de mon code, mon entitymanager et mon requeststack marchent bien.

J'ai juste ajouté au début de ma fonction la lecture de mon fichier INI, et un var_dump pour vérifier si ça marche.

Et en lançant app_dev.php, j'ai le message d'erreur suivant à la ligne '$conf = new Config('list.ini')' :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

The "cptservice" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.

En rajoutant 'public: true' dans la définition de mon service, j'ai le message :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Bundle inheritance is deprecated as of 3.4 and will be removed in 4.0

Avec Symfony 3.4, comment utiliser ce bundle dans un service ?
Est ce que je suis obligé de passer par l'injection de dépendances ?

J'ai lu plusieurs tutos la dessus, voilà ce que j'ai essayé :

Fichier service.yml :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
services:
    cptservice:
        class: projetBundle\Service\projetCompteur
        arguments:
            - "@doctrine.orm.entity_manager"
            - "@request_stack"
            - "@hassankhan"

Et j'ai le message d'erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

The service "cptservice" has a dependency on a non-existent service "hassankhan".

Evidemment que ça ne marche pas, car hassankhan est un bundle et pas un service.
Sauf que maintenant je ne vois pas comment continuer.

Merci d'avance pour votre aide !

**Tsilefy** · 11/02/2019, 20h22

'hassankhan/config est une bibliothèque, pas un bundle symfony à moins que je n'aie mal cherché.

Le plus simple et la manière recommandée est d'injecter.

Sur SF 4, mais ça doit être pareil sur sf 3.4, il faut d'abord que tu déclares l'interface dans le conteneur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
Noodlehaus\ConfigInterface:
      class: 'Noodlehaus\Config'
      arguments:
        $values: '%config_file%'

%config_file% se réfère au paramètre indiquant l'emplacement du fichier de configuration. Il peut être indiqué par exemple dans l'environnement ou un fichier .env, mais supposons qu'il est écrit directement dans les paramètres et qu'il est dans le répertoire config (à adapter en fonction des répertoires de ton projet s'il est différent)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
parameters:    
    config_file: '../config/list.ini'

Tu injectes ensuite l'interface dans le constructeur de ton service

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public function __construct(EntityManager $entitymanager, RequestStack $request_stack, ConfigInterface $config)
    {
        $this->entitymanager = $entitymanager;
        $this->request = $request_stack->getCurrentRequest();
        $this->config = $config;
    }
 
public function modifierCompteur($nom, $sens, $valeur)
    {
        $data = $this->conf->all();
        var_dump($data); //TA-DAA

**merou19** · 11/02/2019, 21h51

Merci beaucoup Tsilefy pour tes explications et codes, j'avance grâce à toi, mais je n'arrive pas encore au TA-DAA final !!!

Voilà ce que j'ai maintenant :

fichier service.yml :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
services:
    cptservice:
        class: projetBundle\Service\projetCompteur
        arguments:
            - "@doctrine.orm.entity_manager"
            - "@request_stack"
            - "@noodlehaus"
    noodlehaus:
        class: 'Noodlehaus\Config'
        arguments:
            $values: '%config_file%'

Fichier config.yml :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
parameters:
    config_file: ip_list.ini

Mon service :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
use Noodlehaus\Config;
use Noodlehaus\Parser\Ini;
 
class projetCompteur
{
    private $entitymanager;
    protected $request_stack;
    private $config;
 
    public function __construct(EntityManager $entitymanager, RequestStack $request_stack, ConfigInterface $config)
    {
        $this->entitymanager = $entitymanager;
        $this->request = $request_stack->getCurrentRequest();
        $this->config = $config;
    }
 
    /**
     * @param string $nom
     * @param string $sens
     * @param smallint $valeur
     */
 
    public function modifierCompteur($nom, $sens, $valeur)
    {
        $data = $this->config->all();
        var_dump($data);

Et je reviens toujours à mon premier message d'erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

The "cptservice" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.

Une idée de ce qui manque ???

Merci d'avance !

**Tsilefy** · 12/02/2019, 00h01

- Si tu es en 3.4, supprime tout ça, c'est inutile avec l'autowiring:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
cptservice:
        class: projetBundle\Service\projetCompteur
        arguments:
            - "@doctrine.orm.entity_manager"
            - "@request_stack"
            - "@noodlehaus"

- mets le nom de la classe (Noodlehaus\ConfigInterface) comme clé et non noodlehaus

- Dans le contrôleur (ou l'autre service) où tu utilises ton service, injecte-le dans le constructeur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

public function __construct(projetCompteur $compteur)

plutôt que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$compteur = $this->container->get('cptservice')

**merou19** · 13/02/2019, 19h45

Merci beaucoup Tsilefy pour ton aide !

J'ai essayé de suivre au mieux tes indications, j'ai essayé plein plein plein de trucs dans tous les sens, mais je n'arrive toujours pas au résultat voulu.

Voilà où j'en suis :

Fichier service.yml

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
services:
#    cptservice:
#        class: projetBundle\Service\projetCompteur
#        arguments:
#            - "@doctrine.orm.entity_manager"
#            - "@request_stack"
#            - "@Noodlehaus"
    Noodlehaus\ConfigInterface:
        class: 'Noodlehaus\Config'
        arguments:
            $values: '%config_file%'

Service projetCompteur.php

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
use Noodlehaus\Config;
use Noodlehaus\Parser\Ini;
 
class projetCompteur
{
    private $entitymanager;
    protected $request_stack;
    private $config;
 
    public function __construct(EntityManager $entitymanager, RequestStack $request_stack, ConfigInterface $config)
    {
        $this->entitymanager = $entitymanager;
        $this->request = $request_stack->getCurrentRequest();
        $this->config = $config;
    }
 
    /**
     * @param string $nom
     * @param string $sens
     * @param smallint $valeur
     */
 
    public function modifierCompteur($nom, $sens, $valeur)
    {
        $data = $this->config->all();
        var_dump($data);

Avant cette évolution, j’appelai mon service par : $compteur = $this->get('cptservice')
Et ça marchait très bien.

Maintenant, j'essaie d'injecter mon service dans le constructeur de mon contrôleur.

Mon contrôleur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class VisitController extends Controller
{
    public $compteur;
 
    public function __construct(projetCompteur $compteur)
    {
        $this->compteur = $compteur;
    }
 
    public function visit_accueilAction(Request $request)
    {
        $entitymanager = $this->getDoctrine()->getManager();
        //$compteur = $this->get('cptservice');
        $compteur->modifierCompteur('cpt-visit-accueil', 'plus', 1);

Et j'ai le message :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Type error: Too few arguments to function projetBundle\Controller\VisitController::__construct(), 0 passed in /home/moi/wwwroot/projet/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Controller/ControllerResolver.php on line 195 and exactly 1 expected

Mon contrôleur attend un argument, que je ne lui donne pas.

Est ce que j'ai bien compris tes indications ?
Mettre un constructeur pour un contrôleur, ça se fait ?
Est ce que, au moins, ce que j'ai fait est juste, et il manque quelque chose ?

Merci encore pour ton aide !

**Tsilefy** · 14/02/2019, 00h11

Un contrôleur est une classe comme une autre, tu peux donc bien sûr utiliser un constructeur. Tu le fais par exemple quand tu as un service qui est utilisé par 2 méthodes, afin d'alléger le nombre de paramètres des méthodes.

Tu n'aurais pas une configuration pour ce contrôleur dans services.yaml? Si c'est le cas, tu dois soit ajouter une entrée "arguments" avec la classe projetBundle\Service\projetCompteur en valeur, ou, le plus simple, supprime carrément la configuration du contrôleur puisque tous les contrôleurs sont désormais des services par défaut en Symfony 3.4, sans besoin de configuration spécifique.

**merou19** · 17/02/2019, 19h22

J'ai encore essayé plein de trucs, sans succès

Je retombe sur la première réponse à mon sujet :

Envoyé par Tsilefy

Sur SF 4, mais ça doit être pareil sur sf 3.4, il faut d'abord que tu déclares l'interface dans le conteneur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
Noodlehaus\ConfigInterface:
      class: 'Noodlehaus\Config'
      arguments:
        $values: '%config_file%'

Et je suis pris d'un gros doute : "déclarer l'interface dans le conteneur", je dois bien mettre ce code dans mon fichier services.yml ?
Ou il y a d'autres actions à faire ?

Merci d'avance !

**Tsilefy** · 19/02/2019, 17h36

Ton problème vient du fait que le conteneur devrait injecter le bon argument, mais il ne le fait pas. Est-ce que tu as upgradé ce projet de Symfony 3.x (avec x < 4) à 3.4?

Parce que en relisant ton code, je vois que tu utilises un ProjectBundle, alors que la manière recommandée depuis 3.4 est de tout mettre dans AppBundle et de ne plus avoir d'autres bundles. Donc c'est normal que mes solutions n'aient pas marché.

On va commencer par tenter un truc, on verra ce que ça donne. Dans ton services.yml, ajoute ceci juste après l'entrée AppBundle.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
ProjectBundle\:
        resource: '../../src/ProjectBundle/*'
        exclude: '../../src/ProjectBundle/{Entity,Repository,Tests}'

Si ça ne marche pas, peux-tu poster une capture d'écran de la structure des répertoires de ton projet, et coller le contenu de services.yml?

**merou19** · 20/02/2019, 19h42

Merci beaucoup pour toute l'aide que tu m'apportes !

Alors, j'avais commencé mon projet à l'époque de symfony 3.0 ou 3.1, je ne me souviens plus !
Et comme c'était mon premier projet symfony, ça fonctionne, mais il y a surement beaucoup de points qui ne sont pas dans les règles de l'art. Je découvre les changements et les bonnes pratiques au fur et à mesure...

J'ai ajouté le code que tu indiques dans app/config/services.yml

Et voilà le message que j'obtiens :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Cannot autowire service "projetBundle\Controller\VisitController": argument "$compteur" of method "__construct()" has type "projetBundle\Controller\projetCompteur" but this class was not found.

D'après ce que je comprends, il cherche ma classe dans le dossier 'Controller', alors qu'elle est dans le dossier 'Service'.

J'ai bidouillé plein de trucs pour essayer de faire pointer cette recherche dans 'Service', sans succès.

Donc je suis revenu à ce message initial, en espérant que tu aies des pistes !

Mon app/config/services.yml actuel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
# Learn more about services, parameters and containers at
# http://symfony.com/doc/current/service_container.html
parameters:
    #parameter_name: value
 
services:
    # default configuration for services in *this* file
    _defaults:
        # automatically injects dependencies in your services
        autowire: true
        # automatically registers your services as commands, event subscribers, etc.
        autoconfigure: true
        # this means you cannot fetch services directly from the container via $container->get()
        # if you need to do this, you can override this setting on individual services
        public: false
 
    # makes classes in src/AppBundle available to be used as services
    # this creates a service per class whose id is the fully-qualified class name
    AppBundle\:
        resource: '../../src/AppBundle/*'
        # you can exclude directories or files
        # but if a service is unused, it's removed anyway
        exclude: '../../src/AppBundle/{Entity,Repository,Tests}'
 
    projetBundle\:
        resource: '../../src/projetBundle/*'
        exclude: '../../src/projetBundle/{Entity,Repository,Tests}'
 
    # controllers are imported separately to make sure they're public
    # and have a tag that allows actions to type-hint services
    AppBundle\Controller\:
        resource: '../../src/AppBundle/Controller'
        public: true
        tags: ['controller.service_arguments']
 
    #service_name:
    #    class: AppBundle\Directory\ClassName
    #    arguments: ['@another_service_name', 'plain_value', '%parameter_name%']
    #twig.extension.intl:
    #    class: Twig_Extensions_Extension_Intl
    #    tags:
    #        - { name: twig.extension }
    AppBundle\EventSubscriber\LocaleSubscriber:
        arguments: ['%kernel.default_locale%']
        # redundant if you're using autoconfigure
        tags: [kernel.event_subscriber]
    # default configuration for services in *this* file
    # default configuration for services in *this* file

Et la structure de mes dossiers :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
app # contient AppKernel.php et Appcache.php et autoload.php
app/Resources # contient des dossiers avec des vues twig
app/config # contient mes fichier yml (dont config.yml et services.yml)
src/AppBundle # contient AppBundle.php
src/AppBundle/Controller # contient DefautController.php
src/AppBundle/Entity # contient user.php
src/AppBundle/EventSubscriber # contient LocalSubscriber.php
src/projetBundle/Controller # contient mes contrôleurs
src/projetBundle/Entity # contient mes entitées
src/projetBundle/Form # contient mes forms
src/projetBundle/Repository # contient mes repositorys
src/projetBundle/Resources # contient projetBundle.php (pour utiliser FOSUserBundle)
src/projetBundle/Resources/config # contient routingg.yml et services.yml
src/projetBundle/Resources/translation # contient mes fichiers de langues
src/projetBundle/Resources/views # contient toutes mes vues twig
src/projetBundle/Service # contient tous mes services, dont projetCompteur.php

Une question qui me vient tant que tu es là stp : tu disais dans ta première réponse à mon sujet que "'hassankhan/config est une bibliothèque, pas un bundle symfony".
Donc je n'ai rien à ajouter dans AppKernel.php pour autoriser le bundle. Tu confirmes ?

Merci encore pour ton aide, c'est super-sympa !

**Tsilefy** · 20/02/2019, 19h49

Cannot autowire service "projetBundle\Controller\VisitController": argument "$compteur" of method "__construct()" has type "projetBundle\Controller\projetCompteur" but this class was not found.

C'est un problème de namespace. Dans VisitController, ajoute en haut

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

use projetBundle\Service\ProjetCompteur;

et ça va régler ce problème particulier (mais pas forcément les autres!).

utiliser un bundle dans un service

Symfony PHP

Vue hybride

Discussions similaires

Partager

Partager