Skip to content

Conversation

@AhmedAlaa4611
Copy link
Contributor

Continuation of: #57478


A quick GitHub search showed other instances of NamedScope.

@AhmedAlaa4611 AhmedAlaa4611 marked this pull request as draft October 23, 2025 06:18
@AhmedAlaa4611
Copy link
Contributor Author

Marking as draft while fixing CI issues.

@AhmedAlaa4611 AhmedAlaa4611 marked this pull request as ready for review October 23, 2025 07:46
@AhmedAlaa4611
Copy link
Contributor Author

Fixed, it is now ready for review. The problem was:

Cannot redeclare method Illuminate\Database\Eloquent\Builder::callScope().

Because the Illuminate\Database\Eloquent\Builder has callNamedScope() and callScope() methods. So, I can not rename callNamedScope() to callScope() in that class.

@taylorotwell
Copy link
Member

Not necessary imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants