Fix NPE in Elm path parameter (!4116) · Merge requests · OpenAPI Tools / openapi-generator

Merged William Cheng requested to merge fix_elm_in_ci into master Oct 10, 2019

PR checklist

Read the contribution guidelines.
Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
Filed the PR against the correct branch: master, 4.1.x, 5.0.x. Default: master.
Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

fix using baseName instead of paramName
re-enable elm in ensure-up-to-date script
minor code formatting (java class)

To repeat the issue, run java -jar ./modules/openapi-generator-cli/target/openapi-generator-cli.jar generate -i modules/openapi-generator/src/test/resources/2_0/petstore-with-fake-endpoints-models-for-testing.yaml -g elm -o /tmp/openapi-generator-test-fake-petstore/2.0/elm

cc @eriktim