Fixed string conversion bug with global function

riganti · Oct 16, 2023 · 2a8200f · 2a8200f
1 parent 3c6ee78
commit 2a8200f
Show file tree

Hide file tree

Showing 7 changed files with 57 additions and 4 deletions.
diff --git a/src/Framework/Framework/Compilation/Javascript/JavascriptTranslatableMethodCollection.cs b/src/Framework/Framework/Compilation/Javascript/JavascriptTranslatableMethodCollection.cs
@@ -760,7 +760,7 @@ JsExpression wrapInRound(JsExpression a) =>
                 }
                 else if (p[0].ParameterType == typeof(string) || p[0].ParameterType == typeof(bool))
                 {
-                    AddMethodTranslator(m, translator: new GenericMethodCompiler(args => wrapInRound(new JsIdentifierExpression("Number").Invoke(args[1]))));
+                    AddMethodTranslator(m, translator: new GenericMethodCompiler(args => wrapInRound(new JsIdentifierExpression("window").Member("Number").Invoke(args[1]))));
                 }
             }
 
@@ -773,7 +773,7 @@ JsExpression wrapInRound(JsExpression a) =>
                     continue;
                 if (p[0].ParameterType.IsNumericType() && p[0].ParameterType != typeof(char))
                 {
-                    AddMethodTranslator(m, translator: new GenericMethodCompiler(args => new JsIdentifierExpression("Boolean").Invoke(args[1])));
+                    AddMethodTranslator(m, translator: new GenericMethodCompiler(args => new JsIdentifierExpression("window").Member("Boolean").Invoke(args[1])));
                 }
             }
         }

diff --git a/src/Framework/Framework/Compilation/Javascript/PrimitiveToStringTranslator.cs b/src/Framework/Framework/Compilation/Javascript/PrimitiveToStringTranslator.cs
@@ -59,7 +59,7 @@ internal static bool CanBeNull(Expression expr)
             if (isNullable)
                 js = new JsBinaryExpression(js, BinaryOperatorType.NullishCoalescing, new JsLiteral(""));
             if (!isStringAlready)
-                js = new JsIdentifierExpression("String").Invoke(js);
+                js = new JsIdentifierExpression("window").Member("String").Invoke(js);
             return js;
         }
     }

diff --git a/src/Samples/Common/DotVVM.Samples.Common.csproj b/src/Samples/Common/DotVVM.Samples.Common.csproj
@@ -98,6 +98,7 @@
     <None Remove="Views\FeatureSamples\CustomPrimitiveTypes\RouteLink.dothtml" />
     <None Remove="Views\FeatureSamples\CustomPrimitiveTypes\TextBox.dothtml" />
     <None Remove="Views\FeatureSamples\CustomPrimitiveTypes\UsedInControls.dothtml" />
+    <None Remove="Views\FeatureSamples\Formatting\ToStringGlobalFunctionBug.dothtml" />
     <None Remove="Views\FeatureSamples\HotReload\ViewChanges.dothtml" />
     <None Remove="Views\FeatureSamples\JavascriptTranslation\ArrayTranslation.dothtml" />
     <None Remove="Views\FeatureSamples\JavascriptTranslation\DateTimeTranslations.dothtml" />

diff --git a/...Samples/Common/ViewModels/FeatureSamples/Formatting/ToStringGlobalFunctionBugViewModel.cs b/...Samples/Common/ViewModels/FeatureSamples/Formatting/ToStringGlobalFunctionBugViewModel.cs
@@ -0,0 +1,18 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using DotVVM.Framework.ViewModel;
+using DotVVM.Framework.Hosting;
+
+namespace DotVVM.Samples.Common.ViewModels.FeatureSamples.Formatting
+{
+    public class ToStringGlobalFunctionBugViewModel : DotvvmViewModelBase
+    {
+        public bool Boolean { get; set; }
+
+        public string String { get; set; }
+    }
+}
+
diff --git a/src/Samples/Common/Views/FeatureSamples/Formatting/ToStringGlobalFunctionBug.dothtml b/src/Samples/Common/Views/FeatureSamples/Formatting/ToStringGlobalFunctionBug.dothtml
@@ -0,0 +1,19 @@
+@viewModel DotVVM.Samples.Common.ViewModels.FeatureSamples.Formatting.ToStringGlobalFunctionBugViewModel, DotVVM.Samples.Common
+
+<!DOCTYPE html>
+
+<html lang="en" xmlns="http://www.w3.org/1999/xhtml">
+<head>
+    <meta charset="utf-8" />
+    <title></title>
+</head>
+<body>
+    <dot:Button Text="Set to true" Click="{staticCommand: Boolean = true}" />
+
+    <p class="result-boolean">Boolean: {{value: Boolean}}</p>
+    <p class="result-string">String: {{value: String}}</p>
+
+</body>
+</html>
+
+
diff --git a/src/Samples/Tests/Abstractions/SamplesRouteUrls.designer.cs b/src/Samples/Tests/Abstractions/SamplesRouteUrls.designer.cs
diff --git a/src/Samples/Tests/Tests/Feature/FormattingTests.cs b/src/Samples/Tests/Tests/Feature/FormattingTests.cs
@@ -55,8 +55,22 @@ public void Feature_Formatting_Formatting()
             });
         }
 
+        [Fact]
+        public void Feature_Formatting_ToStringGlobalFunctionBug()
+        {
+            RunInAllBrowsers(browser => {
+                browser.NavigateToUrl(SamplesRouteUrls.FeatureSamples_Formatting_ToStringGlobalFunctionBug);
+
+                AssertUI.TextEquals(browser.Single(".result-boolean"), "Boolean: false");
+                AssertUI.TextEquals(browser.Single(".result-string"), "String:");
+                browser.Single("input[type=button]").Click();
+                AssertUI.TextEquals(browser.Single(".result-boolean"), "Boolean: true");
+                AssertUI.TextEquals(browser.Single(".result-string"), "String:");
+            });
+        }
+
         public FormattingTests(ITestOutputHelper output) : base(output)
         {
+            }
         }
     }
-}