Skip to content

Conversation

@keulinho
Copy link
Contributor

Q A
Type bug
Fixed issues #7030

Summary

Provide test for dots in quoted idx names

@keulinho keulinho force-pushed the fix-unquoted-index-names branch 3 times, most recently from 8a31939 to d7ff279 Compare July 16, 2025 06:54
@keulinho
Copy link
Contributor Author

The underlying issue was fixed here: 92dea81

Not sure if it makes sense to add the test case just to not accidentally break the behaviour again

anyway I don't know why setting the FK on DB2 and Oracle DBs fails, happy for any insights how to fix

public function testIntrospectTableWithDotInIndexNames(): void
{
$table = Table::editor()
->setQuotedName('test_user')
Copy link
Member

@morozov morozov Sep 21, 2025

Choose a reason for hiding this comment

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

I don't know why setting the FK on DB2 and Oracle DBs fails, happy for any insights how to fix

You declare the table name here as quoted but reference it in the other table as unquoted:

->setUnquotedReferencedTableName('test_user')

On Oracle and Db2, "test_user" (quoted) remains test_user, while test_user (unquoted) becomes TEST_USER. Hence the error message:

"DB2INST1.TEST_USER" is an undefined name

Unless you want to retain the original case of the name on all platforms, you should use unquoted names.

@morozov morozov force-pushed the fix-unquoted-index-names branch from d7ff279 to 83144b1 Compare October 4, 2025 21:09
Otherwise, an extra "not null" assertion is required before comparing
the value returned by OptionallyNamedObject::getObjectName() with the
expected one.
@github-actions
Copy link

github-actions bot commented Jan 3, 2026

There hasn't been any activity on this pull request in the past 90 days, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 7 days.
If you want to continue working on it, please leave a comment.

@github-actions github-actions bot added the Stale label Jan 3, 2026
@github-actions
Copy link

This pull request was closed due to inactivity.

@github-actions github-actions bot closed this Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants