Set toNative() to be not enumerable (#2388)

knex · Jan 3, 2018 · 59f6cba · 59f6cba
1 parent 45f5ffb
commit 59f6cba
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/src/query/compiler.js b/src/query/compiler.js
@@ -52,13 +52,21 @@ assign(QueryCompiler.prototype, {
       timeout: this.timeout,
       cancelOnTimeout: this.cancelOnTimeout,
       bindings: this.formatter.bindings || [],
-      __knexQueryUid: uuid.v4(),
-      toNative: () => ({
-        sql: this.client.positionBindings(query.sql),
-        bindings: this.client.prepBindings(query.bindings)
-      })
+      __knexQueryUid: uuid.v4()
     };
 
+    Object.defineProperties(query, {
+      toNative: {
+        value: () => {
+          return {
+            sql: this.client.positionBindings(query.sql),
+            bindings: this.client.prepBindings(query.bindings)
+          };
+        },
+        enumerable: false
+      }
+    });
+
     if (isString(val)) {
       query.sql = val;
     } else {