Completed
Pull Request — master (#19)
by Matthew
02:56
created
src/AppBundle/Controller/AdminScoreController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -5,7 +5,6 @@
 block discarded – undo
5 5
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
6 6
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
7 7
 use Symfony\Component\HttpFoundation\Request;
8
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
9 8
 use DateTime;
10 9
 
11 10
 class AdminScoreController extends Controller
Please login to merge, or discard this patch.
src/AppBundle/Controller/ArticleInfoController.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -380,7 +380,6 @@
 block discarded – undo
380 380
 
381 381
     /**
382 382
      * Get the size of the diff
383
-     * @param  int $rev The index of the revision within $this->pageHistory
384 383
      * @return int Size of the diff
385 384
      */
386 385
     private function getDiffSize($revIndex)
Please login to merge, or discard this patch.
Unused Use Statements   -4 removed lines patch added patch discarded remove patch
@@ -6,11 +6,7 @@
 block discarded – undo
6 6
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
7 7
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
8 8
 use Symfony\Component\HttpFoundation\Request;
9
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
10 9
 use Symfony\Component\DependencyInjection\ContainerInterface;
11
-use AppBundle\Helper\Apihelper;
12
-use AppBundle\Helper\PageviewsHelper;
13
-use AppBundle\Helper\AutomatedEditsHelper;
14 10
 
15 11
 class ArticleInfoController extends Controller
16 12
 {
Please login to merge, or discard this patch.
src/AppBundle/Controller/DefaultController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
6 6
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
7
-use Symfony\Component\HttpFoundation\Request;
8 7
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
9 8
 
10 9
 class DefaultController extends Controller
Please login to merge, or discard this patch.
src/AppBundle/Controller/PagesController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -5,7 +5,6 @@
 block discarded – undo
5 5
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
6 6
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
7 7
 use Symfony\Component\HttpFoundation\Request;
8
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
9 8
 
10 9
 class PagesController extends Controller
11 10
 {
Please login to merge, or discard this patch.
src/AppBundle/Controller/SimpleEditCounterController.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -4,9 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
6 6
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
7
-use Symfony\Component\Config\Definition\Exception\Exception;
8 7
 use Symfony\Component\HttpFoundation\Request;
9
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
10 8
 
11 9
 class SimpleEditCounterController extends Controller
12 10
 {
Please login to merge, or discard this patch.
src/AppBundle/Helper/LabsHelper.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -25,6 +25,9 @@
 block discarded – undo
25 25
         $this->container = $container;
26 26
     }
27 27
 
28
+    /**
29
+     * @param string $tool
30
+     */
28 31
     public function checkEnabled($tool)
29 32
     {
30 33
         if (!$this->container->getParameter("enable.$tool")) {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -5,7 +5,6 @@
 block discarded – undo
5 5
 use Symfony\Component\Config\Definition\Exception\Exception;
6 6
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
7 7
 use Symfony\Component\DependencyInjection\ContainerInterface;
8
-use Symfony\Component\VarDumper\VarDumper;
9 8
 
10 9
 class LabsHelper
11 10
 {
Please login to merge, or discard this patch.
var/SymfonyRequirements.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -223,7 +223,7 @@  discard block
 block discarded – undo
223 223
      * Adds a mandatory requirement in form of a php.ini configuration.
224 224
      *
225 225
      * @param string        $cfgName           The configuration name used for ini_get()
226
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
226
+     * @param boolean|string $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
227 227
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
228 228
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
229 229
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
      * Adds an optional recommendation in form of a php.ini configuration.
242 242
      *
243 243
      * @param string        $cfgName           The configuration name used for ini_get()
244
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
244
+     * @param string|false $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
245 245
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
246 246
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
247 247
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.
Please login to merge, or discard this patch.
src/AppBundle/Helper/EditCounterHelper.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@
 block discarded – undo
146 146
 
147 147
     /**
148 148
      *
149
-     * @param $username
149
+     * @param string $username
150 150
      * @return integer
151 151
      */
152 152
     public function getPageCounts($username, $totalRevisions)
Please login to merge, or discard this patch.
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -2,12 +2,9 @@
 block discarded – undo
2 2
 
3 3
 namespace AppBundle\Helper;
4 4
 
5
-use AppBundle\Twig\AppExtension;
6
-use DateInterval;
7 5
 use Doctrine\DBAL\Connection;
8 6
 use Exception;
9 7
 use Symfony\Component\DependencyInjection\Container;
10
-use Symfony\Component\VarDumper\VarDumper;
11 8
 
12 9
 class EditCounterHelper extends HelperBase
13 10
 {
Please login to merge, or discard this patch.
src/AppBundle/Helper/AutomatedEditsHelper.php 1 patch
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -4,10 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use Doctrine\DBAL\Connection;
6 6
 use Psr\Cache\CacheItemPoolInterface;
7
-use Symfony\Component\Config\Definition\Exception\Exception;
8
-use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
9 7
 use Symfony\Component\DependencyInjection\ContainerInterface;
10
-use Symfony\Component\VarDumper\VarDumper;
11 8
 
12 9
 class AutomatedEditsHelper
13 10
 {
Please login to merge, or discard this patch.