KIClubinfo / upont

back/src/KI/UserBundle/Entity/CoreUser.php

Doc Comments +3 added lines, -3 removed lines

@@ -218,8 +218,8 @@ 
     /**
      * Add group
      *
-     * @param \KI\UserBundle\Entity\User $group
-     * @return Comment
+     * @param Group $group
+     * @return CoreUser
      */
     public function addGroupUser(\KI\UserBundle\Entity\Group $group)
     {
@@ -232,7 +232,7 @@ 
     /**
      * Remove group
      *
-     * @param \KI\UserBundle\Entity\User $group
+     * @param Group $group
      */
     public function removeGroupUser(\KI\UserBundle\Entity\Group $group)
     {

back/src/KI/UserBundle/Repository/UserRepository.php

Indentation +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
      */
     public function findFollowedEvents($userId, $limit = null, $page = null)
     {
-         $query = $this->getEntityManager()->createQuery('SELECT event FROM
+            $query = $this->getEntityManager()->createQuery('SELECT event FROM
             KIPublicationBundle:Event event,
             KIUserBundle:Club club,
             KIUserBundle:User user

Spacing +3 added lines, -3 removed lines

@@ -33,11 +33,11 @@ 
         ')
             ->setParameter('userId', $userId);
 
-        if($limit !== null && $limit > 0) {
+        if ($limit !== null && $limit > 0) {
             $query->setMaxResults($limit);
 
             if ($page !== null && $page > 0)
-                $query->setFirstResult(($page - 1) * $limit);
+                $query->setFirstResult(($page - 1)*$limit);
         }
 
         return $query->getResult();
@@ -101,7 +101,7 @@ 
     public function getOnlineUsers($delay = 30) {
         return $this->createQueryBuilder('u')
             ->where('u.lastConnect > :date')
-            ->setParameter('date', time() - $delay * 60)
+            ->setParameter('date', time() - $delay*60)
             ->getQuery()
             ->getResult();
     }

back/src/KI/UserBundle/Command/FacegameAchievementsCommand.php

Spacing +4 added lines, -4 removed lines

@@ -47,23 +47,23 @@
             $wrongAnswers = $facegame->getWrongAnswers();
             $duration = $facegame->getDuration();
 
-            if ($wrongAnswers == 0 && $promoGame == $promoUser - 1 && $duration < 60 * 1000) {
+            if ($wrongAnswers == 0 && $promoGame == $promoUser - 1 && $duration < 60*1000) {
 
                 $achievementCheck = new AchievementCheckEvent(Achievement::GAME_BEFORE, $user);
                 $dispatcher->dispatch('upont.achievement', $achievementCheck);
 
-            } else if ($wrongAnswers == 0 && $promoGame == $promoUser && $duration < 60 * 1000) {
+            } else if ($wrongAnswers == 0 && $promoGame == $promoUser && $duration < 60*1000) {
 
                 $achievementCheck = new AchievementCheckEvent(Achievement::GAME_SELF, $user);
                 $dispatcher->dispatch('upont.achievement', $achievementCheck);
 
-            } else if ($wrongAnswers == 0 && $promoGame == $promoUser + 1 && $duration < 60 * 1000) {
+            } else if ($wrongAnswers == 0 && $promoGame == $promoUser + 1 && $duration < 60*1000) {
 
                 $achievementCheck = new AchievementCheckEvent(Achievement::GAME_NEXT, $user);
                 $dispatcher->dispatch('upont.achievement', $achievementCheck);
 
             }
-            if ($wrongAnswers == 0 && $promoGame < $promoUser && $facegame->getHardcore() && $duration < 60 * 1000) {
+            if ($wrongAnswers == 0 && $promoGame < $promoUser && $facegame->getHardcore() && $duration < 60*1000) {
 
                 $achievementCheck = new AchievementCheckEvent(Achievement::GAME_OLD, $user);
                 $dispatcher->dispatch('upont.achievement', $achievementCheck);

back/src/KI/CoreBundle/Helper/FormHelper.php

Unused Use Statements -2 removed lines

@@ -5,9 +5,7 @@
 use Doctrine\ORM\EntityManager;
 use Symfony\Bundle\FrameworkBundle\Routing\Router;
 use Symfony\Component\Form\FormFactory;
-use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\RequestStack;
-use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 
 // Valide les formulaires pour une entité et affiche la réponse à la demande
 class FormHelper

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     public function formData($item, $method, $flush = true)
     {
         // On devine le formulaire à partir du chemin de la classe
-        $formName = str_replace('Entity', 'Form', get_class($item)) . 'Type';
+        $formName = str_replace('Entity', 'Form', get_class($item)).'Type';
         $form = $this->formFactory->create($formName, $item, ['method' => $method]);
         $form->handleRequest($this->request);
         $code = 400;

back/src/KI/CoreBundle/Helper/PaginateHelper.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Doctrine\ORM\EntityManager;
 use Doctrine\ORM\EntityRepository;
-use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpFoundation\Response;
 

Spacing +7 added lines, -7 removed lines

@@ -63,23 +63,23 @@
 
         // On compte le nombre total d'entrées dans la BDD
         $queryBuilder->select('count(o.id)');
-        foreach ($findBy as $key => $values){
+        foreach ($findBy as $key => $values) {
             $andCount = 0;
             $and = '';
-            foreach($values as $value){
-                if($andCount > 0){
+            foreach ($values as $value) {
+                if ($andCount > 0) {
                     $and .= ' OR ';
                 }
-                $and .= 'o.' . $key . ' = :' . $key . $andCount;
-                $queryBuilder->setParameter($key . $andCount, $value);
+                $and .= 'o.'.$key.' = :'.$key.$andCount;
+                $queryBuilder->setParameter($key.$andCount, $value);
 
                 $andCount++;
             }
             $queryBuilder->andWhere($and);
         }
 
-        foreach($sortBy as $field => $order){
-            $queryBuilder->addOrderBy('o.' . $field, $order);
+        foreach ($sortBy as $field => $order) {
+            $queryBuilder->addOrderBy('o.'.$field, $order);
         }
 
         $count = $queryBuilder->getQuery()->getSingleScalarResult();

back/src/KI/DvpBundle/Command/SendOrdersCommand.php

Unused Use Statements +1 added lines, -6 removed lines

@@ -2,17 +2,12 @@
 
 namespace KI\DvpBundle\Command;
 
-use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
-use Doctrine\ORM\NoResultException;
 use KI\DvpBundle\Entity\BasketDate;;
+use Swift_Message;
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
-use Swift_Message;
-
 
 class SendOrdersCommand extends ContainerAwareCommand
 {

Spacing +5 added lines, -5 removed lines

@@ -4,7 +4,7 @@ 
 
 use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
 use Doctrine\ORM\NoResultException;
-use KI\DvpBundle\Entity\BasketDate;;
+use KI\DvpBundle\Entity\BasketDate; ;
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -39,13 +39,13 @@ 
          */
         $basketDate = $query->getOneOrNullResult();
 
-        if($basketDate) {
+        if ($basketDate) {
 
             $orders = $basketDate->getOrders();
 
             // Envoi du mail
             $message = Swift_Message::newInstance()
-                ->setSubject('Commandes du ' . $basketDate->getDateRetrieve()->format('l d F Y'))
+                ->setSubject('Commandes du '.$basketDate->getDateRetrieve()->format('l d F Y'))
                 ->setFrom('noreply@upont.enpc.fr')
                 ->setTo('paniersprimeur@upont.enpc.fr')
                 ->setBody($this->getContainer()->get('twig')->render('KIDvpBundle::basket-orders.html.twig', [
@@ -72,7 +72,7 @@ 
 
         $count = $query->getSingleScalarResult();
 
-        while($count < 4) {
+        while ($count < 4) {
             $prevDate->modify('next thursday');
 
             $dateExists = $query = $em->createQuery('SELECT COUNT(date.id) 
@@ -81,7 +81,7 @@ 
                 ->setParameter('dateAdding', $prevDate)
                 ->getSingleScalarResult();
 
-            if($dateExists == 0) {
+            if ($dateExists == 0) {
                 $nextBasketDate = new BasketDate();
                 $nextBasketDate->setDateRetrieve($prevDate);
                 $em->persist($nextBasketDate);

back/src/KI/DvpBundle/Controller/BasketDatesController.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -3,12 +3,11 @@
 namespace KI\DvpBundle\Controller;
 
 use KI\CoreBundle\Controller\ResourceController;
+use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
-use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 
 class BasketDatesController extends ResourceController
 {

back/src/KI/DvpBundle/Controller/BasketsController.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -3,11 +3,10 @@
 namespace KI\DvpBundle\Controller;
 
 use KI\CoreBundle\Controller\ResourceController;
+use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
-use Nelmio\ApiDocBundle\Annotation\ApiDoc;
 use Symfony\Component\DependencyInjection\ContainerInterface;
-use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 
 class BasketsController extends ResourceController
 {

back/src/KI/UserBundle/Controller/UsersController.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,6 +2,7 @@ 
 
 namespace KI\UserBundle\Controller;
 
+use Exception;
 use KI\CoreBundle\Controller\ResourceController;
 use KI\UserBundle\Entity\Achievement;
 use KI\UserBundle\Entity\User;
@@ -13,7 +14,6 @@ 
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 use Symfony\Component\Security\Core\Exception\AccessDeniedException;
-use Exception;
 
 class UsersController extends ResourceController
 {

Spacing +6 added lines, -6 removed lines

@@ -221,7 +221,7 @@ 
             throw new BadRequestHttpException('Cet utilisateur existe déjà.');
 
         // Si le login existe déjà, on ajoute une lettre du prénom
-        $login = strtolower(str_replace(' ', '-', substr($this->stripAccents($lastName), 0, 7) . $this->stripAccents($firstName)[0]));
+        $login = strtolower(str_replace(' ', '-', substr($this->stripAccents($lastName), 0, 7).$this->stripAccents($firstName)[0]));
         $i = 1;
         while (count($repo->findByUsername($login)) > 0) {
             if (isset($firstName[$i]))
@@ -278,9 +278,9 @@ 
         }
 
         // On récupère le contenu du fichier
-        $path = __DIR__ . '/../../../../web/uploads/tmp/';
+        $path = __DIR__.'/../../../../web/uploads/tmp/';
         $file->move($path, 'users.list');
-        $list = fopen($path . 'users.list', 'r+');
+        $list = fopen($path.'users.list', 'r+');
         if ($list === false)
             throw new BadRequestHttpException('Erreur lors de l\'upload du fichier');
 
@@ -290,7 +290,7 @@ 
         while (!feof($list)) {
             // On enlève le caractère de fin de ligne
             $line = str_replace(["\r", "\n"], ['', ''], fgets($list));
-            if(empty($line))
+            if (empty($line))
                 continue;
 
             $gender = $login = $firstName = $lastName = $email = $promo = $department = $origin = null;
@@ -306,7 +306,7 @@ 
                 $e[] = 'Adresse mail non utilisable';
 
             if (count($e) > 0) {
-                $fails[] = $line . ' : ' . implode(', ', $e);
+                $fails[] = $line.' : '.implode(', ', $e);
             } else {
 
                 /**
@@ -333,7 +333,7 @@ 
                 }
                 $user->setGender($gender);
 
-                $success[] = $firstName . ' ' . $lastName;
+                $success[] = $firstName.' '.$lastName;
             }
         }
 

Doc Comments +3 added lines

@@ -343,6 +343,9 @@
         ], 201);
     }
 
+    /**
+     * @return string
+     */
     private function stripAccents($string)
     {
         return str_replace(