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

[10.x] Fix belongs to many touch using key instead of owner key #50728

Closed

Conversation

GrahamCampbell
Copy link
Member

@GrahamCampbell GrahamCampbell commented Mar 23, 2024

The default is that the owner key is set to the related model's key, however this is just a default, and we need to respect if a non-default value for the owner key name is used.

@taylorotwell
Copy link
Member

Likely need a test for this.

@taylorotwell taylorotwell marked this pull request as draft March 24, 2024 15:03
@driesvints
Copy link
Member

@GrahamCampbell feel free to resend this with a test.

@driesvints driesvints closed this Apr 11, 2024
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.

3 participants