Skip to content

Commit b83ddd6

Browse files
authored
Use instanceof NotFoundError for 404 checks (#7915)
1 parent 0aa25df commit b83ddd6

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

app/routes/crate.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { NotFoundError } from '@ember-data/adapter/error';
12
import Route from '@ember/routing/route';
23
import { inject as service } from '@ember/service';
34

@@ -12,7 +13,7 @@ export default class CrateRoute extends Route {
1213
try {
1314
return await this.store.findRecord('crate', crateName);
1415
} catch (error) {
15-
if (error.errors?.some(e => e.detail === 'Not Found')) {
16+
if (error instanceof NotFoundError) {
1617
let title = `${crateName}: Crate not found`;
1718
this.router.replaceWith('catch-all', { transition, error, title });
1819
} else {

app/routes/team.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { NotFoundError } from '@ember-data/adapter/error';
12
import Route from '@ember/routing/route';
23
import { inject as service } from '@ember/service';
34

@@ -23,7 +24,7 @@ export default class TeamRoute extends Route {
2324

2425
return { crates, team };
2526
} catch (error) {
26-
if (error.errors?.some(e => e.detail === 'Not Found')) {
27+
if (error instanceof NotFoundError) {
2728
this.notifications.error(`Team '${params.team_id}' does not exist`);
2829
return this.router.replaceWith('index');
2930
}

app/routes/user.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import { NotFoundError } from '@ember-data/adapter/error';
12
import Route from '@ember/routing/route';
23
import { inject as service } from '@ember/service';
34

@@ -22,7 +23,7 @@ export default class UserRoute extends Route {
2223

2324
return { crates, user };
2425
} catch (error) {
25-
if (error.errors?.some(e => e.detail === 'Not Found')) {
26+
if (error instanceof NotFoundError) {
2627
this.notifications.error(`User '${params.user_id}' does not exist`);
2728
return this.router.replaceWith('index');
2829
}

0 commit comments

Comments
 (0)