@ember-data/store 4.10.0-beta.2 → 4.10.0-beta.3

package/addon/-private/caches/instance-cache.ts

@@ -476,12 +476,6 @@ export class InstanceCache {
     const isEmpty = _isEmpty(this, identifier);
     const isLoading = _isLoading(this, identifier);
 
-    if (options.preload) {
-      this.store._join(() => {
-        preloadData(this.store, identifier, options.preload!);
-      });
-    }
-
     let promise: Promise<StableRecordIdentifier>;
     if (isEmpty) {
       assertIdentifierHasId(identifier);
@@ -617,7 +611,7 @@ export function recordDataIsFullyDeleted(cache: InstanceCache, identifier: Stabl
     models.
   */
 type PreloadRelationshipValue = RecordInstance | string;
-function preloadData(store: Store, identifier: StableRecordIdentifier, preload: Dict<unknown>) {
+export function preloadData(store: Store, identifier: StableRecordIdentifier, preload: Dict<unknown>) {
   let jsonPayload: JsonApiResource = {};
   //TODO(Igor) consider the polymorphic case
   const schemas = store.getSchemaDefinitionService();

package/addon/-private/legacy-model-support/shim-model-class.ts

@@ -81,16 +81,11 @@ export default class ShimModelClass implements ModelSchema {
     });
   }
 
-  eachTransformedAttribute<T>(
-    callback: (this: T | undefined, key: string, relationship: RelationshipSchema) => void,
-    binding?: T
-  ) {
-    let relationshipDefs = this.__store
-      .getSchemaDefinitionService()
-      .relationshipsDefinitionFor({ type: this.modelName });
-    Object.keys(relationshipDefs).forEach((key) => {
-      if (relationshipDefs[key]!.type) {
-        callback.call(binding, key, relationshipDefs[key] as RelationshipSchema);
+  eachTransformedAttribute<T>(callback: (this: T | undefined, key: string, type: string) => void, binding?: T) {
+    const attrDefs = this.__store.getSchemaDefinitionService().attributesDefinitionFor({ type: this.modelName });
+    Object.keys(attrDefs).forEach((key) => {
+      if (attrDefs[key]!.type) {
+        callback.call(binding, key, attrDefs[key]!.type);
       }
     });
   }

package/addon/-private/store-service.ts

@@ -45,6 +45,7 @@ import { IdentifierCache } from './caches/identifier-cache';
 import {
   InstanceCache,
   peekRecordIdentifier,
+  preloadData,
   recordDataIsFullyDeleted,
   recordIdentifierFor,
   setRecordIdentifier,
@@ -482,6 +483,7 @@ class Store extends Service {
     @return {subclass of Model | ShimModelClass}
     */
   // TODO @deprecate in favor of schema APIs, requires adapter/serializer overhaul or replacement
+  declare _forceShim: boolean;
   modelFor(modelName: string): ShimModelClass | DSModelClass {
     if (DEBUG) {
       assertDestroyedStoreOnly(this, 'modelFor');
@@ -497,7 +499,7 @@ class Store extends Service {
 
       // for factorFor factory/class split
       let klass = maybeFactory && maybeFactory.class ? maybeFactory.class : maybeFactory;
-      if (!klass || !klass.isModel) {
+      if (!klass || !klass.isModel || this._forceShim) {
         assert(
           `No model was found for '${modelName}' and no schema handles the type`,
           this.getSchemaDefinitionService().doesTypeExist(modelName)
@@ -1127,6 +1129,12 @@ class Store extends Service {
     let promise;
     options = options || {};
 
+    if (options.preload) {
+      this._join(() => {
+        preloadData(this, identifier, options!.preload!);
+      });
+    }
+
     // if not loaded start loading
     if (!this._instanceCache.recordIsLoaded(identifier)) {
       promise = this._instanceCache._fetchDataIfNeededForIdentifier(identifier, options);
@@ -1134,6 +1142,7 @@ class Store extends Service {
       // Refetch if the reload option is passed
     } else if (options.reload) {
       assertIdentifierHasId(identifier);
+
       promise = this._fetchManager.scheduleFetch(identifier, options);
     } else {
       let snapshot: Snapshot | null = null;
@@ -2310,8 +2319,8 @@ class Store extends Service {
 
     const saveOptions = Object.assign({ [SaveOp]: operation }, options);
     let fetchManagerPromise = this._fetchManager.scheduleSave(identifier, saveOptions);
-    return fetchManagerPromise.then(
-      (payload) => {
+    return fetchManagerPromise
+      .then((payload) => {
         if (LOG_PAYLOADS) {
           try {
             let data = payload ? JSON.parse(JSON.stringify(payload)) : payload;
@@ -2366,8 +2375,8 @@ class Store extends Service {
           }
         });
         return record;
-      },
-      (e) => {
+      })
+      .catch((e) => {
         let err = e;
         if (!e) {
           err = new Error(`Unknown Error Occurred During Request`);
@@ -2376,8 +2385,7 @@ class Store extends Service {
         }
         adapterDidInvalidate(this, identifier, err);
         throw err;
-      }
-    );
+      });
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ember-data/store",
-  "version": "4.10.0-beta.2",
+  "version": "4.10.0-beta.3",
   "description": "The core of EmberData. Provides the Store service which coordinates the cache with the network and presentation layers.",
   "keywords": [
     "ember-addon"
@@ -14,9 +14,9 @@
   "author": "",
   "directories": {},
   "peerDependencies": {
-    "@ember-data/model": "4.10.0-beta.2",
-    "@ember-data/record-data": "4.10.0-beta.2",
-    "@ember-data/tracking": "4.10.0-beta.2",
+    "@ember-data/model": "4.10.0-beta.3",
+    "@ember-data/record-data": "4.10.0-beta.3",
+    "@ember-data/tracking": "4.10.0-beta.3",
     "@ember/string": "^3.0.0",
     "@glimmer/tracking": "^1.1.2"
   },
@@ -37,8 +37,8 @@
     }
   },
   "dependencies": {
-    "@ember-data/canary-features": "4.10.0-beta.2",
-    "@ember-data/private-build-infra": "4.10.0-beta.2",
+    "@ember-data/canary-features": "4.10.0-beta.3",
+    "@ember-data/private-build-infra": "4.10.0-beta.3",
     "@embroider/macros": "^1.10.0",
     "ember-auto-import": "^2.5.0",
     "ember-cached-decorator-polyfill": "^1.0.1",