jssm 5.49.0 → 5.50.0

package/dist/es6/jssm.d.ts

@@ -97,6 +97,7 @@ declare class 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(newState: StateType, newData: mDT | undefined, wasForced: boolean, wasAction: boolean): 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

@@ -783,46 +783,21 @@ class Machine {
             return false;
         }
     }
-    transition(newState, newData) {
-        // todo whargarbl implement data stuff
-        // todo major incomplete whargarbl comeback
-        if (this.valid_transition(newState, newData)) {
-            if (this._has_hooks) {
-                let hook_permits = undefined;
-                if (this._any_transition_hook !== undefined) {
-                    if (this._any_transition_hook() === false) {
-                        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;
+    transition_impl(newState, newData, wasForced, wasAction) {
+        let valid = false;
+        if (wasForced) {
+            if (this.valid_force_transition(newState, newData)) {
+                valid = true;
             }
         }
         else {
-            return false;
+            if (this.valid_transition(newState, newData)) {
+                valid = true;
+            }
         }
-    }
-    // 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 (valid) {
             if (this._has_hooks) {
                 let hook_permits = undefined;
                 if (this._any_transition_hook !== undefined) {
@@ -835,7 +810,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 });
                 }
                 if (hook_permits !== false) {
                     this._state = newState;
@@ -854,6 +829,13 @@ class Machine {
             return false;
         }
     }
+    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.50.0";
 export { version };