fix style

Author: IonBazan

src/Aggregation/Stage/AbstractReplace.php

@@ -35,11 +35,6 @@ protected function getReplaceExpression(): array|string
         return $this->expression !== null ? $this->convertExpression($this->expression) : $this->expr->getExpression();
     }
 
-    /**
-     * @param mixed[]|string|mixed $expression
-     *
-     * @return mixed[]|string|mixed
-     */
     private function convertExpression(mixed $expression): mixed
     {
         if (is_array($expression)) {

src/Aggregation/Stage/Bucket/AbstractOutput.php

@@ -25,7 +25,7 @@ public function __construct(Builder $builder, protected Stage\AbstractBucket $bu
     {
         parent::__construct($builder);
 
-        $this->expr   = $builder->expr();
+        $this->expr = $builder->expr();
     }
 
     public function getExpression(): array
@@ -39,8 +39,6 @@ public function getExpression(): array
      * @see https://docs.mongodb.com/manual/meta/aggregation-quick-reference/#aggregation-expressions
      * @see Expr::expression
      *
-     * @param mixed|Expr $value
-     *
      * @return $this
      */
     public function expression(mixed $value): static
@@ -55,8 +53,6 @@ public function expression(mixed $value): static
      *
      * @see Expr::field
      *
-     * @param string $fieldName
-     *
      * @return $this
      */
     public function field(string $fieldName): static

src/Aggregation/Stage/MatchStage.php

@@ -124,8 +124,6 @@ public function elemMatch(array|Expr $expression): static
      * Specify an equality match for the current field.
      *
      * @see Expr::equals()
-     *
-     * @param mixed $value
      */
     public function equals(mixed $value): static
     {
@@ -294,8 +292,6 @@ public function getExpression(): ?array
      *
      * @see Expr::gt()
      * @see https://docs.mongodb.com/manual/reference/operator/gt/
-     *
-     * @param mixed $value
      */
     public function gt(mixed $value): static
     {
@@ -309,8 +305,6 @@ public function gt(mixed $value): static
      *
      * @see Expr::gte()
      * @see https://docs.mongodb.com/manual/reference/operator/gte/
-     *
-     * @param mixed $value
      */
     public function gte(mixed $value): static
     {
@@ -361,8 +355,6 @@ public function language(string $language): static
      *
      * @see Expr::lte()
      * @see https://docs.mongodb.com/manual/reference/operator/lte/
-     *
-     * @param mixed $value
      */
     public function lt(mixed $value): static
     {
@@ -376,8 +368,6 @@ public function lt(mixed $value): static
      *
      * @see Expr::lte()
      * @see https://docs.mongodb.com/manual/reference/operator/lte/
-     *
-     * @param mixed $value
      */
     public function lte(mixed $value): static
     {
@@ -391,9 +381,6 @@ public function lte(mixed $value): static
      *
      * @see Expr::mod()
      * @see https://docs.mongodb.com/manual/reference/operator/mod/
-     *
-     * @param float|int $divisor
-     * @param float|int $remainder
      */
     public function mod(float|int $divisor, float|int $remainder = 0): static
     {
@@ -425,8 +412,6 @@ public function not(array|Expr $expression): static
      *
      * @see Expr::notEqual()
      * @see https://docs.mongodb.com/manual/reference/operator/ne/
-     *
-     * @param mixed $value
      */
     public function notEqual(mixed $value): static
     {
@@ -457,9 +442,6 @@ public function notIn(array $values): static
      * and $lt criteria on the upper bound. The upper bound is not inclusive.
      *
      * @see Expr::range()
-     *
-     * @param mixed $start
-     * @param mixed $end
      */
     public function range(mixed $start, mixed $end): static
     {
@@ -510,8 +492,6 @@ public function text(string $search): static
      *
      * @see Expr::type()
      * @see https://docs.mongodb.com/manual/reference/operator/type/
-     *
-     * @param int|string $type
      */
     public function type(int|string $type): static
     {

src/DocumentManager.php

@@ -605,7 +605,6 @@ public function flush(array $options = []): void
      * has its identifier populated. Otherwise a proxy is returned that automatically
      * loads itself on first access.
      *
-     * @param mixed           $identifier
      * @param class-string<T> $documentName
      *
      * @return T
@@ -705,8 +704,6 @@ public function clear(): void
      * Closes the DocumentManager. All documents that are currently managed
      * by this DocumentManager become detached. The DocumentManager may no longer
      * be used after it is closed.
-     *
-     * @return void
      */
     public function close(): void
     {
@@ -717,8 +714,6 @@ public function close(): void
     /**
      * Determines whether a document instance is managed in this DocumentManager.
      *
-     * @param object $object
-     *
      * @return bool TRUE if this DocumentManager currently manages the given document, FALSE otherwise.
      *
      * @throws InvalidArgumentException When the $object param is not an object.

src/Mapping/ClassMetadata.php

@@ -489,7 +489,7 @@
     /**
      * READ-ONLY: The name of the mongo collection the document is mapped to.
      */
-    public string $collection;
+    public ?string $collection = null;
 
     /**
      * READ-ONLY: The name of the GridFS bucket the document is mapped to.
@@ -2207,9 +2207,7 @@ public function getTypeOfField(string $fieldName): ?string
         return $this->fieldMappings[$fieldName]['type'] ?? null;
     }
 
-    /**
-     * @return class-string|null
-     */
+    /** @return class-string|null */
     public function getAssociationTargetClass(string $assocName): ?string
     {
         if (! isset($this->associationMappings[$assocName])) {

src/Mapping/Driver/SimplifiedXmlDriver.php

@@ -13,6 +13,9 @@ class SimplifiedXmlDriver extends XmlDriver
 {
     public const string DEFAULT_FILE_EXTENSION = '.mongodb-odm.xml';
 
+    /**
+     * @param string[]|string $prefixes
+     */
     public function __construct(array|string $prefixes, ?string $fileExtension = self::DEFAULT_FILE_EXTENSION)
     {
         $locator = new SymfonyFileLocator((array) $prefixes, $fileExtension);

src/Persisters/DocumentPersister.php

@@ -767,7 +767,7 @@ private function loadReferenceManyCollectionOwningSide(PersistentCollectionInter
             $class           = $this->dm->getClassMetadata($className);
             $mongoCollection = $this->dm->getDocumentCollection($className);
             $criteria        = $this->cm->merge(
-                ['_id' => ['$in' => array_values($ids)]],
+                ['_id' => ['$in' => $ids]],
                 $this->dm->getFilterCollection()->getFilterCriteria($class),
                 $mapping['criteria'] ?? [],
             );

src/Query/Builder.php

@@ -25,8 +25,6 @@
 use function func_get_args;
 use function in_array;
 use function is_array;
-use function is_bool;
-use function is_callable;
 use function is_string;
 use function strtolower;
 
@@ -186,8 +184,6 @@ public function addOr(array|Expr $expression, array|Expr ...$expressions): self
      * @see Expr::addToSet()
      * @see https://docs.mongodb.com/manual/reference/operator/addToSet/
      * @see https://docs.mongodb.com/manual/reference/operator/each/
-     *
-     * @param mixed|Expr $valueOrExpression
      */
     public function addToSet(mixed $valueOrExpression): self
     {
@@ -811,8 +807,6 @@ public function in(array $values): self
      *
      * @see Expr::inc()
      * @see https://docs.mongodb.com/manual/reference/operator/inc/
-     *
-     * @param float|int $value
      */
     public function inc(float|int $value): self
     {
@@ -932,9 +926,6 @@ public function min(mixed $value): self
      *
      * @see Expr::mod()
      * @see https://docs.mongodb.com/manual/reference/operator/mod/
-     *
-     * @param float|int $divisor
-     * @param float|int $remainder
      */
     public function mod(float|int $divisor, float|int $remainder = 0): self
     {
@@ -950,8 +941,6 @@ public function mod(float|int $divisor, float|int $remainder = 0): self
      *
      * @see Expr::mul()
      * @see https://docs.mongodb.com/manual/reference/operator/update/mul/
-     *
-     * @param float|int $value
      */
     public function mul(float|int $value): self
     {
@@ -971,7 +960,6 @@ public function mul(float|int $value): self
      * @see https://docs.mongodb.com/manual/reference/operator/near/
      *
      * @param float|array<string, mixed>|Point $x
-     * @param float|null                       $y
      */
     public function near(float|array|Point $x, ?float $y = null, ?float $minDistance = null, ?float $maxDistance = null): self
     {
@@ -991,7 +979,6 @@ public function near(float|array|Point $x, ?float $y = null, ?float $minDistance
      * @see https://docs.mongodb.com/manual/reference/operator/nearSphere/
      *
      * @param float|array<string, mixed>|Point $x
-     * @param float|null                       $y
      */
     public function nearSphere(float|array|Point $x, ?float $y = null, ?float $minDistance = null, ?float $maxDistance = null): self
     {
@@ -1007,8 +994,6 @@ public function nearSphere(float|array|Point $x, ?float $y = null, ?float $minDi
      *
      * @see Expr::not()
      * @see https://docs.mongodb.com/manual/reference/operator/not/
-     *
-     * @param array|Expr|mixed $valueOrExpression
      */
     public function not(mixed $valueOrExpression): self
     {
@@ -1123,8 +1108,6 @@ public function prime(bool|callable $primer = true): self
      *
      * @see Expr::pull()
      * @see https://docs.mongodb.com/manual/reference/operator/pull/
-     *
-     * @param mixed|Expr $valueOrExpression
      */
     public function pull(mixed $valueOrExpression): self
     {
@@ -1165,8 +1148,6 @@ public function pullAll(array $values): self
      * @see https://docs.mongodb.com/manual/reference/operator/each/
      * @see https://docs.mongodb.com/manual/reference/operator/slice/
      * @see https://docs.mongodb.com/manual/reference/operator/sort/
-     *
-     * @param mixed|Expr $valueOrExpression
      */
     public function push(mixed $valueOrExpression): self
     {