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:57:28 UTC

svn commit: r1750531 - /ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java

Author: arunpatidar
Date: Tue Jun 28 14:57:28 2016
New Revision: 1750531

URL: http://svn.apache.org/viewvc?rev=1750531&view=rev
Log:
Applied patch from jira issue - OFBIZ-7450 - Enforce noninstantiability to ProductUtilServices Class. Thanks Rishi Solanki and Pratik Kulshreshthfor your contribution.

Modified:
    ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java

Modified: ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java?rev=1750531&r1=1750530&r2=1750531&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java (original)
+++ ofbiz/trunk/applications/product/src/org/ofbiz/product/product/ProductUtilServices.java Tue Jun 28 14:57:28 2016
@@ -54,11 +54,13 @@ import org.ofbiz.service.ServiceUtil;
 /**
  * Product Services
  */
-public class ProductUtilServices {
+public final class ProductUtilServices {
 
     public static final String module = ProductUtilServices.class.getName();
-    public static final String resource = "ProductUiLabels";
-    public static final String resourceError = "ProductErrorUiLabels";
+    private static final String resource = "ProductUiLabels";
+    private static final String resourceError = "ProductErrorUiLabels";
+
+    private ProductUtilServices () {}
 
     /** First expire all ProductAssocs for all disc variants, then disc all virtuals that have all expired variant ProductAssocs */
     public static Map<String, Object> discVirtualsWithDiscVariants(DispatchContext dctx, Map<String, ? extends Object> context) {