@bedrockio/model 0.11.2 → 0.11.3

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.11.3
+
+- Added warning when id field not passed for unique check.
+
 ## 0.11.2
 
 - Fixed issue with non-async hooks hanging.

package/dist/cjs/soft-delete.js

@@ -6,8 +6,10 @@ Object.defineProperty(exports, "__esModule", {
 exports.applySoftDelete = applySoftDelete;
 exports.assertUnique = assertUnique;
 var _lodash = require("lodash");
+var _warn = _interopRequireDefault(require("./warn"));
 var _query = require("./query");
 var _errors = require("./errors");
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 function applySoftDelete(schema) {
   applyQueries(schema);
   applyUniqueConstraints(schema);
@@ -32,7 +34,7 @@ async function assertUnique(options) {
   };
   const exists = await model.exists(query);
   if (exists) {
-    const message = getUniqueErrorMessage(model, field);
+    const message = getUniqueErrorMessage(field, options);
     throw new _errors.UniqueConstraintError(message, {
       model,
       field,
@@ -40,7 +42,11 @@ async function assertUnique(options) {
     });
   }
 }
-function getUniqueErrorMessage(model, field) {
+function getUniqueErrorMessage(field, options) {
+  const {
+    id,
+    model
+  } = options;
   const {
     modelName
   } = model;
@@ -48,6 +54,9 @@ function getUniqueErrorMessage(model, field) {
     const name = field === 'phone' ? 'phone number' : field;
     return `A user with that ${name} already exists.`;
   } else {
+    if (!id) {
+      (0, _warn.default)('Note "id" field was not passed to exclude self for unique check.');
+    }
     return `"${field}" already exists.`;
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bedrockio/model",
-  "version": "0.11.2",
+  "version": "0.11.3",
   "description": "Bedrock utilities for model creation.",
   "type": "module",
   "scripts": {

package/src/soft-delete.js

@@ -1,5 +1,7 @@
 import { isEqual } from 'lodash';
 
+import warn from './warn';
+
 import { wrapQuery } from './query';
 import { UniqueConstraintError } from './errors';
 
@@ -24,8 +26,9 @@ export async function assertUnique(options) {
   };
 
   const exists = await model.exists(query);
+
   if (exists) {
-    const message = getUniqueErrorMessage(model, field);
+    const message = getUniqueErrorMessage(field, options);
     throw new UniqueConstraintError(message, {
       model,
       field,
@@ -34,12 +37,16 @@ export async function assertUnique(options) {
   }
 }
 
-function getUniqueErrorMessage(model, field) {
+function getUniqueErrorMessage(field, options) {
+  const { id, model } = options;
   const { modelName } = model;
   if (modelName === 'User' && !field.includes('.')) {
     const name = field === 'phone' ? 'phone number' : field;
     return `A user with that ${name} already exists.`;
   } else {
+    if (!id) {
+      warn('Note "id" field was not passed to exclude self for unique check.');
+    }
     return `"${field}" already exists.`;
   }
 }

package/types/soft-delete.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"soft-delete.d.ts","sourceRoot":"","sources":["../src/soft-delete.js"],"names":[],"mappings":"AAKA,mDAIC;AAED,0DAuBC"}
+{"version":3,"file":"soft-delete.d.ts","sourceRoot":"","sources":["../src/soft-delete.js"],"names":[],"mappings":"AAOA,mDAIC;AAED,0DAwBC"}