objection 3.0.0-rc.2 → 3.0.0-rc.3

package/lib/queryBuilder/QueryBuilderOperationSupport.js

@@ -3,6 +3,7 @@
 const { isString, isFunction, isRegExp, last } = require('../utils/objectUtils');
 const { QueryBuilderContextBase } = require('./QueryBuilderContextBase');
 const { QueryBuilderUserContext } = require('./QueryBuilderUserContext');
+const { deprecate } = require('../utils/deprecate');
 
 const AllSelector = () => true;
 const SelectSelector =
@@ -494,6 +495,7 @@ class QueryBuilderOperationSupport {
   }
 
   skipUndefined() {
+    deprecate('skipUndefined() is deprecated and will be removed in objection 4.0');
     this.internalOptions().skipUndefined = true;
     return this;
   }

package/lib/queryBuilder/RelationExpression.js

@@ -291,7 +291,7 @@ function modelToNode(model, node) {
   for (let r = 0, lr = relationNames.length; r < lr; ++r) {
     const relName = relationNames[r];
 
-    if (model.hasOwnProperty(relName)) {
+    if (model[relName] !== undefined) {
       let childNode = node[relName];
 
       if (!childNode) {

package/lib/queryBuilder/graph/recursiveUpsert/GraphRecursiveUpsert.js

@@ -18,7 +18,7 @@ class GraphRecursiveUpsert extends GraphOperation {
 
 function hasRelations(obj) {
   for (const relationName of obj.constructor.getRelationNames()) {
-    if (obj.hasOwnProperty(relationName)) {
+    if (obj[relationName] !== undefined) {
       return true;
     }
   }

package/lib/relations/RelationOwner.js

@@ -71,12 +71,10 @@ class RelationOwner {
       if (values) {
         builder.whereInComposite(relatedRefs, values);
       } else {
-        builder.where(false);
-        builder.resolve([]);
+        builder.where(false).resolve([]);
       }
     } else {
-      builder.where(false);
-      builder.resolve([]);
+      builder.where(false).resolve([]);
     }
 
     return builder;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "objection",
-  "version": "3.0.0-rc.2",
+  "version": "3.0.0-rc.3",
   "description": "An SQL-friendly ORM for Node.js",
   "main": "lib/objection.js",
   "license": "MIT",

package/lib/model/modelFilter.js

@@ -1,103 +0,0 @@
-'use strict';
-
-const { isObject } = require('../utils/objectUtils');
-const { isInternalProp } = require('../utils/internalPropUtils');
-
-function omit(model, args) {
-  if (args.length === 1 && isObject(args[0])) {
-    const keys = args[0];
-
-    if (Array.isArray(keys)) {
-      omitArray(model, keys);
-    } else {
-      omitObject(model, keys);
-    }
-  } else {
-    omitArray(model, args);
-  }
-
-  return model;
-}
-
-function pick(model, args) {
-  if (args.length === 1 && isObject(args[0])) {
-    const keys = args[0];
-
-    if (Array.isArray(keys)) {
-      pickArray(model, keys);
-    } else {
-      pickObject(model, keys);
-    }
-  } else {
-    pickArray(model, args);
-  }
-
-  return model;
-}
-
-function omitObject(model, keyObj) {
-  const modelClass = model.constructor;
-  const keys = Object.keys(keyObj);
-
-  for (let i = 0, l = keys.length; i < l; ++i) {
-    const key = keys[i];
-    const value = keyObj[key];
-
-    if (value && !isInternalProp(key) && model.hasOwnProperty(key)) {
-      modelClass.omitImpl(model, key);
-    }
-  }
-}
-
-function omitArray(model, keys) {
-  const modelClass = model.constructor;
-
-  for (let i = 0, l = keys.length; i < l; ++i) {
-    const key = keys[i];
-
-    if (!isInternalProp(key) && model.hasOwnProperty(key)) {
-      modelClass.omitImpl(model, key);
-    }
-  }
-}
-
-function pickObject(model, keyObj) {
-  const modelClass = model.constructor;
-  const keys = Object.keys(model);
-
-  for (let i = 0, l = keys.length; i < l; ++i) {
-    const key = keys[i];
-
-    if (!isInternalProp(key) && !keyObj[key]) {
-      modelClass.omitImpl(model, key);
-    }
-  }
-}
-
-function pickArray(model, pick) {
-  const modelClass = model.constructor;
-  const keys = Object.keys(model);
-
-  for (let i = 0, l = keys.length; i < l; ++i) {
-    const key = keys[i];
-
-    if (!isInternalProp(key) && !contains(pick, key)) {
-      modelClass.omitImpl(model, key);
-    }
-  }
-}
-
-function contains(arr, value) {
-  for (let i = 0, l = arr.length; i < l; ++i) {
-    if (arr[i] === value) {
-      return true;
-    }
-  }
-
-  return false;
-}
-
-module.exports = {
-  omit,
-  pick,
-};