Discussion :

[acesyde]

Bonjour à tous,

J'ai un petit soucis avec Symfony les forms

Contexte :
Deux entités :

• BlogPost
• BlogComment

Le BlogPost contient des BlogComments et vice et versa.

Quand j'affiche mon BlogPost j'ai inclue un formulaire pour rajouter un BlogComment à la suite (comme sur tout blog en faite).

Mais je n'arrive pas à référencer directement le BlogPost au BlogComment pour que ça soit invisible aux yeux de l'utilisateur.

CommentController

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
 
    public function newAction(BlogPost $post=null) {
 
        $comment = new BlogComment();
 
        if ($post) {
            $comment->setPost($post);
        }
 
        $form = $this->createForm(new CommentType(), $comment);
 
        return $this->render('SymbioseBlogBundle:Comment:new.html.twig', array('form' => $form->createView()));
    }
 
    public function createAction() {
        $comment = new BlogComment();
        $request = $this->getRequest();
        $form    = $this->createForm(new CommentType(), $comment);
        $form->bindRequest($request);
 
        if ($form->isValid()) {
            $em = $this->getDoctrine()->getEntityManager();
            $em->persist($comment);
            $em->flush();
 
            return $this->redirect($this->generateUrl('SymbioseBlogBundle_view_post', array('id' => $comment->getPost()->getId())));
 
        }
 
        return $this->render('SymbioseBlogBundle:Comment:new.html.twig', array('form' => $form->createView()));
    }

Ma FormType

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
<?php
 
namespace Symbiose\BlogBundle\Form\Type;
 
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\FormBuilder;
 
class CommentType extends AbstractType
{
    public function buildForm(FormBuilder $builder, array $options)
    {
        $builder->add('text','textarea')
                ->add('post','hidden');
    }
 
    public function getName()
    {
        return 'comment';
    }
 
    public function getDefaultOptions(array $options)
    {
        return array(
            'data_class' => 'Symbiose\BlogBundle\Entity\BlogComment',
        );
    }
}

Ma FormTwig

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<form action="{{ path('SymbioseBlogBundle_create_comment') }}" method="post" {{ form_enctype(form) }} class="new_comment">
    {{ form_errors(form) }}
 
    {{ form_widget(form.text,{ 'attr': {'class': 'comment_field'}}) }}
 
    {{ form_rest(form) }}
 
        <p class="submit">
            <input type="submit" class="button-submit" value="Ajouter" />
        </p>
    </form>

Si quelqu'un à une idée je suis preneur.

Cdt.

[winzou]

Salut,

Pourquoi tu as séparé tes action new et create ? Rassemble les et ca marchera (le $comment->setPost($post) tu dois le faire dans ton create la).

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
 
public function newAction(BlogPost $post=null) {
 
        $comment = new BlogComment();
        $request = $this->getRequest();
 
        if ($post) {
            $comment->setPost($post);
        }
 
        $form = $this->createForm(new CommentType(), $comment);
 
        if($request->getMethod() == 'POST' ) {
             $form->bindRequest($request);
             if ($form->isValid()) {
                $em = $this->getDoctrine()->getEntityManager();
                $em->persist($comment);
                $em->flush();
 
                return $this->redirect($this->generateUrl('SymbioseBlogBundle_view_post', array('id' => $comment->getPost()->getId())));
            }
        }
 
        return $this->render('SymbioseBlogBundle:Comment:new.html.twig', array('form' => $form->createView()));
    }

[acesyde]

Je regarde dans la journée et je te dis si ça fonctionne

[acesyde]

Alors après modification de ce que tu m'as montré j'ai le droit à ceci :

An exception has been thrown during the rendering of a template ("The "SymbioseBlogBundle_new_comment" route has some missing mandatory parameters ("post").") in "SymbioseBlogBundle:Comment:new.html.twig" at line 1.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<form action="{{ path('SymbioseBlogBundle_new_comment') }}" method="post" {{ form_enctype(form) }} class="new_comment">
    {{ form_errors(form) }}
 
    {{ form_widget(form.text,{ 'attr': {'class': 'comment_field'}}) }}
 
    {{ form_rest(form) }}
 
        <p class="submit">
            <input type="submit" class="button-submit" value="Ajouter" />
        </p>
    </form>

Routing

Code :

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

1
2
3
4
 
SymbioseBlogBundle_new_comment:
    pattern:  /new/{post}
    defaults: { _controller: SymbioseBlogBundle:Comment:new, post: null }

[Bleys]

Pourquoi tu as séparé tes action new et create

Si je ne m'abuse ... c'est comme ça que symfony 1.X fonctionne quand tu utilises les generators. C'est un moyen dans ton routing de bien préciser en fonction du type de requête (GET, PUT, POST, ...) quelle action appelée. Je ne suis pas trop friand de cette façon de faire non plus ... mais bon ça se défend.

acesyde > si je peux me permettre, pense qu'il y a là une faille de sécurité plus ou moins importante dans ton système (plus ou moins dépend de comment tu utilises ensuite tes formulaires, la logique de ton site etc). En effet, que se passe-t-il si le user modifie la valeur de ton champ "post" dans ton formulaire (ce n'est pas parce qu'il est hidden qu'il n'est pas modifiable) ? Et bien ça créera le commentaire sur un autre post. Le problème se pose surtout à l'édition de commentaire : méfie-toi des données envoyées par l'utilisateur.
Je te conseille, dans ton formulaire (et en fonction de ton application bien évidemment) d'ajouter :
- soit un token de vérification (qui est un hash de ton id avec un salt soit en session, soit propre à ton action/controller)
- soit conserver l'id original en session ... auquel cas ton champ post dans ton formulaire est inutile.

Je sais on est un peu loin du sujet de base ... mais je pense que de temps en temps il est important de rappeler que "un champ hidden n'est pas inviolable"


Bonne chance pour la suite,

[acesyde]

Salut Bleys,

Je suis preneur pour les bonnes manières, mais à ce niveau là il n'y a pas de documentation, donc forcément (je fais à l'arrache), j'ai pensé aussi au problème de la sécurité, mais c'est assez bizarroide leur système (trop habitué à l'ASPNET surement).

Pour toi le mieux serais de mettre le post courant en session utilisateur ?

Il est vrai que cette méthode est assez séduisante.

Edit:
Je viens de faire cela par le biais d'une session et ça fonctionne au poil

[acesyde]

Bleys,

En faite ça résout plus ou moins mon soucis, avec la session si j'ouvre 2 fenêtres sur le 2 post différent et que je commente le premier ça le met dans le 2éme

Si jamais tu as une idée, voir m'expliquer le token un petit peu plus en profondeur, je suis preneur.

Merci par avance