allejo / bzion

controllers/PlayerController.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 
 use BZIon\Form\Creator\PlayerAdminNotesFormCreator as FormCreator;
-use Carbon\Carbon;
 use Symfony\Component\Form\Form;
 use Symfony\Component\HttpFoundation\Request;
 

src/Event/Event.php

Doc Comments +1 added lines, -1 removed lines

@@ -145,7 +145,7 @@
      *
      * @param mixed    $players A single player/ID or a player/ID list
      * @param string   $type   The type of the event
-     * @param null|\Player|int $except A player who should not receive a notification
+     * @param \Player $except A player who should not receive a notification
      * @param \Player $except
      */
     protected function doNotify($players, $type, $except = null)

models/Match.php

Doc Comments +1 added lines, -1 removed lines

@@ -1326,7 +1326,7 @@
      *
      * @param int[]|Player[] $players
      *
-     * @return float|int
+     * @return integer
      */
     private static function getAveragePlayerElo($players)
     {

models/Player.php

Doc Comments +3 added lines, -2 removed lines

@@ -621,7 +621,7 @@ 
 
     /**
      * Get all of the callsigns a player has used to log in to the website
-     * @return string[] An array containing all of the past callsigns recorded for a player
+     * @return integer[] An array containing all of the past callsigns recorded for a player
      */
     public function getPastCallsigns()
     {
@@ -1176,7 +1176,7 @@ 
 
     /**
      * Get the (victory/total matches) ratio of the player
-     * @return float
+     * @return integer
      */
     public function getMatchWinRatio()
     {
@@ -1309,6 +1309,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $prefix
      */
     public static function getEagerColumns($prefix = null)
     {

controllers/LeagueOverseerHookController.php

Doc Comments +1 added lines, -1 removed lines

@@ -252,7 +252,7 @@
      * @param string $parameter The query parameter to get and split
      * @param string $delimiter A delimiter for splitting the string
      *
-     * @return array
+     * @return string[]
      */
     private function explodeQueryParam($parameter, $delimiter = ',')
     {

src/Composer/ScriptHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
      * booting
      *
      * @param $event Event       Composer's event
-     * @param $env   string|null The environment to clear the cache for, 'all'
+     * @param string $env   string|null The environment to clear the cache for, 'all'
      *                           to clear the cache for all environments, null
      *                           to pick an environment based on command line
      *                           arguments (defaults to 'all')
@@ -270,7 +270,7 @@ 
      * Get the database's configuration
      *
      * @param  bool    $testing Whether to retrieve the test database credentials
-     * @return array|null The configuration as defined in the config.yml file, null if no configuration was found
+     * @return IOInterface The configuration as defined in the config.yml file, null if no configuration was found
      */
     public static function getDatabaseConfig($testing = false)
     {

models/Ban.php

Doc Comments +1 added lines, -1 removed lines

@@ -358,7 +358,7 @@
      *
      * @param int        $playerID    The ID of the victim of the ban
      * @param int        $authorID    The ID of the player responsible for the ban
-     * @param mixed|null $expiration  The expiration of the ban (set to NULL for permanent ban)
+     * @param TimeDate|null $expiration  The expiration of the ban (set to NULL for permanent ban)
      * @param string     $reason      The full reason for the ban (supports markdown)
      * @param string     $serverMsg   A summary of the ban to be displayed on server banlists (max 150 characters)
      * @param string[]   $ipAddresses An array of IPs that have been banned

src/Service.php

Doc Comments +5 added lines, -2 removed lines

@@ -66,6 +66,9 @@ 
         self::$qbConfig = $config;
     }
 
+    /**
+     * @return null|Connection
+     */
     public static function getQueryBuilderConnection()
     {
         if (!self::$qbConnection) {
@@ -119,7 +122,7 @@ 
      * Gets a parameter
      *
      * @param  string $name The parameter name
-     * @return mixed  The parameter value
+     * @return string  The parameter value
      */
     public static function getParameter($name)
     {
@@ -229,7 +232,7 @@ 
     }
 
     /**
-     * @param $kernel
+     * @param AppKernel $kernel
      */
     public static function setKernel($kernel)
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 use BZIon\Cache\ModelCache;
 use Pixie\Connection;
-use Pixie\QueryBuilder\QueryBuilderHandler;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\EventDispatcher\EventDispatcher;
 use Symfony\Component\Form\FormFactory;

src/QueryBuilder/QueryBuilderFlex.php

Doc Comments +2 added lines, -1 removed lines

@@ -81,6 +81,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param integer $limit
      */
     public function limit($limit)
     {
@@ -189,7 +190,7 @@ 
      *
      * @throws \Pixie\Exception
      *
-     * @return array
+     * @return Model[]
      */
     public function getModels($fastFetch = false)
     {