fix(hooks): Update regular hook order as suggested in #97

feathersjs · Jun 4, 2022 · 9b758fc · 9b758fc
1 parent bbd1979
commit 9b758fc
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/main/hooks/src/regular.ts b/main/hooks/src/regular.ts
@@ -21,10 +21,12 @@ export const runHook = (hook: RegularMiddleware, context: any, type?: string) =>
     });
 };
 
-export const runHooks = (hooks: RegularMiddleware[]) => (context: any) => hooks.reduce(
-  (promise, hook) => promise.then(() => runHook(hook, context)),
-  Promise.resolve(context),
-);
+export const runHooks = (hooks: RegularMiddleware[]) =>
+  (context: any) =>
+    hooks.reduce(
+      (promise, hook) => promise.then(() => runHook(hook, context)),
+      Promise.resolve(context),
+    );
 
 export function fromBeforeHook(hook: RegularMiddleware) {
   return (context: any, next: any) => {
@@ -63,5 +65,5 @@ export function collect(
   const afterHooks = [...after].reverse().map(fromAfterHook);
   const errorHooks = error.length ? [fromErrorHook(runHooks(error))] : [];
 
-  return compose([...errorHooks, ...afterHooks, ...beforeHooks]);
+  return compose([...errorHooks, ...beforeHooks, ...afterHooks]);
 }