Add uim.errors dependency and refactor plugin command argument handling

commands/uim/commands/classes/plugins/load.d

@@ -93,7 +93,7 @@ class DPluginLoadCommand : DCommand {
   DConsoleOptionParser buildOptionParser(DConsoleOptionParser parser) {
     with (parser) {
       description("Command for loading plugins.");
-      addArgument("plugin", MapHelper.create!(string, Json)
+/*       addArgument("plugin", MapHelper.create!(string, Json)
           .set("help", "Name of the plugin to load. Must be in CamelCase format. Example: uim plugin load Example")
           .set("required", true)
       );
@@ -105,22 +105,29 @@ class DPluginLoadCommand : DCommand {
           .set("boolean", true)
           .set("help", "Load the plugin only for CLI.")
       );
-      addOption("optional", MapHelper.create!(string, Json)
-          .set("boolean", true)
-          .set("help", "Do not throw an error if the plugin is not available.")
-      );
+      Json[string] optional;
+      optional.set("boolean", true);
+      optional.set("help", "Do not throw an error if the plugin is not available.");
+      addOption("optional", optional);
+
+      Json[string] noBootstrap;
       addOption("no-bootstrap", MapHelper.create!(string, Json)
           .set("boolean", true)
           .set("help", "Do not run the `bootstrap()` hook.")
       );
+      
+      Json[string] noConsole;
       addOption("no-console", MapHelper.create!(string, Json)
           .set("boolean", true)
           .set("help", "Do not run the `console()` hook.")
       );
-      addOption("no-middleware", MapHelper.create!(string, Json)
-          .set("boolean", true)
-          .set("help", "Do not run the `middleware()` hook..")
-      );
+      
+      Json[string] noMiddleware;
+      noMiddleware.set("boolean", true);
+      noMiddleware.set("help", "Do not run the `middleware()` hook..");
+      addOption("no-middleware", noMiddleware);
+
+      Json[string] noRoutes;
       addOption("no-routes", MapHelper.create!(string, Json)
           .set("boolean", true)
           .set("help", "Do not run the `routes()` hook.")
@@ -129,7 +136,7 @@ class DPluginLoadCommand : DCommand {
           .set("boolean", true)
           .set("help", "Do not run the `services()` hook.")
       );
-
+ */
     }
     return parser;
   }

commands/uim/commands/classes/plugins/unload.d

@@ -67,10 +67,10 @@ class DPluginUnloadCommand : DCommand {
     DConsoleOptionParser buildOptionParser(DConsoleOptionParser parsertoUpdate) {
         parsertoUpdate.description("Command for unloading plugins.");
 
-        parsertoUpdate.addArgument("plugin", MapHelper.create!(string, Json)
-                .set("help", "Name of the plugin to unload.")
-                .set("required", true)
-            );
+        Json[string] map;
+        map.set("help", "Name of the plugin to unload.");
+        map.set("required", true);
+        parsertoUpdate.addArgument("plugin", map);
 
         return parsertoUpdate;
     }

core/uim/core/convert/json.d

@@ -0,0 +1,103 @@
+module uim.core.convert.json;
+
+import uim.core;
+@safe:
+
+version (test_uim_core) {
+  unittest {
+    writeln("-----  ", __MODULE__, "\t  -----");
+  }
+}
+
+// #region toJson
+Json toJson(bool value) {
+  return Json(value);
+}
+
+Json toJson(long value) {
+  return Json(value);
+}
+
+Json toJson(double value) {
+  return Json(value);
+}
+
+Json toJson(string value) {
+  return Json(value);
+}
+
+Json toJson(UUID value) {
+  return toJson(value.toString);
+}
+
+Json toJson(string aKey, string aValue) {
+  Json result = Json.emptyObject;
+  result[aKey] = aValue;
+  return result;
+}
+
+unittest {
+  assert(toJson("a", "3")["a"] == "3");
+}
+
+Json toJson(string aKey, UUID aValue) {
+  Json result = Json.emptyObject;
+  result[aKey] = aValue.toString;
+  return result;
+}
+
+unittest {
+  auto id = randomUUID;
+  assert(UUID(toJson("id", id)["id"].get!string) == id);
+}
+
+/// Special case for managing entities
+Json toJson(UUID id, size_t versionNumber) {
+  Json result = toJson("id", id);
+  result["versionNumber"] = versionNumber;
+  return result;
+}
+
+unittest {
+  auto id = randomUUID;
+  assert(toJson(id, 0).getString("id") == id.toString);
+  assert("versionNumber" in toJson(id, 0));
+  assert(toJson(id, 1)["id"].get!string == id.toString);
+  assert(toJson(id, 1)["versionNumber"].get!size_t == 1);
+}
+
+Json toJson(bool[] values) {
+  auto json = Json.emptyArray;
+  values.each!(value => json ~= value);
+  return json;
+}
+
+unittest {
+  assert([true, true, false].toJson.length == 3);
+  assert([true, true, false].toJson[0].getBoolean);
+}
+
+Json toJson(string[] values) {
+  auto json = Json.emptyArray;
+  values.each!(value => json ~= value);
+  return json;
+}
+
+unittest {
+  assert(["a", "b", "c"].toJson.length == 3);
+  assert(["a", "b", "c"].toJson[0] == "a");
+}
+
+Json toJson(string[string] map, string[] excludeKeys = null) {
+  Json json = Json.emptyObject;
+  map.byKeyValue
+    .filter!(kv => !excludeKeys.any!(key => key == kv.key))
+    .each!(kv => json[kv.key] = kv.value);
+  return json;
+}
+
+unittest {
+  assert(["a": "1", "b": "2", "c": "3"].toJson.length == 3);
+  assert(["a": "1", "b": "2", "c": "3"].toJson["a"] == "1");
+}
+// #endregion toJson

core/uim/core/convert/package.d

@@ -1,6 +1,7 @@
 module uim.core.convert;
 
 public {
+  import uim.core.convert.json;
   import uim.core.convert.jsonmap;
   import uim.core.convert.string_;
 }

core/uim/core/datatypes/json/base.d

@@ -1060,98 +1060,6 @@ unittest {
 }
 // #endregion setter
 
-// #region toJson
-Json toJson(bool value) {
-  return Json(value);
-}
-
-Json toJson(long value) {
-  return Json(value);
-}
-
-Json toJson(double value) {
-  return Json(value);
-}
-
-Json toJson(string value) {
-  return Json(value);
-}
-
-Json toJson(UUID value) {
-  return toJson(value.toString);
-}
-
-Json toJson(string aKey, string aValue) {
-  Json result = Json.emptyObject;
-  result[aKey] = aValue;
-  return result;
-}
-
-unittest {
-  assert(toJson("a", "3")["a"] == "3");
-}
-
-Json toJson(string aKey, UUID aValue) {
-  Json result = Json.emptyObject;
-  result[aKey] = aValue.toString;
-  return result;
-}
-
-unittest {
-  auto id = randomUUID;
-  assert(UUID(toJson("id", id)["id"].get!string) == id);
-}
-
-/// Special case for managing entities
-Json toJson(UUID id, size_t versionNumber) {
-  Json result = toJson("id", id);
-  result["versionNumber"] = versionNumber;
-  return result;
-}
-
-unittest {
-  auto id = randomUUID;
-  assert(toJson(id, 0).getString("id") == id.toString);
-  assert("versionNumber" in toJson(id, 0));
-  assert(toJson(id, 1)["id"].get!string == id.toString);
-  assert(toJson(id, 1)["versionNumber"].get!size_t == 1);
-}
-
-Json toJson(bool[] values) {
-  auto json = Json.emptyArray;
-  values.each!(value => json ~= value);
-  return json;
-}
-
-unittest {
-  assert([true, true, false].toJson.length == 3);
-  assert([true, true, false].toJson[0].getBoolean);
-}
-
-Json toJson(string[] values) {
-  auto json = Json.emptyArray;
-  values.each!(value => json ~= value);
-  return json;
-}
-
-unittest {
-  assert(["a", "b", "c"].toJson.length == 3);
-  assert(["a", "b", "c"].toJson[0] == "a");
-}
-
-Json toJson(string[string] map, string[] excludeKeys = null) {
-  Json json = Json.emptyObject;
-  map.byKeyValue
-    .filter!(kv => !excludeKeys.any!(key => key == kv.key))
-    .each!(kv => json[kv.key] = kv.value);
-  return json;
-}
-
-unittest {
-  assert(["a": "1", "b": "2", "c": "3"].toJson.length == 3);
-  assert(["a": "1", "b": "2", "c": "3"].toJson["a"] == "1");
-}
-// #endregion toJson
 
 Json match(K)(Json[K] matchValues, K key, Json defaultValue = Json(null)) {
   return key in matchValues

css/dub.sdl

@@ -13,6 +13,7 @@ dependency "vibe-d" version="~>0.10.0"
 
 dependency "uim:core" path="../"
 dependency "uim:oop" path="../"
+dependency "uim:errors" path="../"
 
 sourcePaths "."
 importPaths "."

css/uim/css/package.d

@@ -10,6 +10,7 @@ public {
 public { // Required uim libraries 
 	import uim.core;
 	import uim.oop;
+	import uim.errors;
 
 	// import uim.filesystems;
 	// import uim.models;