[ofbiz-framework] branch release17.12 updated: Improved: ‘EntityOperator#getId()’ is not used in the framework (OFBIZ-10607)

[mbrohl]

This is an automated email from the ASF dual-hosted git repository.

mbrohl pushed a commit to branch release17.12
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release17.12 by this push:
     new b0aff70  Improved: ‘EntityOperator#getId()’ is not used in the framework (OFBIZ-10607)
b0aff70 is described below

commit b0aff701d05989214a7be932915bcc6c6489fa09
Author: Michael Brohl <[hidden email]>
AuthorDate: Thu Jan 30 20:57:16 2020 +0100

    Improved: ‘EntityOperator#getId()’ is not used in the framework
    (OFBIZ-10607)
   
    Thanks Mathieu Lirzin for reporting and providing the patch.
---
 .../main/java/org/apache/ofbiz/entity/condition/EntityOperator.java | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityOperator.java b/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityOperator.java
index 860b3a5..5762b68 100644
--- a/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityOperator.java
+++ b/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityOperator.java
@@ -25,7 +25,6 @@ import java.util.Iterator;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
-import java.util.concurrent.atomic.AtomicInteger;
 
 import org.apache.ofbiz.base.util.UtilGenerics;
 import org.apache.ofbiz.entity.Delegator;
@@ -57,7 +56,6 @@ public abstract class EntityOperator<L, R, T> extends EntityConditionBase {
     public static final int ID_NOT_IN = 13;
     public static final int ID_NOT_LIKE = 14;
 
-    private static final AtomicInteger dynamicId = new AtomicInteger();
     private static HashMap<String, EntityOperator<?,?,?>> registry = new HashMap<>();
 
     private static <L,R,T> void registerCase(String name, EntityOperator<L,R,T> operator) {
@@ -91,10 +89,6 @@ public abstract class EntityOperator<L, R, T> extends EntityConditionBase {
         return UtilGenerics.cast(operator);
     }
 
-    public static int requestId() {
-        return dynamicId.get();
-    }
-
     public static final EntityComparisonOperator<?,?> EQUALS = new ComparableEntityComparisonOperator<Object>(ID_EQUALS, "=") {
         @Override
         public boolean compare(Comparable<Object> lhs, Object rhs) { return EntityComparisonOperator.compareEqual(lhs, rhs); }