[C++][Pistache] Fixed #2643 (#2653)

* Fixed #2643

Refactored to/from json functions to use universal object serialization method.

* Code review found incorrect indenting and I forgot to remove unused mustache files.

* Removed helpers class because it is not needed anymore.

* Removed helpers package from docs.

* Reverted helper class removal.

Author: SalDiAngelus

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

@@ -81,8 +81,6 @@ public CppPistacheServerCodegen() {
 
         reservedWords = new HashSet<>();
 
-        supportingFiles.add(new SupportingFile("modelbase-header.mustache", "model", modelNamePrefix + "ModelBase.h"));
-        supportingFiles.add(new SupportingFile("modelbase-source.mustache", "model", modelNamePrefix + "ModelBase.cpp"));
         supportingFiles.add(new SupportingFile("helpers-header.mustache", "model", modelNamePrefix + "Helpers.h"));
         supportingFiles.add(new SupportingFile("helpers-source.mustache", "model", modelNamePrefix + "Helpers.cpp"));
         supportingFiles.add(new SupportingFile("main-api-server.mustache", "", modelNamePrefix + "main-api-server.cpp"));
@@ -124,8 +122,6 @@ public void processOpts() {
         if (additionalProperties.containsKey("modelNamePrefix")) {
             additionalProperties().put("prefix", modelNamePrefix);
             supportingFiles.clear();
-            supportingFiles.add(new SupportingFile("modelbase-header.mustache", "model", modelNamePrefix + "ModelBase.h"));
-            supportingFiles.add(new SupportingFile("modelbase-source.mustache", "model", modelNamePrefix + "ModelBase.cpp"));
             supportingFiles.add(new SupportingFile("helpers-header.mustache", "model", modelNamePrefix + "Helpers.h"));
             supportingFiles.add(new SupportingFile("helpers-source.mustache", "model", modelNamePrefix + "Helpers.cpp"));
             supportingFiles.add(new SupportingFile("main-api-server.mustache", "", modelNamePrefix + "main-api-server.cpp"));

modules/openapi-generator/src/main/resources/cpp-pistache-server/README.mustache

@@ -41,7 +41,7 @@ cd build
 ## Libraries required
 - [pistache](http://pistache.io/quickstart)
 - [JSON for Modern C++](https://github.com/nlohmann/json/#integration): Please download the `json.hpp` file and
-put it under the model folder
+put it under the model/nlohmann folder
 
 ## Namespaces
 {{{apiPackage}}}

modules/openapi-generator/src/main/resources/cpp-pistache-server/api-source.mustache

@@ -70,14 +70,7 @@ void {{classname}}::{{operationIdSnakeCase}}_handler(const Pistache::Rest::Reque
     try {
     {{#hasBodyParam}}
     {{#bodyParam}}
-      nlohmann::json request_body = nlohmann::json::parse(request.body());
-    {{^isPrimitiveType}}{{^isContainer}}
-      {{paramName}}.fromJson(request_body);
-    {{/isContainer}}{{/isPrimitiveType}}{{#isContainer}}  {{paramName}} = {{#isListContainer}}  {{prefix}}ArrayHelper{{/isListContainer}}{{#isMapContainer}}  {{prefix}}MapHelper{{/isMapContainer}}::fromJson<{{items.baseType}}>(request_body);{{/isContainer}}
-    {{#isPrimitiveType}}
-      // The conversion is done automatically by the json library
-      {{paramName}} = request_body;
-    {{/isPrimitiveType}}
+      nlohmann::json::parse(request.body()).get_to({{paramName}});
     {{/bodyParam}}
     {{/hasBodyParam}}
       this->{{operationIdSnakeCase}}({{#allParams}}{{paramName}}{{#hasMore}}, {{/hasMore}}{{/allParams}}{{#hasParams}}, {{/hasParams}}response);

modules/openapi-generator/src/main/resources/cpp-pistache-server/cmake.mustache

@@ -20,7 +20,6 @@ ExternalProject_Add(NLOHMANN
 
 )
 include_directories(${EXTERNAL_INSTALL_LOCATION}/include)
-include_directories(${EXTERNAL_INSTALL_LOCATION}/include/nlohmann)
 link_directories(${EXTERNAL_INSTALL_LOCATION}/lib)
 {{/addExternalLibs}}
 

modules/openapi-generator/src/main/resources/cpp-pistache-server/model-header.mustache

@@ -9,10 +9,9 @@
 #define {{classname}}_H_
 
 {{{defaultInclude}}}
-#include "{{prefix}}ModelBase.h"
-
 {{#imports}}{{{this}}}
 {{/imports}}
+#include <nlohmann/json.hpp>
 
 {{#modelNamespaceDeclarations}}
 namespace {{this}} {
@@ -22,19 +21,12 @@ namespace {{this}} {
 /// {{description}}
 /// </summary>
 class {{declspec}} {{classname}}
-    : public {{prefix}}ModelBase
 {
 public:
     {{classname}}();
     virtual ~{{classname}}();
 
-    /////////////////////////////////////////////
-    /// {{prefix}}ModelBase overrides
-
-    void validate() override;
-
-    nlohmann::json toJson() const override;
-    void fromJson(const nlohmann::json& json) override;
+    void validate();
 
     /////////////////////////////////////////////
     /// {{classname}} members
@@ -51,6 +43,8 @@ public:
     {{/required}}
     {{/vars}}
 
+    friend void to_json(nlohmann::json& j, const {{classname}}& o);
+    friend void from_json(const nlohmann::json& j, {{classname}}& o);
 protected:
     {{#vars}}
     {{{dataType}}} m_{{name}};

modules/openapi-generator/src/main/resources/cpp-pistache-server/model-source.mustache

@@ -25,126 +25,26 @@ void {{classname}}::validate()
     // TODO: implement validation
 }
 
-nlohmann::json {{classname}}::toJson() const
+void to_json(nlohmann::json& j, const {{classname}}& o)
 {
-    nlohmann::json val = nlohmann::json::object();
-
-    {{#vars}}{{#isPrimitiveType}}{{^isListContainer}}{{^required}}if(m_{{name}}IsSet)
-    {
-        val["{{baseName}}"] = m_{{name}};
-    }
-    {{/required}}{{#required}}val["{{baseName}}"] = m_{{name}};
-    {{/required}}{{/isListContainer}}{{/isPrimitiveType}}{{#isListContainer}}{
-        nlohmann::json jsonArray;
-        for( const auto& item : m_{{name}} )
-        {
-            jsonArray.push_back({{prefix}}ModelBase::toJson(item));
-        }
-        {{#required}}val["{{baseName}}"] = jsonArray;
-        {{/required}}{{^required}}
-        if(jsonArray.size() > 0)
-        {
-            val["{{baseName}}"] = jsonArray;
-        } {{/required}}
-    }
-    {{/isListContainer}}{{#isMapContainer}}{
-        nlohmann::json jsonObj;
-        for( const auto& item : m_{{name}} )
-        {   {{^items.isContainer}} 
-            jsonObj[item.first] = {{prefix}}ModelBase::toJson(item.second);{{/items.isContainer}} {{#items.isListContainer}} 
-            jsonObj[item.first] = {{prefix}}ArrayHelper::toJson<{{{items.items.datatype}}}>(item.second);
-            {{/items.isListContainer}} {{#items.isMapContainer}} 
-            jsonObj[item.first] = {{prefix}}MapHelper::toJson<{{{items.items.datatype}}}>(item.second); {{/items.isMapContainer}} 
-        }
-        {{#required}}val["{{baseName}}"] = jsonObj; {{/required}}{{^required}}
-        if(jsonObj.size() > 0)
-        {
-            val["{{baseName}}"] = jsonObj;
-        } {{/required}}
-    }
-    {{/isMapContainer}}{{^isContainer}}{{^isPrimitiveType}}{{^required}}if(m_{{name}}IsSet)
-    {
-        val["{{baseName}}"] = {{prefix}}ModelBase::toJson(m_{{name}});
-    }
-    {{/required}}{{#required}}val["{{baseName}}"] = {{prefix}}ModelBase::toJson(m_{{name}});
-    {{/required}}{{/isPrimitiveType}}{{/isContainer}}{{/vars}}
-
-    return val;
+    j = nlohmann::json();
+    {{#vars}}
+    {{#required}}j["{{baseName}}"] = o.m_{{name}};{{/required}}{{^required}}if(o.{{nameInCamelCase}}IsSet())
+        j["{{baseName}}"] = o.m_{{name}};{{/required}}
+    {{/vars}}
 }
 
-void {{classname}}::fromJson(const nlohmann::json& val)
+void from_json(const nlohmann::json& j, {{classname}}& o)
 {
-    {{#vars}}{{#isPrimitiveType}}{{^isListContainer}}{{^required}}if(val.find("{{baseName}}") != val.end())
-    {
-        {{setter}}(val.at("{{baseName}}"));
-    }
-    {{/required}}{{#required}}{{setter}}(val.at("{{baseName}}"));
-    {{/required}}{{/isListContainer}}{{/isPrimitiveType}}{{#isListContainer}}{
-        m_{{name}}.clear();
-        {{^required}}if(val.find("{{baseName}}") != val.end())
-        {
-        {{/required}}
-            for( const auto& item : val["{{baseName}}"] )
-            {
-                {{#isPrimitiveType}}m_{{name}}.push_back(item);
-                {{/isPrimitiveType}}{{^isPrimitiveType}}{{#items.isString}}m_{{name}}.push_back(item);
-                {{/items.isString}}{{^items.isString}}{{#items.isDateTime}}m_{{name}}.push_back(item);
-                {{/items.isDateTime}}{{^items.isDateTime}}
-                if(item.is_null())
-                {
-                    m_{{name}}.push_back( {{{items.datatype}}}() );
-                }
-                else
-                {
-                    {{{items.datatype}}} newItem;
-                    newItem.fromJson(item);
-                    m_{{name}}.push_back( newItem );
-                }
-                {{/items.isDateTime}}{{/items.isString}}{{/isPrimitiveType}}
-            }
-        {{^required}}
-        }
-        {{/required}}
-    }
-    {{/isListContainer}}{{#isMapContainer}}{
-        m_{{name}}.clear();
-        {{^required}}if(val.find("{{baseName}}") != val.end())
-        {
-        {{/required}}
-            if(val["{{baseName}}"].is_object()) { {{^items.isContainer}}
-                m_{{name}} = {{prefix}}MapHelper::fromJson<{{{items.datatype}}}>(val["{{baseName}}"]);
-                {{/items.isContainer}} {{#items.isContainer}}
-                for( const auto& item : val["{{baseName}}"].items() )
-                {   {{#items.isMapContainer}}
-                    {{{items.datatype}}} newItem = {{prefix}}MapHelper::fromJson<{{{items.items.datatype}}}>(item.value());
-                    {{/items.isMapContainer}}{{#items.isListContainer}} 
-                    {{{items.datatype}}} newItem = {{prefix}}ArrayHelper::fromJson<{{{items.items.datatype}}}>(item.value());
-                    {{/items.isListContainer}}
-                    m_{{name}}.insert(m_{{name}}.end(), std::pair< std::string, {{{items.datatype}}} >(item.key(), newItem));
-                } {{/items.isContainer}}
-            }
-        {{^required}}
-        }
-        {{/required}}
-    }
-    {{/isMapContainer}}{{^isContainer}}{{^isPrimitiveType}}{{^required}}if(val.find("{{baseName}}") != val.end())
+    {{#vars}}
+    {{#required}}j.at("{{baseName}}").get_to(o.m_{{name}});{{/required}}{{^required}}if(!j.at("{{baseName}}").is_null())
     {
-        {{#isString}}{{setter}}(val.at("{{baseName}}"));{{/isString}}{{#isByteArray}}{{setter}}(val.at("{{baseName}}"));{{/isByteArray}}{{#isBinary}}{{setter}}(val.at("{{baseName}}"));
-        {{/isBinary}}{{^isString}}{{#isDateTime}}{{setter}}(val.at("{{baseName}}"));
-        {{/isDateTime}}{{^isDateTime}}{{^isByteArray}}{{^isBinary}}if(!val["{{baseName}}"].is_null())
-        {
-            {{{dataType}}} newItem;
-            newItem.fromJson(val["{{baseName}}"]);
-            {{setter}}( newItem );
-        }
-        {{/isBinary}}{{/isByteArray}}{{/isDateTime}}{{/isString}}
-    }
-    {{/required}}{{#required}}{{#isString}}{{setter}}(val.at("{{baseName}}"));
-    {{/isString}}{{^isString}}{{#isDateTime}}{{setter}}(val.at("{{baseName}}"));
-    {{/isDateTime}}{{/isString}}{{/required}}{{/isPrimitiveType}}{{/isContainer}}{{/vars}}
+        j.at("{{baseName}}").get_to(o.m_{{name}});
+        o.m_{{name}}IsSet = true;
+    } {{/required}}
+    {{/vars}}
 }
 
-
 {{#vars}}{{#isContainer}}{{{dataType}}}& {{classname}}::{{getter}}()
 {
     return m_{{name}};