Merge pull request #675 from feathersjs-ecosystem/not-destructure-fea…

…thers-errors

fix(esm): import errors from @feathersjs/errors

src/hooks/alter-items.ts

@@ -1,4 +1,6 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
+
 import type { Hook, HookContext } from '@feathersjs/feathers';
 import { isPromise } from '../common';
 import { getItems } from '../utils/get-items';

src/hooks/disallow.ts

@@ -1,4 +1,5 @@
-import { MethodNotAllowed } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { MethodNotAllowed } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import type { TransportName } from '../types';
 

src/hooks/keep-in-array.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import _get from 'lodash/get';
 import _set from 'lodash/set';
 import _has from 'lodash/has';

src/hooks/keep-query-in-array.ts

@@ -1,5 +1,6 @@
 import { checkContext } from '../utils/check-context';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import _get from 'lodash/get';
 import _set from 'lodash/set';
 import _has from 'lodash/has';

src/hooks/lower-case.ts

@@ -1,5 +1,6 @@
 import _set from 'lodash/set';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 
 import { transformItems } from '../common';
 import { checkContextIf } from './check-context-if';

src/hooks/populate.ts

@@ -1,6 +1,7 @@
 import _get from 'lodash/get';
 import _set from 'lodash/set';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 
 import { hrtime } from 'process';
 

src/hooks/prevent-changes.ts

@@ -2,7 +2,8 @@ import _has from 'lodash/has';
 import _omit from 'lodash/omit';
 
 import { checkContext } from '../utils/check-context';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 
 /**

src/hooks/required.ts

@@ -1,5 +1,6 @@
 import _get from 'lodash/get';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import _has from 'lodash/has';
 
 import { checkContext } from '../utils/check-context';

src/hooks/run-parallel.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 
 /**

src/hooks/set-field.ts

@@ -3,7 +3,8 @@ import _setWith from 'lodash/setWith';
 import _clone from 'lodash/clone';
 import _debug from 'debug';
 import { checkContext } from '../utils/check-context';
-import { Forbidden } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { Forbidden } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import type { SetFieldOptions } from '../types';
 

src/hooks/set-now.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import { setFields as _setFields } from '../common';
 import { getItems } from '../utils/get-items';

src/hooks/set-slug.ts

@@ -1,5 +1,6 @@
 import _set from 'lodash/set';
-import { GeneralError } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { GeneralError } = errors;
 import type { Hook } from '@feathersjs/feathers';
 
 /**

src/hooks/sifter.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import type { SyncContextFunction } from '../types';
 import { checkContext } from '../utils/check-context';

src/hooks/soft-delete.ts

@@ -1,4 +1,5 @@
-import { GeneralError } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { GeneralError } = errors;
 import type { SoftDeleteOptions } from '../types';
 import { checkContext } from '../utils/check-context';
 

src/hooks/stash-before.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import { checkContext } from '../utils/check-context';
 

src/hooks/validate-schema.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import type { AjvOrNewable, ValidateSchemaOptions } from '../types';
 import { getItems } from '../utils/get-items';

src/hooks/validate.ts

@@ -1,4 +1,5 @@
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 import type { Hook } from '@feathersjs/feathers';
 import { isPromise } from '../common';
 import type { ValidatorFn } from '../types';

src/utils/is-not.ts

@@ -1,4 +1,5 @@
-import { MethodNotAllowed } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { MethodNotAllowed } = errors;
 import { isPromise } from '../common';
 import type { PredicateFn } from '../types';
 

src/utils/is-provider.ts

@@ -1,4 +1,5 @@
-import { MethodNotAllowed } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { MethodNotAllowed } = errors;
 import type { SyncContextFunction, TransportName } from '../types';
 
 /**

test/hooks/validate.test.ts

@@ -1,6 +1,7 @@
 import { assert } from 'chai';
 import { validate } from '../../src';
-import { BadRequest } from '@feathersjs/errors';
+import errors from '@feathersjs/errors';
+const { BadRequest } = errors;
 
 let fcnSync: any;
 let fcnPromise: any;