-
-
Notifications
You must be signed in to change notification settings - Fork 68
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1319 from bartocc/ember-data-4-x-deprecation-guides
📝 Add all 4.x ember-data deprecation pages
- Loading branch information
Showing
25 changed files
with
517 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
--- | ||
title: Deprecate Array Like | ||
until: '5.0' | ||
since: '4.7' | ||
displayId: ember-data:deprecate-array-like | ||
--- | ||
|
||
Deprecates Ember "Array-like" methods on `RecordArray` and `ManyArray`. | ||
|
||
These are the arrays returned respectively by `store.peekAll()`, `store.findAll()` and hasMany relationships on instance of Model or `record.hasMany('relationshipName').value()`. | ||
|
||
The appropriate refactor is to treat these arrays as native arrays and to use native array methods. | ||
|
||
For instance, instead of: | ||
|
||
```js | ||
users.firstObject; | ||
``` | ||
|
||
Use: | ||
|
||
```js | ||
users[0]; | ||
// or | ||
users.at(0); | ||
``` |
33 changes: 33 additions & 0 deletions
33
content/ember-data/v4/ember-data-deprecate-early-static.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
--- | ||
displayId: ember-data:deprecate-early-static | ||
title: Deprecate Early Static | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
This deprecation triggers if static computed properties or methods are triggered without looking up the record via the store service's `modelFor` hook. Accessing this static information without looking up the model via the store most commonly occurs when: | ||
|
||
- using ember-cli-mirage (to fix, refactor to not use its auto-discovery of ember-data models) | ||
- importing a model class and accessing its static information via the import | ||
|
||
Instead of: | ||
|
||
```js | ||
import User from 'my-app/models/user'; | ||
|
||
const relationships = User.relationshipsByName; | ||
``` | ||
|
||
Do _at least_ this: | ||
|
||
```js | ||
const relationships = store.modelFor('user').relationshipsByName; | ||
``` | ||
|
||
However, the much more future proof refactor is to not use `modelFor` at all, but instead to utilize the schema service for this static information: | ||
|
||
```js | ||
const relationships = store | ||
.getSchemaDefinitionService() | ||
.relationshipsDefinitionFor({ type: 'user' }); | ||
``` |
12 changes: 12 additions & 0 deletions
12
content/ember-data/v4/ember-data-deprecate-errors-array-to-hash-helper.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
--- | ||
displayId: ember-data:deprecate-errors-hash-to-array-helper | ||
title: Deprecate Errors Hash To Array Helper | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Deprecates `errorsHashToArray` `errorsArrayToHash` and `normalizeModelName`. | ||
|
||
Users making use of these (already private) utilities can trivially copy them into their own codebase to continue using them, though we recommend refactoring to a more direct conversion into the expected errors format for the errors helpers. | ||
|
||
For refactoring normalizeModelName we also recommend following the guidance in [RFC#740 Deprecate Non-Strict Types](https://github.com/emberjs/rfcs/pull/740). |
12 changes: 12 additions & 0 deletions
12
content/ember-data/v4/ember-data-deprecate-errors-hash-to-array-helper.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
--- | ||
displayId: ember-data:deprecate-errors-hash-to-array-helper | ||
title: Deprecate Errors Hash To Array Helper | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Deprecates `errorsHashToArray` `errorsArrayToHash` and `normalizeModelName`. | ||
|
||
Users making use of these (already private) utilities can trivially copy them into their own codebase to continue using them, though we recommend refactoring to a more direct conversion into the expected errors format for the errors helpers. | ||
|
||
For refactoring normalizeModelName we also recommend following the guidance in [RFC#740 Deprecate Non-Strict Types](https://github.com/emberjs/rfcs/pull/740). |
10 changes: 10 additions & 0 deletions
10
content/ember-data/v4/ember-data-deprecate-has-record-for-id.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
displayId: ember-data:deprecate-has-record-for-id | ||
title: Deprecate Has Record For Id | ||
until: '5.0' | ||
since: '4.5' | ||
--- | ||
|
||
Deprecates `store.hasRecordForId(type, id)` in favor of `store.peekRecord({ type, id }) !== null`. | ||
|
||
Broadly speaking, while the ability to query for presence is important, a key distinction exists between these methods that make relying on `hasRecordForId` unsafe, as it may report `true` for a record which is not-yet loaded and un-peekable. `peekRecord` offers a safe mechanism by which to check for whether a record is present in a usable manner. |
28 changes: 28 additions & 0 deletions
28
content/ember-data/v4/ember-data-deprecate-instantiate-record-args.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
--- | ||
displayId: ember-data:deprecate-instantiate-record-args | ||
title: Deprecate Instantiate Record Args | ||
until: '5.0' | ||
since: '4.12' | ||
--- | ||
|
||
Deprecates using the former 3rd and 4th arguments to `Store.instantiateRecord` which are now available as properties on the store. | ||
|
||
Before: | ||
|
||
```ts | ||
{ | ||
instantiateRecord(identifier, createArgs, recordDataFor, notifications) { | ||
const cache = recordDataFor(identifier); | ||
} | ||
} | ||
``` | ||
|
||
After: | ||
|
||
```ts | ||
{ | ||
instantiateRecord(identifier, createArgs) { | ||
const { cache, notifications } = this; | ||
} | ||
} | ||
``` |
27 changes: 27 additions & 0 deletions
27
content/ember-data/v4/ember-data-deprecate-model-reopen.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
--- | ||
displayId: ember-data:deprecate-model-reopen | ||
title: Deprecate Model Reopen | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
For properties known ahead of time, instead of: | ||
|
||
```ts | ||
class User extends Model { | ||
@attr firstName; | ||
} | ||
|
||
User.reopen({ lastName: attr() }); | ||
``` | ||
|
||
Extend `User` again or include it in the initial definition: | ||
|
||
```ts | ||
class User extends Model { | ||
@attr firstName; | ||
@attr lastName; | ||
} | ||
``` | ||
|
||
For properties generated dynamically, consider registering a `SchemaDefinitionService` with the store, as such services are capable of dynamically adjusting their schemas, and utilize the `instantiateRecord` hook to create a Proxy based class that can react to the changes in the schema. Use `Foo extends Model` to extend your class instead. |
28 changes: 28 additions & 0 deletions
28
content/ember-data/v4/ember-data-deprecate-model-reopenclass.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
--- | ||
displayId: ember-data:deprecate-model-reopenclass | ||
title: Deprecate Model Reopenclass | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Instead of reopenClass, define `static` properties with native class syntax or add them to the final object. | ||
|
||
Instead of: | ||
|
||
```js | ||
User.reopenClass({ aStaticMethod() {} }); | ||
``` | ||
|
||
Do this: | ||
|
||
```js | ||
class User { | ||
static aStaticMethod() {} | ||
} | ||
``` | ||
|
||
Or, do this: | ||
|
||
```js | ||
User.aStaticMethod = function () {}; | ||
``` |
112 changes: 112 additions & 0 deletions
112
content/ember-data/v4/ember-data-deprecate-non-strict-relationships.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,112 @@ | ||
--- | ||
displayId: ember-data:deprecate-non-strict-relationships | ||
title: Deprecate Non Strict Relationships | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Deprecates when belongsTo and hasMany relationships are defined without specifying whether the relationship is asynchronous. | ||
|
||
The current behavior is that relationships which do not define this setting are aschronous (`{ async: true }`). | ||
|
||
Instead of: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee') employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@belongsTo('company') company; | ||
} | ||
``` | ||
|
||
Use: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee', { async: true, inverse: 'company' }) employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@belongsTo('company', { async: true, inverse: 'employees' }) company; | ||
} | ||
``` | ||
|
||
Also deprecates when belongsTo and hasMany relationships are defined without specifying the inverse field on the related type. | ||
|
||
The current behavior is that relationships which do not define this setting have their inverse determined at runtime, which is potentially non-deterministic when mixins and polymorphism are involved. | ||
|
||
If an inverse relationship exists and you wish changes on one side to reflect onto the other side, use the inverse key. If you wish to not have changes reflected or no inverse relationship exists, specify `inverse: null`. | ||
|
||
Instead of: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee') employees; | ||
} | ||
class Employee extends Model { | ||
@belongsTo('company') company; | ||
} | ||
``` | ||
|
||
Use: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee', { async: true, inverse: 'company' }) employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@belongsTo('company', { async: true, inverse: 'employees' }) company; | ||
} | ||
``` | ||
|
||
Instead of: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee') employees; | ||
} | ||
class Employee extends Model { | ||
@attr name; | ||
} | ||
``` | ||
|
||
Use: | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee', { async: true, inverse: null }) employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@attr name; | ||
} | ||
``` | ||
|
||
And also deprecates when belongsTo and hasMany relationships are defined without specifying the inverse record's type. | ||
|
||
Instead of | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany() employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@belongsTo() company; | ||
} | ||
``` | ||
|
||
Use | ||
|
||
```js | ||
class Company extends Model { | ||
@hasMany('employee', { async: true, inverse: 'company' }) employees; | ||
} | ||
|
||
class Employee extends Model { | ||
@belongsTo('company', { async: true, inverse: 'employees' }) company; | ||
} | ||
``` |
12 changes: 12 additions & 0 deletions
12
content/ember-data/v4/ember-data-deprecate-normalize-modelname-helper.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
--- | ||
displayId: ember-data:deprecate-promise-many-array-behaviors | ||
title: Deprecate Promise Many Array Behaviors | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Deprecates `errorsHashToArray` `errorsArrayToHash` and `normalizeModelName`. | ||
|
||
Users making use of these (already private) utilities can trivially copy them into their own codebase to continue using them, though we recommend refactoring to a more direct conversion into the expected errors format for the errors helpers. | ||
|
||
For refactoring normalizeModelName we also recommend following the guidance in [RFC#740 Deprecate Non-Strict Types](https://github.com/emberjs/rfcs/pull/740). |
27 changes: 27 additions & 0 deletions
27
content/ember-data/v4/ember-data-deprecate-promise-many-array-behaviors.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
--- | ||
displayId: ember-data:deprecate-promise-many-array-behaviors | ||
title: Deprecate Promise Many Array Behaviors | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
[RFC Documentation](https://rfcs.emberjs.com/id/0745-ember-data-deprecate-methods-on-promise-many-array) | ||
|
||
This deprecation deprecates accessing values on the asynchronous proxy in favor of first "resolving" or "awaiting" the promise to retrieve a synchronous value. | ||
|
||
Template iteration of the asynchronous value will still work and not trigger the deprecation, but all JS access should be avoided and HBS access for anything but `{{#each}}` should also be refactored. | ||
|
||
Recommended approaches include using the addon `ember-promise-helpers`, using Ember's `resource` pattern (including potentially the addon `ember-data-resources`), resolving the value in routes/provider components, or using the references API. | ||
|
||
An example of using the [hasMany](https://api.emberjs.com/ember-data/4.11/classes/Model/methods/hasMany?anchor=hasMany) [reference API](https://api.emberjs.com/ember-data/release/classes/HasManyReference): | ||
|
||
```js | ||
// get the synchronous "ManyArray" value for the asynchronous "friends" relationship. | ||
// note, this will return `null` if the relationship has not been loaded yet | ||
const value = person.hasMany('friends').value(); | ||
|
||
// to get just the list of related IDs | ||
const ids = person.hasMany('friends').ids(); | ||
``` | ||
|
||
References participate in autotracking and getters/cached getters etc. which consume them will recompute if the value changes. |
22 changes: 22 additions & 0 deletions
22
content/ember-data/v4/ember-data-deprecate-promise-proxies.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
--- | ||
displayId: ember-data:deprecate-promise-proxies | ||
title: Deprecate Promise Proxies | ||
until: '5.0' | ||
since: '4.7' | ||
--- | ||
|
||
Additional Reading: [RFC#846 Deprecate Proxies](https://rfcs.emberjs.com/id/0846-ember-data-deprecate-proxies) | ||
|
||
Deprecates using the proxy object/proxy array capabilities of values returned from: | ||
|
||
- `store.findRecord` | ||
- `store.findAll` | ||
- `store.query` | ||
- `store.queryRecord` | ||
- `record.save` | ||
- `recordArray.save` | ||
- `recordArray.update` | ||
|
||
These methods will now return a native Promise that resolves with the value. | ||
|
||
Note that this does not deprecate the proxy behaviors of `PromiseBelongsTo`. See RFC for reasoning. The opportunity should still be taken if available to stop using these proxy behaviors; however, this class will remain until `import Model from '@ember-data/model';` is deprecated more broadly. |
14 changes: 14 additions & 0 deletions
14
content/ember-data/v4/ember-data-deprecate-secret-adapter-fallback.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
--- | ||
displayId: ember-data:deprecate-secret-adapter-fallback | ||
title: Deprecate Secret Adapter Fallback | ||
until: '5.0' | ||
since: '4.5' | ||
--- | ||
|
||
Deprecates the secret `-json-api` fallback adapter in favor or an explicit "catch all" application adapter. In addition to this deprecation ensuring the user has explicitly chosen an adapter, this ensures that the user may choose to use no adapter at all. | ||
|
||
Simplest fix: | ||
|
||
```js {data-filename=/app/adapters/application.js} | ||
export { default } from '@ember-data/adapter/json-api'; | ||
``` |
Oops, something went wrong.