There is a little problem in ruote, route-web2

16 views
Skip to first unread message

gaoxt1983

unread,
Mar 30, 2009, 5:23:54 AM3/30/09
to Ruote (OpenWFEru) users
I have found a little problem about ruote. In ruote, if we define a
sub-process, the sub-process id is xxxxxx.n , n is a number. You
know, in
rails, the url like xxxx.yyy separates into two parts, so if you use
this as
a URL( try ruote-web2, define a process contains sub-process), it
doesn't
work.

The solution is : in vendor/plugins/ruote_plugin/lib_ruote/openwfe/
expool/expressionpool.rb (ruote_web2)

there is a method called "launch_subprocess", modify
raw_exp.fei.wfid =
"#{firing_exp.fei.parent_wfid}.#{firing_exp.get_next_sub_id}"

to whatever you like , but do not use ".".

John Mettraux

unread,
Mar 30, 2009, 5:29:40 AM3/30/09
to openwfe...@googlegroups.com
On Mon, Mar 30, 2009 at 6:23 PM, gaoxt1983 <gaoxtw...@gmail.com> wrote:
>
> I have found a little problem about ruote. In ruote, if we define a
> sub-process, the sub-process id is xxxxxx.n , n is a number. You
> know, in
> rails, the url like xxxx.yyy separates into two parts, so if you use
> this as
> a URL( try ruote-web2, define a process contains sub-process), it
> doesn't
> work.

Hi 高志鑫,

I will fix this issue in a way that is backward compatible.

http://rubyforge.org/tracker/index.php?func=detail&aid=25015&group_id=2609&atid=10195


Thanks a lot and best regards,

--
John Mettraux - http://jmettraux.wordpress.com

John Mettraux

unread,
Apr 2, 2009, 3:58:51 AM4/2/09
to openwfe...@googlegroups.com
2009/3/30 John Mettraux <jmet...@openwfe.org>:

> On Mon, Mar 30, 2009 at 6:23 PM, gaoxt1983 <gaoxtw...@gmail.com> wrote:
>>
>> I have found a little problem about ruote. In ruote, if we define a
>> sub-process, the sub-process id is xxxxxx.n , n is a number. You
>> know, in
>> rails, the url like xxxx.yyy separates into two parts, so if you use
>> this as
>> a URL( try ruote-web2, define a process contains sub-process), it
>> doesn't
>> work.
>
> Hi 高志鑫,
>
> I will fix this issue in a way that is backward compatible.
>
> http://rubyforge.org/tracker/index.php?func=detail&aid=25015&group_id=2609&atid=10195

Hi 高志鑫,

I have studied the issue more carefully. I have decided not to change
the separator from "." to "_" or something else now. It would require
too much "juggling" in the engine itself especially to stay "backward
compatible".

The way to go is to enforce using underscores instead of dots when
generating URLs and I have taken care of that in the latest commit to
ruote / ruote-rest and ruote-web2 :

http://github.com/jmettraux/ruote/commit/6f999c2a4a33f22806b71b4d83dec59c315b5986
http://github.com/jmettraux/ruote-rest/commit/04754202c4c7b984c4799568dc9385fbfa820975
http://github.com/jmettraux/ruote-web2/commit/bd21d7714910af68d3ec2f0a82d1b8df6117fb0d

I invite you to use the new methods OpenWFE.to_dots(s) and
OpenWFE.to_uscores(s) to take care of those _/. issues (follow the
example of ruote-rest and ruote-web2). The links generated by
ruote-rest and ruote-web2 (and later ruote-kit) will use "_".

For the next version of the core of ruote, I will not forget your
request not to use ".".


Thanks again for your report and your patch. Best regards,

Reply all
Reply to author
Forward
0 new messages