Make realKeys available on Mongoose field adapters (#2585)

keystonejs · Mar 26, 2020 · e7e4bc1 · e7e4bc1
1 parent 63e2519
commit e7e4bc1
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 17 deletions.
diff --git a/.changeset/clever-fireants-exercise.md b/.changeset/clever-fireants-exercise.md
@@ -0,0 +1,5 @@
+---
+'@keystonejs/fields': patch
+---
+
+Internal refactor, no functional changes.
diff --git a/docs/guides/apps.md b/docs/guides/apps.md
@@ -90,10 +90,6 @@ class CustomApp {
 
 module.exports = {
   keystone,
-  apps: [
-    new GraphQLApp(),
-    new AdminUIApp(),
-    new CustomApp(),
-  ]
-}
+  apps: [new GraphQLApp(), new AdminUIApp(), new CustomApp()],
+};
 ```
diff --git a/docs/guides/custom-mutations.md b/docs/guides/custom-mutations.md
@@ -39,8 +39,8 @@ If we had a front-end application that updated the view count every time someone
 Using CRUD we'd first have to fetch the current view count:
 
 ```graphql
-query Page($id: ID!){
-  Page(where:{id: $id}) {
+query Page($id: ID!) {
+  Page(where: { id: $id }) {
     views
   }
 }

diff --git a/docs/tutorials/initial-data.md b/docs/tutorials/initial-data.md
@@ -72,7 +72,7 @@ For example:
 keystone.createItems({
   User: [
     { name: 'John Duck', email: 'john@duck.com', password: 'dolphins' },
-    { name: 'Barry', email: 'bartduisters@bartduisters.com', password: 'dolphins' }
+    { name: 'Barry', email: 'bartduisters@bartduisters.com', password: 'dolphins' },
   ],
 });
 ```

diff --git a/docs/tutorials/new-project.md b/docs/tutorials/new-project.md
@@ -109,7 +109,7 @@ In `index.js` add the following above `module.exports` and below `const keystone
 keystone.createList('Todo', {
   fields: {
     name: { type: Text },
-  }
+  },
 });
 ```
 

diff --git a/packages/auth-password/README.md b/packages/auth-password/README.md
@@ -36,6 +36,7 @@ const authStrategy = keystone.createAuthStrategy({
   },
 });
 ```
+
 **Note:** The auth strategy must be created after the User list.
 
 Later, the admin UI authentication handler will do something like this:

diff --git a/packages/fields/src/types/DateTime/Implementation.js b/packages/fields/src/types/DateTime/Implementation.js
@@ -151,21 +151,21 @@ const CommonDateTimeInterface = superclass =>
 export class MongoDateTimeInterface extends CommonDateTimeInterface(MongooseFieldAdapter) {
   constructor() {
     super(...arguments);
-    this.dbPath = `${this.path}_utc`;
+    this.utcPath = `${this.path}_utc`;
+    this.offsetPath = `${this.path}_offset`;
+    this.realKeys = [this.utcPath, this.offsetPath];
+    this.dbPath = this.utcPath;
   }
 
   addToMongooseSchema(schema) {
     const { mongooseOptions } = this.config;
-    const field_path = this.path;
-    const utc_field = `${field_path}_utc`;
-    const offset_field = `${field_path}_offset`;
     schema.add({
       // FIXME: Mongoose needs to know about this field in order for the correct
       // attributes to make it through to the pre-hooks.
-      [field_path]: { type: String, ...mongooseOptions },
+      [this.path]: { type: String, ...mongooseOptions },
       // These are the actual fields we care about storing in the database.
-      [utc_field]: { type: Date, ...mongooseOptions },
-      [offset_field]: { type: String, ...mongooseOptions },
+      [this.utcPath]: { type: Date, ...mongooseOptions },
+      [this.offsetPath]: { type: String, ...mongooseOptions },
     });
   }
 

diff --git a/packages/fields/src/types/Virtual/Implementation.js b/packages/fields/src/types/Virtual/Implementation.js
@@ -47,6 +47,10 @@ const CommonTextInterface = superclass =>
   };
 
 export class MongoVirtualInterface extends CommonTextInterface(MongooseFieldAdapter) {
+  constructor() {
+    super(...arguments);
+    this.realKeys = [];
+  }
   addToMongooseSchema() {}
 }