Skip to content
New issue

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

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

Already on GitHub? # 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

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants