not-node 5.1.5 → 5.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "not-node",
-  "version": "5.1.5",
+  "version": "5.1.8",
   "description": "node complimentary part for client side notFramework.",
   "main": "index.js",
   "scripts": {

package/src/auth/exceptions.js

@@ -0,0 +1,27 @@
+const { notError } = require("not-error");
+
+class RolesExceptionRoleSetIsNotValid extends notError {
+    constructor(name) {
+        super("not-node:auth_roles_role_set_is_not_valid", { name });
+    }
+}
+module.exports.RolesExceptionRoleSetIsNotValid =
+    RolesExceptionRoleSetIsNotValid;
+
+class RolesExceptionNoRolesSupremacyOrder extends notError {
+    constructor() {
+        super("not-node:auth_roles_no_roles_supremacy_order");
+    }
+}
+
+module.exports.RolesExceptionNoRolesSupremacyOrder =
+    RolesExceptionNoRolesSupremacyOrder;
+
+class RolesExceptionSupremacyOrderElementIsNotAString extends notError {
+    constructor() {
+        super("not-node:auth_roles_supremacy_order_element_is_not_a_string");
+    }
+}
+
+module.exports.RolesExceptionSupremacyOrderElementIsNotAString =
+    RolesExceptionSupremacyOrderElementIsNotAString;

package/src/auth/roles.js

@@ -1,4 +1,9 @@
 const ABSTRACT = require("./abstract");
+const {
+    RolesExceptionRoleSetIsNotValid,
+    RolesExceptionNoRolesSupremacyOrder,
+    RolesExceptionSupremacyOrderElementIsNotAString,
+} = require("./exceptions.js");
 
 function compareRolesArrayAgainstArray(userRoles, actionRoles, strict) {
     if (strict) {
@@ -48,7 +53,7 @@ function compareRoles(userRoles, actionRoles, strict = true) {
 
 function sanitizeAndValidateRoleSet(roleSet, name) {
     if (!Array.isArray(roleSet) && !ABSTRACT.isObjectString(roleSet)) {
-        throw new Error(`${name} role set is not valid`);
+        throw new RolesExceptionRoleSetIsNotValid(name);
     } else {
         if (!Array.isArray(roleSet)) {
             roleSet = [roleSet];
@@ -70,14 +75,14 @@ function checkSupremacy(base, against, roles) {
     against = sanitizeAndValidateRoleSet(against, "Against");
 
     if (!Array.isArray(roles)) {
-        throw new Error("No roles supremacy order!");
+        throw new RolesExceptionNoRolesSupremacyOrder();
     }
 
     let baseIndex = -1;
     let againstIndex = -1;
     roles.forEach((role, index) => {
         if (!ABSTRACT.isObjectString(role)) {
-            throw new Error("Supremacy order element is not a string");
+            throw new RolesExceptionSupremacyOrderElementIsNotAString();
         }
         if (baseIndex === -1 && base.indexOf(role) > -1) {
             baseIndex = index;

package/src/exceptions/db.js

@@ -2,7 +2,7 @@ const { notRequestError } = require("not-error");
 
 //delete wasnt successful, or error, or count of deleted documents dont match requested
 class DBExceptionDeleteWasNotSuccessful extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("DB Delete Was Not Successful", { code: 505, ...params }, cause);
     }
 }
@@ -11,7 +11,7 @@ module.exports.DBExceptionDeleteWasNotSuccessful =
 
 //delete wasnt successful, bc document is owned by another user
 class DBExceptionDocumentIsNotOwnerByActiveUser extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super(
             "DB Only Owners Could Delete their documents",
             { code: 403, ...params },

package/src/exceptions/http.js

@@ -4,7 +4,7 @@ const { notRequestError } = require("not-error");
 
 //form of request is not valid
 class HttpExceptionBadRequest extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("Bad Request", { code: 400, ...params }, cause);
     }
 }
@@ -12,7 +12,7 @@ module.exports.HttpExceptionBadRequest = HttpExceptionBadRequest;
 
 //user is not authenticated
 class HttpExceptionUnauthorized extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("Unauthorized", { code: 401, ...params }, cause);
     }
 }
@@ -20,7 +20,7 @@ module.exports.HttpExceptionUnauthorized = HttpExceptionUnauthorized;
 
 //user is too poor to go there
 class HttpExceptionPaymentRequired extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("Payment Required", { code: 402, ...params }, cause);
     }
 }
@@ -28,7 +28,7 @@ module.exports.HttpExceptionPaymentRequired = HttpExceptionPaymentRequired;
 
 //user identity is known, but he has no right to go there
 class HttpExceptionForbidden extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("Forbidden", { code: 403, ...params }, cause);
     }
 }
@@ -36,7 +36,7 @@ module.exports.HttpExceptionForbidden = HttpExceptionForbidden;
 
 //requested address is not exists
 class HttpExceptionNotFound extends notRequestError {
-    constructor({ params = {}, cause = null }) {
+    constructor({ params = {}, cause = null } = {}) {
         super("Not Found", { code: 404, ...params }, cause);
     }
 }

package/src/generic/logic.js

@@ -146,7 +146,7 @@ module.exports = ({
             if (shouldOwn) {
                 query[ownerFieldName] = activeUser._id;
             }
-            const result = await getModel().update(query, data).exec();
+            const result = await getModel().update(query, data);
             LogAction(
                 {
                     action,

package/src/model/exceptions.js

@@ -0,0 +1,40 @@
+const { notError } = require("not-error");
+
+class VersioningExceptionSameOldData extends notError {
+    constructor() {
+        super("not-node:versioning_error_same_old_data");
+    }
+}
+module.exports.VersioningExceptionSameOldData = VersioningExceptionSameOldData;
+
+class VersioningExceptionNoPpreviousVersions extends notError {
+    constructor() {
+        super("not-node:versioning_error_no_previous_versions");
+    }
+}
+module.exports.VersioningExceptionNoPpreviousVersions =
+    VersioningExceptionNoPpreviousVersions;
+
+class IncrementExceptionIDGeneratorRebaseFailed extends notError {
+    constructor() {
+        super("not-node:increment_id_generator_rebase_failed");
+    }
+}
+module.exports.IncrementExceptionIDGeneratorRebaseFailed =
+    IncrementExceptionIDGeneratorRebaseFailed;
+
+class IncrementExceptionIDGenerationFailed extends notError {
+    constructor() {
+        super("not-node:increment_id_generation_failed");
+    }
+}
+module.exports.IncrementExceptionIDGenerationFailed =
+    IncrementExceptionIDGenerationFailed;
+
+class IncrementExceptionFieldsNotExistInDataObject extends notError {
+    constructor(miss) {
+        super("not-node:increment_fields_not_exist_in_data_object", { miss });
+    }
+}
+module.exports.IncrementExceptionFieldsNotExistInDataObject =
+    IncrementExceptionFieldsNotExistInDataObject;

package/src/model/increment.js

@@ -1,6 +1,11 @@
 /** @module Model/Increment */
 
 const { updateResponseSuccess } = require("./utils.js");
+const {
+    IncrementExceptionIDGeneratorRebaseFailed,
+    IncrementExceptionIDGenerationFailed,
+    IncrementExceptionFieldsNotExistInDataObject,
+} = require("./exceptions.js");
 
 const thisSchema = {
     id: {
@@ -84,9 +89,7 @@ function newGetNext() {
             if (miss.length === 0) {
                 id = formId(modelName, filterFields, data);
             } else {
-                throw new Error(
-                    "Fields not exist in data object: " + miss.join(", ")
-                );
+                throw new IncrementExceptionFieldsNotExistInDataObject(miss);
             }
         }
         let which = {
@@ -106,7 +109,7 @@ function newGetNext() {
             const doc = await thisModel.findOne({ id });
             return doc.seq;
         } else {
-            throw new Error("ID generation failed");
+            throw new IncrementExceptionIDGenerationFailed();
         }
     };
 }
@@ -136,7 +139,7 @@ function newRebase() {
         if (updateResponseSuccess(res)) {
             return ID;
         } else {
-            throw new Error("ID generator rebase failed");
+            throw new IncrementExceptionIDGeneratorRebaseFailed();
         }
     };
 }

package/src/model/versioning.js

@@ -11,8 +11,10 @@ const TECH_FIELDS = [
     "__closed",
 ];
 
-const ERR_NO_PREVIOUS_VERSIONS = "No previous version!";
-const ERR_SAME_OLD_DATA = "Same old data";
+const {
+    VersioningExceptioNoPpreviousVersions,
+    VersioningExceptionSameOldData,
+} = require("./exceptions.js");
 
 class ModelVersioning {
     static versionsHistoryExists(data) {
@@ -72,7 +74,7 @@ class ModelVersioning {
                 previous.toObject()
             );
         } else {
-            throw new Error(ERR_NO_PREVIOUS_VERSIONS);
+            throw new VersioningExceptioNoPpreviousVersions();
         }
     }
 
@@ -103,7 +105,7 @@ class ModelVersioning {
                 versionDoc
             );
         }
-        throw new Error(ERR_SAME_OLD_DATA);
+        throw new VersioningExceptionSameOldData();
     }
 
     /**
@@ -162,7 +164,4 @@ class ModelVersioning {
     }
 }
 
-ModelVersioning.ERR_NO_PREVIOUS_VERSIONS = ERR_NO_PREVIOUS_VERSIONS;
-ModelVersioning.ERR_SAME_OLD_DATA = ERR_SAME_OLD_DATA;
-
 module.exports = ModelVersioning;