Discussion :

[mansaychai]

Bonjour à tous,

J'ai un petit problème concernant l'update de données se trouvant dans une table d'association, sur cakephp 3.

Je vous fais un petit topo du contexte pour que tout le monde pige bien :

Au niveau de mes tables, j'ai une table users, une table styles et une table users_styles.

Je suis dans la fonction edit de mon UsersController. Cette fonction renvoie une vue, un formulaire où mon user peut modifier ses informations.

Le problème est le suivant :
A chaque validation du formulaire, les données qui devraient être enregistrées dans la table d'association sont recréées au lieu d'être modifiée. En revanche, sur les données de ma table principale l'update fonctionne sans soucis, sans recréer de nouveau enregistrement.

mon formulaire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
      <?= $this->Form->create($user, ['type' => 'file']); ?>
            <?php
                echo $this->Form->input('email');
 
                echo $this->Form->input('styles._ids', [
                    'label'     => 'Vos styles musicaux favoris',
                    'multiple'  => true
                ]);
            ?>
        <?= $this->Form->button(__("Mettre à jour")) ?>
        <?= $this->Form->end() ?>

mon controller :

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
 
 
    public function edit()
        {
 
            $user = $this->Users->get($this->Auth->user('id'), [
                'contain' => ['Styles']
            ]);
 
            if ($this->request->is(['post', 'put', 'patch'])) {
 
                // c'est ici que ça se joue !
                $user = $this->Users->patchEntity($user,        $this->request->data);
 
                if ($this->Users->save($user)) {
                    $this->Flash->success('Fécilitation, votre profil a été édité avec succès !');
                } else {
                    $this->Flash->error("Oups! L'utilisateur n'a pas pu être enregistré. Veuillez réessayer.");
                }
            }
 
            $styles = $this->Users->Styles->find('list');
            $this->set(compact('user', 'styles'));
        }

Concernant mes Tables :
StylesTables belongsToMany Users et hasMany StylesUsers
UsersTables belongsToMany Styles et hasMany StylesUsers
StylesUsersTables belongsTo Users et Styles

J'ai essayé plusieurs choses et parcouru la doc' mais rien n'y fait, j'ai toujours ce problème. Si quelqu'un a une idée, je ne suis pas contre un petit coup de patte !

[goomazio]

Bonjour,

Que contient $user et $this->request->data avant et après la soumission du formulaire et le passage dans patchEntity() ?

Pour modifier les données, la structure passée à Objet->save() doit contenir l'ID de l'objet, sinon ça en crée un nouveau. Et c'est pareil pour les relations. CakePHP ajoute un champ caché au formulaire pour stocker l'identifiant et le transmettre au contrôleur, mais pas pour les modèles liés. (ou un truc du genre ...)

Voir aussi les options associated pour dire ce qu'on veut sauver comme modèles (on dirait que ça remplace le saveAll) :

• lors de la sauvegarde : $articles->save($entity, ['associated' => ['Comments']]);
• et lors du merged : $articles->patchEntity($article, $this->request->data(), [ 'associated' => ['Tags', 'Comments.Users'] ]);

[mansaychai]

Pour $this->request->data je récupère quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
 
       'email' => 'toto@email.com',
	'styles' => [
		'_ids' => [
			(int) 0 => '3',
			(int) 1 => '4'
		]

pour mon entitée $user je récupère, dans un objet (App\Model\Entity\User), les info relative à mon utilisateur. Puis le champ 'styles', un tableau contenant les différentes entrées de la base + celles envoyées en post via mon formulaire.

Dans mon debug ça ressemble à ça (exemple avec un seul 'style') :

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
 
(int) 0 => object(App\Model\Entity\Style) {
 
			'id' => (int) 2,
			'name' => 'Funk',
			'_joinData' => object(App\Model\Entity\StylesUser) {
 
				'style_id' => '2',
				'id' => (int) 116,
				'user_id' => '2',
				'[new]' => false,
				'[accessible]' => [
					'style_id' => true,
					'user_id' => true,
					'style' => true,
					'user' => true
				],
				'[dirty]' => [],
				'[original]' => [],
				'[virtual]' => [],
				'[errors]' => [],
				'[repository]' => 'StylesUsers'
 
			},
			'[new]' => false,
			'[accessible]' => [
				'name' => true,
				'songs' => true,
				'users' => true
			],
			'[dirty]' => [],
			'[original]' => [],
			'[virtual]' => [],
			'[errors]' => [],
			'[repository]' => 'Styles'

J'ai rajouté les 'associated' comme tu me l'a conseillé, mais malheureusement ça ne fonctionne pas.

Code :

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

1
2
3
4
 
$user = $this->Users->patchEntity($user, $this->request->data(), ['associated' => ['Styles', 'Skills']]);
 
$this->Users->save($user, ['associated' => ['Styles', 'Skills']])

Si je valide deux fois à la suite mon formulaire, j'ai deux champs avec les même clées étrangères dans ma table StylesUsers.

[goomazio]

Est-ce que ce ne serait pas la structure de la base de données le problème ?



J'ai essayé avec ces tables et le code généré avec la console, qui donne quasiment le même code que celui que vous avez posté, et ça fonctionne.

Code :

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

1
2
3
"bin/cake" bake all ingredients
"bin/cake" bake all recettes
"bin/cake" bake all ingredients_recettes

Voici mon debug pour info, avec un seul ingrédient pour le contenu de $recette :

Code :

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

1
2
3
4
5
if ($this->request->is(['patch', 'post', 'put'])) {
    $recette = $this->Recettes->patchEntity($recette, $this->request->data);
    debug($this->request->data);
    debug($recette);
    if ($this->Recettes->save($recette)) {...

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
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
 \src\Controller\RecettesController.php (line 3) [
    'nom' => 'Tarte à la courgette et au fromage de chèvre v5',
    'description' => 'Voir le titre : Tarte à la courgette et au fromage de chèvre. V3 : Ajout des ingrédients 3 et 5. V4 ajout ingrédient 6. V5 : on enlèev le 6 et le 1.',
    'ingredients' => [
        '_ids' => [
            (int) 0 => '3',
            (int) 1 => '5'
        ]
    ]
]

 \src\Controller\RecettesController.php (line 4) object(App\Model\Entity\Recette) {

    'id' => (int) 1,
    'created' => object(Cake\I18n\Time) {

        'time' => '2015-06-13T16:11:31+0000',
        'timezone' => 'UTC',
        'fixedNowTime' => false
    
    },
    'modified' => object(Cake\I18n\Time) {

        'time' => '2015-06-13T16:23:26+0000',
        'timezone' => 'UTC',
        'fixedNowTime' => false
    
    },
    'nom' => 'Tarte à la courgette et au fromage de chèvre v5',
    'description' => 'Voir le titre : Tarte à la courgette et au fromage de chèvre. V3 : Ajout des ingrédients 3 et 5. V4 ajout ingrédient 6. V5 : on enlèev le 6 et le 1.',
    'ingredients' => [
        (int) 0 => object(App\Model\Entity\Ingredient) {

            'id' => (int) 3,
            'created' => object(Cake\I18n\Time) {

                'time' => '2015-06-13T16:09:09+0000',
                'timezone' => 'UTC',
                'fixedNowTime' => false
            
            },
            'modified' => object(Cake\I18n\Time) {

                'time' => '2015-06-13T16:09:09+0000',
                'timezone' => 'UTC',
                'fixedNowTime' => false
            
            },
            'nom' => 'Crème fraiche',
            'description' => 'produits laitier',
            '[new]' => false,
            '[accessible]' => [
                'nom' => true,
                'description' => true,
                'recettes' => true
            ],
            '[dirty]' => [],
            '[original]' => [],
            '[virtual]' => [],
            '[errors]' => [],
            '[repository]' => 'Ingredients'
        
        }
    ],
    '[new]' => false,
    '[accessible]' => [
        'nom' => true,
        'description' => true,
        'ingredients' => true
    ],
    '[dirty]' => [
        'nom' => true,
        'description' => true,
        'ingredients' => true
    ],
    '[original]' => [
        'nom' => 'Tarte à la courgette et au fromage de chèvre v4',
        'description' => 'Voir le titre : Tarte à la courgette et au fromage de chèvre. V3 : Ajout des ingrédients 3 et 5. V4 ajout ingrédient 6.',
        'ingredients' => [
            (int) 0 => object(App\Model\Entity\Ingredient) {

                'id' => (int) 1,
                'created' => object(Cake\I18n\Time) {

                    'time' => '2015-06-13T16:08:31+0000',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false
                
                },
                'modified' => object(Cake\I18n\Time) {

                    'time' => '2015-06-13T16:08:31+0000',
                    'timezone' => 'UTC',
                    'fixedNowTime' => false
                
                },
                'nom' => 'Courgette',
                'description' => 'légume',
                '_joinData' => object(App\Model\Entity\IngredientsRecette) {

                    'ingredient_id' => (int) 1,
                    'id' => (int) 1,
                    'created' => object(Cake\I18n\Time) {

                        'time' => '2015-06-13T16:15:15+0000',
                        'timezone' => 'UTC',
                        'fixedNowTime' => false
                    
                    },
                    'modified' => object(Cake\I18n\Time) {

                        'time' => '2015-06-13T16:15:15+0000',
                        'timezone' => 'UTC',
                        'fixedNowTime' => false
                    
                    },
                    'recette_id' => (int) 1,
                    'quantite' => (float) 0,
                    'unites' => '',
                    '[new]' => false,
                    '[accessible]' => [
                        'ingredient_id' => true,
                        'recette_id' => true,
                        'quantite' => true,
                        'unites' => true,
                        'ingredient' => true,
                        'recette' => true
                    ],
                    '[dirty]' => [],
                    '[original]' => [],
                    '[virtual]' => [],
                    '[errors]' => [],
                    '[repository]' => 'IngredientsRecettes'
                
                },
                '[new]' => false,
                '[accessible]' => [
                    'nom' => true,
                    'description' => true,
                    'recettes' => true
                ],
                '[dirty]' => [],
                '[original]' => [],
                '[virtual]' => [],
                '[errors]' => [],
                '[repository]' => 'Ingredients'
            
            
            }
        ]
    ],
    '[virtual]' => [],
    '[errors]' => [],
    '[repository]' => 'Recettes'}

Note : dans l'objet passé à save, il y a un élément ['original'], qui est surement dû, au pif, à cette ligne dans mon controlleur (que tu n'as pas dans le tiens) : $this->set('_serialize', ['recette']);


Pour mes relations :

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
IngredientsRecettesTable.php
        $this->belongsTo('Ingredients', [
            'foreignKey' => 'ingredient_id',
            'joinType' => 'INNER'
        ]);
        $this->belongsTo('Recettes', [
            'foreignKey' => 'recette_id',
            'joinType' => 'INNER'
        ]);

IngredientsTable.php
        $this->belongsToMany('Recettes', [
            'foreignKey' => 'ingredient_id',
            'targetForeignKey' => 'recette_id',
            'joinTable' => 'ingredients_recettes'
        ]);

RecettesTable.php
        $this->belongsToMany('Ingredients', [
            'foreignKey' => 'recette_id',
            'targetForeignKey' => 'ingredient_id',
            'joinTable' => 'ingredients_recettes'
        ]);

[mansaychai]

C'est bon le problème est résolu ... J'avais effectivement un soucis dans ma BDD, quelle honte je ne l'avais pas remarqué avant !

En fait, j'ai généré mes tables avec l'outil de migration et, pour les clée étrangères, je n'ai pas indiqué de type (j'ai du faire un truc comme style_id:index) ... Cela fait que toutes mes clées, dans les tables d'asso' étaient typée en VARCHAR.
Mais quel naze ! On ne m'y reprendra plus !

Sinon, pour info, le serialize permet de 'serializer' ta variable, pour pouvoir la traiter en xml ou en json.

Merci de m'avoir donné un coup de main, sans ça je serais toujours à chercher le problème ou il n'est pas !