Discussion :

[davidbouv]

Bonjour,

un autre petit soucis sur les sessions flashs (pas une mince à faire cette migration 2.0>2.3 ) alors au vus de la doc le changement ça à pas l'air compliqué.
Auparavant j'avais

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
// Controller
$session->setFlash('foo', 'bar');
$session->getFlash('foo');
// Vue
{% if session.flashBag.get('foo') is defined %}
    {% if session.flashBag.get('foo') %}
        <div class="message">              
                {{ session.getFlash('foo')|trans }}
                {{ session.setFlash('foo', null) }}
        </div>
    {% endif %}
{% endif %}

Selon la doc ça se transforme ainsi :

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
 
// Controller
$session->getFlashBag()->set('foo', 'bar');
$session->getFlashBag()->get('foo');
// Vue
{% if session.flashBag.get('foo') is defined %}
    {% if session.flashBag.get('foo') %}
        {% for message in session.flashBag.get('foo') %}
            <div class="message">              
                {{ message|trans }}
            </div>
        {% endfor %}
        session.flashBag.set('foo', null)
    {% endif %}
{% endif %}

mais alors là au niveau de mon contrôleur pas de soucis ma session flash est bien setter j'ai bien un array avec pour première valeur mon message mais au niveau de ma vue il ne m'affiche rien.
Si ne met pas le foreach dans la vu ma div apparait mais avec marqué "Array".
Je ne comprend pas pourquoi mon foreach n'affiche pas le message.
Si quelqu'un voit quelque chose qui cloche...

[gototog]

ca ne marche pas comme ca? je vois pas l'interet de test pour chaque message flash si on doit l'afficher ou non ^^'

Code :

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

1
2
3
4
5
6
 
{% for message in session.flashBag %}
    <div class="message">              
        {{ message|trans }}
    </div>
{% endfor %}

[davidbouv]

Merci gototog a priori c'est le faite de vérifier que ma session était défini et avait une valeur. Je trouve ça tout de même un peu bizarre mais bon en tous cas ça fonctionne !

[gototog]

Enfait l'affichage des messages flash c'est juste une boucle qui affiche tous les messages que tu envois a l'utilisateur, donc il est inutile de vérifier si chaque type de message existe avant de l'envoyer.

ton code correspondait un peu (en caricaturant un poil)

Code :

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

1
2
3
4
5
6
7
8
9
10
 $is=array(1,2,3,4,5)
foreach($is as $i){
  if($i == 1){
  //action1
  }
  if($i == 2){
  //action2
  }
//etc...
}

ce qui n'est pas spécialement propre

[davidbouv]

Oui je suis d'accord dans les faits mais j'avais fait ça car je n'ai pas tous le temps la session passé en paramètre de ma vue il me faut donc vérifier qu'elle existe j'ai rajouter un test pour voir si

Code :

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

{% if session is defined %}

et j'ai eu une réaction bizarre ça marchai bien sur le coup et le lendemain j'avais de nouveau mes messages flash qui déconné j'ai effacer le cache et ça a de nouveau marché... Y a quand même un truk pas très clair.