fix: Refactor handleJson function parameters to pass JsonVariant by r…

…eference
ZanzyTHEbar · Apr 6, 2024 · e4c9671 · e4c9671
1 parent 5932f2f
commit e4c9671
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/NetworkManager/examples/customHandlers.cpp b/NetworkManager/examples/customHandlers.cpp
@@ -51,7 +51,7 @@ void setupServer() {
     // made to /api/customJson
     async_server.server.addHandler(new AsyncCallbackJsonWebHandler(
         "/api/customJson",
-        [&](AsyncWebServerRequest* request, JsonVariant json) {
+        [&](AsyncWebServerRequest* request, JsonVariant& json) {
             JsonDocument doc;
             doc["hello"] = "world";
             doc["number"] = 42;

diff --git a/NetworkManager/examples/main.cpp b/NetworkManager/examples/main.cpp
@@ -136,7 +136,7 @@ void setupServer() {
 
     async_server.server.addHandler(new AsyncCallbackJsonWebHandler(
         "/api/customJson",
-        [&](AsyncWebServerRequest* request, JsonVariant json) {
+        [&](AsyncWebServerRequest* request, JsonVariant& json) {
             JsonDocument doc;
             doc["hello"] = "world";
             doc["number"] = 42;

diff --git a/NetworkManager/include/api/base_api.hpp b/NetworkManager/include/api/base_api.hpp
@@ -27,7 +27,7 @@ class BaseAPI : public API_Utilities {
     void save(AsyncWebServerRequest* request);
     void rssi(AsyncWebServerRequest* request);
 
-    void handleJson(AsyncWebServerRequest* request, JsonVariant jsonData);
+    void handleJson(AsyncWebServerRequest* request, JsonVariant& jsonData);
 
     using route_method = void (BaseAPI::*)(AsyncWebServerRequest*);
     using route_t = std::unordered_map<std::string, route_method>;

diff --git a/NetworkManager/src/api/base_api.cpp b/NetworkManager/src/api/base_api.cpp
@@ -111,7 +111,7 @@ void BaseAPI::setWiFiTXPower(AsyncWebServerRequest* request) {
     }
 }
 
-void BaseAPI::handleJson(AsyncWebServerRequest* request, JsonVariant jsonData) {
+void BaseAPI::handleJson(AsyncWebServerRequest* request, JsonVariant& jsonData) {
     switch (_networkMethodsMap_enum[request->method()]) {
         case POST: {
             auto jsonObj = jsonData.as<JsonObject>();

diff --git a/NetworkManager/src/api/rest_api_handler.cpp b/NetworkManager/src/api/rest_api_handler.cpp
@@ -49,7 +49,7 @@ void APIServer::begin() {
 
     async_server.server.addHandler(new AsyncCallbackJsonWebHandler(
         json_url.c_str(),
-        [&](AsyncWebServerRequest* request, JsonVariant json) {
+        [&](AsyncWebServerRequest* request, JsonVariant& json) {
             handleJson(request, json);
         }));