@adminforth/crud-approve-plugin 1.0.2 → 1.0.4

package/index.ts

@@ -182,14 +182,24 @@ export default class CRUDApprovePlugin extends AdminForthPlugin {
         updates,
         oldRecord,
       });
-      if (!resp || (!resp.ok && !resp.error)) {
-        throw new Error(`Hook beforeSave must return object with {ok: true} or { error: 'Error' } `);
+      if (!resp || (typeof resp.ok !== 'boolean' && (!resp.error && !resp.newRecordId))) {
+        throw new Error(
+          `Invalid return value from beforeSave hook. Expected: { ok: boolean, error?: string | null, newRecordId?: any }.\n` +
+          `Note: Return { ok: false, error: null, newRecordId } to stop creation and redirect to an existing record.`
+        );
+      }
+      if (resp.ok === false && !resp.error) {
+        const { error, ok, newRecordId } = resp;
+        return {
+          error: error ?? 'Operation aborted by hook',
+          newRecordId: newRecordId
+        };
       }
-
       if (resp.error) {
         return { error: resp.error };
       }
     }
+
     return { ok: true, error: null };
   }
 
@@ -342,7 +352,7 @@ export default class CRUDApprovePlugin extends AdminForthPlugin {
           const beforeSaveResp = await this.callBeforeSaveHooks(
             resource, action as AllowedActionsEnum, diffData['newRecord'], 
             adminUser, diffRecord[this.options.resourceColumns.recordIdColumnName],
-            undefined, diffData['oldRecord'], this.adminforth, extra
+            diffData['newRecord'], diffData['oldRecord'], this.adminforth, extra
           );
           if (beforeSaveResp.error) {
             response.status = 500;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adminforth/crud-approve-plugin",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "publishConfig": {
     "access": "public"
   },