@topce see also discussion in the issue

We can. But I think we first need the spec to account for this (i couldn't generate any client when specifying a delete body because it produced an incorrect spec).
Once it is supported, I think we need to alter client to account for different spec versions and generate the body param for the delete call when using an openapi spec >=1.3 and angular >=12.2.
As it stands now, this is broken dead code.

Any updates on when this will be fixed?

hit same issue, was working okay on 5.3.0 - looking for how to revert

Angular CLI: 12.1.4
Node: 14.18.1
Package Manager: npm 6.14.15
OS: win32 x64

have exact issue as - #10864

so ran this command to go back to previous @openapitools/openapi-generator-cli from 2.4.23 to 2.4.13

npm uninstall @openapitools/openapi-generator-cli
npm i --save-dev @openapitools/[email protected]

then re-run the code gen

openapi-generator-cli generate -g typescript-angular -i swagger.json -o output/typescript-angular

but it still uses version in .openapi/generator/VERSION = 5.3.1

looking to just download the jar file and run the java without @openapitools/openapi-generator-cli as a work-around?

so i made this - hope it helps while the patch/spec/issues are resolved.

#!/bin/bash
rm -rf output
mkdir output
FILE="`pwd`/openapi-generator-cli-5.3.0.jar"
if [ -f "$FILE" ]; then
    echo "$FILE exists."
else
  echo "openapi-generator-cli-5.3.0.jar file not found, downloading..."
  curl "https://repo1.maven.org/maven2/org/openapitools/openapi-generator-cli/5.3.0/openapi-generator-cli-5.3.0.jar" --output "$FILE"
fi

java -jar "`pwd`/openapi-generator-cli-5.3.0.jar" generate -g typescript-angular -i swagger.json -o output/typescript-angular

I'm facing the same issue with 5.3.x and 5.2.x. Do we know that the latest working version is?

I'm facing the same issue with 5.3.x and 5.2.x. Do we know that the latest working version is?

5.1.0 seems to be working properly

@amakhrov rebased, conflict resolved.

Hello guys,

I'm not quite sure about the "requestBody" problem. My specification:

#YAML
 /server/{id}:
    parameters:
      - in: path
        name: id
        required: true
        schema:
          type: integer
          format: int64
    delete:
      tags:
        - Server
      summary: Remove server from check list
      responses:
          '200':
            description: Server successfully removed

and I'm still getting: error TS2554: Expected 1-2 arguments, but got 3.

@macjohnny can we merge this?

Hello guys,
this ticket is not included in 5.4.0 release, is a 5.4.1 in planning?

thanks.

@gong4soft looks like the next upcoming release is 6.0.0:
https://github.com/OpenAPITools/openapi-generator/milestones

But zou can already use it by installing the 6.0.0-SNAPSHOT version

@macjohnny is there a way to use it when you use the @openapitools/openapi-generator-cli?

@mgol I don‘t think the npm package supports snapshot versions, but it would be a great contribution if you implemented that

@mgol You can set downloadUrl in the openapitools.json:

{
  "$schema": "node_modules/@openapitools/openapi-generator-cli/config.schema.json",
  "spaces": 2,
  "generator-cli": {
    "version": "6.0.0-20220330.230127-109",
    "repository": {
      "downloadUrl": "https://oss.sonatype.org/content/repositories/snapshots/org/openapitools/openapi-generator-cli/6.0.0-SNAPSHOT/openapi-generator-cli-${versionName}.jar"
    }
  }
}

@KrickRay Thanks, that's very useful! Although, currently, all that's enough to resolve this issue is to set version to 6.0.0-beta and no downloadUrl needs to be set.

I'm thinking about filing a PR for this issue since I need to generate POST methods with a payload. For not making the same mistakes again / reinventing the wheel: is someone already about to fix this? And what exactly was wrong with this previous PR?

Why not always rely on the request method (instead of post / put / patch etc.)? This way the generated code would be less dependent on Angulars interfaces I guess?

@angelaki I am not aware of anybody actively working on this, but #10976 (review) sums it up.

@macjohnny so there's probably nothing wrong with relying totally on HttpClient.request? So this fix would be pretty easy ...

Would using HttpClient.request possibly ignore any http interceptors, not entirely sure where they get applied.

@maxs-rose Nope, actually all the HttpClient methods are just overloads for the request-method, e.g. https://github.com/angular/angular/blob/main/packages/common/http/src/client.ts#L2565. Is someone knowing the OpenAPICodeGen code able & willing to quickly fix this? I've never contributed to this project and imho it seams pretty easy if you just know what & where to do.

Interesting.

I believe you would want to end it the templates it uses to generate the angular code

https://github.com/OpenAPITools/openapi-generator/tree/master/modules%2Fopenapi-generator%2Fsrc%2Fmain%2Fresources%2Ftypescript-angular

I know I have made a few edits to one of them, a long time ago, then you can tell the generator to use your edited templates. Unfortunately I can't remember how you actually tell the generator to use the custom templates.

@maxs-rose wow, this looks pretty cryptic to me ... I was hoping that I'd just need to replace some post / put / ... by the corresponding request-method. I haven't event found the HttpClient calls...

Yea it's a bit wild, thankfully when I did have to make a change for work it was something obvious. I believe the HttpClient calls are in the api.service.mustache

Oh wow. Now I know why noone is willing / able to make this small fix. I still don't get why request wasn't used from the beginning. All the other methods are just overloads to make it more handy. Never the less it seams like one has to do it. Can you tell me where the variables come from? And is there an easy way to test my changes? Thank you!

Its probably better to move this into its own issue/discussion