The GitHub Actions job "CI" on 
grails-core.git/naming-strategy-table-name-mapping-fix has succeeded.
Run started by GitHub user borinquenkid (triggered by borinquenkid).

Head commit for run:
aa02d74cf6a0b0412da2c1eb3cb33b1af341b053 / Walter Duque de Estrada 
<[email protected]>
Consolidate join-table table-name resolution and fix property-prefix naming

getHibernateAssociatedEntity().getHibernateRootEntity().getTableName(namingStrategy)
was duplicated between resolveJoinTableForeignKeyColumnName() and
joinTableColumName(). Extract it to 
HibernateAssociation#resolveAssociatedEntityTableName
so both to-one and to-many association properties share one implementation.

joinTableColumName() also resolved the collection property-name prefix via
resolveTableName(getName()) even though the result is used as a column, not a
table, on the join table. Under the default snake-case naming strategy this is
indistinguishable from resolveColumnName(), which is why it went unnoticed, but
it produces the wrong prefix under a PhysicalNamingStrategy that treats column
and table naming differently. Switch it to resolveColumnName(getName()).

Co-Authored-By: Claude Sonnet 5 <[email protected]>

Report URL: https://github.com/apache/grails-core/actions/runs/29797170331

With regards,
GitHub Actions via GitBox

Reply via email to