Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-10-02 Thread Jacques Le Roux
Yes saw that yesterday also in another place (at start): [java] 20141002103146184 |catalina-startup-5 |CatalinaContainer |I| file:///C:/Stash/ofbizCloudfeet/applications/commonext/webapp/ordermgr-js/WEB-INF/web.xml not found. It went unnoticed so far. It's also in R13.07 but not

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-10-02 Thread Jacques Le Roux
Hi Jacopo, It's normally fixed in trunk at revision: 1628937. Could you please confirm the issue with sendMailFromScreen service is also fixed? BTW, if someone get a chance, it would be great to have a Junit test for sendMailFromScreen Thanks Jacques Le 02/10/2014 10:50, Jacques Le Roux a

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-10-02 Thread Jacques Le Roux
Ha I forgot to say that finally I did not have to add a controller file, the reason was my 1st empty placeholder web.xml file was not correct. Jacques Le 02/10/2014 13:20, Jacques Le Roux a écrit : Hi Jacopo, It's normally fixed in trunk at revision: 1628937. Could you please confirm the

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-10-02 Thread Jacopo Cappellato
On Oct 2, 2014, at 1:20 PM, Jacques Le Roux jacques.le.r...@les7arts.com wrote: Could you please confirm the issue with sendMailFromScreen service is also fixed? Confirmed Jacopo

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-10-01 Thread Jacopo Cappellato
Since a few weeks I see several occurrences of an error in the logs, here is an example: java.lang.IllegalArgumentException: applications/commonext/webapp/ordermgr-js/WEB-INF/web.xml does not exist. at org.ofbiz.webapp.WebAppUtil.parseWebXmlFile(WebAppUtil.java:148)

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Adrian Crum
This looks really ugly, and I don't understand the need for it. We already share JavaScripts across applications, so what makes this one different? Adrian Crum Sandglass Software www.sandglass-software.com On 7/21/2014 8:32 AM, jler...@apache.org wrote: Author: jleroux Date: Mon Jul 21

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Jacques Le Roux
Please explain if you see a better solution Jacques Le 21/07/2014 09:45, Adrian Crum a écrit : This looks really ugly, and I don't understand the need for it. We already share JavaScripts across applications, so what makes this one different? Adrian Crum Sandglass Software

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Adrian Crum
I can't explain a better solution until I understand the problem. To repeat: We already share JavaScripts across applications (jQuery for example), so what makes this one different? Adrian Crum Sandglass Software www.sandglass-software.com On 7/21/2014 10:53 AM, Jacques Le Roux wrote:

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Jacques Le Roux
I did not want to put applications js scripts in framework (images), that's the only reason. Also js scripts in framework (images) are not well organised ant it's a bit a mess. Did not want to make things more messy I long ago opened a Jira issue to better organised them, it's still open

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Adrian Crum
This is where i am getting confused. You don't need to put application js scripts anywhere - just leave them where they are. Reference them from special purpose applications the same way we do with js scripts in the images component. Adrian Crum Sandglass Software www.sandglass-software.com

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Jacques Le Roux
I thought I explained it enough in my commit comment and when I posted a link to trunk demo. Let me try again... 1) It does not work in trunk because birt component redefines the ordermrg web app (this was necessary after the move of the birt component from framework), but the js scripts are

Re: svn commit: r1612202 - in /ofbiz/trunk: applications/commonext/ applications/commonext/webapp/ordermgr-js/ applications/order/webapp/ordermgr/images/js/ applications/order/widget/ordermgr/ applica

2014-07-21 Thread Adrian Crum
Thank you for the clarification. Now I understand. Adrian Crum Sandglass Software www.sandglass-software.com On 7/21/2014 2:01 PM, Jacques Le Roux wrote: I thought I explained it enough in my commit comment and when I posted a link to trunk demo. Let me try again... 1) It does not work in