Bug fix 3.5/fix view creation errors (arangodb#9599)

KVS85 · web-flow · commit 40c27d283e94 · 2019-07-29T20:41:51.000+03:00
* Fix ArangoSearch view creation errors

* Fix another error
diff --git a/arangod/RestHandler/RestViewHandler.cpp b/arangod/RestHandler/RestViewHandler.cpp
@@ -166,25 +166,30 @@ void RestViewHandler::createView() {
     return;
   }
 
-  auto badParamError = [&]() -> void {
-    generateError(rest::ResponseCode::BAD, TRI_ERROR_BAD_PARAMETER,
-                  "expecting body to be of the form {name: <string>, type: "
-                  "<string>, properties: <object>}");
-  };
-
   if (!body.isObject()) {
-    badParamError();
+    generateError(rest::ResponseCode::BAD, TRI_ERROR_BAD_PARAMETER,
+                  "request body is not an object");
     events::CreateView(_vocbase.name(), "", TRI_ERROR_BAD_PARAMETER);
     return;
   }
 
   auto nameSlice = body.get(StaticStrings::DataSourceName);
   auto typeSlice = body.get(StaticStrings::DataSourceType);
 
-  if (!nameSlice.isString() || !typeSlice.isString()) {
-    badParamError();
+  if (!nameSlice.isString()) {
+    generateError(rest::ResponseCode::BAD, TRI_ERROR_BAD_PARAMETER,
+                  "expecting name parameter to be of the form of \"name: "
+                  "<string>\"");
+    events::CreateView(_vocbase.name(), "", TRI_ERROR_BAD_PARAMETER);
+    return;
+  }
+
+  if (!typeSlice.isString()) {
+    generateError(rest::ResponseCode::BAD, TRI_ERROR_BAD_PARAMETER,
+                  "expecting type parameter to be of the form of \"type: "
+                  "<string>\"");
     events::CreateView(_vocbase.name(),
-                       (nameSlice.isString() ? nameSlice.copyString() : ""),
+                       nameSlice.copyString(),
                        TRI_ERROR_BAD_PARAMETER);
     return;
   }
diff --git a/arangod/RestServer/ViewTypesFeature.cpp b/arangod/RestServer/ViewTypesFeature.cpp
@@ -44,7 +44,7 @@ struct InvalidViewFactory : public arangodb::ViewFactory {
     return arangodb::Result(
         TRI_ERROR_BAD_PARAMETER,
         std::string(
-            "failure to create view without a factory for definition: ") +
+            "invalid type provided to create view with definition: ") +
             definition.toString());
   }
 
@@ -54,7 +54,7 @@ struct InvalidViewFactory : public arangodb::ViewFactory {
     return arangodb::Result(
         TRI_ERROR_BAD_PARAMETER,
         std::string(
-            "failure to instantiate view without a factory for definition: ") +
+            "invalid type provided to instantiate view with definition: ") +
             definition.toString());
   }
 };
