Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Taher Alkhateeb
teeb [mailto:slidingfilame...@gmail.com] >> 发送时间: 2018年7月3日 15:09 >> 收件人: OFBIZ Development Mailing List >> 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn >> commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) >> >> Thank you Shi. I

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Jacques Le Roux
日 23:59 收件人: dev@ofbiz.apache.org 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) +1 to Taher's recommendations. Jacopo On Thu, Jun 28, 2018 at 9:15 AM, Taher Alkhateeb < slidingfilame...@gmail.com wr

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Jacques Le Roux
: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) That is not a constructive thing to say in my opinion. Sounds to me like: hey I don't like reverting, but I know this code is good and now that I reverted I don't care

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Shi Jinghai
> > 发件人: Jacopo Cappellato [mailto:jacopo.cappell...@hotwaxsystems.com] > > 发送时间: 2018年6月28日 23:59 > > 收件人: dev@ofbiz.apache.org > > 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn > commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Taher Alkhateeb
ble code I needed :) > > > -邮件原件- > 发件人: Taher Alkhateeb [mailto:slidingfilame...@gmail.com] > 发送时间: 2018年7月3日 15:09 > 收件人: OFBIZ Development Mailing List > 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn > commit: r1834389 - in /ofbiz/ofbi

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Taher Alkhateeb
--- > >> 发件人: Jacopo Cappellato [mailto:jacopo.cappell...@hotwaxsystems.com] > >> 发送时间: 2018年6月28日 23:59 > >> 收件人: dev@ofbiz.apache.org > >> 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn > commit: r1834389 - in /ofbiz/ofbiz-framework/trunk

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Jacques Le Roux
- 发件人: Jacopo Cappellato [mailto:jacopo.cappell...@hotwaxsystems.com] 发送时间: 2018年6月28日 23:59 收件人: dev@ofbiz.apache.org 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) +1 to Taher's recommendations. Jacopo

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Shi Jinghai
; 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn commit: > r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) > > +1 to Taher's recommendations. > > Jacopo > > On Thu, Jun 28, 2018 at 9:15 AM, Taher Alkhateeb > wrote: > >> A few comments: &

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Taher Alkhateeb
Add method attribute to request-map (Was: svn commit: > r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) > > +1 to Taher's recommendations. > > Jacopo > > On Thu, Jun 28, 2018 at 9:15 AM, Taher Alkhateeb > wrote: > >> A few comments: >> >> 1- I wou

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-07-03 Thread Shi Jinghai
Reverted in rev 1834917. -邮件原件- 发件人: Jacopo Cappellato [mailto:jacopo.cappell...@hotwaxsystems.com] 发送时间: 2018年6月28日 23:59 收件人: dev@ofbiz.apache.org 主题: Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...) +1

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-06-28 Thread Jacopo Cappellato
+1 to Taher's recommendations. Jacopo On Thu, Jun 28, 2018 at 9:15 AM, Taher Alkhateeb wrote: > A few comments: > > 1- I would suggest to try and avoid in the future committing any > design changes to the framework without discussing it properly in the > mailing list first > 2- I think it

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-06-28 Thread Arun Patidar
+1 Taher On Thu, Jun 28, 2018 at 2:01 PM Michael Brohl wrote: > A strong +1 to your recommendations. > > There were several commits in the OFBiz core recently which were not > properly discussed before they were committed. We should avoid this. > > People should have in mind that other

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-06-28 Thread Michael Brohl
A strong +1 to your recommendations. There were several commits in the OFBiz core recently which were not properly discussed before they were committed. We should avoid this. People should have in mind that other committers might not have the time to review, think and discuss these patches

Re: [Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-06-28 Thread Taher Alkhateeb
A few comments: 1- I would suggest to try and avoid in the future committing any design changes to the framework without discussing it properly in the mailing list first 2- I think it would be better to revert this work. I noticed in the JIRA for example that Mathieu Lirzin asked for some time to

[Discussion]: Add method attribute to request-map (Was: svn commit: r1834389 - in /ofbiz/ofbiz-framework/trunk/framework...)

2018-06-27 Thread Shi Jinghai
Hi all, Thanks Jacques, Taher and Nicolas mentioned our community rule, "a proper discussion". I created an issue "Add method attribute to request-map to controll a uri can be called GET or POST only" a week ago: https://issues.apache.org/jira/browse/OFBIZ-10438 Thanks Mathieu, he submitted