Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-29 Thread Julien NICOLAS
+1 to move forward. Julien. Le 09/08/2018 à 17:31, Taher Alkhateeb a écrit : Hello Everyone, We want to make this formal with proper community consensus and I will try to keep this short. In [1] Mathieu Lirzin is providing some (in my opinion) good work on converting the Control Servlet

Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-29 Thread deepak nigam
Hi Shi, Verified the fix by Mathieu on the local machine, the system is working fine now. Thanks & Regards -- Deepak Nigam HotWax Systems Pvt. Ltd. On Wed, Aug 29, 2018 at 11:01 AM Jacques Le Roux < jacques.le.r...@les7arts.com> wrote: > Hi Taher, > > Thank you for summarizing. I agree with

Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-28 Thread Jacques Le Roux
Hi Taher, Thank you for summarizing. I agree with this proposal After I have applied Mathieu's last OFBIZ-10438 at r1839451, I think we are ready to continue this work Jacques Le 09/08/2018 à 17:31, Taher Alkhateeb a écrit : Hello Everyone, We want to make this formal with proper community

Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-19 Thread Shi Jinghai
Thanks Taher! I noticed this bug report, and I have tested it, I cannot reproduce it in my local and online trunk demo environments. As Rishi Solanki suggested, if it's a bug, please reopen the OFBIZ-10438 or create a new jira. I'm waiting a further feedback from Deepak Nigam or anyone else.

Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-19 Thread Taher Alkhateeb
Well, I guess Shi rushed another commit without a proper review. And now we have a bug reported in [1]. The strange thing is that Shi suggested he's no longer interested in this work! [1]

Re: [Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-10 Thread Shi Jinghai
Thanks Taher for the great proposal! +1. I tested Swagger OpenAPI editor[1], both 2.0 and 3.0.n regulations support multiple methods for one uri/path, here is my test code (yaml format): openapi: "3.0.1" info: title: OFBiz OpenAPI Test description: OFBiz OpenAPI Test version: 1.0.0

[Proposal] Enhance the controller logic to accommodate HTTP methods and prepare for REST

2018-08-09 Thread Taher Alkhateeb
Hello Everyone, We want to make this formal with proper community consensus and I will try to keep this short. In [1] Mathieu Lirzin is providing some (in my opinion) good work on converting the Control Servlet logic to be able to handle the different HTTP methods (GET, POST, etc ...) The