Commentage du code ;

Intégration de la locale dans l'appel ajax

src/ScenarisationProcessBundle/Entity/ScenarisationProcess.php

@@ -213,6 +213,12 @@ class ScenarisationProcess
       return $this;
     }
 
+  /**
+    * This method check if the current ScenarisationProcess is already
+    * liked to a ScenarisationStage given.
+    *
+    * This method is only used by the view ScenarioBundle:scenarisationstage:processManagement.html.twig
+    */
     public function isAlreadyLinkedToStage(ScenarisationStage $scenarisationStage)
     {
       // Faire une requête à partir d'une entité est une très mauvaise pratique, j'abandonne donc l'idée
@@ -229,9 +235,11 @@ class ScenarisationProcess
       $resultat = $dql->getSingleScalarResult();
       */
 
+      // Booléen qui informe sur l'existance d'un lien entre la démarche et l'étape
       $etapeTrouvee = false;
       $scenarisationStageOrderTab = $this->getScenarisationStageOrder();
 
+      // On initialise avant le parcours de toutes les étapes de scénarisation liées à la démarche courante
       $indiceCourant = 0;
       $tailleTableau = sizeof($scenarisationStageOrderTab);
 

src/ScenarisationProcessBundle/Resources/translations/messages.en.yml

@@ -18,6 +18,7 @@ scenarisationStage:
     questionsManagement: "Questions management"
     otherQuestions: "Other questions available"
     associateToAprocess: "Associate to a process"
+    processesManagement: "Associate a stage to a process"
 
 
 question:

src/ScenarisationProcessBundle/Resources/translations/messages.es.yml

@@ -19,6 +19,7 @@ scenarisationStage:
     questionsManagement: ""
     otherQuestions: ""
     associateToAprocess: ""
+    processesManagement: ""
 
 question:
     new: ""

src/ScenarisationProcessBundle/Resources/translations/messages.fr.yml

@@ -19,6 +19,7 @@ scenarisationStage:
     questionsManagement: "Définir les questions associées"
     otherQuestions: "Autres questions disponibles"
     associateToAprocess: "Associer à une démarche"
+    processesManagement: "Associer une étape à une démarche"
 
 question:
     new: "Ajouter une question"

src/ScenarisationProcessBundle/Resources/views/scenarisationprocess/stagesEdit.html.twig

 {% extends 'base.html.twig' %}
 
+{# The ajax call need to know the locale to be correctly redirected #}
+{% set locale = app.request.locale %}
+
 {% block others_javascripts %}
 <script>
     {#  Le script qui change la position des étapes en Drag&Drop a besoin de connaître lID de la
         démarche de scénarisation lors de lappel de la fonction en Ajax #}
   var scen_process_current_id={{ scenarisationProcess.id}};
-  var urlAppelAjax = "/txuleton/web/app_dev.php/es/pedagogical-engineer/scenarisation/process/" + scen_process_current_id +"/";
+  var urlAppelAjax = "/txuleton/web/app_dev.php/{{locale}}/pedagogical-engineer/scenarisation/process/" + scen_process_current_id +"/";
 </script>
 <script src="{{ asset('js/sort_drag_and_drop.js')}}"></script>
 {% endblock %}

src/ScenarisationProcessBundle/Resources/views/scenarisationstage/processManagement.html.twig

 {% extends 'base.html.twig' %}
+
+{# On a besoin de stocker le numéro de la page courante pour pouvoir revenir dessus après ajout dans une démarche #}
 {% set currentPage = app.request.query.get('page') %}
 
 {% block body %}
@@ -7,25 +9,23 @@
 
     <h2>{{ scenarisationStage.title }}</h2>
     <hr/>
-
     {# Si l'application contient des démarches de scénarisation #}
     {% if processes is not empty %}
-
       {# On parcourt la liste des démarche #}
       {% for process in processes %}
         <p>
           <a href="{{path('scenarisation_process_stage_management', {'id' : process.id} ) }}">
             {{ process.title }}
           </a>
-
+          {# Si la l'étape n'est pas déjà dans la démarche, on affiche le bouton '+' #}
           {% if not process.isAlreadyLinkedToStage(scenarisationStage) %}
             &nbsp;|&nbsp;
             <a href="{{ path('add_scenarisation_stage_in_a_process', { 'page': currentPage, 'scenarisationStage': scenarisationStage.id, 'scenarisationProcess': process.id }) ~ '#' ~ process.id }}">
               <i class="fa fa-plus" aria-hidden="true"></i>
             </a>
           {% endif %}
-
         </p>
+        {# Si la démarche contient des étapes, on crée un tableau #}
         {% if (process.scenarisationStageOrder) is not empty %}
           <table id="{{process.id}}" class="table table-striped">
                 <thead>
@@ -35,6 +35,7 @@
                     </tr>
                 </thead>
                 <tbody>
+                  {# On affiche chacune des étapes #}
                   {% for scenarisationStageOrder in process.scenarisationStageOrder %}
                     <tr>
                       <td>{{ scenarisationStageOrder.position }}</td>
@@ -48,9 +49,11 @@
         {% endif %}
 
       {% endfor %}
+        {# Composant KNP pour pouvoir naviguer entre les différentes page #}
         <div class="navigation text-center">
           {{ knp_pagination_render(processes) }}
         </div>
+
     {% else %}
       <p> <i>Aucune démarche de scénarisation trouvée.</i> </p>
     {% endif %}

src/ScenarisationProcessBundle/Resources/views/scenarisationstage/questionsEdit.html.twig

@@ -16,7 +16,7 @@
     {#  Le script qui change la position des étapes en Drag&Drop a besoin de connaître lID de la
         démarche de scénarisation lors de lappel de la fonction en Ajax #}
   var scen_stage_current_id={{ scenarisationStage.id}};
-  var urlAppelAjax = "/txuleton/web/app_dev.php/fr/pedagogical-engineer/scenarisation/stage/" + scen_stage_current_id +"/";
+  var urlAppelAjax = "/txuleton/web/app_dev.php/{{locale}}/pedagogical-engineer/scenarisation/stage/" + scen_stage_current_id +"/";
 
 </script>
 <script src="{{ asset('js/sort_drag_and_drop.js')}}"></script>