RE: Re: [OSWF] Status of JIRA WF-198?
"Patrick Lightbody" <[email protected]> Sat, 11 Oct 2003 12:24:43 -0700
| Newsgroups | gmane.comp.java.open-symphony.devel |
|---|---|
| Message-ID | <000201c3902d$52b900d0$6901a8c0@PLAPTOP> |
I dig this, +1 -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Eric Pugh Sent: Saturday, October 11, 2003 4:50 AM To: [email protected] Subject: RE: [Opensymphony-developers] Re: [OSWF] Status of JIRA WF-198? I have to say, one of the things that attracted me to OSWF was the simplicity.. And I think that what Nick is proposing is simpler because it removes the concept of local versus global actions, and provides a more compact way of referencing a action over and over. Also, while what he is proposing is to solve using the same action over and over, if you have an action used 3 times, this is a much compactor, simpler way to describe that as well. If we change it, lets definitly make sure that the global-actions syntax is clearly highlighted as deprecated. Eric > -----Original Message----- > From: [email protected] > [mailto:[email protected]]On > Behalf Of > Nick Dellamaggiore > Sent: Friday, October 10, 2003 8:47 PM > To: [email protected] > Subject: [Opensymphony-developers] Re: [OSWF] Status of JIRA WF-198? > > > Ok. Before I patch, I want to know how global-actions play > into the new > "workflow state" semantics of v2.6. Here is the section that > worries me: > > " If the caller does not explicitly alters the instance > state, the workflow > will remain in this state until it is unambigiously completed. This is > defined to be the case when the workflow cannot possibly > perform any further > actions. In this case, the workflow will automatically move > to the COMPLETED > state. " > > So, once we reach an action with no 'local actions' > available, the workflow > is marked completed. Thus, no further actions can be > performed due to the > following code in doAction(): > > if( entry.getState() != WorkflowEntry.ACTIVATED ) { > return; > } > > So, the notion of global actions doesn't apply here. There > ARE potential > (global) actions to be fired, but the workflow state > disallows it since > there are no local actions left for the current step. Fair enough. > > Actually, in a true finite state machine, each state has a > set of actions > (as you've correctly modelled). There is no notion of a > 'global-action'. > Should we just do away with this feature entirely (in a > future release)? > That would be fine by me as long as there was a feature in > the XML file > allowing me to refactor a common (not global) action out that > can then be > compactly referenced by id in each step it can be used. At > runtime, it > would simply appear as another ActionDescriptor for the Step. The XML > notation below would just be shorthand (macro) for writing > the action out > over and over again: > > <workflow> > > <common-actions> > <action id="99" name="Cancel Review"> > <pre-functions> > <!-- ... --> > </pre-functions> > <results> > <unconditional-result old-status = "Interrupted" > status = "Cancelled" > step = "100" > owner = "${caller}" > > <post-functions> > <!-- lots of post function definitions I don't > want to have to > copy/paste into each action below --> > </post-functions> > </unconditional-result> > </results> > </action> > </common-actions> > > <step id="10" name="Initial Review" > <action id="101" name="Complete Review"> > <!-- regular action xml goes here --> > </action> > > <!-- Workflow Loader code will resolve this action from > common-actions above since the action body is missing --> > <action id="99" /> > > <!-- OR give it a special tag to aid in > parsability/readability --> > <common-action id="99" /> > </step> > </workflow> > > > I can write the code for this. I'd leave the current global-action > semantics intact to allow backward-compatibility and just > change the Loader > so it recognized the new shorthard notation. What do you think? > > -nick > > > "Hani Suleiman" <[email protected]> wrote in message > news:[email protected]... > > Please do provide a patch, the splits/join stuff is rather > murky for me > > and I'd rather someone who is actively using it provide > patches than be > > guessing what it's supposed to be doing. > > > > On Wednesday, October 8, 2003, at 08:58 PM, Nicholas J. > Dellamaggiore > > wrote: > > > > > I posted a JIRA about 2 months ago regarding a pretty serious > > > OSWorkflow corruption issue. In a nutshell, if you're in a split > > > state (i.e 2 current steps are active) and you fire a > global action > > > before the split is resolved, the global action creates > its new step > > > and leaves the splits steps alive, thus resulting in THREE current > > > steps. The offending code is in AbstractWorkflow > (actually, I think > > > all non-getter/setter OSWF code lives here) and I detailed the > > > potentially buggy areas in the JIRA. I noticed that the > JIRA hasn't > > > been placed into the 2.6 release. Were you planning to > fix it for the > > > upcoming release? If not, I plan to just go ahead and > patch it myself. > > > Thanks. > > > > > > http://jira.opensymphony.com/secure/ViewIssue.jspa?key=WF-198 > > > > > > -nick > > > > > > > > > > > > ------------------------------------------------------- > > > This SF.net email is sponsored by: SF.net Giveback Program. > > > SourceForge.net hosts over 70,000 Open Source Projects. > > > See the people who have HELPED US provide better services: > > > Click here: http://sourceforge.net/supporters.php > > > _______________________________________________ > > > Opensymphony-developers mailing list > > > [email protected] > > > > https://lists.sourceforge.net/lists/listinfo/opensymphony-developers > > > > > > > > > > > ------------------------------------------------------- > > This SF.net email is sponsored by: SF.net Giveback Program. > > SourceForge.net hosts over 70,000 Open Source Projects. > > See the people who have HELPED US provide better services: > > Click here: http://sourceforge.net/supporters.php > > > > > > ------------------------------------------------------- > This SF.net email is sponsored by: SF.net Giveback Program. > SourceForge.net hosts over 70,000 Open Source Projects. > See the people who have HELPED US provide better services: > Click here: http://sourceforge.net/supporters.php > _______________________________________________ > Opensymphony-developers mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/opensymphony-developers ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. SourceForge.net hosts over 70,000 Open Source Projects. See the people who have HELPED US provide better services: Click here: http://sourceforge.net/supporters.php _______________________________________________ Opensymphony-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensymphony-developers ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. SourceForge.net hosts over 70,000 Open Source Projects. See the people who have HELPED US provide better services: Click here: http://sourceforge.net/supporters.php