Improvement Number Format (#13744)

* Improvement Number Format

* updating scope

* minor refactoring

* Number Format to Double and String values based on the preference

* comment changes and refactoring

Author: jesusalvino

src/DynamoCore/Models/DynamoModel.cs

@@ -1663,7 +1663,7 @@ private IPreferences CreateOrLoadPreferences(IPreferences preferences)
 
         private static void InitializePreferences(IPreferences preferences)
         {
-            DynamoUnits.Display.PrecisionFormat = preferences.NumberFormat;
+            ProtoCore.Mirror.MirrorData.PrecisionFormat = DynamoUnits.Display.PrecisionFormat = preferences.NumberFormat;
 
             var settings = preferences as PreferenceSettings;
             if (settings != null)
@@ -1683,8 +1683,8 @@ private void PreferenceSettings_PropertyChanged(object sender, PropertyChangedEv
         {
             switch (e.PropertyName)
             {
-                case "NumberFormat":
-                    DynamoUnits.Display.PrecisionFormat = PreferenceSettings.NumberFormat;
+                case nameof(PreferenceSettings.NumberFormat):
+                    ProtoCore.Mirror.MirrorData.PrecisionFormat = DynamoUnits.Display.PrecisionFormat = PreferenceSettings.NumberFormat;
                     break;
             }
         }

src/DynamoCoreWpf/Utilities/PreferencesPanelUtilities.cs

@@ -1,4 +1,4 @@
-using System.Linq;
+using System.Linq;
 using System.Windows;
 using System.Windows.Controls;
 using Dynamo.Controls;

src/DynamoCoreWpf/ViewModels/Preview/WatchViewModel.cs

@@ -250,6 +250,8 @@ private static string GetStringFromObject(object obj)
                     return ObjectToLabelString(obj);
                 case TypeCode.Double:
                     return ((double)obj).ToString(numberFormat, CultureInfo.InvariantCulture);
+                //TODO: uncomment this once https://jira.autodesk.com/browse/DYN-5101 is complete
+                //return ((double)obj).ToString(ProtoCore.Mirror.MirrorData.PrecisionFormat, CultureInfo.InvariantCulture);
                 case TypeCode.Int32:
                     return ((int)obj).ToString(CultureInfo.InvariantCulture);
                 case TypeCode.Int64:
@@ -259,6 +261,10 @@ private static string GetStringFromObject(object obj)
                 case TypeCode.Object:
                     return ObjectToLabelString(obj);
                 default:
+                    if (double.TryParse(obj.ToString(), out double d))
+                    {
+                        return Convert.ToDouble(obj).ToString(ProtoCore.Mirror.MirrorData.PrecisionFormat, CultureInfo.InvariantCulture);
+                    }
                     return (string)obj;
             };
         }

src/DynamoCoreWpf/Views/Preview/PreviewControl.xaml.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;
@@ -383,8 +383,7 @@ private void RefreshExpandedDisplay(Action refreshDisplay)
 
                     newViewModel = nodeViewModel.DynamoViewModel.WatchHandler.GenerateWatchViewModelForData(
                         nodeViewModel.NodeModel.CachedValue, preferredDictionaryOrdering,
-                        null, nodeViewModel.NodeModel.AstIdentifierForPreview.Name, false);
-
+                        null, nodeViewModel.NodeModel.AstIdentifierForPreview.Name, false);                    
                 },
                 (m) =>
                 {
@@ -434,7 +433,7 @@ private void RefreshExpandedDisplay(Action refreshDisplay)
                 }
             );
         }
-
+        
         /// <summary>
         /// It's used to apply Collapsed and Expanded events for TreeViewItems.
         /// </summary>

src/Engine/ProtoCore/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Runtime.CompilerServices;
 
 // General Information about an assembly is controlled through the following 
@@ -12,6 +12,7 @@
 [assembly: InternalsVisibleTo("ProtoTest")]
 [assembly: InternalsVisibleTo("ProtoTestFx")]
 [assembly:InternalsVisibleTo("DynamoCore")]
+[assembly: InternalsVisibleTo("DynamoCoreWpf")]
 [assembly: InternalsVisibleTo("DynamoCoreTests")]
 [assembly: InternalsVisibleTo("DynamoCoreWpfTests")]
 [assembly: InternalsVisibleTo("ProtoImperative")]

src/Engine/ProtoCore/Reflection/MirrorData.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Globalization;
 using System.Linq;
@@ -208,6 +208,8 @@ internal static object GetData(StackValue sv, RuntimeCore runtimeCore)
                 return null;
             }
 
+            internal static string PrecisionFormat { get; set; } = "f3";
+
             /// <summary>
             /// Returns string representation of data
             /// </summary>
@@ -230,10 +232,23 @@ public string StringData
                         // https://msdn.microsoft.com/en-us/library/3hfd35ad(v=vs.110).aspx
                         // We should always use invariant culture format for formattable 
                         // object.
+
+                        //TODO: uncomment this once https://jira.autodesk.com/browse/DYN-5101 is complete
+                        //if (Data is double)
+                        //{
+                        //    return (Data as IFormattable).ToString(PrecisionFormat, CultureInfo.InvariantCulture);
+                        //}
+                        //else
+                        //{
                         return (Data as IFormattable).ToString(null, CultureInfo.InvariantCulture);
+                        //}
                     }
                     else
                     {
+                        if (double.TryParse(Data.ToString(), out double d))
+                        {
+                            return Convert.ToDouble(Data).ToString(PrecisionFormat, CultureInfo.InvariantCulture);
+                        }
                         return Data.ToString();
                     }
                 }