clarify types

emberjs · Dec 6, 2019 · ba7c14c · ba7c14c
1 parent b8d1496
commit ba7c14c
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/packages/store/addon/-private/system/snapshot.ts b/packages/store/addon/-private/system/snapshot.ts
@@ -163,6 +163,7 @@ export default class Snapshot {
     if (CUSTOM_MODEL_CLASS) {
       attrs.forEach(keyName => {
         if (schemaIsDSModel(this.type)) {
+          // if the schema is for a DSModel then the instance is too
           attributes[keyName] = get(record as DSModel, keyName);
         } else {
           attributes[keyName] = recordDataFor(this._internalModel).getAttr(keyName);
@@ -457,6 +458,7 @@ export default class Snapshot {
         callback.call(binding, key, attrDefs[key]);
       });
     } else {
+      // in the non CUSTOM_MODEL_CLASS world we only have DSModel instances
       (this.record as DSModel).eachAttribute(callback, binding);
     }
   }
@@ -484,6 +486,7 @@ export default class Snapshot {
         callback.call(binding, key, relationshipDefs[key]);
       });
     } else {
+      // in the non CUSTOM_MODEL_CLASS world we only have DSModel instances
       (this.record as DSModel).eachRelationship(callback, binding);
     }
   }