You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@isis.apache.org by da...@apache.org on 2020/01/17 01:11:46 UTC

[isis] 10/36: ISIS-2264: isis.value.format["long"] -> isis.value-types.}java-lang.long.format

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

danhaywood pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/isis.git

commit d9ebfc6c395dc14aa0e07e4a66afecaea848bdcd
Author: danhaywood <da...@haywood-associates.co.uk>
AuthorDate: Thu Jan 16 23:22:09 2020 +0100

    ISIS-2264: isis.value.format["long"] -> isis.value-types.}java-lang.long.format
---
 .../toc/modules/mignotes/pages/migrating-to-2.0.0.adoc           | 4 ++++
 .../main/java/org/apache/isis/core/config/IsisConfiguration.java | 9 ++++++++-
 .../facets/value/longs/LongValueSemanticsProviderAbstract.java   | 9 ++++++++-
 3 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/antora/components/toc/modules/mignotes/pages/migrating-to-2.0.0.adoc b/antora/components/toc/modules/mignotes/pages/migrating-to-2.0.0.adoc
index 738cf54..00c693c 100644
--- a/antora/components/toc/modules/mignotes/pages/migrating-to-2.0.0.adoc
+++ b/antora/components/toc/modules/mignotes/pages/migrating-to-2.0.0.adoc
@@ -640,6 +640,10 @@ isis.value-types. +
 |isis.value-types. +
 {nbsp}{nbsp}java-lang.float.format
 
+|isis.value.format["long"]
+|isis.value-types. +
+{nbsp}{nbsp}java-lang.long.format
+
 |
 |
 
diff --git a/core/config/src/main/java/org/apache/isis/core/config/IsisConfiguration.java b/core/config/src/main/java/org/apache/isis/core/config/IsisConfiguration.java
index 5242b75..41871d4 100644
--- a/core/config/src/main/java/org/apache/isis/core/config/IsisConfiguration.java
+++ b/core/config/src/main/java/org/apache/isis/core/config/IsisConfiguration.java
@@ -1204,6 +1204,13 @@ public class IsisConfiguration {
             public static class Float {
                 private String format;
             }
+
+            // capitalized to avoid clash with keyword
+            private final Long Long = new Long();
+            @Data
+            public static class Long {
+                private String format;
+            }
         }
 
         private final JavaMath javaMath = new JavaMath();
@@ -1274,7 +1281,7 @@ public class IsisConfiguration {
             TIMESTAMP, 
             TIME,
             
-            LONG, SHORT,
+            SHORT,
             PERCENTAGE
         }
         
diff --git a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/facets/value/longs/LongValueSemanticsProviderAbstract.java b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/facets/value/longs/LongValueSemanticsProviderAbstract.java
index d1592b6..7fe5644 100644
--- a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/facets/value/longs/LongValueSemanticsProviderAbstract.java
+++ b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/facets/value/longs/LongValueSemanticsProviderAbstract.java
@@ -22,9 +22,11 @@ package org.apache.isis.core.metamodel.facets.value.longs;
 import java.text.DecimalFormat;
 import java.text.NumberFormat;
 import java.text.ParseException;
+import java.util.Locale;
 import java.util.Map;
 
 import org.apache.isis.core.config.IsisConfiguration.Value.FormatIdentifier;
+import org.apache.isis.core.metamodel.commons.LocaleUtil;
 import org.apache.isis.core.metamodel.facetapi.Facet;
 import org.apache.isis.core.metamodel.facetapi.FacetHolder;
 import org.apache.isis.core.metamodel.facets.object.parseable.TextEntryParseException;
@@ -46,9 +48,14 @@ public abstract class LongValueSemanticsProviderAbstract extends ValueSemanticsP
 
     public LongValueSemanticsProviderAbstract(final FacetHolder holder, final Class<Long> adaptedClass) {
         super(type(), holder, adaptedClass, TYPICAL_LENGTH, MAX_LENGTH, Immutability.IMMUTABLE, EqualByContent.HONOURED, DEFAULT_VALUE);
-        format = determineNumberFormat(FormatIdentifier.LONG);
+        final String formatRequired = getConfiguration().getValueTypes().getJavaLang().getLong().getFormat();
+
+        format = formatRequired != null
+                ? new DecimalFormat(formatRequired)
+                : NumberFormat.getNumberInstance(getConfiguration().getCore().getRuntime().getLocale().map(LocaleUtil::findLocale).orElse(Locale.getDefault()));
     }
 
+
     // //////////////////////////////////////////////////////////////////
     // Parser
     // //////////////////////////////////////////////////////////////////