Jacques, Pierre,

if you look at the code I have removed, that simplified is:

            String ofbizHome = System.getProperty("ofbiz.home");
            if (UtilValidate.isEmpty(ofbizHome)) {
                ofbizHome = System.getProperty("ofbiz.home");
                System.setProperty("ofbiz.home", ofbizHome);
            }

you will see that it is completely meaningless.

Jacopo


On Feb 26, 2015, at 10:35 AM, Pierre Smits <[email protected]> wrote:

> That must be related to the functionality in the appserver component?
> 
> Best regards,
> 
> Pierre Smits
> 
> *ORRTIZ.COM <http://www.orrtiz.com>*
> Services & Solutions for Cloud-
> Based Manufacturing, Professional
> Services and Retail & Trade
> http://www.orrtiz.com
> 
> On Thu, Feb 26, 2015 at 10:29 AM, Jacques Le Roux <
> [email protected]> wrote:
> 
>> It was used by https://cwiki.apache.org/confluence/display/OFBIZ/
>> Geronimo+and+IBM+Websphere+Application+Server+Community+Edition and R4.0
>> 
>> I guess nobody is indeed now using this. Even then, it was more an
>> internal company political affair anyway, was fun though.
>> 
>> Jacques
>> 
>> Le 26/02/2015 10:09, [email protected] a écrit :
>> 
>> Author: jacopoc
>>> Date: Thu Feb 26 09:09:04 2015
>>> New Revision: 1662393
>>> 
>>> URL: http://svn.apache.org/r1662393
>>> Log:
>>> Removed some old code that, as can be easily noticed by reviewing the
>>> lines I am removing, was completely useless; it was probably an half baked
>>> experiment that has been committed by error.
>>> 
>>> Modified:
>>>     ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/
>>> ContextFilter.java
>>>     ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/web.xml
>>> 
>>> Modified: ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/
>>> ContextFilter.java
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/
>>> src/org/ofbiz/webapp/control/ContextFilter.java?rev=
>>> 1662393&r1=1662392&r2=1662393&view=diff
>>> ============================================================
>>> ==================
>>> --- 
>>> ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/ContextFilter.java
>>> (original)
>>> +++ 
>>> ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/ContextFilter.java
>>> Thu Feb 26 09:09:04 2015
>>> @@ -407,15 +407,6 @@ public class ContextFilter implements Fi
>>>              if (Debug.verboseOn()) Debug.logVerbose("Adding web.xml
>>> context-param to application attribute with name [" + initParamName + "]
>>> and value [" + initParamValue + "]", module);
>>>              config.getServletContext().setAttribute(initParamName,
>>> initParamValue);
>>>          }
>>> -        String GeronimoMultiOfbizInstances = (String)
>>> config.getServletContext().getAttribute("GeronimoMultiOfbizInstances");
>>> -        if (UtilValidate.isNotEmpty(GeronimoMultiOfbizInstances)) {
>>> -            String ofbizHome = System.getProperty("ofbiz.home");
>>> -            if (GeronimoMultiOfbizInstances.equalsIgnoreCase("true") &&
>>> UtilValidate.isEmpty(ofbizHome)) {
>>> -                ofbizHome = System.getProperty("ofbiz.home"); // This
>>> is only used in case of Geronimo or WASCE using OFBiz multi-instances. It
>>> allows to retrieve ofbiz.home value set in JVM env
>>> -                System.out.println("Set OFBIZ_HOME to - " + ofbizHome);
>>> -                System.setProperty("ofbiz.home", ofbizHome);
>>> -            }
>>> -        }
>>>      }
>>>        protected String getServerId() {
>>> 
>>> Modified: ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/web.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/
>>> webtools/webapp/webtools/WEB-INF/web.xml?rev=1662393&r1=
>>> 1662392&r2=1662393&view=diff
>>> ============================================================
>>> ==================
>>> --- ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/web.xml
>>> (original)
>>> +++ ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/web.xml Thu
>>> Feb 26 09:09:04 2015
>>> @@ -44,12 +44,6 @@ under the License.
>>>      <description>The location of the main-decorator screen to use for
>>> this webapp; referred to as a context variable in screen def XML
>>> files.</description>
>>>    </context-param>
>>>  -  <context-param>
>>> -    <param-name>GeronimoMultiOfbizInstances</param-name>
>>> -    <param-value>true</param-value>
>>> -    <description>This is only used in case of Geronimo or WASCE using
>>> OFBiz multi-instances. It allows to retrieve ofbiz.home value set in JVM
>>> env</description>
>>> -  </context-param>
>>> -
>>>      <filter>
>>>          <filter-name>ContextFilter</filter-name>
>>>          <display-name>ContextFilter</display-name>
>>> 
>>> 
>>> 
>>> 

Reply via email to