Remove redundant php-docs

Signed-off-by: Jefersson Nathan <[email protected]>
XTAIN · Jun 27, 2021 · e6d9f99 · e6d9f99
1 parent 244e7c7
commit e6d9f99
Show file tree

Hide file tree

Showing 13 changed files with 10 additions and 52 deletions.
diff --git a/lib/Doctrine/ORM/Cache/Lock.php b/lib/Doctrine/ORM/Cache/Lock.php
@@ -23,9 +23,6 @@
 use function time;
 use function uniqid;
 
-/**
- * Cache Lock
- */
 class Lock
 {
     /** @var string */
@@ -34,11 +31,7 @@ class Lock
     /** @var int */
     public $time;
 
-    /**
-     * @param string $value
-     * @param int    $time
-     */
-    public function __construct($value, $time = null)
+    public function __construct(string $value, ?int $time = null)
     {
         $this->value = $value;
         $this->time  = $time ? : time();

diff --git a/lib/Doctrine/ORM/Cache/Logging/CacheLoggerChain.php b/lib/Doctrine/ORM/Cache/Logging/CacheLoggerChain.php
@@ -24,9 +24,6 @@
 use Doctrine\ORM\Cache\EntityCacheKey;
 use Doctrine\ORM\Cache\QueryCacheKey;
 
-/**
- * Cache logger chain
- */
 class CacheLoggerChain implements CacheLogger
 {
     /** @var array<CacheLogger> */

diff --git a/lib/Doctrine/ORM/Cache/QueryCacheKey.php b/lib/Doctrine/ORM/Cache/QueryCacheKey.php
@@ -48,18 +48,13 @@ class QueryCacheKey extends CacheKey
      */
     public $timestampKey;
 
-    /**
-     * @param string $hash      Result cache id
-     * @param int    $lifetime  Query lifetime
-     * @param int    $cacheMode Query cache mode
-     */
     public function __construct(
-        $hash,
-        $lifetime = 0,
-        $cacheMode = Cache::MODE_NORMAL,
+        string $cacheId,
+        int $lifetime = 0,
+        int $cacheMode = Cache::MODE_NORMAL,
         ?TimestampCacheKey $timestampKey = null
     ) {
-        $this->hash         = $hash;
+        $this->hash         = $cacheId;
         $this->lifetime     = $lifetime;
         $this->cacheMode    = $cacheMode;
         $this->timestampKey = $timestampKey;

diff --git a/lib/Doctrine/ORM/Persisters/Collection/AbstractCollectionPersister.php b/lib/Doctrine/ORM/Persisters/Collection/AbstractCollectionPersister.php
@@ -47,11 +47,7 @@ abstract class AbstractCollectionPersister implements CollectionPersister
      */
     protected $platform;
 
-    /**
-     * The quote strategy.
-     *
-     * @var QuoteStrategy
-     */
+    /** @var QuoteStrategy */
     protected $quoteStrategy;
 
     /**

diff --git a/lib/Doctrine/ORM/Persisters/Collection/CollectionPersister.php b/lib/Doctrine/ORM/Persisters/Collection/CollectionPersister.php
@@ -24,7 +24,6 @@
 use Doctrine\ORM\PersistentCollection;
 
 /**
- * Collection persister interface
  * Define the behavior that should be implemented by all collection persisters.
  */
 interface CollectionPersister

diff --git a/lib/Doctrine/ORM/PessimisticLockException.php b/lib/Doctrine/ORM/PessimisticLockException.php
@@ -22,11 +22,6 @@
 
 use Doctrine\ORM\Exception\ORMException;
 
-/**
- * Pessimistic Lock Exception
- *
- * @link        www.doctrine-project.com
- */
 class PessimisticLockException extends ORMException
 {
     /**

diff --git a/lib/Doctrine/ORM/Query/AST/BetweenExpression.php b/lib/Doctrine/ORM/Query/AST/BetweenExpression.php
@@ -20,11 +20,6 @@
 
 namespace Doctrine\ORM\Query\AST;
 
-/**
- * Description of BetweenExpression.
- *
- * @link    www.doctrine-project.org
- */
 class BetweenExpression extends Node
 {
     /** @var ArithmeticExpression */

diff --git a/lib/Doctrine/ORM/Query/AST/Functions/ConcatFunction.php b/lib/Doctrine/ORM/Query/AST/Functions/ConcatFunction.php
@@ -57,7 +57,7 @@ public function getSql(SqlWalker $sqlWalker)
             $args[] = $sqlWalker->walkStringPrimary($expression);
         }
 
-        return call_user_func_array([$platform, 'getConcatExpression'], $args);
+        return $platform->getConcatExpression(...$args);
     }
 
     /**

diff --git a/lib/Doctrine/ORM/Query/AST/GroupByClause.php b/lib/Doctrine/ORM/Query/AST/GroupByClause.php
@@ -20,11 +20,8 @@
 
 namespace Doctrine\ORM\Query\AST;
 
-/**
- * Description of GroupByClause.
- *
- * @link    www.doctrine-project.org
- */
+use Doctrine\ORM\Query\Expr\GroupBy;
+
 class GroupByClause extends Node
 {
     /** @var mixed[] */

diff --git a/lib/Doctrine/ORM/Query/AST/HavingClause.php b/lib/Doctrine/ORM/Query/AST/HavingClause.php
@@ -20,11 +20,6 @@
 
 namespace Doctrine\ORM\Query\AST;
 
-/**
- * Description of HavingClause.
- *
- * @link    www.doctrine-project.org
- */
 class HavingClause extends Node
 {
     /** @var ConditionalExpression */

diff --git a/lib/Doctrine/ORM/Query/Filter/SQLFilter.php b/lib/Doctrine/ORM/Query/Filter/SQLFilter.php
@@ -59,8 +59,6 @@ abstract class SQLFilter
     private $parameters = [];
 
     /**
-     * Constructs the SQLFilter object.
-     *
      * @param EntityManagerInterface $em The entity manager.
      */
     final public function __construct(EntityManagerInterface $em)

diff --git a/lib/Doctrine/ORM/Query/QueryExpressionVisitor.php b/lib/Doctrine/ORM/Query/QueryExpressionVisitor.php
@@ -54,8 +54,6 @@ class QueryExpressionVisitor extends ExpressionVisitor
     private $parameters = [];
 
     /**
-     * Constructor
-     *
      * @param mixed[] $queryAliases
      */
     public function __construct($queryAliases)

diff --git a/tests/Doctrine/Tests/ORM/Mapping/AnnotationDriverTest.php b/tests/Doctrine/Tests/ORM/Mapping/AnnotationDriverTest.php
@@ -175,7 +175,7 @@ public function testInvalidMappedSuperClassWithManyToManyAssociation(): void
             "mapped superclass 'Doctrine\Tests\ORM\Mapping\InvalidMappedSuperClass#users'"
         );
 
-        $usingInvalidMsc = $factory->getMetadataFor(UsingInvalidMappedSuperClass::class);
+        $factory->getMetadataFor(UsingInvalidMappedSuperClass::class);
     }
 
     /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -57,7 +57,7 @@ public function getSql(SqlWalker $sqlWalker) @@
                 $args[] = $sqlWalker->walkStringPrimary($expression);
             }
-            return call_user_func_array([$platform, 'getConcatExpression'], $args);
+            return $platform->getConcatExpression(...$args);
         }
         /**
@@ Expand Down @@