Author: jleroux
Date: Tue Aug 16 06:32:59 2011 New Revision: 1158125 URL: http://svn.apache.org/viewvc?rev=1158125&view=rev Log: "Applied fix from trunk for revision: 1158124" ------------------------------------------------------------------------ r1158124 | jleroux | 2011-08-16 08:24:26 +0200 (mar., 16 août 2011) | 8 lines A patch from Leon "incorrect url of root webapp in menu" https://issues.apache.org/jira/browse/OFBIZ-4368 If I enable the "app-bar-display" for root webapp which is mounted to "/", the url of root webapp in menu is "https://control/main" and not the correct "https://localhost:8443/". That's because, e.g. in themes/tomahawk/includes/appbarOpen.ftl, it uses expression <#if thisApp != "/"> to judge whether the app is root or not, while the variable thisApp is java String and it'll be encoded as iso code "/", so the "if" expression mentioned above will always return false. In the patch it use StringUtil.wrapString to wrap variable theApp to avoid encode. ------------------------------------------------------------------------ Modified: ofbiz/branches/release11.04/ (props changed) ofbiz/branches/release11.04/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml (props changed) ofbiz/branches/release11.04/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy (props changed) ofbiz/branches/release11.04/applications/product/script/org/ofbiz/product/test/InventoryTests.xml (props changed) ofbiz/branches/release11.04/framework/common/widget/HelpScreens.xml (props changed) ofbiz/branches/release11.04/framework/security/data/PasswordSecurityData.xml (props changed) ofbiz/branches/release11.04/specialpurpose/ebaystore/lib/ (props changed) ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy (props changed) ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy (props changed) ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy (props changed) ofbiz/branches/release11.04/themes/bizznesstime/includes/appbar.ftl ofbiz/branches/release11.04/themes/bizznesstime/includes/secondary-appbar.ftl ofbiz/branches/release11.04/themes/bluelight/includes/appbarOpen.ftl ofbiz/branches/release11.04/themes/droppingcrumbs/includes/appbarOpen.ftl ofbiz/branches/release11.04/themes/flatgrey/includes/appbar.ftl ofbiz/branches/release11.04/themes/tomahawk/includes/appbarOpen.ftl Propchange: ofbiz/branches/release11.04/ ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4:951708-952957 /ofbiz/branches/jquery:952958-1044489 /ofbiz/branches/multitenant20100310:921280-927264 -/ofbiz/trunk:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1133353,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1133353,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml:951708-952957 /ofbiz/branches/jquery/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml:952958-1044489 /ofbiz/branches/multitenant20100310/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml:921280-927264 -/ofbiz/trunk/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/applications/order/script/org/ofbiz/order/customer/CheckoutMapProcs.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy:951708-952957 /ofbiz/branches/jquery/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy:952958-1044489 /ofbiz/branches/multitenant20100310/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy:921280-927264 -/ofbiz/trunk/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/applications/party/webapp/partymgr/WEB-INF/actions/party/PartyFinancialHistory.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/applications/product/script/org/ofbiz/product/test/InventoryTests.xml ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/applications/product/script/org/ofbiz/product/test/InventoryTests.xml:951708-952957 /ofbiz/branches/jquery/applications/product/script/org/ofbiz/product/test/InventoryTests.xml:952958-1044489 /ofbiz/branches/multitenant20100310/applications/product/script/org/ofbiz/product/test/InventoryTests.xml:921280-927264 -/ofbiz/trunk/applications/product/script/org/ofbiz/product/test/InventoryTests.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/applications/product/script/org/ofbiz/product/test/InventoryTests.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/framework/common/widget/HelpScreens.xml ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/framework/common/widget/HelpScreens.xml:951708-952957 /ofbiz/branches/jquery/framework/common/widget/HelpScreens.xml:952958-1044489 /ofbiz/branches/multitenant20100310/framework/common/widget/HelpScreens.xml:921280-927264 -/ofbiz/trunk/framework/common/widget/HelpScreens.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/framework/common/widget/HelpScreens.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/framework/security/data/PasswordSecurityData.xml ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/applications/securityext/data/PasswordSecurityData.xml:951708-952957 /ofbiz/branches/jquery/applications/securityext/data/PasswordSecurityData.xml:952958-1044489 /ofbiz/branches/multitenant20100310/applications/securityext/data/PasswordSecurityData.xml:921280-927264 -/ofbiz/trunk/framework/security/data/PasswordSecurityData.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/framework/security/data/PasswordSecurityData.xml:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/specialpurpose/ebaystore/lib/ ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/specialpurpose/ebaystore/lib:951708-952957 /ofbiz/branches/jquery/specialpurpose/ebaystore/lib:952958-1044489 /ofbiz/branches/multitenant20100310/specialpurpose/ebaystore/lib:921280-927264 -/ofbiz/trunk/specialpurpose/ebaystore/lib:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/specialpurpose/ebaystore/lib:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy:951708-952957 /ofbiz/branches/jquery/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy:952958-1044489 /ofbiz/branches/multitenant20100310/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy:921280-927264 -/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/Facilities.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy:951708-952957 /ofbiz/branches/jquery/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy:952958-1044489 /ofbiz/branches/multitenant20100310/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy:921280-927264 -/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductList.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Propchange: ofbiz/branches/release11.04/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Tue Aug 16 06:32:59 2011 @@ -2,4 +2,4 @@ /ofbiz/branches/dojo1.4/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy:951708-952957 /ofbiz/branches/jquery/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy:952958-1044489 /ofbiz/branches/multitenant20100310/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy:921280-927264 -/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768 +/ofbiz/trunk/specialpurpose/hhfacility/webapp/hhfacility/WEB-INF/actions/ProductStockTake.groovy:1100197,1100880,1104423,1131144,1131396,1132496,1132749,1134990,1135199,1135686,1135929,1137201,1137433,1137435,1138463,1138485,1139346,1139385,1139504,1139521,1140358,1140362,1140375,1140469,1144537,1144791,1153768,1158124 Modified: ofbiz/branches/release11.04/themes/bizznesstime/includes/appbar.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/bizznesstime/includes/appbar.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/bizznesstime/includes/appbar.ftl (original) +++ ofbiz/branches/release11.04/themes/bizznesstime/includes/appbar.ftl Tue Aug 16 06:32:59 2011 @@ -43,6 +43,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> Modified: ofbiz/branches/release11.04/themes/bizznesstime/includes/secondary-appbar.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/bizznesstime/includes/secondary-appbar.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/bizznesstime/includes/secondary-appbar.ftl (original) +++ ofbiz/branches/release11.04/themes/bizznesstime/includes/secondary-appbar.ftl Tue Aug 16 06:32:59 2011 @@ -42,6 +42,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> Modified: ofbiz/branches/release11.04/themes/bluelight/includes/appbarOpen.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/bluelight/includes/appbarOpen.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/bluelight/includes/appbarOpen.ftl (original) +++ ofbiz/branches/release11.04/themes/bluelight/includes/appbarOpen.ftl Tue Aug 16 06:32:59 2011 @@ -54,6 +54,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> @@ -83,6 +84,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> Modified: ofbiz/branches/release11.04/themes/droppingcrumbs/includes/appbarOpen.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/droppingcrumbs/includes/appbarOpen.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/droppingcrumbs/includes/appbarOpen.ftl (original) +++ ofbiz/branches/release11.04/themes/droppingcrumbs/includes/appbarOpen.ftl Tue Aug 16 06:32:59 2011 @@ -54,6 +54,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> @@ -83,6 +84,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> Modified: ofbiz/branches/release11.04/themes/flatgrey/includes/appbar.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/flatgrey/includes/appbar.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/flatgrey/includes/appbar.ftl (original) +++ ofbiz/branches/release11.04/themes/flatgrey/includes/appbar.ftl Tue Aug 16 06:32:59 2011 @@ -44,6 +44,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> @@ -80,6 +81,7 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> <#assign thisURL = thisURL + "/control/main"> Modified: ofbiz/branches/release11.04/themes/tomahawk/includes/appbarOpen.ftl URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/themes/tomahawk/includes/appbarOpen.ftl?rev=1158125&r1=1158124&r2=1158125&view=diff ============================================================================== --- ofbiz/branches/release11.04/themes/tomahawk/includes/appbarOpen.ftl (original) +++ ofbiz/branches/release11.04/themes/tomahawk/includes/appbarOpen.ftl Tue Aug 16 06:32:59 2011 @@ -58,9 +58,10 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> - <#assign thisURL = StringUtil.wrapString(thisURL) + "/control/main"> + <#assign thisURL = thisURL + "/control/main"> </#if> <#if layoutSettings.suppressTab?exists && display.name == layoutSettings.suppressTab> <!-- do not display this component--> @@ -87,9 +88,10 @@ under the License. <#if thisApp == contextPath || contextPath + "/" == thisApp> <#assign selected = true> </#if> + <#assign thisApp = StringUtil.wrapString(thisApp)> <#assign thisURL = thisApp> <#if thisApp != "/"> - <#assign thisURL = StringUtil.wrapString(thisURL) + "/control/main"> + <#assign thisURL = thisURL + "/control/main"> </#if> <#if layoutSettings.suppressTab?exists && display.name == layoutSettings.suppressTab> <!-- do not display this component--> |
Free forum by Nabble | Edit this page |