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

Fixed #35024 -- Fixed model instance creation crash on GeneratedField output_field with backend converters. #17587

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

felixxm
Copy link
Member

@felixxm felixxm commented Dec 7, 2023

Regression in d9de741.

ticket-35024

This is a long standing issue, however it caused a crash of GeneratedField for all output fields that have backend-specific converters when RETURNING clause is not supported (MySQL and SQLite < 3.35). That's why severity was exacerbated.

@felixxm
Copy link
Member Author

felixxm commented Dec 7, 2023

buildbot, test on oracle.

@felixxm felixxm requested a review from a team December 7, 2023 17:12
Copy link
Member

@charettes charettes left a comment

Choose a reason for hiding this comment

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

Crazy to think this flew under the radar for so long.

Copy link
Contributor

@nessita nessita left a comment

Choose a reason for hiding this comment

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

LGTM, I added some comments. Thank you for the great find and super fast fix! 🏃

docs/releases/5.0.1.txt Outdated Show resolved Hide resolved
django/db/models/sql/compiler.py Show resolved Hide resolved
….output_field with backend converters.

Regression in d9de741.

This is a long standing issue, however it caused a crash of
GeneratedField for all output fields that have backend-specific
converters when RETURNING clause is not supported (MySQL and
SQLite < 3.35). That's why severity was exacerbated.
@felixxm felixxm merged commit 5b3b791 into django:main Dec 7, 2023
35 checks passed
@felixxm felixxm deleted the issue-35024 branch December 7, 2023 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants