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 in R12.04
Though it has any functional nor other code implications, a web.xml file is
needed. I will put a placeholder
I will need one for the controller as well because when I put an empty web.xml
file I get an error at start.
Jacques
Le 02/10/2014 07:24, Jacopo Cappellato a écrit :
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)
~[ofbiz-webapp.jar:?]
at org.ofbiz.webapp.WebAppUtil.getWebXml(WebAppUtil.java:131)
~[ofbiz-webapp.jar:?]
at org.ofbiz.webapp.WebAppUtil.getWebSiteId(WebAppUtil.java:117)
~[ofbiz-webapp.jar:?]
at
org.ofbiz.webapp.WebAppUtil.getWebappInfoFromWebsiteId(WebAppUtil.java:100)
~[ofbiz-webapp.jar:?]
at
org.ofbiz.common.email.NotificationServices.setBaseUrl(NotificationServices.java:258)
[ofbiz-common.jar:?]
at
org.ofbiz.common.email.EmailServices.sendMailFromScreen(EmailServices.java:458)
[ofbiz-common.jar:?]
...
Jacopo
On Jul 21, 2014, at 3:38 PM, Adrian Crum <adrian.c...@sandglass-software.com>
wrote:
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 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 not under the
redefined webapp
2) If you move the js scripts from order component to bit component it
will work... Until you use an upcoming release where the birt component
does not exist
3) If you copy js scripts from order component to bit component it will
work in all cases. But it will introduce duplication, so maintenance
hazards.
4) I did not want to move them to framework images for reasons explained
below
Hence my solution, I hope it's more clear. And yes it's a bit ugly :)
But there are more stuff more ugly than that. An example?
http://markmail.org/message/pd5ouy64lxbybr2m It seems you did not care
then...
Jacques
Le 21/07/2014 13:49, Adrian Crum a écrit :
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
On 7/21/2014 12:40 PM, Jacques Le Roux wrote:
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 https://issues.apache.org/jira/browse/OFBIZ-1319
Jacques
Le 21/07/2014 12:09, Adrian Crum a écrit :
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:
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
www.sandglass-software.com
On 7/21/2014 8:32 AM, jler...@apache.org wrote:
Author: jleroux
Date: Mon Jul 21 07:32:10 2014
New Revision: 1612202
URL: http://svn.apache.org/r1612202
Log:
Since we removed the specialpurpose components (but ecommerce),
there
was an issue with ordermgr js scripts.
The problem was double. It'd have been easy to add the missing
"/js:"
patch in allowedPaths in the web.xml of birt component from where
the
js scripts are now expected to be loaded in trunk.
But then you would need to also copy the js scripst there (in birt
component). You could not simply move them because for upcoming
branches we also need them in ordermgr component (they were there in
images/js) else they would miss again.
This commit rather adds them under a simple ordermgr-js new webapp
(simple, ie like framework/images, only to load resources) under
commonext: applications/commonext/webapp/ordermgr-js to prevent the
duplication maintenance hazards.
Added:
ofbiz/trunk/applications/commonext/webapp/ordermgr-js/
- copied from r1611410,
ofbiz/trunk/applications/order/webapp/ordermgr/images/js/
Removed:
ofbiz/trunk/applications/order/webapp/ordermgr/images/js/
Modified:
ofbiz/trunk/applications/commonext/ofbiz-component.xml
ofbiz/trunk/applications/order/widget/ordermgr/OrderReturnScreens.xml
ofbiz/trunk/applications/order/widget/ordermgr/OrderViewScreens.xml
ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
ofbiz/trunk/specialpurpose/ecommerce/widget/CustomerScreens.xml
ofbiz/trunk/specialpurpose/ecommerce/widget/OrderScreens.xml
Modified: ofbiz/trunk/applications/commonext/ofbiz-component.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/commonext/ofbiz-component.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
--- ofbiz/trunk/applications/commonext/ofbiz-component.xml
(original)
+++ ofbiz/trunk/applications/commonext/ofbiz-component.xml Mon
Jul 21
07:32:10 2014
@@ -44,4 +44,12 @@ under the License.
location="webapp/ofbizsetup"
base-permission="OFBTOOLS,SETUP"
mount-point="/ofbizsetup"/>
+
+ <webapp name="ordermgr-js"
+ title="ordermgr-js"
+ server="default-server"
+ location="webapp/ordermgr-js"
+ mount-point="/ordermgr-js"
+ app-bar-display="false"/>
+
</ofbiz-component>
Modified:
ofbiz/trunk/applications/order/widget/ordermgr/OrderReturnScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/widget/ordermgr/OrderReturnScreens.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
---
ofbiz/trunk/applications/order/widget/ordermgr/OrderReturnScreens.xml
(original)
+++
ofbiz/trunk/applications/order/widget/ordermgr/OrderReturnScreens.xml
Mon Jul 21 07:32:10 2014
@@ -88,7 +88,7 @@ under the License.
<set field="titleProperty"
value="PageTitleReturnHeader"/>
<set field="headerItem" value="return"/>
<set field="tabButtonItem"
value="OrderReturnHeader"/>
- <set field="layoutSettings.javaScripts[+0]"
value="/ordermgr/images/js/return.js" global="true"/>
+ <set field="layoutSettings.javaScripts[+0]"
value="/ordermgr-js/return.js" global="true"/>
<property-to-field field="defaultCurrencyUomId"
resource="general" property="currency.uom.id.default"
default="USD"/>
<script
location="component://order/webapp/ordermgr/WEB-INF/actions/return/ReturnHeader.groovy"/>
</actions>
Modified:
ofbiz/trunk/applications/order/widget/ordermgr/OrderViewScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/widget/ordermgr/OrderViewScreens.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
---
ofbiz/trunk/applications/order/widget/ordermgr/OrderViewScreens.xml
(original)
+++
ofbiz/trunk/applications/order/widget/ordermgr/OrderViewScreens.xml
Mon Jul 21 07:32:10 2014
@@ -52,9 +52,9 @@ under the License.
<actions>
<set field="titleProperty"
value="OrderViewOrder"/>
<set field="headerItem" value="findorders"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/order.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/OrderShippingInfo.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/geoAutoCompleter.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/order.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/OrderShippingInfo.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/geoAutoCompleter.js" global="true"/>
<script
location="component://order/webapp/ordermgr/WEB-INF/actions/order/OrderView.groovy"/>
<script
location="component://order/webapp/ordermgr/WEB-INF/actions/order/OrderViewWebSecure.groovy"/>
</actions>
Modified:
ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
---
ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
(original)
+++
ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
Mon Jul 21 07:32:10 2014
@@ -1093,7 +1093,7 @@ under the License.
<entity-one entity-name="Facility"
value-field="facility"/>
<property-map resource="ProductUiLabels"
map-name="uiLabelMap"/>
<set field="title"
value="${uiLabelMap.ProductReceiveInventory}"/>
- <set field="layoutSettings.javaScripts[+0]"
value="/ordermgr/images/js/ConvertUom.js" global="true"/>
+ <set field="layoutSettings.javaScripts[+0]"
value="/ordermgr-js/ConvertUom.js" global="true"/>
</actions>
<widgets>
<decorator-screen name="CommonFacilityDecorator"
location="${parameters.commonFacilityDecoratorLocation}">
Modified:
ofbiz/trunk/specialpurpose/ecommerce/widget/CustomerScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/widget/CustomerScreens.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/widget/CustomerScreens.xml
(original)
+++ ofbiz/trunk/specialpurpose/ecommerce/widget/CustomerScreens.xml
Mon Jul 21 07:32:10 2014
@@ -553,7 +553,7 @@ under the License.
<property-map resource="SecurityextUiLabels"
map-name="uiLabelMap" global="true"/>
<set field="titleProperty"
value="PageTitleNewCustomer"/>
<set field="layoutSettings.javaScripts[]"
value="/ecommerce/images/profile.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/geoAutoCompleter.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/geoAutoCompleter.js" global="true"/>
</actions>
<widgets>
<decorator-screen name="main-decorator"
location="${parameters.mainDecoratorLocation}">
@@ -616,7 +616,7 @@ under the License.
<set field="partyId"
from-field="userLogin.partyId"/>
<set field="layoutSettings.javaScripts[]"
value="/ecommerce/images/profile.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/geoAutoCompleter.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/geoAutoCompleter.js" global="true"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/EditShippingAddress.groovy"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/EditBillingAddress.groovy"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/ViewProfile.groovy"/>
Modified:
ofbiz/trunk/specialpurpose/ecommerce/widget/OrderScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/widget/OrderScreens.xml?rev=1612202&r1=1612201&r2=1612202&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/widget/OrderScreens.xml
(original)
+++ ofbiz/trunk/specialpurpose/ecommerce/widget/OrderScreens.xml Mon
Jul 21 07:32:10 2014
@@ -612,7 +612,7 @@ under the License.
<actions>
<set field="titleProperty"
value="EcommerceOnePageCheckout"/>
<set field="layoutSettings.javaScripts[]"
value="/ecommerce/images/checkoutProcess.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/ordermgr/images/js/geoAutoCompleter.js" global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/ordermgr-js/geoAutoCompleter.js" global="true"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/EditShippingAddress.groovy"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/EditBillingAddress.groovy"/>
<script
location="component://ecommerce/webapp/ecommerce/WEB-INF/actions/customer/EditEmailAndTelecomNumber.groovy"/>
