Skip to content

[12.x] Avoid class name conflict #10470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 5, 2025
Merged

Conversation

AhmedAlaa4611
Copy link
Contributor

Description

This PR fixes a class name conflict caused by importing two classes with the same name:

  • App\Contracts\Repository
  • Illuminate\Contracts\Cache\Repository

Since both share the same base name (Repository), PHP throws the following error:

Cannot use Illuminate\Contracts\Cache\Repository as Repository because the name is already in use

To resolve this, I updated the constructor to use a custom UserRepository interface in place of the generic Repository contract from our app domain. This makes the intent clearer and avoids naming collisions.

@taylorotwell taylorotwell merged commit 918c798 into laravel:12.x Jun 5, 2025
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