Author: deepak
Date: Sat May 27 11:42:27 2017
New Revision: 1796382

URL: http://svn.apache.org/viewvc?rev=1796382&view=rev
Log:
Fixed:  After the implementation of new servlet filter for controller related 
logic, the multisite feature was not working properly due to URL modification. 
This control filter can not be applied to cmssite because that web application 
is intended to use dynamically publish URL. So we can remove the control filter 
from web.xml of cmssite.
  Thanks Jacopo Cappellato for discussing the possible solution on this issue. 
(OFBIZ-7120)
  
 Thanks Mohammed Rehan Khan for your contribution.

Modified:
    
ofbiz/ofbiz-plugins/trunk/cmssite/src/main/java/org/apache/ofbiz/cmssite/multisite/WebSiteFilter.java
    ofbiz/ofbiz-plugins/trunk/cmssite/webapp/cmssite/WEB-INF/web.xml

Modified: 
ofbiz/ofbiz-plugins/trunk/cmssite/src/main/java/org/apache/ofbiz/cmssite/multisite/WebSiteFilter.java
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/cmssite/src/main/java/org/apache/ofbiz/cmssite/multisite/WebSiteFilter.java?rev=1796382&r1=1796381&r2=1796382&view=diff
==============================================================================
--- 
ofbiz/ofbiz-plugins/trunk/cmssite/src/main/java/org/apache/ofbiz/cmssite/multisite/WebSiteFilter.java
 (original)
+++ 
ofbiz/ofbiz-plugins/trunk/cmssite/src/main/java/org/apache/ofbiz/cmssite/multisite/WebSiteFilter.java
 Sat May 27 11:42:27 2017
@@ -80,7 +80,7 @@ public class WebSiteFilter implements Fi
 
         GenericValue webSite = null;
         try {
-            if (UtilValidate.isNotEmpty(webSiteAlias) && webSite != null) {
+            if (UtilValidate.isNotEmpty(webSiteAlias) && webSite == null) {
                 webSite = 
EntityQuery.use(delegator).from("WebSite").where("hostedPathAlias", 
webSiteAlias).cache().queryFirst();
             }
             if (UtilValidate.isEmpty(webSite)) {

Modified: ofbiz/ofbiz-plugins/trunk/cmssite/webapp/cmssite/WEB-INF/web.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/cmssite/webapp/cmssite/WEB-INF/web.xml?rev=1796382&r1=1796381&r2=1796382&view=diff
==============================================================================
--- ofbiz/ofbiz-plugins/trunk/cmssite/webapp/cmssite/WEB-INF/web.xml (original)
+++ ofbiz/ofbiz-plugins/trunk/cmssite/webapp/cmssite/WEB-INF/web.xml Sat May 27 
11:42:27 2017
@@ -40,19 +40,6 @@ under the License.
     </context-param>
 
     <filter>
-        <display-name>ControlFilter</display-name>
-        <filter-name>ControlFilter</filter-name>
-        
<filter-class>org.apache.ofbiz.webapp.control.ControlFilter</filter-class>
-        <init-param>
-            <param-name>allowedPaths</param-name>
-            
<param-value>/cms:/select:/index.html:/index.jsp:/default.html:/default.jsp:/images</param-value>
-        </init-param>
-        <init-param>
-            <param-name>redirectPath</param-name>
-            <param-value>/cms</param-value>
-        </init-param>
-    </filter>
-    <filter>
         <display-name>ContextFilter</display-name>
         <filter-name>ContextFilter</filter-name>
         
<filter-class>org.apache.ofbiz.webapp.control.ContextFilter</filter-class>
@@ -63,10 +50,6 @@ under the License.
         
<filter-class>org.apache.ofbiz.cmssite.multisite.WebSiteFilter</filter-class>
     </filter>
     <filter-mapping>
-        <filter-name>ControlFilter</filter-name>
-        <url-pattern>/*</url-pattern>
-    </filter-mapping>
-    <filter-mapping>
         <filter-name>ContextFilter</filter-name>
         <url-pattern>/*</url-pattern>
     </filter-mapping>


Reply via email to