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/06/28 14:41:22 UTC
svn commit: r1750526 -
/ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java
Author: arunpatidar
Date: Tue Jun 28 14:41:22 2016
New Revision: 1750526
URL: http://svn.apache.org/viewvc?rev=1750526&view=rev
Log:
Applied patch from jira issue - OFBIZ-7445 - Enforce noninstantiability to ProductConfigWorker Class. Thanks Rishi Solanki and Amit Gadaley for your contribution.
Modified:
ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java
Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java?rev=1750526&r1=1750525&r2=1750526&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/config/ProductConfigWorker.java Tue Jun 28 14:41:22 2016
@@ -47,11 +47,12 @@ import org.ofbiz.webapp.website.WebSiteW
/**
* Product Config Worker class to reduce code in templates.
*/
-public class ProductConfigWorker {
+public final class ProductConfigWorker {
public static final String module = ProductConfigWorker.class.getName();
- public static final String resource = "ProductUiLabels";
- public static final String SEPARATOR = "::"; // cache key separator
+ private static final String SEPARATOR = "::"; // cache key separator
+
+ private ProductConfigWorker () {}
private static final UtilCache<String, ProductConfigWrapper> productConfigCache = UtilCache.createUtilCache("product.config", true); // use soft reference to free up memory if needed