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/27 14:09:55 UTC
svn commit: r1750343 -
/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java
Author: arunpatidar
Date: Mon Jun 27 14:09:55 2016
New Revision: 1750343
URL: http://svn.apache.org/viewvc?rev=1750343&view=rev
Log:
Applied patch from jira issue - OFBIZ-7408 - Enforce noninstantiability to ProductDisplayWorker Class. Thanks Rishi Solanki and Rohit Koushal for your contribution.
Modified:
ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java
Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java?rev=1750343&r1=1750342&r2=1750343&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/shoppingcart/product/ProductDisplayWorker.java Mon Jun 27 14:09:55 2016
@@ -48,10 +48,12 @@ import org.ofbiz.product.category.Catego
import org.ofbiz.product.product.ProductWorker;
-public class ProductDisplayWorker {
+public final class ProductDisplayWorker {
public static final String module = ProductDisplayWorker.class.getName();
+ private ProductDisplayWorker() {}
+
/* ========================================================================================*/
/* ============================= Special Data Retrieval Methods ===========================*/
@@ -306,7 +308,7 @@ public class ProductDisplayWorker {
return result;
}
- static class ProductByMapComparator implements Comparator<Object> {
+ private static class ProductByMapComparator implements Comparator<Object> {
private Map<String, Object> orderByMap;
private boolean descending;