Add return type to Closures.

loophp · Aug 21, 2019 · a2b9366 · a2b9366
1 parent e50bbfd
commit a2b9366
Show file tree

Hide file tree

Showing 20 changed files with 40 additions and 20 deletions.
diff --git a/src/Operation/Append.php b/src/Operation/Append.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$items] = $this->parameters;
 
         return Collection::with(
-            static function () use ($items, $collection) {
+            static function () use ($items, $collection): \Generator
+            {
                 foreach ($collection as $item) {
                     yield $item;
                 }

diff --git a/src/Operation/Chunk.php b/src/Operation/Chunk.php
@@ -24,7 +24,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         }
 
         return Collection::with(
-            static function () use ($size, $collection) {
+            static function () use ($size, $collection): \Generator
+            {
                 $iterator = $collection->getIterator();
 
                 while ($iterator->valid()) {

diff --git a/src/Operation/Collapse.php b/src/Operation/Collapse.php
@@ -18,7 +18,8 @@ final class Collapse extends Operation
     public function run(CollectionInterface $collection): CollectionInterface
     {
         return Collection::with(
-            static function () use ($collection) {
+            static function () use ($collection): \Generator
+            {
                 foreach ($collection as $item) {
                     if (\is_array($item) || $item instanceof CollectionInterface) {
                         foreach ($item as $value) {

diff --git a/src/Operation/Combine.php b/src/Operation/Combine.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$keys] = $this->parameters;
 
         return Collection::with(
-            static function () use ($keys, $collection) {
+            static function () use ($keys, $collection): \Generator
+            {
                 $original = $collection->getIterator();
                 $keysIterator = Collection::with($keys)->getIterator();
 

diff --git a/src/Operation/Filter.php b/src/Operation/Filter.php
@@ -26,7 +26,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         }
 
         return Collection::with(
-            static function () use ($callbacks, $collection) {
+            static function () use ($callbacks, $collection): \Generator
+            {
                 foreach ($callbacks as $callback) {
                     foreach ($collection->getIterator() as $key => $value) {
                         if (true === (bool) $callback($value, $key)) {

diff --git a/src/Operation/Flatten.php b/src/Operation/Flatten.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         $depth = $this->parameters[0];
 
         return Collection::with(
-            static function () use ($depth, $collection) {
+            static function () use ($depth, $collection): \Generator
+            {
                 $iterator = $collection->getIterator();
 
                 foreach ($iterator as $item) {

diff --git a/src/Operation/Flip.php b/src/Operation/Flip.php
@@ -18,7 +18,8 @@ final class Flip extends Operation
     public function run(CollectionInterface $collection): CollectionInterface
     {
         return Collection::with(
-            static function () use ($collection) {
+            static function () use ($collection): \Generator
+            {
                 foreach ($collection as $key => $value) {
                     yield $value => $key;
                 }

diff --git a/src/Operation/Forget.php b/src/Operation/Forget.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$keys] = $this->parameters;
 
         return Collection::with(
-            static function () use ($keys, $collection) {
+            static function () use ($keys, $collection): \Generator
+            {
                 $keys = \array_flip($keys);
 
                 foreach ($collection->getIterator() as $key => $value) {

diff --git a/src/Operation/Keys.php b/src/Operation/Keys.php
@@ -18,7 +18,8 @@ final class Keys extends Operation
     public function run(CollectionInterface $collection): CollectionInterface
     {
         return Collection::with(
-            static function () use ($collection) {
+            static function () use ($collection): \Generator
+            {
                 foreach ($collection as $key => $value) {
                     yield $key;
                 }

diff --git a/src/Operation/Limit.php b/src/Operation/Limit.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         $limit = $this->parameters[0];
 
         return Collection::with(
-            static function () use ($limit, $collection) {
+            static function () use ($limit, $collection): \Generator
+            {
                 $iterator = $collection->getIterator();
 
                 for (; (true === $iterator->valid()) && (0 !== $limit--); $iterator->next()) {

diff --git a/src/Operation/Merge.php b/src/Operation/Merge.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$sources] = $this->parameters;
 
         return Collection::with(
-            static function () use ($sources, $collection) {
+            static function () use ($sources, $collection): \Generator
+            {
                 foreach ($collection as $item) {
                     yield $item;
                 }

diff --git a/src/Operation/Nth.php b/src/Operation/Nth.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$step, $offset] = $this->parameters;
 
         return Collection::with(
-            static function () use ($step, $offset, $collection) {
+            static function () use ($step, $offset, $collection): \Generator
+            {
                 $position = 0;
 
                 foreach ($collection->getIterator() as $key => $item) {

diff --git a/src/Operation/Only.php b/src/Operation/Only.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$keys] = $this->parameters;
 
         return Collection::with(
-            static function () use ($keys, $collection) {
+            static function () use ($keys, $collection): \Generator
+            {
                 if ([] === $keys) {
                     yield from $collection;
                 } else {

diff --git a/src/Operation/Pad.php b/src/Operation/Pad.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$size, $value] = $this->parameters;
 
         return Collection::with(
-            static function () use ($size, $value, $collection) {
+            static function () use ($size, $value, $collection): \Generator
+            {
                 $y = 0;
 
                 foreach ($collection->getIterator() as $key => $item) {

diff --git a/src/Operation/Prepend.php b/src/Operation/Prepend.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$items] = $this->parameters;
 
         return Collection::with(
-            static function () use ($items, $collection) {
+            static function () use ($items, $collection): \Generator
+            {
                 foreach ($items as $item) {
                     yield $item;
                 }

diff --git a/src/Operation/Skip.php b/src/Operation/Skip.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$counts] = $this->parameters;
 
         return Collection::with(
-            static function () use ($counts, $collection) {
+            static function () use ($counts, $collection): \Generator
+            {
                 $iterator = $collection->getIterator();
                 $counts = \array_sum($counts);
 

diff --git a/src/Operation/Slice.php b/src/Operation/Slice.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$offset, $length] = $this->parameters;
 
         return Collection::with(
-            static function () use ($offset, $length, $collection) {
+            static function () use ($offset, $length, $collection): \Generator
+            {
                 if (null === $length) {
                     yield from $collection->skip($offset);
                 } else {

diff --git a/src/Operation/Sort.php b/src/Operation/Sort.php
@@ -22,7 +22,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$callback] = $this->parameters;
 
         return Collection::with(
-            static function () use ($callback, $collection) {
+            static function () use ($callback, $collection): \Generator
+            {
                 $array = \iterator_to_array($collection->getIterator());
 
                 \uasort($array, $callback);

diff --git a/src/Operation/Walk.php b/src/Operation/Walk.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$callbacks] = $this->parameters;
 
         return Collection::with(
-            static function () use ($callbacks, $collection) {
+            static function () use ($callbacks, $collection): \Generator
+            {
                 $callback = static function ($carry, $callback) {
                     return $callback($carry);
                 };

diff --git a/src/Operation/Zip.php b/src/Operation/Zip.php
@@ -20,7 +20,8 @@ public function run(CollectionInterface $collection): CollectionInterface
         [$iterables] = $this->parameters;
 
         return Collection::with(
-            static function () use ($iterables, $collection) {
+            static function () use ($iterables, $collection): \Generator
+            {
                 $iterators =
                     Collection::empty()
                         ->append($collection, ...$iterables)