You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by de...@apache.org on 2026/07/01 05:55:58 UTC

(ofbiz-framework) 02/03: Replaced the mutable HashSet initialization of NUMERIC_FIELD_TYPES with Set.of(...).

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

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

commit e9750fb937c7dc23774d9f304ba731172bf09a69
Author: Deepak Dixit <de...@apache.org>
AuthorDate: Wed Jul 1 11:09:04 2026 +0530

    Replaced the mutable HashSet initialization of NUMERIC_FIELD_TYPES with Set.of(...).
---
 .../java/org/apache/ofbiz/entity/model/ModelViewEntity.java    | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/framework/entity/src/main/java/org/apache/ofbiz/entity/model/ModelViewEntity.java b/framework/entity/src/main/java/org/apache/ofbiz/entity/model/ModelViewEntity.java
index 20b69aa3f7..25cefcbf5d 100644
--- a/framework/entity/src/main/java/org/apache/ofbiz/entity/model/ModelViewEntity.java
+++ b/framework/entity/src/main/java/org/apache/ofbiz/entity/model/ModelViewEntity.java
@@ -60,8 +60,14 @@ public class ModelViewEntity extends ModelEntity {
     private static final Map<String, String> FUNCTION_PREFIX_MAP = new HashMap<>();
     private static final Set<String> NUMERIC_FUNCTION_SET = new HashSet<>(); // names of functions that return a numeric type
     // OFBiz field types that map to numeric Java types — default values for these do not need SQL quoting
-    private static final Set<String> NUMERIC_FIELD_TYPES = new HashSet<>(Arrays.asList(
-            "numeric", "integer", "floating-point", "fixed-point", "currency-amount", "currency-precise"));
+    private static final Set<String> NUMERIC_FIELD_TYPES = Set.of(
+        "numeric",
+        "integer",
+        "floating-point",
+        "fixed-point",
+        "currency-amount",
+        "currency-precise"
+    );
     static {
         FUNCTION_PREFIX_MAP.put("min", "MIN(");
         FUNCTION_PREFIX_MAP.put("max", "MAX(");