You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ar...@apache.org on 2016/08/27 11:05:48 UTC
svn commit: r1757989 -
/ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java
Author: arunpatidar
Date: Sat Aug 27 11:05:48 2016
New Revision: 1757989
URL: http://svn.apache.org/viewvc?rev=1757989&view=rev
Log:
Applied patch from jira issue - OFBIZ-7983 - Enforce noninstantiability to JNDIConfigUtil class. Thanks Rishi Solanki and Rohit Koushal for your contribution.
Modified:
ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java
Modified: ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java?rev=1757989&r1=1757988&r2=1757989&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java (original)
+++ ofbiz/trunk/framework/base/src/main/java/org/apache/ofbiz/base/config/JNDIConfigUtil.java Sat Aug 27 11:05:48 2016
@@ -28,11 +28,12 @@ import org.w3c.dom.Element;
* JNDIConfigUtil
*
*/
-public class JNDIConfigUtil {
+public final class JNDIConfigUtil {
public static final String module = JNDIConfigUtil.class.getName();
- public static final String JNDI_CONFIG_XML_FILENAME = "jndiservers.xml";
+ private static final String JNDI_CONFIG_XML_FILENAME = "jndiservers.xml";
private static final ConcurrentHashMap<String, JndiServerInfo> jndiServerInfos = new ConcurrentHashMap<String, JndiServerInfo>();
+ private JNDIConfigUtil() {};
private static Element getXmlRootElement() throws GenericConfigException {
try {