Skip to content
GitLab

[cpp-restbed] Added "out-of-the-box" functionality

Merged Administrator requested to merge github/fork/AIexG/improve_cpp_restbed into master

Created by: AIexG

With those changes the cpp-restbed Codegen should generate a server which automatically stores data received through a request in objects and also creates necessary instances of objects. The goal was to have the server compile out of the box, to offer a quick start with projects.

  • Fixed missing objects created from request bodies in api class files.
  • Fixed member variable names starting with an underscore ("_") generating the same getters and setters as members without the underscore, which led to duplicate generation of the same functions.
  • Added default value make_shared<Object>() to shared pointers to allow automated object generation
  • Added classes getting derived from their respective interfaces
  • Added override of updateAllModels() to clean interfaces of ambiguity
  • Improves ENUM handling by getting only saved if value is present in spezification

Requires the changes from my PRs #4753 and #4758.

PR checklist

  • Read the contribution guidelines.
  • If contributing template-only or documentation-only changes which will change sample output, build the project before.
  • Run the shell script(s) under ./bin/ (or Windows batch scripts under.\bin\windows) to update Petstore samples related to your fix. This is important, as CI jobs will verify all generator outputs of your HEAD commit, and these must match the expectations made by your contribution. You only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the code or mustache templates for a language ({LANG}) (e.g. php, ruby, python, etc).
  • File the PR against the correct branch: master, 4.3.x, 5.0.x. Default: master.
  • Copy the technical committee to review the pull request if your PR is targeting a particular programming language.

Additional context

Not sure if I should commit the generated server files for the samples as well, please tell me or remove them if necessary.

Requires the changes from my PRs #4753 to compile.

First pull request(s), hopefully doing everything right!

technical committee

@ravinikam (2017/07) @stkrwork (2017/07) @etherealjoy (2018/02) @MartinDelille (2018/03) @muttleyxd (2019/08)