allejo / bzion

models/Ban.php

Doc Comments +1 added lines, -1 removed lines

@@ -354,7 +354,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 so that it never expires)
+     * @param TimeDate|null $expiration      The expiration of the ban (set to NULL so that it never expires)
      * @param string      $reason          The full reason for the ban
      * @param string      $srvmsg          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

models/Role.php

Doc Comments +2 added lines, -2 removed lines

@@ -211,7 +211,7 @@ 
     /**
      * Add a permission to a role
      *
-     * @param string|Permission $perm_name The name of the permission to add
+     * @param Permission $perm_name The name of the permission to add
      *
      * @return bool Whether or not the operation was successful
      */
@@ -265,7 +265,7 @@ 
     /**
      * Revoke a permission from a role
      *
-     * @param string|Permission $perm_name The permission to remove
+     * @param Permission $perm_name The permission to remove
      *
      * @return bool Whether or not the operation was successful
      */

models/Player.php

Doc Comments +2 added lines, -2 removed lines

@@ -379,7 +379,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()
     {
@@ -871,7 +871,7 @@ 
 
     /**
      * Get the (victory/total matches) ratio of the player
-     * @return float
+     * @return integer
      */
     public function getMatchWinRatio()
     {

src/Model/Model.php

Doc Comments +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
     /**
      * Find if two objects represent the same model
      *
-     * @param  object   $model The model to compare
+     * @param  null|Model   $model The model to compare
      * @return bool
      */
     public function isSameAs($model)
@@ -168,7 +168,7 @@ 
      * Useful for form validation
      *
      * @param  string $property The name of the property to change
-     * @param  mixed  $value    The value of the property
+     * @param  string  $value    The value of the property
      * @return self
      */
     protected function inject($property, $value)

src/Model/BaseModel.php

Doc Comments +2 added lines, -2 removed lines

@@ -275,7 +275,7 @@ 
      * @param string          $column           The name of the column that should be tested
      * @param array|mixed     $possible_values  List of acceptable values
      * @param bool            $negate           Whether to search if the value of $column does NOT belong to the $possible_values array
-     * @param string|string[] $select           The name of the column(s) that the returned array should contain
+     * @param string $select           The name of the column(s) that the returned array should contain
      * @param string          $additional_query Additional parameters to be passed to the MySQL query (e.g. `WHERE id = 5`)
      * @param string          $table            The database table which will be used for queries
      *
@@ -342,7 +342,7 @@ 
      * Get a formatted string with a comma separated column list with table/alias prefixes if necessary.
      *
      * @param string|null $prefix  The table name or SQL alias to be prepend to these columns
-     * @param array       $columns The columns to format
+     * @param string[]       $columns The columns to format
      *
      * @return string
      */

src/QueryBuilder/QueryBuilder.php

Doc Comments +3 added lines, -2 removed lines

@@ -275,7 +275,7 @@ 
     /**
      * Request that a timestamp is before the specified time
      *
-     * @param string|TimeDate $time      The timestamp to compare to
+     * @param Carbon\Carbon $time      The timestamp to compare to
      * @param bool            $inclusive Whether to include the given timestamp
      * @param bool            $reverse   Whether to reverse the results
      *
@@ -368,7 +368,7 @@ 
     /**
      * Request that a specific model is not returned
      *
-     * @param  Model|int $model The ID or model you don't want to get
+     * @param  Player $model The ID or model you don't want to get
      * @return static
      */
     public function except($model)
@@ -821,6 +821,7 @@ 
 
     /**
      * Generates all the WHERE conditions for the query
+     * @param string $mode
      * @return string
      */
     private function createQueryConditions($mode)

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;
 

models/Match.php

Doc Comments +1 added lines, -2 removed lines

@@ -664,7 +664,6 @@ 
      * Set the server address of the server where this match took place
      *
      * @param  string|null $server The server hostname
-     * @param  int|null    $port   The server port
      * @return self
      */
     public function setServerAddress($server = null)
@@ -1099,7 +1098,7 @@ 
      *
      * @param int[] $players
      *
-     * @return float|int
+     * @return integer
      */
     private static function getAveragePlayerElo($players)
     {