Codegen with no Prefix Overwrites Bean Files

[rrmckinley]

If codegen is run with empty string as a prefix then the bean and the "Q" class will have the same file name. Please consider separate table-model-prefix, table-model-suffix, bean-prefix, bean-suffix and independent bean-package settings to help the generated code fit a wide range of naming and package styles.

Alternately, it might be easier for just two settings that are templates for table model and bean, "org.my.db.sales.Q{0}" and "org.my.db.sales.beans.{0}Bean". Those patterns could be parsed to get the package names and the class name from the NamingStrategy could be applied in place of "{0}" or similar placeholder.

MetaDataExporter.java

    private void handleTable(DatabaseMetaData md, ResultSet tables) throws SQLException {
        String tableName = tables.getString(TABLE_NAME);
        String className = namingStrategy.getClassName(namePrefix, tableName);
        if (beanSerializer != null){
            className = className.substring(namePrefix.length());
        }
        EntityType classModel = createEntityType(tableName, className);
    ...
    }

[Timo Westkämper (timo-westkamper)]

Thanks for the bug report.

These are good additions, but I will take them on the roadmap after the 2.1.0 release.

[Timo Westkämper (timo-westkamper)]

Will be implemented namePrefix + nameSuffix + beanPrefix + beanSuffix, since this is more compatible with the rest of Querydsl

[Timo Westkämper (timo-westkamper)]

I just added the following properties namePrefix + nameSuffix + beanPrefix + beanSuffix + beanPackageName to MetaDataExporter. McKinley, could you make sure that the changes work for you?

[Timo Westkämper (timo-westkamper)]

McKinley, have you been to test the changes?

[rrmckinley (rrmckinley)]

Not yet, but likely today.

Cheers,

Rich

Sent from my phone.

On Feb 15, 2011, at 10:09 AM, Timo Westkämper <email address hidden> wrote:

> McKinley, have you been to test the changes?
>
> --
> You received this bug notification because you are a direct subscriber
> of the bug.
> https://bugs.launchpad.net/bugs/704638
>
> Title:
> Codegen with no Prefix Overwrites Bean Files
>
> Status in Querydsl:
> Fix Committed
>
> Bug description:
> If codegen is run with empty string as a prefix then the bean and the
> "Q" class will have the same file name. Please consider separate
> table-model-prefix, table-model-suffix, bean-prefix, bean-suffix and
> independent bean-package settings to help the generated code fit a
> wide range of naming and package styles.
>
> Alternately, it might be easier for just two settings that are
> templates for table model and bean, "org.my.db.sales.Q{0}" and
> "org.my.db.sales.beans.{0}Bean". Those patterns could be parsed to get
> the package names and the class name from the NamingStrategy could be
> applied in place of "{0}" or similar placeholder.
>
> MetaDataExporter.java
>
> private void handleTable(DatabaseMetaData md, ResultSet tables) throws SQLException {
> String tableName = tables.getString(TABLE_NAME);
> String className = namingStrategy.getClassName(namePrefix, tableName);
> if (beanSerializer != null){
> className = className.substring(namePrefix.length());
> }
> EntityType classModel = createEntityType(tableName, className);
> ...
> }
>
> To unsubscribe from this bug, go to:
> https://bugs.launchpad.net/querydsl/+bug/704638/+subscribe

[Timo Westkämper (timo-westkamper)]

Released in 2.1.2