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

Trim quotes in unqualified foreign table names when diffing foreign keys #6773

Open
wants to merge 1 commit into
base: 3.9.x
Choose a base branch
from

Conversation

cristi-contiu
Copy link

@cristi-contiu cristi-contiu commented Feb 6, 2025

Q A
Type bug
Fixed issues #6772

Summary

Trims the quotes in ForeignKeyConstraint::getUnqualifiedForeignTableName, only if quoted. Used in Comparator::diffForeignKey when comparing foreign keys for changes. Fixes dropping and re-adding of foreign keys when foreign table name is simultaneously a reserved word (quoted) and in a non-default schema (not in default search path schemas).

@cristi-contiu cristi-contiu marked this pull request as ready for review February 6, 2025 12:50
@cristi-contiu cristi-contiu changed the title PostgreSQL: Fix re-adding of foreign keys when table name is quoted and in a custom schema PostgreSQL: Fix re-adding of foreign keys when foreign table name is quoted and in a custom schema Feb 6, 2025
@cristi-contiu cristi-contiu changed the title PostgreSQL: Fix re-adding of foreign keys when foreign table name is quoted and in a custom schema Trim quotes in unqualified foreign table name when diffing foreign keys Feb 7, 2025
@cristi-contiu cristi-contiu changed the title Trim quotes in unqualified foreign table name when diffing foreign keys Trim quotes in unqualified foreign table names when diffing foreign keys Feb 7, 2025
@cristi-contiu cristi-contiu force-pushed the fix_postgres_foreign_key_table_custom_schema_reserverd_word branch from b0de455 to d0c53b4 Compare February 12, 2025 09:09
@cristi-contiu cristi-contiu force-pushed the fix_postgres_foreign_key_table_custom_schema_reserverd_word branch from 056a8be to a009e2c Compare February 14, 2025 07:58
greg0ire
greg0ire previously approved these changes Feb 14, 2025
@cristi-contiu cristi-contiu force-pushed the fix_postgres_foreign_key_table_custom_schema_reserverd_word branch from a009e2c to ad1b24a Compare February 14, 2025 12:11
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