Skip to content

fix: correct regex mappings for duplicate and foreign key errors - #266

Merged
cofin merged 1 commit into
litestar-org:mainfrom
noamsto:main
Oct 22, 2024
Merged

cofin merged 1 commit into
litestar-org:mainfrom
noamsto:main

Conversation

@noamsto

@noamsto noamsto commented Oct 20, 2024

Copy link
Copy Markdown
Contributor

Description

Swap the variable names for DUPLICATE_KEY_REGEXES and FOREIGN_KEY_REGEXES to correctly match their contents.
This ensures that the error detection for duplicate keys and foreign key violations works as intended across different database backends.

Closes

#262

@noamsto
noamsto requested review from a team as code owners October 20, 2024 13:57
Swap the variable names for DUPLICATE_KEY_REGEXES and FOREIGN_KEY_REGEXES to correctly match their contents. This ensures that the error detection for duplicate keys and foreign key violations works as intended across different database backends.

Signed-off-by: Noam Stolero <noam@factify.com>
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.

Bug: Violation of unique constraints raises ForeignKeyError exception instead of IntegrityError

2 participants