Evolution #704: Tests: résoudre fucking perists truc

src/Muzich/CoreBundle/Controller/CoreController.php

@@ -109,18 +109,6 @@ class CoreController extends Controller
     
     $user = $this->getUser();
     
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     // Vérifications préléminaires
     if ($user->getPersonalHash($id) != $token 
         || !in_array($type, array('user', 'group')) 
@@ -239,19 +227,6 @@ class CoreController extends Controller
         
     if ($form->isValid())
     {
-
-      /**
-       * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-       * Docrine le voit si on faire une requete directe.
-       */
-      if ($this->container->getParameter('env') == 'test')
-      {
-        $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-          $this->container->get('security.context')->getToken()->getUser()->getId(),
-          array()
-        )->getSingleResult();
-      }
-
       // On utilise le gestionnaire d'élément
       $factory = new ElementManager($element, $em, $this->container);
       $factory->proceedFill($user);
@@ -535,17 +510,6 @@ class CoreController extends Controller
     }
     
     $user = $this->getUser();
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
     
     // On contrôle au préalable que le tag ne fait pas déjà partie des favoris de 
     // l'utilisateur

src/Muzich/CoreBundle/Controller/ElementController.php

@@ -116,13 +116,6 @@ class ElementController extends Controller
      * Docrine le voit si on faire une requete directe.
      */
     $user = $this->getUser();
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
     
     $element = $this->checkExistingAndOwned($element_id);
     // Si il y a un groupe on le retire pour le bind
@@ -230,13 +223,6 @@ class ElementController extends Controller
       * Docrine le voit si on faire une requete directe.
       */
       $user = $this->getUser();
-      if ($this->container->getParameter('env') == 'test')
-      {
-        $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-          $this->container->get('security.context')->getToken()->getUser()->getId(),
-          array()
-        )->getSingleResult();
-      }
       
       // On signale que cet user a modifié ses diffusions
       $user->setData(User::DATA_DIFF_UPDATED, true);
@@ -660,13 +646,6 @@ class ElementController extends Controller
      * Docrine le voit si on faire une requete directe.
      */
     $user = $this->getUser();
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
     
     $proposition = new ElementTagsProposition();
     $proposition->setElement($element);
@@ -897,13 +876,6 @@ class ElementController extends Controller
       * Docrine le voit si on faire une requete directe.
       */
     $user = $this->getUser();
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
     
     // Pour le repartage on crée un nouvel élément
     $element_reshared = new Element();

src/Muzich/CoreBundle/Resources/public/css/main.css

@@ -2195,9 +2195,14 @@ div#helpbox input.tag_prompt
   width: 480px;
 }
 
-div#helpbox  input.button
+div#helpbox input.button
 {
   float: right;
   margin-right: 11px;
   margin-top: 5px;
-}
+}
+
+div#helpbox div.center input.button
+{
+  float: none;
+}

src/Muzich/CoreBundle/lib/Controller.php

@@ -62,10 +62,6 @@ class Controller extends BaseController
   protected function isVisitor()
   {
     $user = $this->getUser();
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getUserRefreshed();
-    }
     if ($user === 'anon.')
     {
       return true;
@@ -162,6 +158,11 @@ class Controller extends BaseController
    */
   public function getUser($personal_query = false, $params = array(), $force_refresh = false)
   {
+    if ($this->container->getParameter('env') == 'test')
+    {
+      return $this->container->get('security.context')->getToken()->getUser();
+    }
+    
     if (!$personal_query)
     {
       // Si on demande l'utilisateur sans forcer la réactualisation et que l'utilisateur
@@ -208,15 +209,7 @@ class Controller extends BaseController
    */
   protected function getUserId($return_null_if_visitor = false)
   {
-    /** Bug */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getUserRefreshed();
-    }
-    else
-    {
-      $user = $this->getUser();
-    }
+    $user = $this->getUser();
     
     if ($user !== 'anon.')
     {

src/Muzich/FavoriteBundle/Controller/FavoriteController.php

@@ -38,19 +38,6 @@ class FavoriteController extends Controller
     }
     
     $user = $this->getUser();
-    
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     $em = $this->getEntityManager();
     
     if ($user->getPersonalHash($id) != $token || !is_numeric($id)

src/Muzich/GroupBundle/Controller/DefaultController.php

@@ -64,18 +64,6 @@ if (($non_condition = $this->userHaveNonConditionToMakeAction(SecurityContext::A
       throw $this->createNotFoundException();
     }
     
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     $em = $this->getDoctrine()->getEntityManager();
     
     $new_group = new Group();

src/Muzich/UserBundle/Controller/UserController.php

@@ -37,23 +37,7 @@ class UserController extends Controller
   
   protected function getPreferencesForm()
   {
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    else
-    {
-      $user = $this->getUser();
-    }
-     
-    return $this->createFormBuilder($user)
+    return $this->createFormBuilder($this->getUser())
       ->add('mail_newsletter', 'checkbox', array('required' => false))
       ->add('mail_partner', 'checkbox', array('required' => false))
       ->getForm()
@@ -79,19 +63,6 @@ class UserController extends Controller
     if ($this->tags_favorites === null || $force)
     {
       $user = $this->getUser();
-      
-      /**
-       * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-       * Docrine le voit si on faire une requete directe.
-       */
-      if ($this->container->getParameter('env') == 'test')
-      {
-        $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-          $this->container->get('security.context')->getToken()->getUser()->getId(),
-          array()
-        )->getSingleResult();
-      }
-      
       $this->tags_favorites = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')
         ->getTagsFavorites($user->getId())
       ;
@@ -297,13 +268,6 @@ class UserController extends Controller
   public function changePasswordAction(Request $request)
   {
     $user = $this->getUser();
-    
-    /** Bug */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getUserRefreshed();
-    }
-    
     $form = $this->getChangePasswordForm($user);
     $form->bind($request);
     
@@ -363,19 +327,6 @@ class UserController extends Controller
   {
     $request = $this->getRequest();
     $user = $this->getUser(true, array('join' => array('favorites_tags')));
-    
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     $form = $this->getTagsFavoritesForm($user);
     
     if ($request->getMethod() == 'POST')
@@ -453,18 +404,6 @@ class UserController extends Controller
     $em = $this->getDoctrine()->getEntityManager();
     $user = $this->getUser();
     
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     $request = $this->getRequest();
     $change_email_form = $this->getChangeEmailForm();
     
@@ -551,18 +490,6 @@ class UserController extends Controller
     $um = $this->get('muzich_user_manager');
     $user = $this->getUser();
     
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
-    
     $token_ = hash('sha256', $user->getConfirmationToken().($email = $user->getEmailRequested()));
     
     // Le token est-il valide
@@ -595,18 +522,7 @@ class UserController extends Controller
       return $response;
     }
     
-    /**
-     * Bug lors des tests: L'user n'est pas 'lié' a celui en base par doctrine.
-     * Docrine le voit si on faire une requete directe.
-     */
     $user = $this->getUser();
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getDoctrine()->getRepository('MuzichCoreBundle:User')->findOneById(
-        $this->container->get('security.context')->getToken()->getUser()->getId(),
-        array()
-      )->getSingleResult();
-    }
     
     $errors = array();
     if ($user->getPersonalHash() != $token)
@@ -833,12 +749,6 @@ class UserController extends Controller
   {
     $user = $this->getUser();
     
-    /** Bug */
-    if ($this->container->getParameter('env') == 'test')
-    {
-      $user = $this->getUserRefreshed();
-    }
-    
     if ($token == hash('sha256', $user->getConfirmationToken().$user->getEmail()))
     {
       $user->setEmailConfirmed(true);