Skip to content

Don't detect join table with other incoming references as a simple join table - #32627

Merged
ajcvickers merged 1 commit into
mainfrom
231214_RebelYell
Dec 16, 2023
Merged

ajcvickers merged 1 commit into
mainfrom
231214_RebelYell

Conversation

@ajcvickers

Copy link
Copy Markdown
Contributor

Fixes #28905

@ajcvickers
ajcvickers requested a review from a team December 15, 2023 18:17
@ErikEJ

ErikEJ commented Dec 15, 2023 •

Copy link
Copy Markdown
Contributor

Wonder if this also affects #32571 ?

@ajcvickers

Copy link
Copy Markdown
Contributor Author

@ErikEJ Yep. :-) I ran into that exception while testing this. I need to investigate whether both sides are expecting the same thing with regards to generating navigations.

@ajcvickers
ajcvickers merged commit 9b0f169 into main Dec 16, 2023
@ajcvickers
ajcvickers deleted the 231214_RebelYell branch December 16, 2023 11:03
ajcvickers added a commit that referenced this pull request Jan 5, 2024
ajcvickers added a commit that referenced this pull request Jan 10, 2024
…as a simple join table (#32729)

* Don't detect join table with other incoming references as a simple join table (#32627)

Fixes #28905

* Add quirk
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.

Incorrect simple join table determination in reverse engineering leads to incorrect code generation

3 participants