Messages d'erreurs françisé, amélioration dom ...

app/Resources/translations/validators.en.yml

@@ -0,0 +1,5 @@
+
+error:
+  element:
+    name:
+      notblank:          Choose a name please

app/Resources/translations/validators.fr.yml

@@ -24,6 +24,15 @@ error:
   changeemail:
     email:
       invalid:           L'email saisie n'est pas valide.
+  element:
+    url:
+      notblank:          Vous devez saisir une url (adresse http://)
+      tolong:            L'url (adresse http://) est trop longue
+      invalid:           L'url (adresse http://) n'est pas correcte
+    name:
+      notblank:          Vous devez saisir un nom
+      toshort:           Le nom est trop court
+      tolong:            Le nom est trop long
       
 registration:
   token:

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -248,7 +248,7 @@ class CoreController extends Controller
 
         foreach ($errorList as $error)
         {
-          $errors[] = $error->getMessage();
+          $errors[] = $this->trans($error->getMessage(), array(), 'validators');
         }
         
         return $this->jsonResponse(array(

src/Muzich/CoreBundle/Controller/ElementController.php

@@ -139,7 +139,7 @@ class ElementController extends Controller
       
       foreach ($errorList as $error)
       {
-        $errors[] = $error->getMessage();
+        $errors[] = $this->trans($error->getMessage(), array(), 'validators');
       }
     }
     

src/Muzich/CoreBundle/Entity/Element.php

@@ -77,9 +77,9 @@ class Element
    * L'url est l'url du media. 
    * 
    * @ORM\Column(type="string", length=1024)
-   * @Assert\NotBlank()
-   * @Assert\MaxLength(1024)
-   * @Assert\Url()
+   * @Assert\NotBlank(message = "error.element.url.notblank")
+   * @Assert\MaxLength(limit = 1024, message = "error.element.url.tolong")
+   * @Assert\Url(message = "error.element.url.invalid")
    * @var type string
    */
   protected $url;
@@ -87,10 +87,10 @@ class Element
   /**
    * Libellé du media
    * 
-   * @ORM\Column(type="string", length=128)
-   * @Assert\NotBlank()
-   * @Assert\MinLength(limit=3)
-   * @Assert\MaxLength(64)
+   * @ORM\Column(type = "string", length = 128)
+   * @Assert\NotBlank(message = "error.element.name.notblank")
+   * @Assert\MinLength(limit = 3, message = "error.element.name.toshort")
+   * @Assert\MaxLength(limit = 64, message = "error.element.name.tolong")
    * @var type string
    */
   protected $name;

src/Muzich/CoreBundle/Managers/GroupManager.php

@@ -51,8 +51,10 @@ class GroupManager
     {
       $this->em->remove($group_tag);
     }
+    $this->em->flush();
     // Pour etablir les nouveaux liens
     $this->group->setTagsWithIds($this->em, $tags_ids);
+    $this->em->flush();
   }
   
 }

src/Muzich/CoreBundle/Resources/views/Element/ajax.element.edit.html.twig

@@ -1,5 +1,5 @@
 
-<form class="edit_element" name="{{ form_name }}" action="{{ path('element_update', {'element_id': element_id}) }}" method="post" {{ form_enctype(form) }}>
+<form novalidate class="edit_element" name="{{ form_name }}" action="{{ path('element_update', {'element_id': element_id}) }}" method="post" {{ form_enctype(form) }}>
 
   {% include "MuzichCoreBundle:Element:form.edit.html.twig" with { 
     'form'          : form, 

src/Muzich/CoreBundle/Resources/views/Element/element.edit.html.twig

@@ -6,7 +6,7 @@
 
   <span class="sinfo">{{ 'system.infos.normally_js'|trans({}, 'userui') }}</span>
 
-  <form name="{{ form_name }}" action="{{ path('element_update', {'element_id': element_id}) }}" method="post" {{ form_enctype(form) }}>
+  <form novalidate name="{{ form_name }}" action="{{ path('element_update', {'element_id': element_id}) }}" method="post" {{ form_enctype(form) }}>
 
     {% include "MuzichCoreBundle:Element:form.edit.html.twig" with { 
       'form'          : form, 

src/Muzich/CoreBundle/lib/Controller.php

@@ -295,6 +295,13 @@ class Controller extends BaseController
     return $this->container->get('templating.helper.assets')->getUrl($path, $packageName);
   }
   
+  /**
+   *
+   * @param strin $string
+   * @param array $params
+   * @param string $package
+   * @return string 
+   */
   protected function trans($string, $params = array(), $package = null)
   {
     return $this->get('translator')->trans($string, $params, $package);

src/Muzich/GroupBundle/Controller/DefaultController.php

@@ -36,9 +36,10 @@ class DefaultController extends Controller
     $form_new = $this->getGroupForm($new_group);
     
     return array(
-      'groups'        => $user->getGroupsOwned(),
-      'form_new'      => $form_new->createView(),
-      'form_new_name' => $form_new->getName()
+      'groups'         => $user->getGroupsOwned(),
+      'form_new'       => $form_new->createView(),
+      'form_new_name'  => $form_new->getName(),
+      'open_add_group' => false
     );
   }
   
@@ -94,9 +95,10 @@ class DefaultController extends Controller
       return $this->render(
         'MuzichGroupBundle:Default:myList.html.twig', 
          array(
-           'groups'        => $user->getGroupsOwned(),
-           'form_new'      => $form_new->createView(),
-           'form_new_name' => $form_new->getName()
+           'groups'         => $user->getGroupsOwned(),
+           'form_new'       => $form_new->createView(),
+           'form_new_name'  => $form_new->getName(),
+           'open_add_group' => true
          )
       );
     }
@@ -138,7 +140,7 @@ class DefaultController extends Controller
       $prompt_tags[$tag->getTag()->getId()] = $tag->getTag()->getName();
     }
     
-    $group->setTagsToIds();
+    $group->setTags($group->getTagsIdsJson());
     $form = $this->getGroupForm($group);
     
     return array(
@@ -159,9 +161,14 @@ class DefaultController extends Controller
       throw $this->createNotFoundException('Vous n\'ête pas le créateur de ce groupe.');
     }
 
-    // Pour être compatible avec le formulaire, la collection de tags dois être
-    // une collection d'id
-    $group->setTagsToIds();
+    $prompt_tags = array();
+    foreach ($group->getTags() as $tag)
+    {
+      $prompt_tags[$tag->getTag()->getId()] = $tag->getTag()->getName();
+    }
+    
+    // Pour être compatible avec le formulaire
+    $group->setTags($group->getTagsIdsJson());
     $form = $this->getGroupForm($group);
     
     $form->bindRequest($request);
@@ -178,14 +185,14 @@ class DefaultController extends Controller
       return $this->redirect($this->generateUrl('show_group', array('slug' => $group->getSlug())));
     }
     else
-    {
-      $this->setFlash('error', 'group.update.failure');
-      
+    {      
       return $this->render(
-        'GroupBundle:Default:edit.html.twig', 
+        'MuzichGroupBundle:Default:edit.html.twig', 
          array(
-           'form_new'      => $form->createView(),
-           'form_new_name' => $form_new->getName()
+          'group'       => $group,
+          'form'        => $form->createView(),
+          'form_name'   => 'group',
+          'search_tags' => $prompt_tags
          )
       );
     }

src/Muzich/GroupBundle/Resources/views/Default/myList.html.twig

@@ -34,7 +34,7 @@
 
   {% endif %}
 
-  <div id="group_add_box" style="display: none;">
+  <div id="group_add_box" style="{% if not open_add_group %}display: none;{% endif %}">
 
     <a href="#" id="group_add_close_link" class="button">
       &lt;&lt; {{ 'group.add_group_box.close'|trans({}, 'navigationui') }}