Merge branch '2.3.x' into 3.0.x

* 2.3.x:
  Bump doctrine/.github from 5.0.0 to 5.0.1
  Set label of GH actions dependabot PRs to CI
  Bump ramsey/composer-install from 2 to 3 (#417)
  Bump actions/checkout from 3 to 4
  Pass CODECOV_TOKEN to the shared workflow
  Bump doctrine/.github from 1.5.0 to 5.0.0
  Enable dependabot
  Add tests
  Repeat closure type
  Add isNotNull to ExpressionBuilder (#408)

.github/dependabot.yml

@@ -0,0 +1,8 @@
+version: 2
+updates:
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    labels:
+      - "CI"

.github/workflows/coding-standards.yml

@@ -12,6 +12,6 @@ on:
 jobs:
   coding-standards:
     name: "Coding Standards"
-    uses: "doctrine/.github/.github/workflows/coding-standards.yml@1.5.0"
+    uses: "doctrine/.github/.github/workflows/[email protected].1"
     with:
       php-version: "8.1"

.github/workflows/composer-lint.yml

@@ -15,6 +15,6 @@ on:
 jobs:
   composer-lint:
     name: "Composer Lint"
-    uses: "doctrine/.github/.github/workflows/composer-lint.yml@1.5.0"
+    uses: "doctrine/.github/.github/workflows/[email protected].1"
     with:
       php-version: "8.1"

.github/workflows/continuous-integration.yml

@@ -13,6 +13,8 @@ on:
 jobs:
   phpunit:
     name: "PHPUnit"
-    uses: "doctrine/.github/.github/workflows/continuous-integration.yml@1.5.0"
+    uses: "doctrine/.github/.github/workflows/[email protected].1"
     with:
       php-versions: '["8.1"]'
+    secrets:
+      CODECOV_TOKEN: "${{ secrets.CODECOV_TOKEN }}"

.github/workflows/documentation.yml

@@ -21,7 +21,7 @@ jobs:
 
     steps:
       - name: "Checkout code"
-        uses: "actions/checkout@v3"
+        uses: "actions/checkout@v4"
 
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
@@ -36,7 +36,7 @@ jobs:
         run: "composer require --dev phpdocumentor/guides-cli --no-update"
 
       - name: "Install dependencies with Composer"
-        uses: "ramsey/composer-install@v2"
+        uses: "ramsey/composer-install@v3"
         with:
           dependency-versions: "highest"
 

.github/workflows/release-on-milestone-closed.yml

@@ -8,7 +8,7 @@ on:
 jobs:
   release:
     name: "Git tag, release & create merge-up PR"
-    uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@1.5.0"
+    uses: "doctrine/.github/.github/workflows/[email protected].1"
     secrets:
       GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }}
       GIT_AUTHOR_NAME: ${{ secrets.GIT_AUTHOR_NAME }}

.github/workflows/static-analysis.yml

@@ -12,6 +12,6 @@ on:
 jobs:
   static-analysis:
     name: "Static Analysis"
-    uses: "doctrine/.github/.github/workflows/static-analysis.yml@1.5.0"
+    uses: "doctrine/.github/.github/workflows/[email protected].1"
     with:
       php-version: "8.1"

docs/en/expression-builder.rst

@@ -124,6 +124,16 @@ isNull
 
     $collection->matching(new Criteria($expression));
 
+isNotNull
+---------
+
+.. code-block:: php
+    $expressionBuilder = Criteria::expr();
+
+    $expression = $expressionBuilder->isNotNull('foo');
+
+    $collection->matching(new Criteria($expression));
+
 in
 ---
 

phpstan.neon.dist

@@ -2,6 +2,7 @@ parameters:
     level: 8
     paths:
         - src
+        - tests/StaticAnalysis
 
     ignoreErrors:
         # https://github.com/phpstan/phpstan-src/pull/1289

psalm.xml.dist

@@ -9,6 +9,7 @@
 >
     <projectFiles>
         <directory name="src" />
+        <directory name="tests/StaticAnalysis" />
         <ignoreFiles>
             <directory name="vendor" />
             <directory name="src/Expr"/>

src/ArrayCollection.php

@@ -314,6 +314,8 @@ public function reduce(Closure $func, $initial = null): mixed
     /**
      * {@inheritDoc}
      *
+     * @psalm-param Closure(T, TKey):bool $p
+     *
      * @return static
      * @psalm-return static<TKey,T>
      */

src/Collection.php

@@ -90,14 +90,18 @@ public function map(Closure $func): self;
     /**
      * {@inheritDoc}
      *
+     * @psalm-param Closure(T, TKey):bool $p
+     *
      * @return Collection<mixed> A collection with the results of the filter operation.
      * @psalm-return Collection<TKey, T>
      */
     public function filter(Closure $p): self;
 
     /**
      * {@inheritDoc}
-
+     *
+     * @psalm-param Closure(TKey, T):bool $p
+     *
      * @return Collection<mixed>[] An array with two elements. The first element contains the collection
      *                      of elements where the predicate returned TRUE, the second element
      *                      contains the collection of elements where the predicate returned FALSE.

src/ExpressionBuilder.php

@@ -68,6 +68,11 @@ public function isNull(string $field): Comparison
         return new Comparison($field, Comparison::EQ, new Value(null));
     }
 
+    public function isNotNull(string $field): Comparison
+    {
+        return new Comparison($field, Comparison::NEQ, new Value(null));
+    }
+
     /** @param mixed[] $values */
     public function in(string $field, array $values): Comparison
     {

tests/ExpressionBuilderTest.php

@@ -114,6 +114,14 @@ public function testIsNull(): void
         self::assertEquals(Comparison::EQ, $expr->getOperator());
     }
 
+    public function testIsNotNull(): void
+    {
+        $expr = $this->builder->isNotNull('a');
+
+        self::assertInstanceOf(Comparison::class, $expr);
+        self::assertEquals(Comparison::NEQ, $expr->getOperator());
+    }
+
     public function testContains(): void
     {
         $expr = $this->builder->contains('a', 'b');

tests/StaticAnalysis/CustomCollection.php

@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\Tests\Common\Collections\StaticAnalysis;
+
+use Closure;
+use Doctrine\Common\Collections\ArrayCollection;
+use Doctrine\Common\Collections\Collection;
+
+/**
+ * @phpstan-template TKey of array-key
+ * @phpstan-template T of object
+ * @phpstan-implements Collection<TKey, T>
+ */
+abstract class CustomCollection implements Collection
+{
+    /** @var ArrayCollection<TKey, T> */
+    private ArrayCollection $collection;
+
+    /** @param ArrayCollection<TKey, T> $arrayCollection */
+    public function __construct(ArrayCollection $arrayCollection)
+    {
+        $this->collection = $arrayCollection;
+    }
+
+    /**
+     * @psalm-param Closure(T, TKey):bool $p
+     *
+     * @return Collection<TKey, T>
+     */
+    public function filter(Closure $p)
+    {
+        return $this->collection->filter($p);
+    }
+
+    /**
+     * @psalm-param Closure(TKey, T):bool $p
+     *
+     * @psalm-return array{0: Collection<TKey, T>, 1: Collection<TKey, T>}
+     */
+    public function partition(Closure $p)
+    {
+        return $this->collection->partition($p);
+    }
+}