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

Prevent sealed accessed on related queryset indexing. #43

Merged
merged 3 commits into from
Feb 24, 2020
Merged

Conversation

charettes
Copy link
Owner

fixes #42 cc @fish-face

Avoid fiddling with __mro__ and rely on an attribute assigned at dynamic class
creation instead.
Fixes #42, thanks Chris Le Sueur for the detailed report.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 97.625% when pulling 52eb4ee on issue-42 into bb99715 on master.

@charettes charettes merged commit 52eb4ee into master Feb 24, 2020
@charettes charettes deleted the issue-42 branch February 24, 2020 03:58
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.

Indexing into a related manager's queryset fails
2 participants