@e22m4u/js-repository 0.8.4 → 0.8.6

package/src/relations/has-one-resolver.js

@@ -27,42 +27,48 @@ export class HasOneResolver extends Service {
     foreignKey,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includeTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasOneResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includeTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -70,14 +76,17 @@ export class HasOneResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasOneResolver.includeTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -94,7 +103,9 @@ export class HasOneResolver extends Service {
       filter.limit = 1;
       promises.push(
         targetRepository.find(filter).then(result => {
-          if (result.length) targetBySourceId.set(sourceId, result[0]);
+          if (result.length) {
+            targetBySourceId.set(sourceId, result[0]);
+          }
         }),
       );
     });
@@ -127,48 +138,55 @@ export class HasOneResolver extends Service {
     discriminator,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includePolymorphicTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!foreignKey || typeof foreignKey !== 'string')
+    }
+    if (!foreignKey || typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "foreignKey" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         foreignKey,
       );
-    if (!discriminator || typeof discriminator !== 'string')
+    }
+    if (!discriminator || typeof discriminator !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "discriminator" of HasOneResolver.includePolymorphicTo requires ' +
           'a non-empty String, but %v was given.',
         discriminator,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includePolymorphicTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const sourcePkPropName =
       this.getService(ModelDefinitionUtils).getPrimaryKeyAsPropertyName(
@@ -176,14 +194,17 @@ export class HasOneResolver extends Service {
       );
     const sourceIds = [];
     entities.forEach(entity => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of HasOneResolver.includePolymorphicTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const sourceId = entity[sourcePkPropName];
-      if (sourceIds.includes(sourceId)) return;
+      if (sourceIds.includes(sourceId)) {
+        return;
+      }
       sourceIds.push(sourceId);
     });
 
@@ -203,7 +224,9 @@ export class HasOneResolver extends Service {
       filter.limit = 1;
       promises.push(
         targetRepository.find(filter).then(result => {
-          if (result.length) targetBySourceId.set(sourceId, result[0]);
+          if (result.length) {
+            targetBySourceId.set(sourceId, result[0]);
+          }
         }),
       );
     });
@@ -234,47 +257,53 @@ export class HasOneResolver extends Service {
     targetRelationName,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (!targetRelationName || typeof targetRelationName !== 'string')
+    }
+    if (!targetRelationName || typeof targetRelationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetRelationName" of HasOneResolver.includePolymorphicByRelationName requires ' +
           'a non-empty String, but %v was given.',
         targetRelationName,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of HasOneResolver.includePolymorphicByRelationName ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
 
     const targetRelationDef = this.getService(
       ModelDefinitionUtils,
     ).getRelationDefinitionByName(targetName, targetRelationName);
-    if (targetRelationDef.type !== RelationType.BELONGS_TO)
+    if (targetRelationDef.type !== RelationType.BELONGS_TO) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasOne" relation, ' +
           'so it requires the target relation %v to be a polymorphic "belongsTo", ' +
@@ -284,7 +313,8 @@ export class HasOneResolver extends Service {
         targetRelationName,
         targetRelationDef.type,
       );
-    if (!targetRelationDef.polymorphic)
+    }
+    if (!targetRelationDef.polymorphic) {
       throw new InvalidArgumentError(
         'The relation %v of the model %v is a polymorphic "hasOne" relation, ' +
           'so it requires the target relation %v to be a polymorphic too.',
@@ -292,6 +322,7 @@ export class HasOneResolver extends Service {
         sourceName,
         targetRelationName,
       );
+    }
     const foreignKey =
       targetRelationDef.foreignKey || `${targetRelationName}Id`;
     const discriminator =

package/src/relations/references-many-resolver.js

@@ -27,59 +27,69 @@ export class ReferencesManyResolver extends Service {
     foreignKey = undefined,
     scope = undefined,
   ) {
-    if (!entities || !Array.isArray(entities))
+    if (!entities || !Array.isArray(entities)) {
       throw new InvalidArgumentError(
         'The parameter "entities" of ReferencesManyResolver.includeTo requires ' +
           'an Array of Object, but %v was given.',
         entities,
       );
-    if (!sourceName || typeof sourceName !== 'string')
+    }
+    if (!sourceName || typeof sourceName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "sourceName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         sourceName,
       );
-    if (!targetName || typeof targetName !== 'string')
+    }
+    if (!targetName || typeof targetName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "targetName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         targetName,
       );
-    if (!relationName || typeof relationName !== 'string')
+    }
+    if (!relationName || typeof relationName !== 'string') {
       throw new InvalidArgumentError(
         'The parameter "relationName" of ReferencesManyResolver.includeTo requires ' +
           'a non-empty String, but %v was given.',
         relationName,
       );
-    if (foreignKey && typeof foreignKey !== 'string')
+    }
+    if (foreignKey && typeof foreignKey !== 'string') {
       throw new InvalidArgumentError(
         'The provided parameter "foreignKey" of ReferencesManyResolver.includeTo ' +
           'should be a String, but %v was given.',
         foreignKey,
       );
-    if (scope && (typeof scope !== 'object' || Array.isArray(scope)))
+    }
+    if (scope && (typeof scope !== 'object' || Array.isArray(scope))) {
       throw new InvalidArgumentError(
         'The provided parameter "scope" of ReferencesManyResolver.includeTo ' +
           'should be an Object, but %v was given.',
         scope,
       );
+    }
     if (foreignKey == null) {
       const singularRelationName = singularize(relationName);
       foreignKey = `${singularRelationName}Ids`;
     }
     const targetIds = entities.reduce((acc, entity) => {
-      if (!entity || typeof entity !== 'object' || Array.isArray(entity))
+      if (!entity || typeof entity !== 'object' || Array.isArray(entity)) {
         throw new InvalidArgumentError(
           'The parameter "entities" of ReferencesManyResolver.includeTo requires ' +
             'an Array of Object, but %v was given.',
           entity,
         );
+      }
       const ids = entity[foreignKey];
-      if (Array.isArray(ids))
+      if (Array.isArray(ids)) {
         ids.forEach(id => {
-          if (id == null || acc.includes(id)) return;
+          if (id == null || acc.includes(id)) {
+            return;
+          }
           acc.push(id);
         });
+      }
       return acc;
     }, []);
 
@@ -102,11 +112,14 @@ export class ReferencesManyResolver extends Service {
     entities.forEach(entity => {
       const ids = entity[foreignKey];
       entity[relationName] = [];
-      if (Array.isArray(ids))
+      if (Array.isArray(ids)) {
         targets.forEach(target => {
           const targetId = target[targetPkPropName];
-          if (ids.includes(targetId)) entity[relationName].push(target);
+          if (ids.includes(targetId)) {
+            entity[relationName].push(target);
+          }
         });
+      }
     });
   }
 }

package/src/repository/repository-registry.js

@@ -51,7 +51,9 @@ export class RepositoryRegistry extends Service {
   getRepository(modelName) {
     const modelKey = modelNameToModelKey(modelName);
     let repository = this._repositories[modelKey];
-    if (repository) return repository;
+    if (repository) {
+      return repository;
+    }
     repository = new this._repositoryCtor(this.container, modelName);
     this._repositories[modelKey] = repository;
     return repository;

package/src/repository/repository.js

@@ -51,11 +51,12 @@ export class Repository extends Service {
     this._modelName = modelName;
     const modelDef = this.getService(DefinitionRegistry).getModel(modelName);
     const datasourceName = modelDef.datasource;
-    if (!datasourceName)
+    if (!datasourceName) {
       throw new InvalidArgumentError(
         'The model %v does not have a specified datasource.',
         modelName,
       );
+    }
     this._datasourceName = datasourceName;
   }
 

package/src/utils/capitalize.js

@@ -5,6 +5,8 @@
  * @returns {string}
  */
 export function capitalize(string) {
-  if (!string || typeof string !== 'string') return string;
+  if (!string || typeof string !== 'string') {
+    return string;
+  }
   return string.charAt(0).toUpperCase() + string.slice(1);
 }

package/src/utils/clone-deep.js

@@ -6,7 +6,9 @@
  * @returns {*}
  */
 export function cloneDeep(value) {
-  if (!value) return value; // null, undefined values check
+  if (!value) {
+    return value;
+  } // null, undefined values check
 
   const types = [Number, String, Boolean];
   let result;
@@ -14,7 +16,9 @@ export function cloneDeep(value) {
   // normalizing primitives if someone did new String('aaa'),
   // or new Number('444');
   types.forEach(type => {
-    if (value instanceof type) result = type(value);
+    if (value instanceof type) {
+      result = type(value);
+    }
   });
 
   if (result === undefined) {

package/src/utils/exclude-object-keys.js

@@ -8,11 +8,12 @@ import {InvalidArgumentError} from '../errors/index.js';
  * @returns {object}
  */
 export function excludeObjectKeys(obj, keys) {
-  if (typeof obj !== 'object' || !obj || Array.isArray(obj))
+  if (typeof obj !== 'object' || !obj || Array.isArray(obj)) {
     throw new InvalidArgumentError(
       'Cannot exclude keys from a non-Object value, %v was given.',
       obj,
     );
+  }
   const result = {...obj};
   keys = Array.isArray(keys) ? keys : [keys];
   keys.forEach(key => delete result[key]);

package/src/utils/get-value-by-path.js

@@ -7,8 +7,12 @@
  * @returns {*}
  */
 export function getValueByPath(obj, path, orElse = undefined) {
-  if (!obj || typeof obj !== 'object') return orElse;
-  if (!path || typeof path !== 'string') return orElse;
+  if (!obj || typeof obj !== 'object') {
+    return orElse;
+  }
+  if (!path || typeof path !== 'string') {
+    return orElse;
+  }
   const keys = path.split('.');
   let value = obj;
   for (const key of keys) {

package/src/utils/is-deep-equal.js

@@ -13,23 +13,33 @@ export function isDeepEqual(firstValue, secondValue) {
     // operator; since the typeof primitive null is object, check
     // if one of the inputs is equal to null. If one of the two
     // inputs is primitive, then I can compare them by reference.
-    if (a === null || b === null) return a === b;
-    if (typeof a !== 'object' || typeof b !== 'object') return a === b;
+    if (a === null || b === null) {
+      return a === b;
+    }
+    if (typeof a !== 'object' || typeof b !== 'object') {
+      return a === b;
+    }
     // Check if the data type of the two inputs are the same,
     // both are arrays or objects. If they are not, return false.
     const dataTypeA = Array.isArray(a) ? 'array' : 'object';
     const dataTypeB = Array.isArray(b) ? 'array' : 'object';
-    if (dataTypeA !== dataTypeB) return false;
+    if (dataTypeA !== dataTypeB) {
+      return false;
+    }
     // Use Object.keys and Object.getOwnPropertySymbols to get
     // all of enumerable and not-inherited properties of the two
     // inputs. Compare their size respectively, if one of them
     // is not equal, return false.
     const keysA = Object.keys(a);
     const keysB = Object.keys(b);
-    if (keysA.length !== keysB.length) return false;
+    if (keysA.length !== keysB.length) {
+      return false;
+    }
     const symbolsA = Object.getOwnPropertySymbols(a);
     const symbolsB = Object.getOwnPropertySymbols(b);
-    if (symbolsA.length !== symbolsB.length) return false;
+    if (symbolsA.length !== symbolsB.length) {
+      return false;
+    }
     // To handle the circular reference, initialize a WeakMap
     // that is going to keep track of the objects or arrays
     // that have been seen, in which each key is an object
@@ -58,10 +68,14 @@ export function isDeepEqual(firstValue, secondValue) {
     // the property value.
     const propertyNamesA = [...keysA, ...symbolsA];
     for (const propertyNameA of propertyNamesA) {
-      if (!Object.prototype.hasOwnProperty.call(b, propertyNameA)) return false;
+      if (!Object.prototype.hasOwnProperty.call(b, propertyNameA)) {
+        return false;
+      }
       const propertyValueA = a[propertyNameA];
       const propertyValueB = b[propertyNameA];
-      if (!compare(propertyValueA, propertyValueB)) return false;
+      if (!compare(propertyValueA, propertyValueB)) {
+        return false;
+      }
     }
     // If we get out of the loop without
     // returning false, return true.

package/src/utils/is-promise.js

@@ -7,7 +7,11 @@
  * @returns {boolean}
  */
 export function isPromise(value) {
-  if (!value) return false;
-  if (typeof value !== 'object') return false;
+  if (!value) {
+    return false;
+  }
+  if (typeof value !== 'object') {
+    return false;
+  }
   return typeof value.then === 'function';
 }

package/src/utils/model-name-to-model-key.js

@@ -7,11 +7,12 @@ import {InvalidArgumentError} from '../errors/index.js';
  * @returns {string}
  */
 export function modelNameToModelKey(modelName) {
-  if (!modelName || typeof modelName !== 'string' || /\s/.test(modelName))
+  if (!modelName || typeof modelName !== 'string' || /\s/.test(modelName)) {
     throw new InvalidArgumentError(
       'The model name should be a non-empty String ' +
         'without spaces, but %v was given.',
       modelName,
     );
+  }
   return modelName.toLowerCase().replace(/[-_]/g, '');
 }

package/src/utils/select-object-keys.js

@@ -8,30 +8,35 @@ import {InvalidArgumentError} from '../errors/index.js';
  * @returns {object}
  */
 export function selectObjectKeys(obj, keys) {
-  if (!obj || typeof obj !== 'object' || Array.isArray(obj))
+  if (!obj || typeof obj !== 'object' || Array.isArray(obj)) {
     throw new InvalidArgumentError(
       'The first argument of selectObjectKeys ' +
         'should be an Object, but %v was given.',
       obj,
     );
-  if (!Array.isArray(keys))
+  }
+  if (!Array.isArray(keys)) {
     throw new InvalidArgumentError(
       'The second argument of selectObjectKeys ' +
         'should be an Array of String, but %v was given.',
       keys,
     );
+  }
   keys.forEach(key => {
-    if (typeof key !== 'string')
+    if (typeof key !== 'string') {
       throw new InvalidArgumentError(
         'The second argument of selectObjectKeys ' +
           'should be an Array of String, but %v was given.',
         key,
       );
+    }
   });
   const result = {};
   const allKeys = Object.keys(obj);
   allKeys.forEach(key => {
-    if (keys.includes(key)) result[key] = obj[key];
+    if (keys.includes(key)) {
+      result[key] = obj[key];
+    }
   });
   return result;
 }

package/src/utils/singularize.js

@@ -5,7 +5,9 @@
  * @returns {string}
  */
 export function singularize(noun) {
-  if (!noun || typeof noun !== 'string') return noun;
+  if (!noun || typeof noun !== 'string') {
+    return noun;
+  }
   const endings = {
     ves: 'fe',
     ies: 'y',