[TS][Inverisify] Adding support for RxJS 6

[gualtierim]

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\.
• Filed the PR against the correct branch: master, 3.4.x, 4.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

Adding support to rxjs6 and patch http method for typescript-inverisify's language

[gualtierim]

@nicokoenig @TiFu @taxpon @sebastianhaas @macjohnny

[macjohnny]

I think map should be added to the reserved keywords

[gualtierim]

I used the standard import of map operator of rxjs as suggested in the documentation.

[macjohnny]

yes, but if a user defines an api parameter named map, this clashes, so we should add map to the list of reserved keywords, similar to this:

openapi-generator/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractTypeScriptClientCodegen.java

Lines 64 to 69 in 50fddc7

	reservedWords.addAll(Arrays.asList(
	// local variable names used in API methods (endpoints)
	"varLocalPath", "queryParameters", "headerParams", "formParams", "useFormData", "varLocalDeferred",
	"requestOptions",
	// Typescript reserved words
	"abstract", "await", "boolean", "break", "byte", "case", "catch", "char", "class", "const", "continue", "debugger", "default", "delete", "do", "double", "else", "enum", "export", "extends", "false", "final", "finally", "float", "for", "function", "goto", "if", "implements", "import", "in", "instanceof", "int", "interface", "let", "long", "native", "new", "null", "package", "private", "protected", "public", "return", "short", "static", "super", "switch", "synchronized", "this", "throw", "transient", "true", "try", "typeof", "var", "void", "volatile", "while", "with", "yield"));

[gualtierim]

I thought you refered to the reserved key keywords of javascript and I could not understand you. Thanks for the example. I added map to reserved keywords.

[wing328]

I've filed #3020 to rename "map" to "GlobalImportOperators" so that map doesn't need to be a reserved word.

[macjohnny]

i suggest to use double quotes as in the rest of the code

[macjohnny]

why is this changed? it is not related to typescript-inversify

[gualtierim]

I added a config's propertyuseRxJS6 to allow the user to decide the using of rxjs6 or rxjs5

[macjohnny]

here you are modifying the typescript-angular file

[gualtierim]

Sorry, I did a big mistake

[gualtierim]

@macjohnny In the list of api client the typescript-inversify is not listed. Please can you insert it?

[macjohnny]

@gualtierim which list do you mean? you can add it yourself as part of this PR if you find the corresponding file.

[wing328]

@gualtierim thanks again for your contribution, which has been included in the v4.0.1 release (https://twitter.com/oas_generator/status/1135534738658062336)