[MAVEN] #3284: Made modelNamePrefix and -Suffix available through maven

[d0x]

As described in #3284

[wing328]

@d0x thanks for the PR.

cc @ePaul

[ePaul]

Looks nice. (I didn't test is yet.)

Previously I just passed this argument through the configOptions → additionalProperties parameter, but this way is certainly nicer (and documented).

[d0x]

Thanks.
What exactly you mean by "just passed it through configOptions -> additionalProperties"?

This wasn't successful for me:

 <plugin>
                <groupId>io.swagger</groupId>
                <artifactId>swagger-codegen-maven-plugin</artifactId>
                <version>2.2.0-SNAPSHOT</version>
                <executions>
                    <execution>
                        <goals>
                            <goal>generate</goal>
                        </goals>
                        <configuration>
                            <!-- specify the swagger yaml -->
                            <inputSpec>swagger.yaml</inputSpec>

                            <!-- target to generate -->
                            <language>java</language>

                            <!-- pass any necessary config options -->
                            <configOptions>
                                <dateLibrary>java8</dateLibrary>
                                <modelNameSuffix>DTO</modelNameSuffix>
                            </configOptions>

                            <!-- override the default library to jersey2 -->
                            <library>jersey2</library>
                        </configuration>
                    </execution>
                </executions>
            </plugin>

And because it is a parameter in the cli I thought it makes sense to put it there as well.

If there is a way over the configOptions which you prefer, I could live with that too.

[ePaul]

@d0x I think I used something like this (I don't have a project using this here right now):

                  <configOptions>
                      <additionalProperties>modelNameSuffix=DTO</additionalProperties>
                  </configOptions>

I actually prefer your way, I just was too lazy to build this in when adding the modelNameSuffix feature.

[ePaul]

You have a copy+paste error here, this should be configurator.setModelNameSuffix(...).

[ePaul]

With the fixed configurator call, it does what it pretends to do.

So 👍 depending on fixing what I mentioned.

Thankyou for the pull request, and sorry for taking so long to review it.

[d0x]

I fixed the c&p error. Sorry for that.

Have a nice sunday.

[ePaul]

👍 – can be merged from my point of view.

[wing328]

@d0x thanks for the contribution. PR merged into master (ready for 2.2.0 stable release)

@ePaul thanks for reviewing the change.