Issue Details (XML | Word | Printable)

Key: SHALE-267
Type: Task Task
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Unassigned
Reporter: Rahul Akolkar
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Shale

Promote the best practice of avoiding spaces in dialog state names

Created: 31/Aug/06 02:53 PM   Updated: 23/Jan/07 04:40 PM
Return to search
Component/s: Dialog
Affects Version/s: 1.0.3
Fix Version/s: 1.0.4


 Description  « Hide
More of a task reminder for myself.

Discussion thread:

http://www.nabble.com/-dialog--Name-attribute-best-practice-tf2166188.html

Summary (fallback in case archives link fails):

On 8/25/06, Craig McClanahan <craigmcc@apache.org> wrote:
> On 8/25/06, Rahul Akolkar <rahul.akolkar@gmail.com> wrote:
> >
> > On 8/25/06, Craig McClanahan <craigmcc@apache.org> wrote:
> > > On 8/25/06, Rahul Akolkar <rahul.akolkar@gmail.com> wrote:
> > > >
> > > > I would like to propose a best practice for the name attribute used in
> > > > various bits of the XML vocabulary for Shale dialogs that recommends
> > > > restricting these to alphanumeric characters (no spaces etc.)
> > >
> > >
> > > I suppose there's a technical reason for this ... I really like the
> > > readability of these if we can keep them and it was why I chose "name"
> > > instead of "id' for that attribute in the first place.
> > >
> > <snip/>
> >
> > Yes, the state IDs are meant to be IDREFs in space separated lists on
> > transition targets when the SCXML <parallel> element is used, where it
> > will be possible to have more than one transition target, as long as
> > the targets belong to the regions of the same parallel.
> >
> > Ofcourse, when using a space separated list, having spaces in
> > individual tokens will throw a wrench in the works.
>
>
> Yah, that makes sense. Go ahead and do the patch, and I'll apply it.
>
> -Rahul
>
>
> Craig
>


 All   Comments   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Wendy Smoak added a comment - 06/Sep/06 11:33 PM
URL: http://svn.apache.org/viewvc?rev=439525&view=rev
Log:
Per discussion on the dev list, make dialog and state names omit embedded spaces.


Wendy Smoak made changes - 06/Sep/06 11:33 PM
Field Original Value New Value
Fix Version/s 1.0.4-SNAPSHOT [ 21740 ]
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Rahul Akolkar made changes - 23/Jan/07 04:40 PM
Fix Version/s 1.0.4 [ 21790 ]
Fix Version/s 1.0.4-SNAPSHOT [ 21740 ]
Jeff Turner made changes - 09/Aug/07 07:17 AM
Workflow Struts [ 38692 ] Struts - editable closed status [ 42426 ]
Antonio Petrelli made changes - 08/Jan/09 08:57 AM
Workflow Struts - editable closed status [ 42426 ] Struts - editable closed status (temporary) [ 46301 ]
Antonio Petrelli made changes - 08/Jan/09 09:08 AM
Workflow Struts - editable closed status (temporary) [ 46301 ] Struts - editable closed status [ 52934 ]