From 7dffc11ab4cc72ddac1a48daad5b44413f33fb50 Mon Sep 17 00:00:00 2001
From: LinusCenterstrom <linuscent@gmail.com>
Date: Wed, 6 Nov 2024 16:26:50 +0100
Subject: [PATCH] fix: Better support for phrases which contain a \n (#176)

* Better support for phrases which contain a \n

* Removed temporary test
---
 MN.L10n.BuildTasks/MN.L10n.BuildTasks.csproj         |  2 +-
 .../MN.L10n.Javascript.Core.csproj                   |  2 +-
 .../MN.L10n.Javascript.Shared.csproj                 |  2 +-
 MN.L10n.Javascript/MN.L10n.Javascript.csproj         |  4 ++--
 MN.L10n.Tests/ParserTests.cs                         | 12 +++++++++++-
 MN.L10n/L10nParser.cs                                | 10 +++++++++-
 MN.L10n/MN.L10n.csproj                               |  4 ++--
 7 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/MN.L10n.BuildTasks/MN.L10n.BuildTasks.csproj b/MN.L10n.BuildTasks/MN.L10n.BuildTasks.csproj
index 0a23feb..07112e9 100644
--- a/MN.L10n.BuildTasks/MN.L10n.BuildTasks.csproj
+++ b/MN.L10n.BuildTasks/MN.L10n.BuildTasks.csproj
@@ -6,7 +6,7 @@
     <OutputType>Exe</OutputType>
     <StartupObject>MN.L10n.BuildTasks.Program</StartupObject>
     <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
-    <Version>4.0.0</Version>
+    <Version>4.0.1</Version>
     <Authors>Chris Gårdenberg</Authors>
     <Company>MultiNet Interactive AB</Company>
   </PropertyGroup>
diff --git a/MN.L10n.Javascript.Core/MN.L10n.Javascript.Core.csproj b/MN.L10n.Javascript.Core/MN.L10n.Javascript.Core.csproj
index af0ae51..69ffcec 100644
--- a/MN.L10n.Javascript.Core/MN.L10n.Javascript.Core.csproj
+++ b/MN.L10n.Javascript.Core/MN.L10n.Javascript.Core.csproj
@@ -7,7 +7,7 @@
     <PackageProjectUrl>https://github.com/MultinetInteractive/MN.L10n</PackageProjectUrl>
     <RepositoryType>git</RepositoryType>
     <Copyright>© 20XX MultiNet Interactive AB</Copyright>
-    <Version>3.0.1</Version>
+    <Version>3.0.2</Version>
   </PropertyGroup>
 
   <ItemGroup>
diff --git a/MN.L10n.Javascript.Shared/MN.L10n.Javascript.Shared.csproj b/MN.L10n.Javascript.Shared/MN.L10n.Javascript.Shared.csproj
index e8e36a1..0f8defe 100644
--- a/MN.L10n.Javascript.Shared/MN.L10n.Javascript.Shared.csproj
+++ b/MN.L10n.Javascript.Shared/MN.L10n.Javascript.Shared.csproj
@@ -7,7 +7,7 @@
     <PackageProjectUrl>https://github.com/MultinetInteractive/MN.L10n</PackageProjectUrl>
     <RepositoryType>git</RepositoryType>
     <Copyright>© 20XX MultiNet Interactive AB</Copyright>
-    <Version>2.0.0</Version>
+    <Version>2.0.1</Version>
     <AutoIncrementPackageRevision>True</AutoIncrementPackageRevision>
   </PropertyGroup>
 
diff --git a/MN.L10n.Javascript/MN.L10n.Javascript.csproj b/MN.L10n.Javascript/MN.L10n.Javascript.csproj
index 6c45550..7e6e0a1 100644
--- a/MN.L10n.Javascript/MN.L10n.Javascript.csproj
+++ b/MN.L10n.Javascript/MN.L10n.Javascript.csproj
@@ -3,8 +3,8 @@
   <PropertyGroup>
     <TargetFrameworks>net48</TargetFrameworks>
     <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
-    <Version>2.0.0</Version>
-    <AssemblyVersion>2.0.0.0</AssemblyVersion>
+    <Version>2.0.1</Version>
+    <AssemblyVersion>2.0.0.1</AssemblyVersion>
   </PropertyGroup>
 
   <ItemGroup>
diff --git a/MN.L10n.Tests/ParserTests.cs b/MN.L10n.Tests/ParserTests.cs
index 653d352..9a5ef98 100644
--- a/MN.L10n.Tests/ParserTests.cs
+++ b/MN.L10n.Tests/ParserTests.cs
@@ -1,4 +1,5 @@
-using System.Linq;
+using System.IO;
+using System.Linq;
 using Xunit;
 
 namespace MN.L10n.Tests
@@ -105,5 +106,14 @@ public void TestWorksWithLiteralTemplateStrings()
             var result = parser.Parse(src);
             Assert.Single(result);
         }
+
+        [Fact]
+        public void LineBreakCharInCall()
+        {
+            var parser = new L10nParser();
+            var result = parser.Parse("_s('Hello\\nBrother')");
+
+            Assert.Collection(result, x => Assert.Equal("Hello\nBrother", x.Phrase));
+        }
     }
 }
diff --git a/MN.L10n/L10nParser.cs b/MN.L10n/L10nParser.cs
index c68dd47..d021e77 100644
--- a/MN.L10n/L10nParser.cs
+++ b/MN.L10n/L10nParser.cs
@@ -181,7 +181,15 @@ bool TryPeek(int forward)
                                 }
                             }
 
-                            _tokenContent.Append(source[_pos]);
+                            if (inToken && !isVerbatim && source[_pos] == '\\' && TryPeek(1) && source[_pos + 1] == 'n')
+                            {
+                                _pos++;
+                                _tokenContent.Append('\n');
+                            }
+                            else
+                            {
+                                _tokenContent.Append(source[_pos]);
+                            }
                         }
                         break;
                 }
diff --git a/MN.L10n/MN.L10n.csproj b/MN.L10n/MN.L10n.csproj
index c8be3b5..ca9b0f2 100644
--- a/MN.L10n/MN.L10n.csproj
+++ b/MN.L10n/MN.L10n.csproj
@@ -12,13 +12,13 @@ Translation package</Description>
 		<PackageProjectUrl>https://github.com/MultinetInteractive/MN.L10n</PackageProjectUrl>
 		<RepositoryType>git</RepositoryType>
 		<Copyright>© 20XX MultiNet Interactive AB</Copyright>
-		<Version>4.0.0</Version>
+		<Version>4.0.1</Version>
 	<AutoIncrementPackageRevision>True</AutoIncrementPackageRevision>
 	<PackageReleaseNotes>Now includes analyzer</PackageReleaseNotes>
 	<ApplicationIcon />
 	<OutputType>Library</OutputType>
 	<StartupObject></StartupObject>
-	<AssemblyVersion>4.0.0</AssemblyVersion>
+	<AssemblyVersion>4.0.1</AssemblyVersion>
 	</PropertyGroup>
 
 	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">