On Thu, Oct 27, 2011 at 17:21, Burak Arslan <burak.ars...@arskom.com.tr>wrote:
> On 10/27/11 18:07, Derrell Lipman wrote:
> > Hi Burak,
> >
> > I don't recall making any change to that effect. The target parameter
> > is not documented as optional, and I don't think it ever was.
> >
>
> Hi Derrell,
>
> This is actually from an older part of our codebase. Once we initialize
> the fsm, we send it its initialization event:
>
> fsm.start();
> fsm.fireImmediateEvent("application_entry");
>
> The relevant excerpt from the initialization code:
>
> state = new qx.util.fsm.State("entry_point", {
> "onentry" : function(fsm, event) { },
> "onexit" : function(fsm, event) { },
> "events" : {
> "application_entry" : "to_login_screen"
> }
> });
> fsm.addState(state);
>
> trans = new qx.util.fsm.Transition("to_login_screen", {
> "predicate" : true,
> "nextState" : "awaiting_login",
> "ontransition": function(fsm_, event) {
> // stuff
> }
> });
> state.addTransition(trans);
>
> One should not need to pass anything to the fireImmediateEvent. The fsm
> object has the potential to either handle the event given its current
> state or reject it by either throwing an exception or ignoring it if no
> edges depart from the current state with the named event. (which makes
> qx.util.fsm.FiniteStateMachne a finite state transducer rather than a
> finite state machine but anyway) So i think the documentation should be
> fixed as well.
>
> I've fixed the bug, It's a simple, one-line patch:
> https://github.com/qooxdoo/qooxdoo/pull/4
Yes, it's an innocuous change. It can't hurt existing code. I've pulled it.
Thanks!
Derrell
------------------------------------------------------------------------------
The demand for IT networking professionals continues to grow, and the
demand for specialized networking skills is growing even more rapidly.
Take a complimentary Learning@Cisco Self-Assessment and learn
about Cisco certifications, training, and career opportunities.
http://p.sf.net/sfu/cisco-dev2dev
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
