[11.x] Added generics to paginators

src/Illuminate/Contracts/Pagination/LengthAwarePaginator.php

@@ -2,6 +2,13 @@
 
 namespace Illuminate\Contracts\Pagination;
 
+/**
+ * @template TKey of array-key
+ *
+ * @template-covariant TValue
+ *
+ * @extends Paginator<TKey, TValue>
+ */
 interface LengthAwarePaginator extends Paginator
 {
     /**

src/Illuminate/Contracts/Pagination/Paginator.php

@@ -2,6 +2,11 @@
 
 namespace Illuminate\Contracts\Pagination;
 
+/**
+ * @template TKey of array-key
+ *
+ * @template-covariant TValue
+ */
 interface Paginator
 {
     /**
@@ -46,7 +51,7 @@ public function previousPageUrl();
     /**
      * Get all of the items being paginated.
      *
-     * @return array
+     * @return array<TKey, TValue>
      */
     public function items();
 

src/Illuminate/Pagination/AbstractPaginator.php

@@ -12,7 +12,11 @@
 use Traversable;
 
 /**
- * @mixin \Illuminate\Support\Collection
+ * @template TKey of array-key
+ *
+ * @template TValue
+ *
+ * @mixin \Illuminate\Support\Collection<TKey, TValue>
  */
 abstract class AbstractPaginator implements Htmlable, Stringable
 {
@@ -21,7 +25,7 @@ abstract class AbstractPaginator implements Htmlable, Stringable
     /**
      * All of the items being paginated.
      *
-     * @var \Illuminate\Support\Collection
+     * @var \Illuminate\Support\Collection<TKey, TValue>
      */
     protected $items;
 
@@ -310,7 +314,7 @@ public function loadMorphCount($relation, $relations)
     /**
      * Get the slice of items being paginated.
      *
-     * @return array
+     * @return array<TKey, TValue>
      */
     public function items()
     {
@@ -649,7 +653,7 @@ public static function useBootstrapFive()
     /**
      * Get an iterator for the items.
      *
-     * @return \ArrayIterator
+     * @return \ArrayIterator<TKey, TValue>
      */
     public function getIterator(): Traversable
     {
@@ -689,7 +693,7 @@ public function count(): int
     /**
      * Get the paginator's underlying collection.
      *
-     * @return \Illuminate\Support\Collection
+     * @return \Illuminate\Support\Collection<TKey, TValue>
      */
     public function getCollection()
     {
@@ -699,7 +703,7 @@ public function getCollection()
     /**
      * Set the paginator's underlying collection.
      *
-     * @param  \Illuminate\Support\Collection  $collection
+     * @param  \Illuminate\Support\Collection<TKey, TValue>  $collection
      * @return $this
      */
     public function setCollection(Collection $collection)

src/Illuminate/Pagination/LengthAwarePaginator.php

@@ -11,6 +11,16 @@
 use IteratorAggregate;
 use JsonSerializable;
 
+/**
+ * @template TKey of array-key
+ *
+ * @template TValue
+ *
+ * @implements Arrayable<TKey, TValue>
+ * @implements ArrayAccess<TKey, TValue>
+ * @implements LengthAwarePaginatorContract<TKey, TValue>
+ * @implements IteratorAggregate<TKey, TValue>
+ */
 class LengthAwarePaginator extends AbstractPaginator implements Arrayable, ArrayAccess, Countable, IteratorAggregate, Jsonable, JsonSerializable, LengthAwarePaginatorContract
 {
     /**
@@ -30,7 +40,7 @@ class LengthAwarePaginator extends AbstractPaginator implements Arrayable, Array
     /**
      * Create a new paginator instance.
      *
-     * @param  mixed  $items
+     * @param  \Illuminate\Contracts\Support\Arrayable<TKey, TValue>|iterable<TKey, TValue>|Collection<TKey, TValue>|null  $items
      * @param  int  $total
      * @param  int  $perPage
      * @param  int|null  $currentPage

src/Illuminate/Pagination/Paginator.php

@@ -11,6 +11,16 @@
 use IteratorAggregate;
 use JsonSerializable;
 
+/**
+ * @template TKey of array-key
+ *
+ * @template-covariant TValue
+ *
+ * @implements Arrayable<TKey, TValue>
+ * @implements ArrayAccess<TKey, TValue>
+ * @implements IteratorAggregate<TKey, TValue>
+ * @implements PaginatorContract<TKey, TValue>
+ */
 class Paginator extends AbstractPaginator implements Arrayable, ArrayAccess, Countable, IteratorAggregate, Jsonable, JsonSerializable, PaginatorContract
 {
     /**
@@ -23,7 +33,7 @@ class Paginator extends AbstractPaginator implements Arrayable, ArrayAccess, Cou
     /**
      * Create a new paginator instance.
      *
-     * @param  mixed  $items
+     * @param  TValue[]  $items
      * @param  int  $perPage
      * @param  int|null  $currentPage
      * @param  array  $options  (path, query, fragment, pageName)
@@ -60,7 +70,7 @@ protected function setCurrentPage($currentPage)
     /**
      * Set the items for the paginator.
      *
-     * @param  mixed  $items
+     * @param  \Illuminate\Contracts\Support\Arrayable<TKey, TValue>|iterable<TKey, TValue>|Collection<TKey, TValue>|null  $items
      * @return void
      */
     protected function setItems($items)

tests/Pagination/PaginatorTest.php

@@ -9,6 +9,7 @@ class PaginatorTest extends TestCase
 {
     public function testSimplePaginatorReturnsRelevantContextInformation()
     {
+        /** @var Paginator<int, string> $p */
         $p = new Paginator(['item3', 'item4', 'item5'], 2, 2);
 
         $this->assertEquals(2, $p->currentPage());

types/Pagination/Paginator.php

@@ -0,0 +1,37 @@
+<?php
+
+use Illuminate\Pagination\LengthAwarePaginator;
+use Illuminate\Pagination\Paginator;
+use function PHPStan\Testing\assertType;
+
+$items = [new Post(), new Post(), new Post()];
+
+/** @var Paginator<int, Post> $paginator */
+$paginator = new Paginator($items, 1, 1);
+
+assertType('Post[]', $paginator->items());
+assertType('array<int, Post>', $paginator->toArray());
+assertType('ArrayIterator<TKey, TValue>', $paginator->getIterator());
+
+$paginator->each(function ($post) {
+    assertType('Post', $post);
+});
+
+foreach ($paginator as $post) {
+    assertType('Post', $post);
+}
+
+/** @var LengthAwarePaginator<int, Post> $lengthAwarePaginator */
+$lengthAwarePaginator = new LengthAwarePaginator($items, 1, 1);
+
+assertType('Post[]', $lengthAwarePaginator->items());
+assertType('array<int, Post>', $lengthAwarePaginator->toArray());
+assertType('ArrayIterator<int, Post>', $lengthAwarePaginator->getIterator());
+
+$lengthAwarePaginator->each(function ($post) {
+    assertType('Post', $post);
+});
+
+foreach ($lengthAwarePaginator as $post) {
+    assertType('Post', $post);
+}