composer update 時のエラー

Question

下記のサイトを参考にlaravel のアップグレードを実施しておりました。

5.8から8.xに段階的にアップグレードする

6から7へアップグレードしていた時に、このようなエラーが出てしまいました。

In QueryRecorder.php line 22:
  Argument 2 passed to Facade\Ignition\QueryRecorder\QueryRecorder::__constru
  ct() must be of the type bool, null given, called in vendor/facade/ignition/src/IgnitionServiceProvider.php on line 386

このQueryRecorder.php line 22の場所はこのようになっておりました。

class QueryRecorder

{

    /** @var \Facade\Ignition\QueryRecorder\Query|[] */

    protected $queries = [];

 

    /** @var \Illuminate\Contracts\Foundation\Application */

    protected $app;

 

    /** @var bool */

    private $reportBindings;

 

    /** @var int|null */

    private $maxQueries;

 

    public function __construct(　　//ここが２２行目となります。

        Application $app,

        bool $reportBindings = true,

        ?int $maxQueries = null

    ) {

        $this->app = $app;

        $this->reportBindings = $reportBindings;

        $this->maxQueries = $maxQueries;

    }

 

    public function register()

    {

        $this->app['events']->listen(QueryExecuted::class, [$this, 'record']);

 

        return $this;

    }

laravel6から7へのアップグレードする前に、エラーにも表記されている"facade/ignition"をcomposer.jsonにも依存パッケージとして更新（追記？）しております。

composer.jsonはlaravelの公式で掲載している通りの記述にしているかと思います。

composer.json

{
    "name": "laravel/laravel",
    "type": "project",
    "description": "The Laravel Framework.",
    "keywords": [
        "framework",
        "laravel"
    ],
    "license": "MIT",
    "require": {
        "php": "^7.1.3",
        "abraham/twitteroauth": "^3.1",
        "fabpot/goutte": "^4.0",
        "fideloper/proxy": "^4.0",
        "fruitcake/laravel-cors": "^2.0",
        "google/cloud-vision": "^1.5",
        "guzzlehttp/guzzle": "^7.4",
        "intervention/image": "^2.7",
        "laravel/framework": "^7.0",
        "laravel/tinker": "^2.0",
        "laravel/ui": "2.0"
    },
    "require-dev": {
        "beyondcode/laravel-dump-server": "^1.0",
        "filp/whoops": "^2.0",
        "fzaninotto/faker": "^1.4",
        "mockery/mockery": "^1.0",
        "nunomaduro/collision": "^4.1",
        "phpunit/phpunit": "^8.5",
        "facade/ignition": "^2.0"
    },
    "config": {
        "optimize-autoloader": true,
        "preferred-install": "dist",
        "sort-packages": true
    },
    "extra": {
        "laravel": {
            "dont-discover": []
        }
    },
    "autoload": {
        "psr-4": {
            "App\": "app/"
        },
        "classmap": [
            "database/seeds",
            "database/factories"
        ]
    },
    "autoload-dev": {
        "psr-4": {
            "Tests\": "tests/"
        }
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "scripts": {
        "post-autoload-dump": [
            "Illuminate\Foundation\ComposerScripts::postAutoloadDump",
            "@php artisan package:discover --ansi"
        ],
        "post-root-package-install": [
            "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
        ],
        "post-create-project-cmd": [
            "@php artisan key:generate --ansi"
        ]
    }
}

現在の環境です
windows10
Laravel Framework 6.20.40　　→　php artisanが使えないため、現在はこのバージョンを確認できません。
teraterm

すみません、この手のエラーを探したのですが見つけられなかったため、今回も質問させていただきました。
よろしくお願いします。

Answer 1

まず最初に。

laravel6未満のfilp/whoopsに代わって、
laravel6以上ではfacade/ignitionになったので、

両方入れる必要はないんじゃないかな・・と思いました。
どちらも「エラー表示を見やすくするツール」なので、片方あれば十分なはず。。。

次に。

この「エラー表示を見やすくするツール」であるfacade/ignitionは、
require-devに含まれているものなので、システム全体の動きには影響しない。。。
なんなら、なくても大丈夫なヤツです。

facade/ignitionにエラー表示してもらおうと、
エラーを渡している元の部分の方が、根本的な問題なのでは？と思うので、
エラーメッセージを少し遡ってみてはどうでしょうか。。

いっそ、filp/whoopsもfacade/ignitionも削除して、動作検証してみるのも手かもしれません。

（もし、filp/whoopsもfacade/ignitionを両方削除して、エラーがなくなるのであれば、同じ機能のツールが競合して問題が発生していた。。という可能性もあるかも？？）

以上、検証したわけではありませんので、あくまで想像の範囲ですが、
ご参考までに。。。

Comments

[of_the_Europa]

ありがとうございます。

composer.jsonのfacade/ignitionの記述を削除してから composer update したところ、無事７にアップデートできました。サイトの表示や動きも今のところ問題ないようです。

filp/whoops と facade/ignition が同じエラーメッセージツールだということは、正直分かっておりませんでした。
根本的はエラーの原因は調べていないのですが、もう少し心に余裕ができてから再度検証してみたいと思いますm(__)m