modification et suppression compatible sans ajax

app/Resources/translations/flash.fr.yml

@@ -20,10 +20,14 @@ group:
 element:
   add:
     error:        Une erreur est survenus lors de l'ajout.
+  remove:
+    success:      Element supprimé avec succés.
+    error:        Une errur est survenue durant la suppression de l'élément.
     
 user:
   changeemail:
     mail_send:     Un email a été envoyé a l'adresse communiqué pour confirmation.
     token_invalid: Le lien pour mettre a jour votre email est périmé, veuillez relancer la procédure.
     success:       Votre email a correctement été mis a jour.
-    wait:          Vous devez attendre avant de pouvoir faire une nouvelle demande de changement d'email.
+    wait:          Vous devez attendre avant de pouvoir faire une nouvelle demande de changement d'email.
+    

app/Resources/translations/navigationui.fr.yml

@@ -35,6 +35,8 @@ my_groups:
   
 title:
   hello:                Découvrez et partagez la musique !
+  element:
+    edit:               Modification d'un élément
   
   
 bottom_menu:

app/Resources/translations/userui.fr.yml

@@ -60,4 +60,11 @@ tags_filter:
   
 index:
   beta:
-    warning:  L'accès au service est privé pendant le développement de l'application
+    warning:  L'accès au service est privé pendant le développement de l'application
+    
+system:
+  infos:
+    normally_js:   |
+                    Vous avez ouvert cette page dans un nouvel onglet/nouvelle page
+                    Sachez qu'avez le clic gauche de votre souris la modification
+                    de l'élément est intégré dans la page.

src/Muzich/CoreBundle/Controller/ElementController.php

@@ -14,12 +14,7 @@ class ElementController extends Controller
    * @return Muzich\CoreBundle\Entity\Element 
    */
   protected function checkExistingAndOwned($element_id)
-  {
-    if (!$this->getRequest()->isXmlHttpRequest())
-    {
-      throw $this->createNotFoundException('Not allowed');
-    }
-    
+  {    
     if (!($element = $this->getDoctrine()->getRepository('MuzichCoreBundle:Element')
       ->findOneById($element_id)))
     {
@@ -38,7 +33,7 @@ class ElementController extends Controller
    * 
    */
   public function editAction($element_id)
-  {
+  {    
     $element = $this->checkExistingAndOwned($element_id);
     
     $element_tags = $element->getTags();
@@ -51,19 +46,30 @@ class ElementController extends Controller
       $search_tags[$tag->getId()] = $tag->getName();
     }
     
-    $html = $this->render('MuzichCoreBundle:Element:element.edit.html.twig', array(
+    $template = 'MuzichCoreBundle:Element:ajax.element.edit.html.twig'; 
+    if (!$this->getRequest()->isXmlHttpRequest())
+    {
+      $template = 'MuzichCoreBundle:Element:element.edit.html.twig'; 
+    }
+    
+    $response = $this->render($template, array(
       'form'        => $form->createView(),
       'form_name'   => 'element_'.$element->getId(),
       'element_id'  => $element->getId(),
       'search_tags' => $search_tags
-    ))->getContent();
-    
-    return $this->jsonResponse(array(
-      'status'    => 'success',
-      'form_name' => 'element_'.$element->getId(),
-      'tags'      => $search_tags,
-      'html'      => $html
     ));
+    
+    if ($this->getRequest()->isXmlHttpRequest())
+    {
+      return $this->jsonResponse(array(
+        'status'    => 'success',
+        'form_name' => 'element_'.$element->getId(),
+        'tags'      => $search_tags,
+        'html'      => $response->getContent()
+      ));
+    }
+    
+    return $response;
   }
   
   /**
@@ -109,10 +115,31 @@ class ElementController extends Controller
       }
     }
     
-    return $this->jsonResponse(array(
-      'status'  => $status,
-      'html'    => $html,
-      'errors'  => $errors
+    if ($this->getRequest()->isXmlHttpRequest())
+    {
+      return $this->jsonResponse(array(
+        'status'  => $status,
+        'html'    => $html,
+        'errors'  => $errors
+      ));
+    }
+    
+    if ($status == 'success')
+    {
+      return $this->redirect($this->generateUrl('home'));
+    }
+    
+    
+    $element->setTagsWithIds(
+      $this->getDoctrine()->getEntityManager(), 
+      json_decode($element->getTags())
+    );
+    
+    return $this->render('MuzichCoreBundle:Element:element.edit.html.twig', array(
+      'form'        => $form->createView(),
+      'form_name'   => 'element_'.$element->getId(),
+      'element_id'  => $element->getId(),
+      'search_tags' => $element->getTagsIdsJson()
     ));
   }
   
@@ -124,11 +151,21 @@ class ElementController extends Controller
       $em->remove($element);
       $em->flush();
       
-      return $this->jsonResponse(array('status' => 'success'));
+      if ($this->getRequest()->isXmlHttpRequest())
+      {
+        return $this->jsonResponse(array('status' => 'success'));
+      }
+      $this->setFlash('success', 'element.remove.success');
+      return $this->redirect($this->container->get('request')->headers->get('referer'));
     } 
     catch(Exception $e)
     {
-      return $this->jsonResponse(array('status' => 'error'));
+      if ($this->getRequest()->isXmlHttpRequest())
+      {
+        return $this->jsonResponse(array('status' => 'error'));
+      }
+      $this->setFlash('error', 'element.remove.error');
+      return $this->redirect($this->container->get('request')->headers->get('referer'));
     }
   }
   

src/Muzich/CoreBundle/Entity/Element.php

@@ -224,9 +224,12 @@ class Element
   public function getTagsIdsJson()
   {
     $ids = array();
-    foreach ($this->getTags() as $tag)
+    if (count($this->getTags()))
     {
-      $ids[] = $tag->getId();
+      foreach ($this->getTags() as $tag)
+      {
+        $ids[] = $tag->getId();
+      }
     }
     return json_encode($ids);
   }

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

@@ -0,0 +1,10 @@
+
+<form 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, 
+    'form_name'     : form_name
+  } %}
+
+  <input type="submit" value="{{ 'element_edit.submit.value'|trans({}, 'userui') }}" />
+</form>

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

@@ -1,9 +1,19 @@
-<form name="{{ form_name }}" action="{{ path('element_update', {'element_id': element_id}) }}" method="post" {{ form_enctype(form) }}>
+{% extends "MuzichHomeBundle::layout.html.twig" %}
 
-  {% include "MuzichCoreBundle:Element:form.edit.html.twig" with { 
-    'form'          : form, 
-    'form_name'     : form_name
-  } %}
+{% block title %}{{ 'title.element.edit'|trans({}, 'userui') }}{% endblock %}
 
-  <input type="submit" value="{{ 'element_edit.submit.value'|trans({}, 'userui') }}" />
-</form>
+{% block content %}
+
+  <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) }}>
+
+    {% include "MuzichCoreBundle:Element:form.edit.html.twig" with { 
+      'form'          : form, 
+      'form_name'     : form_name
+    } %}
+
+    <input type="submit" value="{{ 'element_edit.submit.value'|trans({}, 'userui') }}" />
+  </form>
+  
+{% endblock %}