Evolution #31: html: erreurs en couleurs

app/Resources/translations/validators.fr.yml

@@ -2,15 +2,19 @@
 'The credentials were changed from another session.' :  "La page a été innactive trop longtemps, cuillez recharger la page."
 'The presented password cannot be empty.'            :  "Vous devez saisir un mot de passe."
 'The presented password is invalid.'                 :  "Le mot de passe est invalide."
+'Bad credentials'                                    :  "Vos identifiants sont incorrects."
 
 'The username is already used'   :   Le nom d'utilisateur est déjà utilisé.
-'The email is already used'      :   Cette adresse email est déjà utilisé sur Muzi.ch
+'The email is already used'      :   Cette adresse email est déjà utilisé.
+
 
 error:
   group:
     name:
-      unique:            Ce nom est déjà utilisé pour un autre groupe
+      unique:            Ce nom est déjà utilisé pour un autre groupe.
   registration:
     username:
-      min:               Votre nom d'utilisateur doit contenir au moins %limit% caractères 
-      max:               Votre nom d'utilisateur doit contenir au maximum %limit% caractères 
+      min:               Votre nom d'utilisateur doit contenir au moins %limit% caractères.
+      max:               Votre nom d'utilisateur doit contenir au maximum %limit% caractères.
+    password:
+      notsame:           Vous avez saisie deux mot de passes différents.

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -214,7 +214,7 @@ class CoreController extends Controller
         }
         else
         {
-          $this->setFlash('error', 'element.add.error');
+          //$this->setFlash('error', 'element.add.error');
           
           $search_object = $this->getElementSearcher();
           $user = $this->getUser(true, array('join' => array(

src/Muzich/CoreBundle/Resources/views/Element/form.add.html.twig

@@ -1,9 +1,23 @@
+{% form_theme form 'MuzichCoreBundle:Form:errors.html.twig' %}
+
 {{ form_errors(form) }}
 
-{{ form_row(form.name) }}
+<div class="field">
+  {{ form_errors(form.name) }}
+  {{ form_label(form.name, 'name'|trans({}, 'elementform')) }}
+  {{ form_widget(form.name) }}
+</div>
 
-{{ form_row(form.url) }}
+<div class="field">
+  {{ form_errors(form.url) }}
+  {{ form_label(form.url, 'url'|trans({}, 'elementform')) }}
+  {{ form_widget(form.url) }}
+</div>
 
-{{ form_row(form.tags) }}
+<div class="field">
+  {{ form_errors(form.tags) }}
+  {{ form_label(form.tags, 'tags'|trans({}, 'elementform')) }}
+  {{ form_widget(form.tags) }}
+</div>
 
 {{ form_rest(form) }}

src/Muzich/CoreBundle/Resources/views/Form/errors.html.twig

@@ -0,0 +1,11 @@
+{% block field_errors %}
+{% spaceless %}
+  {% if errors|length > 0 %}
+  <ul class="error_list">
+    {% for error in errors %}
+      <li>{{ error.messageTemplate|trans(error.messageParameters, 'validators') }}</li>
+    {% endfor %}
+  </ul>
+  {% endif %}
+{% endspaceless %}
+{% endblock field_errors %}

src/Muzich/GroupBundle/Controller/DefaultController.php

@@ -88,7 +88,7 @@ class DefaultController extends Controller
         'groups_owned'
       )));
       
-      $this->setFlash('error', 'group.create.failure');
+      //$this->setFlash('error', 'group.create.failure');
       
       return $this->render(
         'MuzichGroupBundle:Default:myList.html.twig', 

src/Muzich/GroupBundle/Resources/views/Form/form.html.twig

@@ -1,28 +1,30 @@
+{% form_theme form 'MuzichCoreBundle:Form:errors.html.twig' %}
+
 {{ form_errors(form) }}
 
 <div class="field">
-  {{ form_label(form.name, 'name'|trans({}, 'groupform')) }}
   {{ form_errors(form.name) }}
+  {{ form_label(form.name, 'name'|trans({}, 'groupform')) }}
   {{ form_widget(form.name) }}
 </div>
 
 <div class="field">
-  {{ form_label(form.description, 'description'|trans({}, 'groupform')) }}
   {{ form_errors(form.description) }}
+  {{ form_label(form.description, 'description'|trans({}, 'groupform')) }}
   {{ form_widget(form.description) }}
 </div>
 
   <i>{{ 'openhelp'|trans({}, 'groupform') }}</i>
 
 <div class="field">
-  {{ form_label(form.open, 'open'|trans({}, 'groupform')) }}
   {{ form_errors(form.open) }}
+  {{ form_label(form.open, 'open'|trans({}, 'groupform')) }}
   {{ form_widget(form.open) }}
 </div>
 
 <div class="field">
-  {{ form_label(form.tags, 'tags'|trans({}, 'groupform')) }}
   {{ form_errors(form.tags) }}
+  {{ form_label(form.tags, 'tags'|trans({}, 'groupform')) }}
   {{ form_widget(form.tags) }}
 </div>
 

src/Muzich/MynetworkBundle/Resources/views/Mynetwork/search.html.twig

@@ -12,13 +12,17 @@
   
   <form action="{{ path('mynetwork_search') }}" method="post" {{ form_enctype(search_form) }}>
     
+    {% form_theme search_form 'MuzichCoreBundle:Form:errors.html.twig' %}
     {{ form_errors(search_form) }}
 
-    {{ form_row(search_form.string) }}
+    <div class="field">
+      {{ form_errors(search_form.string) }}
+      {{ form_widget(search_form.string) }}
+      <input type="submit" value="{{ 'search.form.submit_value'|trans({}, 'network') }}" />
+    </div>
 
     {{ form_rest(search_form) }}
 
-    <input type="submit" value="{{ 'search.form.submit_value'|trans({}, 'network') }}" />
   </form>
   
   {% if results.users|length or results.groups|length %}

src/Muzich/UserBundle/Controller/UserController.php

@@ -41,12 +41,15 @@ class UserController extends Controller
    * Un bug étrange empêche la mise ne place de contraintes sur le formulaire
    * d'inscription. On effectue alors les vérifications ici.
    * 
+   * C'est sale, mais ça marche ...
+   * 
    * @return array of string errors
    */
   protected function checkRegistrationInformations($form)
   {
     $errors = array();
     $form->bindRequest($this->getRequest());
+    $form_values = $this->getRequest()->request->get($form->getName());
     $user = $form->getData();
     
     /*
@@ -72,6 +75,18 @@ class UserController extends Controller
       );
     }
     
+    /**
+     * Mot de passes indentiques
+     */
+    if ($form_values['plainPassword']['first'] != $form_values['plainPassword']['second'])
+    {
+      $errors[] = $this->get('translator')->trans(
+        'error.registration.password.notsame', 
+        array(),
+        'validators'
+      );
+    }
+    
     return $errors;
   }
   

src/Muzich/UserBundle/Resources/views/Registration/register.html.twig

@@ -4,15 +4,9 @@
     class="fos_user_registration_register"
   >
         
+    {% form_theme form 'MuzichCoreBundle:Form:errors.html.twig' %}
     
-    {% if form.get("errors")|length > 0 %}
-    <ul class="error_list">
-      {% for error in form.get("errors") %}
-        <li>{{ error.messageTemplate|trans(error.messageParameters, 'validators') }}</li>
-      {% endfor %}
-    </ul>
-    {% endif %}
-    
+    {{ form_errors(form) }}
     {% if registration_errors_pers|length > 0 %}
     <ul class="error_list">
       {% for error in registration_errors_pers %}
@@ -21,12 +15,16 @@
     </ul>
     {% endif %}
     
+    {{ form_errors(form.username) }}
     {{ form_row(form.username) }}
     
+    {{ form_errors(form.email) }}
     {{ form_row(form.email) }}
     
+    {{ form_errors(form.plainPassword.first) }}
     {{ form_row(form.plainPassword.first) }}
     
+    {{ form_errors(form.plainPassword.second) }}
     {{ form_row(form.plainPassword.second) }}
 
     {{ form_rest(form) }}

src/Muzich/UserBundle/Resources/views/Security/login.html.twig

@@ -1,6 +1,6 @@
 <div class="login">
   {% if error %}
-      <div>{{ error|trans({}, 'validators') }}</div>
+      <ul class="error_list"><li>{{ error|trans({}, 'validators') }}</li></ul>
   {% endif %}
 
   <form action="{{ path("fos_user_security_check") }}" method="post">

web/bundles/muzichcore/css/main.css

@@ -273,4 +273,20 @@ ul.inline li
 {
   text-decoration: none;
   color: black;
+}
+
+.error_list
+{
+  font-size: 87%;
+  color: #dd0101;
+  
+  margin-left: 24px;
+  margin-top: 2px;
+  margin-bottom: 2px;
+  margin-right: 2px;
+}
+
+.error_list li
+{
+  list-style: url(/bundles/muzichcore/img/1323874685_warning.png);
 }