JavaScript: Errornous generation of content type values containing "="

[arikon]

Description

swagger-codegen produces such a code in API call methods:

      var contentTypes = ['application/json; charset=utf-8'];
      var accepts = ['application/json; charset=utf-8'];

The expected code is:

      var contentTypes = ['application/json; charset=utf-8'];
      var accepts = ['application/json; charset=utf-8'];

The part of spec source is:

consumes:
  - "application/json; charset=utf-8"

produces:
  - "application/json; charset=utf-8"

Swagger-codegen version

Latest master. Not sure if it is a regression.

Swagger declaration file content or url

https://gist.github.com/arikon/2f078fcf15aae07984c7e1d35def90bf

Command line used for generation

SWAGGER_JSON=swagger-new.json
SWAGGER_OUTPUT=sandbox-json-api

export JAVA_HOME=$(/usr/libexec/java_home -v 1.7)
export PATH=${JAVA_HOME}/bin:$PATH

java -jar swagger-codegen/modules/swagger-codegen-cli/target/swagger-codegen-cli.jar generate \
    -i ${SWAGGER_JSON} \
    -l javascript \
    -o ${SWAGGER_OUTPUT} \
    --additional-properties usePromises=true

Steps to reproduce

Related issues

Suggest a Fix

[wing328]

@arikon thanks for reporting the issue, which will be addressed by #2921

[wing328]

@arikon fix merged into master. Please pull the latest and give it a try.

[arikon]

@wing328 It doesn't help =(

[arikon]

@wing328 It should be fixed here also https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/resources/Javascript/api.mustache#L77-L78

[wing328]

@arikon you're right. I missed that due to the fact that it's not using the typical {{mediaType}} mustache tag. Another PR on the way to address it.

[arikon]

@wing328 Is it on purpose (different syntax)?

[wing328]

If the language itself uses a lot of {{ or }}, then mustache allows you to map {{ and }} to something else to avoid confusion.

I did a check and found only Javascript and Clojure map <, > with {{, }} respectively.