From 3b75048d1216128bb046859322e12613bc76dec8 Mon Sep 17 00:00:00 2001
From: Karel Faille <shaffe.fr@gmail.com>
Date: Wed, 4 Aug 2021 11:13:23 +0200
Subject: [PATCH] Support variadic parameters in models command (#1234)

---
 CHANGELOG.md                                  |  3 ++
 README.md                                     |  1 +
 src/Console/ModelsCommand.php                 |  3 +-
 .../ModelsCommand/Variadic/Models/Simple.php  | 20 ++++++++++++
 tests/Console/ModelsCommand/Variadic/Test.php | 24 ++++++++++++++
 .../Variadic/__snapshots__/Test__test__1.php  | 32 +++++++++++++++++++
 6 files changed, 82 insertions(+), 1 deletion(-)
 create mode 100644 tests/Console/ModelsCommand/Variadic/Models/Simple.php
 create mode 100644 tests/Console/ModelsCommand/Variadic/Test.php
 create mode 100644 tests/Console/ModelsCommand/Variadic/__snapshots__/Test__test__1.php

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 248b88a..0be42ad 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,6 +10,9 @@ All notable changes to this project will be documented in this file.
 ### Fixed
 - Use platformName to determine db type when casting boolean types [\#1212 / stockalexander](https://github.com/barryvdh/laravel-ide-helper/pull/1212)
 
+### Added
+- Add support of variadic parameters in `ide-helper:models` [\#1234 / shaffe-fr](https://github.com/barryvdh/laravel-ide-helper/pull/1234)
+
 2021-04-09, 2.10.0
 ------------------
 ### Added
diff --git a/README.md b/README.md
index c227fe7..91906c9 100644
--- a/README.md
+++ b/README.md
@@ -166,6 +166,7 @@ php artisan ide-helper:models "App\Models\Post"
  * @method static \Illuminate\Database\Eloquent\Builder|\App\Models\Post newQuery()
  * @method static \Illuminate\Database\Eloquent\Builder|\App\Models\Post query()
  * @method static \Illuminate\Database\Eloquent\Builder|\App\Models\Post whereTitle($value)
+ * @method static \Illuminate\Database\Eloquent\Builder|\App\Models\Post forAuthors(\User ...$authors)
  * …
  */
 ```
diff --git a/src/Console/ModelsCommand.php b/src/Console/ModelsCommand.php
index 3f3cd31..57b8fea 100644
--- a/src/Console/ModelsCommand.php
+++ b/src/Console/ModelsCommand.php
@@ -935,7 +935,8 @@ class ModelsCommand extends Command
         $paramsWithDefault = [];
         /** @var \ReflectionParameter $param */
         foreach ($method->getParameters() as $param) {
-            $paramStr = '$' . $param->getName();
+            $paramStr = $param->isVariadic() ? '...$' . $param->getName() : '$' . $param->getName();
+
             if ($paramType = $this->getParamType($method, $param)) {
                 $paramStr = $paramType . ' ' . $paramStr;
             }
diff --git a/tests/Console/ModelsCommand/Variadic/Models/Simple.php b/tests/Console/ModelsCommand/Variadic/Models/Simple.php
new file mode 100644
index 0000000..7932295
--- /dev/null
+++ b/tests/Console/ModelsCommand/Variadic/Models/Simple.php
@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Barryvdh\LaravelIdeHelper\Tests\Console\ModelsCommand\Variadic\Models;
+
+use DateTime;
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Model;
+
+class Simple extends Model
+{
+    public function scopeWhereVariadic(Builder $query, ...$values): void
+    {
+    }
+
+    public function scopeWhereTypedVariadic(Builder $query, int ...$values): void
+    {
+    }
+}
diff --git a/tests/Console/ModelsCommand/Variadic/Test.php b/tests/Console/ModelsCommand/Variadic/Test.php
new file mode 100644
index 0000000..7883d83
--- /dev/null
+++ b/tests/Console/ModelsCommand/Variadic/Test.php
@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Barryvdh\LaravelIdeHelper\Tests\Console\ModelsCommand\Variadic;
+
+use Barryvdh\LaravelIdeHelper\Console\ModelsCommand;
+use Barryvdh\LaravelIdeHelper\Tests\Console\ModelsCommand\AbstractModelsCommand;
+
+class Test extends AbstractModelsCommand
+{
+    public function test(): void
+    {
+        $command = $this->app->make(ModelsCommand::class);
+
+        $tester = $this->runCommand($command, [
+            '--write' => true,
+        ]);
+
+        $this->assertSame(0, $tester->getStatusCode());
+        $this->assertStringContainsString('Written new phpDocBlock to', $tester->getDisplay());
+        $this->assertMatchesMockedSnapshot();
+    }
+}
diff --git a/tests/Console/ModelsCommand/Variadic/__snapshots__/Test__test__1.php b/tests/Console/ModelsCommand/Variadic/__snapshots__/Test__test__1.php
new file mode 100644
index 0000000..80d773a
--- /dev/null
+++ b/tests/Console/ModelsCommand/Variadic/__snapshots__/Test__test__1.php
@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Barryvdh\LaravelIdeHelper\Tests\Console\ModelsCommand\Variadic\Models;
+
+use DateTime;
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Model;
+
+/**
+ * Barryvdh\LaravelIdeHelper\Tests\Console\ModelsCommand\Variadic\Models\Simple
+ *
+ * @property integer $id
+ * @method static Builder|Simple newModelQuery()
+ * @method static Builder|Simple newQuery()
+ * @method static Builder|Simple query()
+ * @method static Builder|Simple whereId($value)
+ * @method static Builder|Simple whereTypedVariadic(int ...$values)
+ * @method static Builder|Simple whereVariadic(...$values)
+ * @mixin \Eloquent
+ */
+class Simple extends Model
+{
+    public function scopeWhereVariadic(Builder $query, ...$values): void
+    {
+    }
+
+    public function scopeWhereTypedVariadic(Builder $query, int ...$values): void
+    {
+    }
+}