Anomalie #185: Message pour proposition de tag

app/Resources/translations/userui.fr.yml

@@ -54,6 +54,10 @@ tags:
     inputs:
       cancel:       Annuler
       submit:       Faire la demande
+    errors:
+      min:              Le tag doit être de %limit% caractéres minimum
+      max:              Le tag doit être de %limit% caractéres maximum
+      unknow:           Une erreur est survenue
   new:
     has_news:       %count% éléments viennent d'être ajoutés, %link%
     has_news_one:   %count% élément vient d'être ajouté, %link%

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -433,15 +433,49 @@ class CoreController extends Controller
   /**
    * Ajout d'un tag en base.
    */
-  public function addTagAction($name, $arguments = null)
+  public function addTagAction()
   {
-    if (($response = $this->mustBeConnected()))
+    if (($response = $this->mustBeConnected(true)))
     {
       return $response;
     }
     
+    if (strlen((($tag_name = $this->getRequest()->request->get('tag_name')))) 
+      < $this->container->getParameter('tag_add_min_length'))
+    {
+      return $this->jsonResponse(array(
+        'status' => 'error',
+        'errors' => array($this->trans(
+          'tags.add.errors.min', 
+          array(
+            '%limit%' => $this->container->getParameter('tag_add_min_length')
+          ), 
+          'userui'
+        )
+      )));
+    }
+    
+    if (strlen($tag_name) > $this->container->getParameter('tag_add_max_length'))
+    {
+      return $this->jsonResponse(array(
+        'status' => 'error',
+        'errors' => array($this->trans(
+          'tags.add.errors.max', 
+          array(
+            '%limit%' => $this->container->getParameter('tag_add_max_length')
+          ), 
+          'userui'
+        )
+      )));
+    }
+    
     $tagManager = new TagManager();
-    $tag = $tagManager->addTag($this->getDoctrine(), $name, $this->getUser(), $arguments);
+    $tag = $tagManager->addTag(
+      $this->getDoctrine(), 
+      $tag_name, 
+      $this->getUser(), 
+      $this->getRequest()->request->get('argument')
+    );
     
     return $this->jsonResponse(array(
       'status'   => 'success',

src/Muzich/CoreBundle/Entity/Tag.php

@@ -58,7 +58,7 @@ class Tag
   /**
    * Nom du tag
    * 
-   * @ORM\Column(type="string", length=64, unique=true)
+   * @ORM\Column(type="string", length=64, unique=true, nullable=false)
    * @var type string
    */
   protected $name;

src/Muzich/CoreBundle/Managers/TagManager.php

@@ -78,6 +78,7 @@ class TagManager
     }
     else
     {
+      
       // Sinon on l'ajoute en base
       $tag = new Tag();
       $tag->setName(ucfirst(strtolower($name)));

src/Muzich/CoreBundle/Resources/config/routing.yml

@@ -45,12 +45,8 @@ ajax_get_favorites_tags:
   defaults: { _controller: MuzichCoreBundle:Core:getFavoriteTags }
 
 ajax_add_tag:
-  pattern: /ajax/add-tag/{name}
-  defaults: { _controller: MuzichCoreBundle:Core:addTag, name: null }
-  
-ajax_add_tag_arguments:
-  pattern: /ajax/add-tag/{name}/{arguments}
-  defaults: { _controller: MuzichCoreBundle:Core:addTag, name: null, arguments: null }
+  pattern: /ajax/add-tag
+  defaults: { _controller: MuzichCoreBundle:Core:addTag }
   
 ajax_element_add_vote_good:
   pattern: /ajax/element/vote/add/good/{element_id}/{token}

web/bundles/muzichcore/js/muzich.js

@@ -817,6 +817,55 @@ $(document).ready(function(){
                     .addClass('popin_block')
                     .css('width', '400px')
                       //.append($('<h2>').append(string_tag_add_title))
+                   .append($('<form>')
+                     .attr('action', url_add_tag)
+                     .attr('method', 'post')
+                     .attr('name', 'add_tag')
+                     .ajaxForm(function(response) {
+                       /*
+                        *
+                        */
+
+                        if (response.status == 'mustbeconnected')
+                        {
+                          $(location).attr('href', url_index);
+                        }
+
+                        if (response.status == 'success')
+                        {
+                          var tag_id   = response.tag_id;
+                          var tag_name = response.tag_name;
+
+                          $('input#tags_selected_tag_'+form_name).val(tag_id);
+                          inputTag.val(tag_name);
+                          // Et on execute l'évènement selectTag de l'input
+                          inputTag.trigger("selectTag");
+                          // On cache la liste puisque le choix vient d'être fait
+                          divtags.hide();
+                          inputTag.val(tag_text_help); 
+
+                          link_add_tag.parents('div.search_tag_list').find('img.tag_loader').hide();
+
+                          $('#fade').fadeOut(400, function(){$('#fade').remove();});
+                          $('#add_tag').remove();
+                        }
+                        
+                        if (response.status == 'error')
+                        {
+                          $('form[name="add_tag"]').find('ul.error_list').remove();
+                          var ul_errors = $('<ul>').addClass('error_list');
+
+                          for (i in response.errors)
+                          {
+                            ul_errors.append($('<li>').append(response.errors[i]));
+                          }
+
+                          $('form[name="add_tag"]').prepend(ul_errors);
+                        }
+                        
+                        return false;
+                     })
+                     
                       .append($('<div>').addClass('tag')
                         .append($('<ul>')
                           .append($('<li>').addClass('button')
@@ -837,58 +886,17 @@ $(document).ready(function(){
                           })
                         )
                         .append($('<input>')
-                          .attr('type', 'button')
+                          .attr('type', 'submit')
                           .attr('value', string_tag_add_inputs_submit)
                           .addClass('button')
                           .click(function(){
                             
-                            var arguments = $('#add_tag textarea').val();
-                            
-                            $('#fade').fadeOut(400, function(){$('#fade').remove();});
-                            $('#add_tag').remove();
-                            
-                            // On récupère le nom du tag
-                            var name = link_add_tag.attr('href').substr(1,link_add_tag.attr('href').length);
-                            name = name.substr(strpos(name, '#')+1, name.length);
-
                             link_add_tag.parents('div.search_tag_list').find('img.tag_loader').show();
-
-                            var url;
-                            if (arguments)
-                            {
-                              url = url_add_tag+'/'+name+'/'+arguments;
-                            }
-                            else
-                            {
-                              url = url_add_tag+'/'+name;
-                            }
-
-                            // La on fait l'ajout en base en tant que nouveau tag
-                            $.getJSON(url, function(response){
-
-                              if (response.status == 'mustbeconnected')
-                              {
-                                $(location).attr('href', url_index);
-                              }
-
-                              var tag_id   = response.tag_id;
-                              var tag_name = response.tag_name;
-
-                              $('input#tags_selected_tag_'+form_name).val(tag_id);
-                              inputTag.val(tag_name);
-                              // Et on execute l'évènement selectTag de l'input
-                              inputTag.trigger("selectTag");
-                              // On cache la liste puisque le choix vient d'être fait
-                              divtags.hide();
-                              inputTag.val(tag_text_help); 
-
-                              link_add_tag.parents('div.search_tag_list').find('img.tag_loader').hide();
-                            });
-                            
-                            return false;
+                          
                           })
                         )
-                      )
+                        .append($('<input>').attr('type', 'hidden').attr('name', 'tag_name').val($(this).text()))
+                      ))
                     ;
                     
                     // Il faut ajouter le popup au dom avant de le positionner en css