Revamp DefaultCodegenConfig fromRequestBody()

[devplayer0]

This makes the behaviour of fromRequestBody() similar to that of fromParameter() (by replacing most of the existing code with code from fromParameter()) in order to fix a number of issues. Notably this includes a lack of handling for enums, both as references (as can be seen in #95) and inline.

Also:

• Add postProcessRequestBody() hook for code generators
• Fix Incorrect dataType output when using a $ref to an enum as the schema in the requestBody #95

[devplayer0]

Hi @HugoMario, I don't think it's actually a problem. (The correct schema is obtained via getSchemaFromBody() and dataType is set accordingly) Here's a snippet from the generated swift4 code:

/**

 - parameter body: (body) Pet to add to the store 
 - parameter completion: completion handler to receive the data and the error objects
 */
open class func addPet(body: NewPet, completion: @escaping ((_ data: Pet?,_ error: Error?) -> Void)) {
    addPetWithRequestBuilder(body: body).execute { (response, error) -> Void in
        completion(response?.body, error)
    }
}

Here is the corresponding section of the OpenAPI spec:

post:
      description: Creates a new pet in the store.  Duplicates are allowed
      operationId: addPet
      requestBody:
        description: Pet to add to the store
        required: true
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/NewPet'
      responses:
        '200':
          description: pet response
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Pet'
        default:
          description: unexpected error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Error'

[HugoMario]

Hey @devplayer0, sorry for this delay, i'm wondering what you mean, if it's a problem or not.

in case you wanna merge this chance, is it possible you update the PR or you think we can close this one?

[jacqueminv]

Lack of handling for enum in the requestBody is an issue for me too. Is there any plan to merge this PR?