fix: falsy IDs breaking relationships (#730)

postlight · Jul 2, 2018 · 56225da · 56225da
1 parent 667febd
commit 56225da
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/packages/database/query/runner/utils/build-results.js b/src/packages/database/query/runner/utils/build-results.js
@@ -108,7 +108,7 @@ export default async function buildResults<T: Model>({
         .reduce((r, entry) => {
           let [key, value] = entry;
 
-          if (!value && pkPattern.test(key)) {
+          if (value == null && pkPattern.test(key)) {
             return r;
           } else if (key.indexOf('.') >= 0) {
             const [a, b] = key.split('.');

diff --git a/src/packages/serializer/index.js b/src/packages/serializer/index.js
@@ -616,7 +616,7 @@ class Serializer<T: Model> {
                   })
                 )
               };
-            } else if (related && related.id) {
+            } else if (related && related.id != null) {
               return this.formatRelationship({
                 domain,
                 included,