Call to an undefined method Illuminate\Database\Eloquent\Builder<Illuminate\Database\Eloquent\Model>::allowedFields()

[giacomomasseron]

Hi,
PHPStan level 2 returns the error as title with this code:

return QueryBuilder::for(MyModel::query())
    ->where('is_disabled', false)
    ->allowedFields([
        ...
    ])
    ->allowedFilters([
        ...
    ])
    ->defaultSort(AllowedSort::custom('campaign', new SaleCampaignSort))
    ->allowedSorts([
        ...
    ])
    ->with(['entities'])
    ->when(
        $this->data('with'),
        fn (BuilderContract $builder) => $builder->with($this->data('with')),
    )
    ->when(
        $this->data('scopes'),
        function (BuilderContract $builder) {
            foreach ($this->data('scopes') as $scope => $parameters) {
                if (is_int($scope)) {
                    $builder->$parameters();
                    continue;
                }

                $builder->$scope(...$parameters);
            }
        }
    );

[OpShaid]

The line

$builder->$scope(...$parameters);

is what’s breaking PHPStan. It can’t verify that $scope is a real method.
Fix:
Add a guard and the ignore comment:

if (method_exists($builder, $scope)) {
// PHPStan false positive — dynamic scope call is fine.
// @phpstan-ignore-next-line
$builder->$scope(...$parameters);
}

That’s it stops PHPStan whining and makes runtime safer :)

[giacomomasseron]

@Shaid-T which file you are talking about?

[OpShaid]

This Line Right here: $builder->$scope(...$parameters); - sorry im not sure about which file because it doesn't let me open the file perhaps you had deleted it or renamed the file the issue was assigned to.

[giacomomasseron]

I'm sorry but I do not see any $builder->scope() in the code I pasted

[OpShaid]

PHPStan is complaining about this line
its down where your closing your brackets:
$builder->$scope(...$parameters);

The problem isn’t that your code is “wrong” it’s that PHPStan cannot verify dynamically called methods. $scope is a string that will be used as a method name at runtime, so static analysis tools like PHPStan don’t know if that method actually exists on $builder.

this should do the job:

foreach ($this->data('scopes') as $scope => $parameters) {
if (is_int($scope)) {
$builder->$parameters();
continue;
}

if (method_exists($builder, $scope)) {
    // PHPStan false positive — dynamic scope call is fine.
    // @phpstan-ignore-next-line
    $builder->$scope(...$parameters);
}

}

method_exists($builder, $scope) ensures you only call real methods at runtime.

@phpstan-ignore-next-line tells PHPStan to ignore the false positive.

Alternative: If you are certain all $scopes are valid, you could just add the ignore comment:

// @phpstan-ignore-next-line
$builder->$scope(...$parameters);

…but using method_exists is safer because it prevents runtime errors if bad input ever sneaks in. Hope this helped :) 👍

[giacomomasseron]

Aaaaah, I got it. Sorry I did not see my code :(

And thanks.

[OpShaid]

haha no problem i feel you on that when you got to many files and just forget, happens to all of us man. Let me know if it worked tho :)