Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi Janaka, +1 for that approach. Thanks On Tue, Nov 6, 2012 at 12:37 AM, Janaka Ranabahu wrote: > Hi Shelan, > > IMO, #1 is the best approach to fix this. There are couple of reasons for > me to say that. > >1. This has been already implemented and it is working without any >issue. Therefore the only reason we need to change that is to comeup with a >better solution. >2. IMO, using full stop (".") to separate the words is more user >friendly than using camel case phrases. >3. If we change this, there is an aspect of data migration since this >will change the properties of some resources(which have been associated >with a lifecycle) and will change the content of all the lifecycle >configurations. > > Thanks, > Janaka > > > On Mon, Nov 5, 2012 at 5:24 PM, Shelan Perera wrote: > >> Hi, >> >> Could you please give the feedback on the suggested approach? >> >> 1) Let the user to add the fullstops and prevent adding spaces. >> >> 2) If user adds spaces remove it and do the parsing (Since it is not >> compliant with SCXML and then add the use the one with spaces.) >> >> I prefer 1st approach since 2nd seems like a hack. >> >> Thanks >> >> >> On Thu, Nov 1, 2012 at 3:35 PM, Shelan Perera wrote: >> >>> Hi, >>> >>> So shall we adhere to that method? i.e using fullstops instead of >>> spaces. ? (I tested with fullstops and it worked) >>> >>> Thanks >>> >>> >>> On Thu, Nov 1, 2012 at 9:37 AM, Evanthika Amarasiri >>> wrote: >>> Yes. We came across this issue while testing G-Reg 4.0.0 release and then the solution given was to use fullstops instead of spaces. In fact, in latest G-Reg packs, the default ServiceLifeCycle has a state in this format. : >>> class="org.wso2.carbon.governance.registry.extensions.executors.DemoteActionExecutor"> >>> class="org.wso2.carbon.governance.registry.extensions.executors.apistore.ApiStoreExecutor"> : Regards, Evanthika On Wed, Oct 31, 2012 at 11:36 PM, Janaka Ranabahu wrote: > Hi Shelan, > > Did you try defining the state ID by replacing all the whitespaces > with '.' character. > > *Eg:- Ready For QA ==> Ready.For.QA* > > IIRC, it should show that correctly in the UI. Also IIRC, similar > scenarios have been tested against lifecycles in past releases. Maybe QA > folks can share more information on that. > > Thanks, > Janaka > > > On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: > >> Hi, >> >> For the issue [1] we cannot specify whitespaces in state ID. It is a >> constraint added by SCXML and it gives an exception when it tries to >> parse. >> According to this [2] release note they have mentioned >> not to add white spaces in state IDs. A fix would be to add a >> temporary element which will complaint with parsing and then again >> replace >> it with white space. Is that approach clean enough to introduce? >> >> Thanks >> >> >> [1] https://wso2.org/jira/browse/REGISTRY-1378 >> >> [2] >> http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt >> -- >> *Shelan Perera* >> >> Software Engineer >> ** >> *WSO2, Inc. : wso2.com* >> lean.enterprise.middleware. >> >> *Home Page* :shelan.org >> *Blog* : blog.shelan.org >> *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 >> *Twitter* :https://twitter.com/#!/shelan >> >> *Mobile* : +94 772 604 402 >> >> >> >> ___ >> Dev mailing list >> Dev@wso2.org >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > Janaka Ranabahu > Software Engineer > WSO2 Inc. > > Mobile +94 718370861 > Email : jan...@wso2.com > Blog : janakaranabahu.blogspot.com > > >>> >>> >>> -- >>> *Shelan Perera* >>> >>> Software Engineer >>> ** >>> *WSO2, Inc. : wso2.com* >>> lean.enterprise.middleware. >>> >>> *Home Page* :shelan.org >>> *Blog* : blog.shelan.org >>> *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 >>> *Twitter* :https://twitter.com/#!/shelan >>> >>> *Mobile* : +94 772 604 402 >>> >>> >>> >> >> >> -- >> *Shelan Perera* >> >> Software Engineer >> ** >> *WSO2, Inc. : wso2.com* >> lean.enterprise.middleware. >> >> *Home Page* :shelan.org >> *Blog* : blog.shelan.org >> *Linked-i*n :http://www.link
Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi Shelan, IMO, #1 is the best approach to fix this. There are couple of reasons for me to say that. 1. This has been already implemented and it is working without any issue. Therefore the only reason we need to change that is to comeup with a better solution. 2. IMO, using full stop (".") to separate the words is more user friendly than using camel case phrases. 3. If we change this, there is an aspect of data migration since this will change the properties of some resources(which have been associated with a lifecycle) and will change the content of all the lifecycle configurations. Thanks, Janaka On Mon, Nov 5, 2012 at 5:24 PM, Shelan Perera wrote: > Hi, > > Could you please give the feedback on the suggested approach? > > 1) Let the user to add the fullstops and prevent adding spaces. > > 2) If user adds spaces remove it and do the parsing (Since it is not > compliant with SCXML and then add the use the one with spaces.) > > I prefer 1st approach since 2nd seems like a hack. > > Thanks > > > On Thu, Nov 1, 2012 at 3:35 PM, Shelan Perera wrote: > >> Hi, >> >> So shall we adhere to that method? i.e using fullstops instead of spaces. >> ? (I tested with fullstops and it worked) >> >> Thanks >> >> >> On Thu, Nov 1, 2012 at 9:37 AM, Evanthika Amarasiri >> wrote: >> >>> Yes. We came across this issue while testing G-Reg 4.0.0 release and >>> then the solution given was to use fullstops instead of spaces. In fact, in >>> latest G-Reg packs, the default ServiceLifeCycle has a state in this >>> format. >>> >>> : >>> >>> >>> >>> >> class="org.wso2.carbon.governance.registry.extensions.executors.DemoteActionExecutor"> >>> >>> >> class="org.wso2.carbon.governance.registry.extensions.executors.apistore.ApiStoreExecutor"> >>> >>> >>> >>> >>> >>> >>> >>> >>> : >>> >>> Regards, >>> Evanthika >>> >>> >>> On Wed, Oct 31, 2012 at 11:36 PM, Janaka Ranabahu wrote: >>> Hi Shelan, Did you try defining the state ID by replacing all the whitespaces with '.' character. *Eg:- Ready For QA ==> Ready.For.QA* IIRC, it should show that correctly in the UI. Also IIRC, similar scenarios have been tested against lifecycles in past releases. Maybe QA folks can share more information on that. Thanks, Janaka On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: > Hi, > > For the issue [1] we cannot specify whitespaces in state ID. It is a > constraint added by SCXML and it gives an exception when it tries to > parse. > According to this [2] release note they have mentioned > not to add white spaces in state IDs. A fix would be to add a > temporary element which will complaint with parsing and then again replace > it with white space. Is that approach clean enough to introduce? > > Thanks > > > [1] https://wso2.org/jira/browse/REGISTRY-1378 > > [2] > http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt > -- > *Shelan Perera* > > Software Engineer > ** > *WSO2, Inc. : wso2.com* > lean.enterprise.middleware. > > *Home Page* :shelan.org > *Blog* : blog.shelan.org > *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 > *Twitter* :https://twitter.com/#!/shelan > > *Mobile* : +94 772 604 402 > > > > ___ > Dev mailing list > Dev@wso2.org > http://wso2.org/cgi-bin/mailman/listinfo/dev > > -- Janaka Ranabahu Software Engineer WSO2 Inc. Mobile +94 718370861 Email : jan...@wso2.com Blog : janakaranabahu.blogspot.com >>> >> >> >> -- >> *Shelan Perera* >> >> Software Engineer >> ** >> *WSO2, Inc. : wso2.com* >> lean.enterprise.middleware. >> >> *Home Page* :shelan.org >> *Blog* : blog.shelan.org >> *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 >> *Twitter* :https://twitter.com/#!/shelan >> >> *Mobile* : +94 772 604 402 >> >> >> > > > -- > *Shelan Perera* > > Software Engineer > ** > *WSO2, Inc. : wso2.com* > lean.enterprise.middleware. > > *Home Page* :shelan.org > *Blog* : blog.shelan.org > *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 > *Twitter* :https://twitter.com/#!/shelan > > *Mobile* : +94 772 604 402 > > > -- Janaka Ranabahu Software Engineer WSO2 Inc. Mobile +94 718370861 Email : jan...@wso2.com Blog : janakaranabahu.blogspot.com __
Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi, Could you please give the feedback on the suggested approach? 1) Let the user to add the fullstops and prevent adding spaces. 2) If user adds spaces remove it and do the parsing (Since it is not compliant with SCXML and then add the use the one with spaces.) I prefer 1st approach since 2nd seems like a hack. Thanks On Thu, Nov 1, 2012 at 3:35 PM, Shelan Perera wrote: > Hi, > > So shall we adhere to that method? i.e using fullstops instead of spaces. > ? (I tested with fullstops and it worked) > > Thanks > > > On Thu, Nov 1, 2012 at 9:37 AM, Evanthika Amarasiri wrote: > >> Yes. We came across this issue while testing G-Reg 4.0.0 release and then >> the solution given was to use fullstops instead of spaces. In fact, in >> latest G-Reg packs, the default ServiceLifeCycle has a state in this >> format. >> >> : >> >> >> >> > class="org.wso2.carbon.governance.registry.extensions.executors.DemoteActionExecutor"> >> >> > class="org.wso2.carbon.governance.registry.extensions.executors.apistore.ApiStoreExecutor"> >> >> >> >> >> >> >> >> >> : >> >> Regards, >> Evanthika >> >> >> On Wed, Oct 31, 2012 at 11:36 PM, Janaka Ranabahu wrote: >> >>> Hi Shelan, >>> >>> Did you try defining the state ID by replacing all the whitespaces with >>> '.' character. >>> >>> *Eg:- Ready For QA ==> Ready.For.QA* >>> >>> IIRC, it should show that correctly in the UI. Also IIRC, similar >>> scenarios have been tested against lifecycles in past releases. Maybe QA >>> folks can share more information on that. >>> >>> Thanks, >>> Janaka >>> >>> >>> On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: >>> Hi, For the issue [1] we cannot specify whitespaces in state ID. It is a constraint added by SCXML and it gives an exception when it tries to parse. According to this [2] release note they have mentioned not to add white spaces in state IDs. A fix would be to add a temporary element which will complaint with parsing and then again replace it with white space. Is that approach clean enough to introduce? Thanks [1] https://wso2.org/jira/browse/REGISTRY-1378 [2] http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt -- *Shelan Perera* Software Engineer ** *WSO2, Inc. : wso2.com* lean.enterprise.middleware. *Home Page* :shelan.org *Blog* : blog.shelan.org *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 *Twitter* :https://twitter.com/#!/shelan *Mobile* : +94 772 604 402 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev >>> >>> >>> -- >>> Janaka Ranabahu >>> Software Engineer >>> WSO2 Inc. >>> >>> Mobile +94 718370861 >>> Email : jan...@wso2.com >>> Blog : janakaranabahu.blogspot.com >>> >>> >> > > > -- > *Shelan Perera* > > Software Engineer > ** > *WSO2, Inc. : wso2.com* > lean.enterprise.middleware. > > *Home Page* :shelan.org > *Blog* : blog.shelan.org > *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 > *Twitter* :https://twitter.com/#!/shelan > > *Mobile* : +94 772 604 402 > > > -- *Shelan Perera* Software Engineer ** *WSO2, Inc. : wso2.com* lean.enterprise.middleware. *Home Page* :shelan.org *Blog* : blog.shelan.org *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 *Twitter* :https://twitter.com/#!/shelan *Mobile* : +94 772 604 402 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi, So shall we adhere to that method? i.e using fullstops instead of spaces. ? (I tested with fullstops and it worked) Thanks On Thu, Nov 1, 2012 at 9:37 AM, Evanthika Amarasiri wrote: > Yes. We came across this issue while testing G-Reg 4.0.0 release and then > the solution given was to use fullstops instead of spaces. In fact, in > latest G-Reg packs, the default ServiceLifeCycle has a state in this > format. > > : > > > > class="org.wso2.carbon.governance.registry.extensions.executors.DemoteActionExecutor"> > > class="org.wso2.carbon.governance.registry.extensions.executors.apistore.ApiStoreExecutor"> > > > > > > > > > : > > Regards, > Evanthika > > > On Wed, Oct 31, 2012 at 11:36 PM, Janaka Ranabahu wrote: > >> Hi Shelan, >> >> Did you try defining the state ID by replacing all the whitespaces with >> '.' character. >> >> *Eg:- Ready For QA ==> Ready.For.QA* >> >> IIRC, it should show that correctly in the UI. Also IIRC, similar >> scenarios have been tested against lifecycles in past releases. Maybe QA >> folks can share more information on that. >> >> Thanks, >> Janaka >> >> >> On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: >> >>> Hi, >>> >>> For the issue [1] we cannot specify whitespaces in state ID. It is a >>> constraint added by SCXML and it gives an exception when it tries to parse. >>> According to this [2] release note they have mentioned >>> not to add white spaces in state IDs. A fix would be to add a temporary >>> element which will complaint with parsing and then again replace it with >>> white space. Is that approach clean enough to introduce? >>> >>> Thanks >>> >>> >>> [1] https://wso2.org/jira/browse/REGISTRY-1378 >>> >>> [2] >>> http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt >>> -- >>> *Shelan Perera* >>> >>> Software Engineer >>> ** >>> *WSO2, Inc. : wso2.com* >>> lean.enterprise.middleware. >>> >>> *Home Page* :shelan.org >>> *Blog* : blog.shelan.org >>> *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 >>> *Twitter* :https://twitter.com/#!/shelan >>> >>> *Mobile* : +94 772 604 402 >>> >>> >>> >>> ___ >>> Dev mailing list >>> Dev@wso2.org >>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>> >>> >> >> >> -- >> Janaka Ranabahu >> Software Engineer >> WSO2 Inc. >> >> Mobile +94 718370861 >> Email : jan...@wso2.com >> Blog : janakaranabahu.blogspot.com >> >> > -- *Shelan Perera* Software Engineer ** *WSO2, Inc. : wso2.com* lean.enterprise.middleware. *Home Page* :shelan.org *Blog* : blog.shelan.org *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 *Twitter* :https://twitter.com/#!/shelan *Mobile* : +94 772 604 402 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Yes. We came across this issue while testing G-Reg 4.0.0 release and then the solution given was to use fullstops instead of spaces. In fact, in latest G-Reg packs, the default ServiceLifeCycle has a state in this format. : : Regards, Evanthika On Wed, Oct 31, 2012 at 11:36 PM, Janaka Ranabahu wrote: > Hi Shelan, > > Did you try defining the state ID by replacing all the whitespaces with > '.' character. > > *Eg:- Ready For QA ==> Ready.For.QA* > > IIRC, it should show that correctly in the UI. Also IIRC, similar > scenarios have been tested against lifecycles in past releases. Maybe QA > folks can share more information on that. > > Thanks, > Janaka > > > On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: > >> Hi, >> >> For the issue [1] we cannot specify whitespaces in state ID. It is a >> constraint added by SCXML and it gives an exception when it tries to parse. >> According to this [2] release note they have mentioned >> not to add white spaces in state IDs. A fix would be to add a temporary >> element which will complaint with parsing and then again replace it with >> white space. Is that approach clean enough to introduce? >> >> Thanks >> >> >> [1] https://wso2.org/jira/browse/REGISTRY-1378 >> >> [2] >> http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt >> -- >> *Shelan Perera* >> >> Software Engineer >> ** >> *WSO2, Inc. : wso2.com* >> lean.enterprise.middleware. >> >> *Home Page* :shelan.org >> *Blog* : blog.shelan.org >> *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 >> *Twitter* :https://twitter.com/#!/shelan >> >> *Mobile* : +94 772 604 402 >> >> >> >> ___ >> Dev mailing list >> Dev@wso2.org >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > Janaka Ranabahu > Software Engineer > WSO2 Inc. > > Mobile +94 718370861 > Email : jan...@wso2.com > Blog : janakaranabahu.blogspot.com > > ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
Re: [Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi Shelan, Did you try defining the state ID by replacing all the whitespaces with '.' character. *Eg:- Ready For QA ==> Ready.For.QA* IIRC, it should show that correctly in the UI. Also IIRC, similar scenarios have been tested against lifecycles in past releases. Maybe QA folks can share more information on that. Thanks, Janaka On Wed, Oct 31, 2012 at 2:18 PM, Shelan Perera wrote: > Hi, > > For the issue [1] we cannot specify whitespaces in state ID. It is a > constraint added by SCXML and it gives an exception when it tries to parse. > According to this [2] release note they have mentioned > not to add white spaces in state IDs. A fix would be to add a temporary > element which will complaint with parsing and then again replace it with > white space. Is that approach clean enough to introduce? > > Thanks > > > [1] https://wso2.org/jira/browse/REGISTRY-1378 > > [2] > http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt > -- > *Shelan Perera* > > Software Engineer > ** > *WSO2, Inc. : wso2.com* > lean.enterprise.middleware. > > *Home Page* :shelan.org > *Blog* : blog.shelan.org > *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 > *Twitter* :https://twitter.com/#!/shelan > > *Mobile* : +94 772 604 402 > > > > ___ > Dev mailing list > Dev@wso2.org > http://wso2.org/cgi-bin/mailman/listinfo/dev > > -- Janaka Ranabahu Software Engineer WSO2 Inc. Mobile +94 718370861 Email : jan...@wso2.com Blog : janakaranabahu.blogspot.com ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
[Dev] Best approach to fix https://wso2.org/jira/browse/REGISTRY-1378
Hi, For the issue [1] we cannot specify whitespaces in state ID. It is a constraint added by SCXML and it gives an exception when it tries to parse. According to this [2] release note they have mentioned not to add white spaces in state IDs. A fix would be to add a temporary element which will complaint with parsing and then again replace it with white space. Is that approach clean enough to introduce? Thanks [1] https://wso2.org/jira/browse/REGISTRY-1378 [2] http://svn.apache.org/repos/asf/commons/proper/scxml/tags/SCXML_0_7/RELEASE-NOTES.txt -- *Shelan Perera* Software Engineer ** *WSO2, Inc. : wso2.com* lean.enterprise.middleware. *Home Page* :shelan.org *Blog* : blog.shelan.org *Linked-i*n :http://www.linkedin.com/pub/shelan-perera/a/194/465 *Twitter* :https://twitter.com/#!/shelan *Mobile* : +94 772 604 402 ___ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev