]> git.immae.eu Git - github/wallabag/wallabag.git/commitdiff
Merge pull request #1383 from wallabag/cs
authorNicolas Lœuillet <nicolas@loeuillet.org>
Thu, 20 Aug 2015 06:22:39 +0000 (08:22 +0200)
committerNicolas Lœuillet <nicolas@loeuillet.org>
Thu, 20 Aug 2015 06:22:39 +0000 (08:22 +0200)
CS

22 files changed:
app/AppKernel.php
src/Wallabag/ApiBundle/Controller/WallabagRestController.php
src/Wallabag/ApiBundle/Security/Authentication/Provider/WsseProvider.php
src/Wallabag/ApiBundle/Security/Firewall/WsseListener.php
src/Wallabag/CoreBundle/Controller/EntryController.php
src/Wallabag/CoreBundle/Controller/SecurityController.php
src/Wallabag/CoreBundle/Controller/StaticController.php
src/Wallabag/CoreBundle/Controller/TagController.php
src/Wallabag/CoreBundle/DataFixtures/ORM/LoadConfigData.php
src/Wallabag/CoreBundle/DataFixtures/ORM/LoadEntryData.php
src/Wallabag/CoreBundle/DataFixtures/ORM/LoadUserData.php
src/Wallabag/CoreBundle/Entity/Tag.php
src/Wallabag/CoreBundle/Entity/User.php
src/Wallabag/CoreBundle/Filter/EntryFilterType.php
src/Wallabag/CoreBundle/Form/Type/ChangePasswordType.php
src/Wallabag/CoreBundle/Form/Type/ResetPasswordType.php
src/Wallabag/CoreBundle/ParamConverter/UsernameRssTokenConverter.php
src/Wallabag/CoreBundle/Repository/TagRepository.php
src/Wallabag/CoreBundle/Security/Authentication/Encoder/WallabagPasswordEncoder.php
src/Wallabag/CoreBundle/Service/Extractor.php
src/Wallabag/CoreBundle/Tests/Controller/EntryControllerTest.php
src/Wallabag/CoreBundle/Tests/Controller/TagControllerTest.php

index 953614c0f53bb09158506b1252bb14489825cc98..00206ddf21e9d25e881b862cda9b7afcee0f0fd5 100644 (file)
@@ -2,7 +2,6 @@
 
 use Symfony\Component\HttpKernel\Kernel;
 use Symfony\Component\Config\Loader\LoaderInterface;
-use Wallabag\ApiBundle\WallabagApiBundle;
 
 class AppKernel extends Kernel
 {
index 692a4ae0e5de37d449f85d60a5be1a87aed7c31f..02a6de64c9297d69809bdb4b71f32bee62f6fb25 100644 (file)
@@ -84,12 +84,12 @@ class WallabagRestController extends Controller
     public function getEntriesAction(Request $request)
     {
         $isArchived = $request->query->get('archive');
-        $isStarred  = $request->query->get('star');
-        $sort       = $request->query->get('sort', 'created');
-        $order      = $request->query->get('order', 'desc');
-        $page       = (int) $request->query->get('page', 1);
-        $perPage    = (int) $request->query->get('perPage', 30);
-        $tags       = $request->query->get('tags', []);
+        $isStarred = $request->query->get('star');
+        $sort = $request->query->get('sort', 'created');
+        $order = $request->query->get('order', 'desc');
+        $page = (int) $request->query->get('page', 1);
+        $perPage = (int) $request->query->get('perPage', 30);
+        $tags = $request->query->get('tags', []);
 
         $pager = $this
             ->getDoctrine()
@@ -99,7 +99,7 @@ class WallabagRestController extends Controller
         $pager->setCurrentPage($page);
         $pager->setMaxPerPage($perPage);
 
-        $pagerfantaFactory   = new PagerfantaFactory('page', 'perPage');
+        $pagerfantaFactory = new PagerfantaFactory('page', 'perPage');
         $paginatedCollection = $pagerfantaFactory->createRepresentation(
             $pager,
             new Route('api_get_entries', [], $absolute = true)
@@ -188,9 +188,9 @@ class WallabagRestController extends Controller
     {
         $this->validateUserAccess($entry->getUser()->getId(), $this->getUser()->getId());
 
-        $title      = $request->request->get('title');
+        $title = $request->request->get('title');
         $isArchived = $request->request->get('is_archived');
-        $isStarred  = $request->request->get('is_starred');
+        $isStarred = $request->request->get('is_starred');
 
         if (!is_null($title)) {
             $entry->setTitle($title);
index db73ae2a44093fa9c1f79848de6eaa21be3d9034..9bf8b377f6b6416f9d47f7b382f48f9865dd6efc 100644 (file)
@@ -17,7 +17,7 @@ class WsseProvider implements AuthenticationProviderInterface
     public function __construct(UserProviderInterface $userProvider, $cacheDir)
     {
         $this->userProvider = $userProvider;
-        $this->cacheDir     = $cacheDir;
+        $this->cacheDir = $cacheDir;
 
         // If cache directory does not exist we create it
         if (!is_dir($this->cacheDir)) {
index 5058783798e4cdf1576d9c7bd098f131e0bf0b7b..2fcbe014e67da4613cfb8ccbd908c4fe62298295 100644 (file)
@@ -36,9 +36,9 @@ class WsseListener implements ListenerInterface
         $token = new WsseUserToken();
         $token->setUser($matches[1]);
 
-        $token->digest   = $matches[2];
-        $token->nonce    = $matches[3];
-        $token->created  = $matches[4];
+        $token->digest = $matches[2];
+        $token->nonce = $matches[3];
+        $token->created = $matches[4];
 
         try {
             $authToken = $this->authenticationManager->authenticate($token);
index 93ce650e16abd2075fac5b46f0d4e68734ecec62..006fa396476019c3055ed9c83da58b6f542acdba 100644 (file)
@@ -136,9 +136,9 @@ class EntryController extends Controller
         return $this->render(
             'WallabagCoreBundle:Entry:entries.html.twig',
             array(
-                'form'          => $form->createView(),
-                'entries'       => $entries,
-                'currentPage'   => $page
+                'form' => $form->createView(),
+                'entries' => $entries,
+                'currentPage' => $page,
             )
         );
     }
@@ -178,9 +178,9 @@ class EntryController extends Controller
         return $this->render(
             'WallabagCoreBundle:Entry:entries.html.twig',
             array(
-                'form'          => $form->createView(),
-                'entries'       => $entries,
-                'currentPage'   => $page
+                'form' => $form->createView(),
+                'entries' => $entries,
+                'currentPage' => $page,
             )
         );
     }
@@ -220,9 +220,9 @@ class EntryController extends Controller
         return $this->render(
             'WallabagCoreBundle:Entry:entries.html.twig',
             array(
-                'form'          => $form->createView(),
-                'entries'       => $entries,
-                'currentPage'   => $page
+                'form' => $form->createView(),
+                'entries' => $entries,
+                'currentPage' => $page,
             )
         );
     }
@@ -265,7 +265,7 @@ class EntryController extends Controller
 
         $this->get('session')->getFlashBag()->add(
             'notice',
-            'Entry ' . ($entry->isArchived() ? 'archived' : 'unarchived')
+            'Entry '.($entry->isArchived() ? 'archived' : 'unarchived')
         );
 
         return $this->redirect($request->headers->get('referer'));
@@ -290,7 +290,7 @@ class EntryController extends Controller
 
         $this->get('session')->getFlashBag()->add(
             'notice',
-            'Entry ' . ($entry->isStarred() ? 'starred' : 'unstarred')
+            'Entry '.($entry->isStarred() ? 'starred' : 'unstarred')
         );
 
         return $this->redirect($request->headers->get('referer'));
index a61a898bf1d07598fb9f54d36c5b0519db7b1dab..f0a7ab6d8dd5126bd1dcdbd039b19b9334a2fbb0 100644 (file)
@@ -25,7 +25,7 @@ class SecurityController extends Controller
         return $this->render('WallabagCoreBundle:Security:login.html.twig', array(
             // last username entered by the user
             'last_username' => $session->get(SecurityContext::LAST_USERNAME),
-            'error'         => $error,
+            'error' => $error,
         ));
     }
 
index 09be1b74f87606d0cb0b064665c572db6a41ee16..3b844b44c500054cb47bd1412d9c7215399941d2 100644 (file)
@@ -10,13 +10,14 @@ class StaticController extends Controller
     /**
      * @Route("/howto", name="howto")
      */
-     public function howtoAction()
-     {
-         return $this->render(
-             'WallabagCoreBundle:Static:howto.html.twig',
-             array()
-         );
-     }
+    public function howtoAction()
+    {
+        return $this->render(
+            'WallabagCoreBundle:Static:howto.html.twig',
+            array()
+        );
+    }
+
     /**
      * @Route("/about", name="about")
      */
index e448cea1ebd181be873e1c2f6ca1e6e20794f7fb..a342ec0b91e21a44fa6c8ad7d3aa13a1c5c2817c 100644 (file)
@@ -23,9 +23,8 @@ class TagController extends Controller
         return $this->render(
             'WallabagCoreBundle:Tag:tags.html.twig',
             array(
-                'tags'       => $tags
+                'tags' => $tags,
             )
         );
     }
-
 }
index 3bb5c5f2acbb423dce8c612d470521c083cc9fa7..cb0c52c499394fd2c674f1b3c6d10cf05fa1635e 100644 (file)
@@ -10,7 +10,7 @@ use Wallabag\CoreBundle\Entity\Config;
 class LoadConfigData extends AbstractFixture implements OrderedFixtureInterface
 {
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function load(ObjectManager $manager)
     {
@@ -36,7 +36,7 @@ class LoadConfigData extends AbstractFixture implements OrderedFixtureInterface
     }
 
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function getOrder()
     {
index 547d6753e179135c5e6839d69caf63f6c1319990..6cd24d442081432cdad86c4c5d5a6026b307d9ee 100644 (file)
@@ -11,7 +11,7 @@ use Wallabag\CoreBundle\Entity\Tag;
 class LoadEntryData extends AbstractFixture implements OrderedFixtureInterface
 {
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function load(ObjectManager $manager)
     {
@@ -91,7 +91,7 @@ class LoadEntryData extends AbstractFixture implements OrderedFixtureInterface
     }
 
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function getOrder()
     {
index d99412f482e906f7be584716d5cee81e90c5f0a7..1c7f9dfae03947c19f7d2f2d2ea2d4d2118ddddc 100644 (file)
@@ -10,7 +10,7 @@ use Wallabag\CoreBundle\Entity\User;
 class LoadUserData extends AbstractFixture implements OrderedFixtureInterface
 {
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function load(ObjectManager $manager)
     {
@@ -38,7 +38,7 @@ class LoadUserData extends AbstractFixture implements OrderedFixtureInterface
     }
 
     /**
-     * {@inheritDoc}
+     * {@inheritdoc}
      */
     public function getOrder()
     {
index 92442df6ddb00ad823f43bc53f0c534d6be0760a..6f005314f69fd8e33a2e253a0d135bf74913171c 100644 (file)
@@ -48,7 +48,7 @@ class Tag
 
     public function __construct(User $user)
     {
-        $this->user    = $user;
+        $this->user = $user;
         $this->entries = new ArrayCollection();
     }
 
index 00eb808a626839c1e12d5c879f1d19d6a197878b..510a1594e2a38821619fb2ee6aa3ce544d647229 100644 (file)
@@ -123,9 +123,9 @@ class User implements AdvancedUserInterface, \Serializable
     public function __construct()
     {
         $this->isActive = true;
-        $this->salt     = md5(uniqid(null, true));
-        $this->entries  = new ArrayCollection();
-        $this->tags     = new ArrayCollection();
+        $this->salt = md5(uniqid(null, true));
+        $this->entries = new ArrayCollection();
+        $this->tags = new ArrayCollection();
     }
 
     /**
@@ -176,7 +176,7 @@ class User implements AdvancedUserInterface, \Serializable
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getSalt()
     {
@@ -184,7 +184,7 @@ class User implements AdvancedUserInterface, \Serializable
     }
 
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function getRoles()
     {
@@ -323,7 +323,7 @@ class User implements AdvancedUserInterface, \Serializable
         return $this->tags;
     }
     /**
-     * @inheritDoc
+     * {@inheritdoc}
      */
     public function eraseCredentials()
     {
index d1057fb56f7d9a4a6231fd11e2cdf70a9d6c71e9..771daef1799edb9c02637570a82b89877e93bbac 100644 (file)
@@ -5,7 +5,6 @@ namespace Wallabag\CoreBundle\Filter;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
-use Lexik\Bundle\FormFilterBundle\Filter\FilterOperands;
 use Lexik\Bundle\FormFilterBundle\Filter\Query\QueryInterface;
 
 class EntryFilterType extends AbstractType
@@ -17,26 +16,29 @@ class EntryFilterType extends AbstractType
             ->add('createdAt', 'filter_date_range', array(
                     'left_date_options' => array(
                         'attr' => array(
-                            'placeholder' => 'dd/mm/yyyy'),
+                            'placeholder' => 'dd/mm/yyyy',
+                        ),
                         'format' => 'dd/MM/yyyy',
-                        'widget' => 'single_text'
+                        'widget' => 'single_text',
                     ),
                     'right_date_options' => array(
                         'attr' => array(
-                            'placeholder' => 'dd/mm/yyyy'),
+                            'placeholder' => 'dd/mm/yyyy',
+                        ),
                         'format' => 'dd/MM/yyyy',
-                        'widget' => 'single_text'
-                )))
+                        'widget' => 'single_text',
+                ),
+            ))
             ->add('domainName', 'filter_text', array(
-                'apply_filter' => function (QueryInterface $filterQuery, $field, $values)
-                    {
+                'apply_filter' => function (QueryInterface $filterQuery, $field, $values) {
                         $value = $values['value'];
                         if (strlen($value) <= 3 || empty($value)) {
-                            return null;
+                            return;
                         }
                         $expression = $filterQuery->getExpr()->like($field, $filterQuery->getExpr()->literal('%'.$value.'%'));
+
                         return $filterQuery->createCondition($expression);
-                    }
+                    },
             ));
     }
 
@@ -48,8 +50,8 @@ class EntryFilterType extends AbstractType
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults(array(
-            'csrf_protection'   => false,
-            'validation_groups' => array('filtering')
+            'csrf_protection' => false,
+            'validation_groups' => array('filtering'),
         ));
     }
 }
index b4224e3df69df004f15e148afb10777abf8770d5..82e1954d1a871906c5220ff6a5bbf39edbf01e53 100644 (file)
@@ -19,7 +19,7 @@ class ChangePasswordType extends AbstractType
                 'type' => 'password',
                 'invalid_message' => 'The password fields must match.',
                 'required' => true,
-                'first_options'  => array('label' => 'New password'),
+                'first_options' => array('label' => 'New password'),
                 'second_options' => array('label' => 'Repeat new password'),
                 'constraints' => array(
                     new Constraints\Length(array(
index a5d683c46f36fc2b2943c928d640f71468e6f20f..38f1a1054f513cd775c626996f23968d6335413e 100644 (file)
@@ -15,7 +15,7 @@ class ResetPasswordType extends AbstractType
                 'type' => 'password',
                 'invalid_message' => 'The password fields must match.',
                 'required' => true,
-                'first_options'  => array('label' => 'New password'),
+                'first_options' => array('label' => 'New password'),
                 'second_options' => array('label' => 'Repeat new password'),
                 'constraints' => array(
                     new Constraints\Length(array(
index ea2bda1750db3584d9973df1037cd11f74d20499..2771cf116ca64d7be5e70a68410cfec847ad8e50 100644 (file)
@@ -67,7 +67,7 @@ class UsernameRssTokenConverter implements ParamConverterInterface
     public function apply(Request $request, ParamConverter $configuration)
     {
         $username = $request->attributes->get('username');
-        $rssToken  = $request->attributes->get('token');
+        $rssToken = $request->attributes->get('token');
 
         // Check, if route attributes exists
         if (null === $username || null === $rssToken) {
index 51f1cd42924b22a88eb879fd6b5b3b14cfff058f..9c4096072626273dd98372a69f95f85a07c61262 100644 (file)
@@ -11,7 +11,7 @@ class TagRepository extends EntityRepository
     /**
      * Find Tags.
      *
-     * @param int    $userId
+     * @param int $userId
      *
      * @return array
      */
index e7c81fc0ae08961bd9a34652abbca7042d565a7c..98b4e86b2222bd95a5db08068e1a35a95e830e1f 100644 (file)
@@ -52,7 +52,7 @@ class WallabagPasswordEncoder extends BasePasswordEncoder
         $digest = hash($this->algorithm, $salted, true);
 
         // "stretch" hash
-        for ($i = 1; $i < $this->iterations; $i++) {
+        for ($i = 1; $i < $this->iterations; ++$i) {
             $digest = hash($this->algorithm, $digest.$salted, true);
         }
 
index 961ac20ad7ab3c779f2bd836ae684fd40034c41a..4c067d3afef985fa5226c0359bc81260c2eb7eec 100644 (file)
@@ -33,8 +33,8 @@ final class Extractor
         $REAL = array();
         foreach ($GLOBALS as $key => $value) {
             if ($key != 'GLOBALS' && $key != '_SESSION' && $key != 'HTTP_SESSION_VARS') {
-                $GLOBALS[$key]  = array();
-                $REAL[$key]     = $value;
+                $GLOBALS[$key] = array();
+                $REAL[$key] = $value;
             }
         }
         // Saving and clearing session
index 8d013ec002c626b73b3c9fbd59b70ad374e4e380..86a19f616e351490be7e640ada6233e79d8afdf9 100644 (file)
@@ -252,7 +252,7 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $data = array(
             'entry_filter[readingTime][right_number]' => 11,
-            'entry_filter[readingTime][left_number]' => 11
+            'entry_filter[readingTime][left_number]' => 11,
         );
 
         $crawler = $client->submit($form, $data);
@@ -271,7 +271,7 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $data = array(
             'entry_filter[createdAt][left_date]' => date('d/m/Y'),
-            'entry_filter[createdAt][right_date]' => date('d/m/Y', strtotime("+1 day"))
+            'entry_filter[createdAt][right_date]' => date('d/m/Y', strtotime('+1 day')),
         );
 
         $crawler = $client->submit($form, $data);
@@ -280,13 +280,12 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $data = array(
             'entry_filter[createdAt][left_date]' => '01/01/1970',
-            'entry_filter[createdAt][right_date]' => '01/01/1970'
+            'entry_filter[createdAt][right_date]' => '01/01/1970',
         );
 
         $crawler = $client->submit($form, $data);
 
         $this->assertCount(0, $crawler->filter('div[class=entry]'));
-
     }
 
     public function testPaginationWithFilter()
@@ -318,7 +317,7 @@ class EntryControllerTest extends WallabagCoreTestCase
         $crawler = $client->request('GET', '/unread/list');
         $form = $crawler->filter('button[id=submit-filter]')->form();
         $data = array(
-            'entry_filter[domainName]' => 'monde'
+            'entry_filter[domainName]' => 'monde',
         );
 
         $crawler = $client->submit($form, $data);
@@ -326,7 +325,7 @@ class EntryControllerTest extends WallabagCoreTestCase
 
         $form = $crawler->filter('button[id=submit-filter]')->form();
         $data = array(
-            'entry_filter[domainName]' => 'wallabag'
+            'entry_filter[domainName]' => 'wallabag',
         );
 
         $crawler = $client->submit($form, $data);
index 34faf70976e6a818bde71af0a9779fbdaded7e67..4a43e04928124d1718673936d1322019e9d5932c 100644 (file)
@@ -3,7 +3,6 @@
 namespace Wallabag\CoreBundle\Tests\Controller;
 
 use Wallabag\CoreBundle\Tests\WallabagCoreTestCase;
-use Doctrine\ORM\AbstractQuery;
 
 class TagControllerTest extends WallabagCoreTestCase
 {