[swagger-codegen-maven-plugin:2.2.1] Use both java8 date-time and model name suffix will result wrong date-time class name

[fayndee]

Description

When using swagger-codegen-maven-plugin:2.2.1, if both dateLibrary=java8 and modelNameSuffix=Resource options are specified, it will accidentally generate fields with type for example: OffsetDateTimeResource.

Swagger-codegen version

2.2.1

Command line used for generation

Maven config:

<plugin>
    <groupId>io.swagger</groupId>
    <artifactId>swagger-codegen-maven-plugin</artifactId>
    <version>2.2.1</version>
    <executions>
        <execution>
            <goals>
                <goal>generate</goal>
            </goals>
            <configuration>
                <inputSpec>http://petstore.swagger.io/v2/swagger.json</inputSpec>
                <language>java</language>
                <library>retrofit2</library>
                <configOptions>
                    <dateLibrary>java8</dateLibrary>
                </configOptions>
                <modelNameSuffix>Resource</modelNameSuffix>
            </configuration>
        </execution>
    </executions>
</plugin>

Steps to reproduce

Run mvn package.

[wing328]

@fayndee thanks for reporting the issue.

cc @cbornet

[liqidator]

+1

[wing328]

@fayndee @liqidator it should be fixed in the latest master. Please give it a try.

If not, we'll reopen this issue and look into it again.

[liqidator]

When I use format: uuid (#4739) and modelNameSuffix=Gen, the outcome type is UuidGen, which is not compilable.
Probably best to re-open.

[wing328]

@liqidator which version are you using? 2.2.2 (released a few days ago)?

[liqidator]

Yes, I used 2.2.2

[wing328]

Reopened the issue. If you remove 'format: uuid" (which means it's just a string), would that work?

[liqidator]

Thanks. When I removed format uuid, it worked fine. But I had to use just type string for uuid

[wing328]

Ok, so I think that's an edge case my fix/work didn't handle properly.

Would you have time to contribute the fix? I can show you some good starting points.

[ePaul]

I start to regret having contributed this feature (modelNamePrefix/Suffix) :-(
... or at least this way of implementing it.