jssm 5.49.0 → 5.52.0

package/dist/es6/jssm.d.ts

@@ -96,7 +96,8 @@ declare class Machine<mDT> {
     hook(from: string, to: string, handler: HookHandler): Machine<mDT>;
     hook_action(from: string, to: string, action: string, handler: HookHandler): Machine<mDT>;
     hook_any_transition(handler: HookHandler): Machine<mDT>;
-    action(name: StateType, newData?: mDT): boolean;
+    transition_impl(newStateOrAction: StateType, newData: mDT | undefined, wasForced: boolean, wasAction: boolean): boolean;
+    action(actionName: StateType, newData?: mDT): boolean;
     transition(newState: StateType, newData?: mDT): boolean;
     force_transition(newState: StateType, newData?: mDT): boolean;
     current_action_for(action: StateType): number;

package/dist/es6/jssm.js

@@ -746,47 +746,30 @@ class Machine {
     // remove_hook(HookDesc: HookDescription) {
     //   throw 'TODO: Should remove hook here';
     // }
-    action(name, newData) {
-        // todo whargarbl implement hooks
-        // todo whargarbl implement data stuff
-        // todo major incomplete whargarbl comeback
-        if (this.valid_action(name, newData)) {
-            const edge = this.current_action_edge_for(name);
-            if (this._has_hooks) {
-                let hook_permits = undefined;
-                if (this._any_transition_hook !== undefined) {
-                    if (this._any_transition_hook() === false) {
-                        return false;
-                    }
-                }
-                const nhn = named_hook_name(this._state, edge.to, name), maybe_hook = this._named_hooks.get(nhn);
-                if (maybe_hook === undefined) {
-                    hook_permits = true;
-                }
-                else {
-                    hook_permits = maybe_hook({ from: this._state, to: edge.to, action: name });
-                }
-                if (hook_permits !== false) {
-                    this._state = edge.to;
-                    return true;
-                }
-                else {
-                    return false;
-                }
+    transition_impl(newStateOrAction, newData, wasForced, wasAction) {
+        let valid = false, newState;
+        if (wasForced) {
+            if (this.valid_force_transition(newStateOrAction, newData)) {
+                valid = true;
+                newState = newStateOrAction;
             }
-            else {
-                this._state = edge.to;
-                return true;
+        }
+        else if (wasAction) {
+            if (this.valid_action(newStateOrAction, newData)) {
+                const edge = this.current_action_edge_for(newStateOrAction);
+                valid = true;
+                newState = edge.to;
             }
         }
         else {
-            return false;
+            if (this.valid_transition(newStateOrAction, newData)) {
+                valid = true;
+                newState = newStateOrAction;
+            }
         }
-    }
-    transition(newState, newData) {
         // todo whargarbl implement data stuff
         // todo major incomplete whargarbl comeback
-        if (this.valid_transition(newState, newData)) {
+        if (valid) {
             if (this._has_hooks) {
                 let hook_permits = undefined;
                 if (this._any_transition_hook !== undefined) {
@@ -794,39 +777,15 @@ class Machine {
                         return false;
                     }
                 }
-                const hn = hook_name(this._state, newState), maybe_hook = this._hooks.get(hn);
-                if (maybe_hook === undefined) {
-                    hook_permits = true;
-                }
-                else {
-                    hook_permits = maybe_hook({ from: this._state, to: newState });
-                }
-                if (hook_permits !== false) {
-                    this._state = newState;
-                    return true;
-                }
-                else {
-                    return false;
-                }
-            }
-            else {
-                this._state = newState;
-                return true;
-            }
-        }
-        else {
-            return false;
-        }
-    }
-    // can leave machine in inconsistent state.  generally do not use
-    force_transition(newState, newData) {
-        // todo whargarbl implement data stuff
-        // todo major incomplete whargarbl comeback
-        if (this.valid_force_transition(newState, newData)) {
-            if (this._has_hooks) {
-                let hook_permits = undefined;
-                if (this._any_transition_hook !== undefined) {
-                    if (this._any_transition_hook() === false) {
+                if (wasAction) {
+                    const nhn = named_hook_name(this._state, newState, newStateOrAction), n_maybe_hook = this._named_hooks.get(nhn);
+                    if (n_maybe_hook === undefined) {
+                        hook_permits = true;
+                    }
+                    else {
+                        hook_permits = n_maybe_hook({ from: this._state, to: newState, action: newStateOrAction });
+                    }
+                    if (!(hook_permits)) {
                         return false;
                     }
                 }
@@ -835,7 +794,7 @@ class Machine {
                     hook_permits = true;
                 }
                 else {
-                    hook_permits = maybe_hook({ from: this._state, to: newState, forced: true });
+                    hook_permits = maybe_hook({ from: this._state, to: newState, forced: wasForced, action: wasAction ? newStateOrAction : undefined });
                 }
                 if (hook_permits !== false) {
                     this._state = newState;
@@ -854,6 +813,16 @@ class Machine {
             return false;
         }
     }
+    action(actionName, newData) {
+        return this.transition_impl(actionName, newData, false, true);
+    }
+    transition(newState, newData) {
+        return this.transition_impl(newState, newData, false, false);
+    }
+    // can leave machine in inconsistent state.  generally do not use
+    force_transition(newState, newData) {
+        return this.transition_impl(newState, newData, true, false);
+    }
     current_action_for(action) {
         const action_base = this._actions.get(action);
         return action_base

package/dist/es6/version.js

@@ -1,2 +1,2 @@
-const version = "5.49.0";
+const version = "5.52.0";
 export { version };