@ember-data/serializer 4.8.0-alpha.2 → 4.8.0-alpha.3

package/addon/-private/embedded-records-mixin.js

@@ -1,6 +1,5 @@
 import { A } from '@ember/array';
 import { warn } from '@ember/debug';
-import { get, set } from '@ember/object';
 import Mixin from '@ember/object/mixin';
 import { camelize } from '@ember/string';
 import { typeOf } from '@ember/utils';
@@ -544,7 +543,7 @@ export default Mixin.create({
   },
 
   attrsOption(attr) {
-    let attrs = this.get('attrs');
+    let attrs = this.attrs;
     return attrs && (attrs[camelize(attr)] || attrs[attr]);
   },
 
@@ -571,7 +570,7 @@ export default Mixin.create({
    @private
   */
   _extractEmbeddedHasMany(store, key, hash, relationshipMeta) {
-    let relationshipHash = get(hash, `data.relationships.${key}.data`);
+    let relationshipHash = hash.data?.relationships?.[key]?.data;
 
     if (!relationshipHash) {
       return;
@@ -592,7 +591,7 @@ export default Mixin.create({
     }
 
     let relationship = { data: hasMany };
-    set(hash, `data.relationships.${key}`, relationship);
+    hash.data.relationships[key] = relationship;
   },
 
   /**
@@ -600,7 +599,7 @@ export default Mixin.create({
    @private
   */
   _extractEmbeddedBelongsTo(store, key, hash, relationshipMeta) {
-    let relationshipHash = get(hash, `data.relationships.${key}.data`);
+    let relationshipHash = hash.data?.relationships?.[key]?.data;
     if (!relationshipHash) {
       return;
     }
@@ -615,7 +614,7 @@ export default Mixin.create({
     let belongsTo = { id: data.id, type: data.type };
     let relationship = { data: belongsTo };
 
-    set(hash, `data.relationships.${key}`, relationship);
+    hash.data.relationships[key] = relationship;
   },
 
   /**

package/addon/-private/index.js

@@ -3,7 +3,6 @@
 */
 
 export { default as EmbeddedRecordsMixin } from './embedded-records-mixin';
-export { modelHasAttributeOrRelationshipNamedType } from './utils';
 
 export { default as Transform } from './transforms/transform';
 export { default as BooleanTransform } from './transforms/boolean';

package/addon/json-api.js

@@ -710,7 +710,7 @@ const JSONAPISerializer = JSONSerializer.extend({
         }
 
         // only serialize has many relationships that are not new
-        let nonNewHasMany = hasMany.filter((item) => item.record && !item.record.get('isNew'));
+        let nonNewHasMany = hasMany.filter((item) => item.record && !item.record.isNew);
         let data = new Array(nonNewHasMany.length);
 
         for (let i = 0; i < nonNewHasMany.length; i++) {

package/addon/json.js

@@ -3,15 +3,12 @@
  */
 import { getOwner } from '@ember/application';
 import { assert, warn } from '@ember/debug';
-import { get } from '@ember/object';
 import { dasherize } from '@ember/string';
 import { isNone, typeOf } from '@ember/utils';
 
 import Serializer from '@ember-data/serializer';
 import { coerceId } from '@ember-data/store/-private';
 
-import { modelHasAttributeOrRelationshipNamedType } from './-private';
-
 const SOURCE_POINTER_REGEXP = /^\/?data\/(attributes|relationships)\/(.*)/;
 const SOURCE_POINTER_PRIMARY_REGEXP = /^\/?data/;
 const PRIMARY_ATTRIBUTE_KEY = 'base';
@@ -196,7 +193,7 @@ const JSONSerializer = Serializer.extend({
    @return {Object} data The transformed data object
   */
   applyTransforms(typeClass, data) {
-    let attributes = get(typeClass, 'attributes');
+    let attributes = typeClass.attributes;
 
     typeClass.eachTransformedAttribute((key, typeClass) => {
       if (data[key] === undefined) {
@@ -576,7 +573,7 @@ const JSONSerializer = Serializer.extend({
 
     export default class ApplicationSerializer extends JSONSerializer {
       normalize(typeClass, hash) {
-        let fields = get(typeClass, 'fields');
+        let fields = typeClass.fields;
 
         fields.forEach(function(type, field) {
           let payloadField = underscore(field);
@@ -629,7 +626,7 @@ const JSONSerializer = Serializer.extend({
     @return {String}
   */
   extractId(modelClass, resourceHash) {
-    let primaryKey = get(this, 'primaryKey');
+    let primaryKey = this.primaryKey;
     let id = resourceHash[primaryKey];
     return coerceId(id);
   },
@@ -685,7 +682,7 @@ const JSONSerializer = Serializer.extend({
       }
 
       let modelClass = this.store.modelFor(relationshipModelName);
-      if (relationshipHash.type && !modelHasAttributeOrRelationshipNamedType(modelClass)) {
+      if (relationshipHash.type && !modelClass.fields.has('type')) {
         relationshipHash.type = this.modelNameFromPayloadKey(relationshipHash.type);
       }
 
@@ -830,7 +827,7 @@ const JSONSerializer = Serializer.extend({
     @private
   */
   normalizeUsingDeclaredMapping(modelClass, hash) {
-    let attrs = get(this, 'attrs');
+    let attrs = this.attrs;
     let normalizedKey;
     let payloadKey;
 
@@ -842,11 +839,11 @@ const JSONSerializer = Serializer.extend({
           continue;
         }
 
-        if (get(modelClass, 'attributes').has(key)) {
+        if (modelClass.attributes.has(key)) {
           normalizedKey = this.keyForAttribute(key, 'deserialize');
         }
 
-        if (get(modelClass, 'relationshipsByName').has(key)) {
+        if (modelClass.relationshipsByName.has(key)) {
           normalizedKey = this.keyForRelationship(key, modelClass, 'deserialize');
         }
 
@@ -874,13 +871,13 @@ const JSONSerializer = Serializer.extend({
         '` on `' +
         modelClass.modelName +
         '`. Check your serializers attrs hash.',
-      get(modelClass, 'attributes').has(key) || get(modelClass, 'relationshipsByName').has(key),
+      modelClass.attributes.has(key) || modelClass.relationshipsByName.has(key),
       {
         id: 'ds.serializer.no-mapped-attrs-key',
       }
     );
 
-    let attrs = get(this, 'attrs');
+    let attrs = this.attrs;
     let mappedKey;
     if (attrs && attrs[key]) {
       mappedKey = attrs[key];
@@ -907,7 +904,7 @@ const JSONSerializer = Serializer.extend({
     @return {boolean} true if the key can be serialized
   */
   _canSerialize(key) {
-    let attrs = get(this, 'attrs');
+    let attrs = this.attrs;
 
     return !attrs || !attrs[key] || attrs[key].serialize !== false;
   },
@@ -923,7 +920,7 @@ const JSONSerializer = Serializer.extend({
     @return {boolean} true if the key must be serialized
   */
   _mustSerialize(key) {
-    let attrs = get(this, 'attrs');
+    let attrs = this.attrs;
 
     return attrs && attrs[key] && attrs[key].serialize === true;
   },
@@ -1110,7 +1107,7 @@ const JSONSerializer = Serializer.extend({
     if (options && options.includeId) {
       const id = snapshot.id;
       if (id) {
-        json[get(this, 'primaryKey')] = id;
+        json[this.primaryKey] = id;
       }
     }
 

package/addon/rest.js

@@ -11,8 +11,6 @@ import { singularize } from 'ember-inflector';
 import JSONSerializer from '@ember-data/serializer/json';
 import { coerceId } from '@ember-data/store/-private';
 
-import { modelHasAttributeOrRelationshipNamedType } from './-private';
-
 function makeArray(value) {
   return Array.isArray(value) ? value : [value];
 }
@@ -198,7 +196,7 @@ const RESTSerializer = JSONSerializer.extend({
     let serializer = primarySerializer;
     let modelClass = primaryModelClass;
 
-    let primaryHasTypeAttribute = modelHasAttributeOrRelationshipNamedType(primaryModelClass);
+    let primaryHasTypeAttribute = primaryModelClass.fields.has('type');
 
     if (!primaryHasTypeAttribute && hash.type) {
       // Support polymorphic records in async relationships

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ember-data/serializer",
-  "version": "4.8.0-alpha.2",
+  "version": "4.8.0-alpha.3",
   "description": "The default blueprint for ember-cli addons.",
   "keywords": [
     "ember-addon"
@@ -18,15 +18,15 @@
     "test:node": "mocha"
   },
   "dependencies": {
-    "@ember-data/private-build-infra": "4.8.0-alpha.2",
-    "@ember-data/store": "4.8.0-alpha.2",
+    "@ember-data/private-build-infra": "4.8.0-alpha.3",
+    "@ember-data/store": "4.8.0-alpha.3",
     "ember-auto-import": "^2.4.2",
     "ember-cli-babel": "^7.26.11",
     "ember-cli-test-info": "^1.0.0",
     "ember-cli-typescript": "^5.1.0"
   },
   "devDependencies": {
-    "@ember-data/unpublished-test-infra": "4.8.0-alpha.2",
+    "@ember-data/unpublished-test-infra": "4.8.0-alpha.3",
     "@ember/optional-features": "^2.0.0",
     "@ember/string": "^3.0.0",
     "@ember/test-helpers": "~2.7.0",

package/addon/-private/utils.js

@@ -1,10 +0,0 @@
-import { get } from '@ember/object';
-
-/*
-  Check if the passed model has a `type` attribute or a relationship named `type`.
- */
-function modelHasAttributeOrRelationshipNamedType(modelClass) {
-  return get(modelClass, 'attributes').has('type') || get(modelClass, 'relationshipsByName').has('type');
-}
-
-export { modelHasAttributeOrRelationshipNamedType };