feathers-utils 1.9.2 → 1.9.5

package/dist/esm/hooks/runPerItem.d.ts

@@ -1,3 +1,3 @@
 import type { HookRunPerItemOptions, ReturnAsyncHook, Promisable } from "../types";
 import type { HookContext } from "@feathersjs/feathers";
-export declare const runPerItem: (actionPerItem: (item: any, context: HookContext) => Promisable<any>, options: HookRunPerItemOptions) => ReturnAsyncHook;
+export declare const runPerItem: (actionPerItem: (item: any, context: HookContext) => Promisable<any>, _options?: HookRunPerItemOptions | undefined) => ReturnAsyncHook;

package/dist/esm/hooks/runPerItem.js

@@ -8,8 +8,8 @@ const makeOptions = (options) => {
 };
 export const runPerItem = (
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-actionPerItem, options) => {
-    options = makeOptions(options);
+actionPerItem, _options) => {
+    const options = makeOptions(_options);
     return async (context) => {
         if (shouldSkip("runForItems", context)) {
             return context;

package/dist/esm/index.d.ts

@@ -4,7 +4,7 @@ import { runPerItem } from "./hooks/runPerItem";
 export declare const hooks: {
     checkMulti: typeof checkMulti;
     setData: typeof setData;
-    runPerItem: (actionPerItem: (item: any, context: import("@feathersjs/feathers").HookContext<any, import("@feathersjs/feathers").Service<any>>) => any, options: import("./types").HookRunPerItemOptions) => import("./types").ReturnAsyncHook;
+    runPerItem: (actionPerItem: (item: any, context: import("@feathersjs/feathers").HookContext<any, import("@feathersjs/feathers").Service<any>>) => any, _options?: import("./types").HookRunPerItemOptions | undefined) => import("./types").ReturnAsyncHook;
 };
 export { checkMulti };
 export { setData };

package/dist/esm/utils/getItemsIsArray.js

@@ -1,10 +1,18 @@
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export const getItemsIsArray = (context) => {
-    let itemOrItems = context.type === "before" ? context.data : context.result;
-    itemOrItems = itemOrItems && context.method === "find" ? (itemOrItems.data || itemOrItems) : itemOrItems;
+    let itemOrItems = context.type === "before"
+        ? context.data
+        : context.result;
+    itemOrItems = itemOrItems && context.method === "find"
+        ? (itemOrItems.data || itemOrItems)
+        : itemOrItems;
     const isArray = Array.isArray(itemOrItems);
     return {
-        items: (isArray) ? itemOrItems : [itemOrItems],
+        items: (isArray)
+            ? itemOrItems
+            : (itemOrItems != null)
+                ? [itemOrItems]
+                : [],
         isArray
     };
 };

package/dist/hooks/runPerItem.d.ts

@@ -1,3 +1,3 @@
 import type { HookRunPerItemOptions, ReturnAsyncHook, Promisable } from "../types";
 import type { HookContext } from "@feathersjs/feathers";
-export declare const runPerItem: (actionPerItem: (item: any, context: HookContext) => Promisable<any>, options: HookRunPerItemOptions) => ReturnAsyncHook;
+export declare const runPerItem: (actionPerItem: (item: any, context: HookContext) => Promisable<any>, _options?: HookRunPerItemOptions | undefined) => ReturnAsyncHook;

package/dist/hooks/runPerItem.js

@@ -20,8 +20,8 @@ const makeOptions = (options) => {
 };
 const runPerItem = (
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-actionPerItem, options) => {
-    options = makeOptions(options);
+actionPerItem, _options) => {
+    const options = makeOptions(_options);
     return (context) => __awaiter(void 0, void 0, void 0, function* () {
         if ((0, shouldSkip_1.shouldSkip)("runForItems", context)) {
             return context;

package/dist/index.d.ts

@@ -4,7 +4,7 @@ import { runPerItem } from "./hooks/runPerItem";
 export declare const hooks: {
     checkMulti: typeof checkMulti;
     setData: typeof setData;
-    runPerItem: (actionPerItem: (item: any, context: import("@feathersjs/feathers").HookContext<any, import("@feathersjs/feathers").Service<any>>) => any, options: import("./types").HookRunPerItemOptions) => import("./types").ReturnAsyncHook;
+    runPerItem: (actionPerItem: (item: any, context: import("@feathersjs/feathers").HookContext<any, import("@feathersjs/feathers").Service<any>>) => any, _options?: import("./types").HookRunPerItemOptions | undefined) => import("./types").ReturnAsyncHook;
 };
 export { checkMulti };
 export { setData };

package/dist/index.js

@@ -1,7 +1,11 @@
 "use strict";
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
 }) : (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
     o[k2] = m[k];

package/dist/utils/getItemsIsArray.js

@@ -3,11 +3,19 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.getItemsIsArray = void 0;
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const getItemsIsArray = (context) => {
-    let itemOrItems = context.type === "before" ? context.data : context.result;
-    itemOrItems = itemOrItems && context.method === "find" ? (itemOrItems.data || itemOrItems) : itemOrItems;
+    let itemOrItems = context.type === "before"
+        ? context.data
+        : context.result;
+    itemOrItems = itemOrItems && context.method === "find"
+        ? (itemOrItems.data || itemOrItems)
+        : itemOrItems;
     const isArray = Array.isArray(itemOrItems);
     return {
-        items: (isArray) ? itemOrItems : [itemOrItems],
+        items: (isArray)
+            ? itemOrItems
+            : (itemOrItems != null)
+                ? [itemOrItems]
+                : [],
         isArray
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "feathers-utils",
-  "version": "1.9.2",
+  "version": "1.9.5",
   "description": "Some utils for projects using '@feathersjs/feathers'",
   "author": "fratzinger",
   "repository": {
@@ -46,22 +46,22 @@
   },
   "devDependencies": {
     "@istanbuljs/nyc-config-typescript": "^1.0.2",
-    "@types/lodash": "^4.14.178",
+    "@types/lodash": "^4.14.179",
     "@types/mocha": "^9.1.0",
-    "@types/node": "^17.0.14",
-    "@typescript-eslint/eslint-plugin": "^5.10.2",
-    "@typescript-eslint/parser": "^5.10.2",
+    "@types/node": "^17.0.21",
+    "@typescript-eslint/eslint-plugin": "^5.13.0",
+    "@typescript-eslint/parser": "^5.13.0",
     "cross-env": "^7.0.3",
-    "eslint": "^8.8.0",
+    "eslint": "^8.10.0",
     "eslint-import-resolver-typescript": "^2.5.0",
     "eslint-plugin-import": "^2.25.4",
     "eslint-plugin-security": "^1.4.0",
     "feathers-memory": "^4.1.0",
-    "mocha": "^9.2.0",
+    "mocha": "^9.2.1",
     "np": "^7.6.0",
     "nyc": "^15.1.0",
     "shx": "^0.3.4",
-    "ts-node": "^10.4.0",
-    "typescript": "^4.5.5"
+    "ts-node": "^10.5.0",
+    "typescript": "^4.6.2"
   }
 }

package/src/hooks/runPerItem.ts

@@ -5,7 +5,7 @@ import type { HookContext } from "@feathersjs/feathers";
 import { getItemsIsArray } from "../utils/getItemsIsArray";
 
 const makeOptions = (
-  options: HookRunPerItemOptions
+  options?: HookRunPerItemOptions
 ): Required<HookRunPerItemOptions> => {
   options = options || {};
   return Object.assign({
@@ -16,9 +16,9 @@ const makeOptions = (
 export const runPerItem = (
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   actionPerItem: (item: any, context: HookContext) => Promisable<any>, 
-  options: HookRunPerItemOptions
+  _options?: HookRunPerItemOptions
 ): ReturnAsyncHook => {
-  options = makeOptions(options);
+  const options = makeOptions(_options);
   return async (context: HookContext): Promise<HookContext> => {
     if (shouldSkip("runForItems", context)) { return context; }
     const { items } = getItemsIsArray(context);

package/src/utils/getItemsIsArray.ts

@@ -5,11 +5,19 @@ import type { GetItemsIsArrayOptions } from "..";
 export const getItemsIsArray = <T = any>(
   context: HookContext
 ): GetItemsIsArrayOptions<T> => {
-  let itemOrItems = context.type === "before" ? context.data : context.result;
-  itemOrItems = itemOrItems && context.method === "find" ? (itemOrItems.data || itemOrItems) : itemOrItems;
+  let itemOrItems = context.type === "before" 
+    ? context.data 
+    : context.result;
+  itemOrItems = itemOrItems && context.method === "find" 
+    ? (itemOrItems.data || itemOrItems) 
+    : itemOrItems;
   const isArray = Array.isArray(itemOrItems);
   return {
-    items: (isArray) ? itemOrItems : [itemOrItems],
+    items: (isArray) 
+      ? itemOrItems 
+      : (itemOrItems != null) 
+        ? [itemOrItems] 
+        : [],
     isArray
   };
 };