Skip to content
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

PHPORM-219 Deprecate Connection::collection() and Schema::collection() #3062

Merged
merged 4 commits into from
Jul 23, 2024

Conversation

GromNaN
Copy link
Member

@GromNaN GromNaN commented Jul 22, 2024

Fix PHPORM-219

The method MongoDB\Laravel\Connection::collection() is an alias for Laravel’s Illuminate\Database\ConnectionInterface::table(). I guess it was introduced when the package was created to fit MongoDB terminology.

Removing this custom name fit with Laravel and remove the one of the 2 ways of doing the same thing.

Checklist

  • Add tests and ensure they pass
  • Add an entry to the CHANGELOG.md file
  • Update documentation for new features

@GromNaN GromNaN added this to the 4.8 milestone Jul 22, 2024
@GromNaN GromNaN requested review from a team as code owners July 22, 2024 14:33
@GromNaN GromNaN requested a review from alcaeus July 22, 2024 14:33
@github-actions github-actions bot added the docs label Jul 22, 2024
@GromNaN GromNaN force-pushed the PHPORM-219 branch 3 times, most recently from 439ae56 to c17b69a Compare July 22, 2024 15:11
Copy link
Member

@alcaeus alcaeus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small wording suggestion, LGTM otherwise

src/Connection.php Outdated Show resolved Hide resolved
Co-authored-by: Andreas Braun <git@alcaeus.org>
@github-actions github-actions bot added the docs label Jul 23, 2024
Copy link
Contributor

@rustagir rustagir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added some commits to catch some other uses of collection(). LGTM !

@GromNaN
Copy link
Member Author

GromNaN commented Jul 23, 2024

Thank you @rustagir, I really appreciate your intervention.

@GromNaN GromNaN merged commit 979cf52 into mongodb:4.8 Jul 23, 2024
26 checks passed
@GromNaN GromNaN deleted the PHPORM-219 branch July 23, 2024 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants