From 58298c9187374f3cf06efdff71ee302199ee4cbe Mon Sep 17 00:00:00 2001
From: Nick Randolph <nick@builttoroam.com>
Date: Tue, 1 Nov 2022 16:50:16 +1100
Subject: [PATCH] feat: Adding reactive flag

---
 .../.template.config/dotnetcli.host.json      |  4 ++
 .../.template.config/ide.host.json            | 12 +++++
 .../.template.config/template.json            |  6 +++
 .../unoapp-extensions/Directory.Build.props   |  1 +
 .../Directory.Packages.props                  |  1 +
 .../MyExtensionsApp.UI/App.xaml.host.cs       | 11 ++--
 .../MyExtensionsApp.UI/Views/MainPage.xaml    | 18 ++++---
 .../MyExtensionsApp.UI/Views/SecondPage.xaml  |  5 ++
 .../MyExtensionsApp/Business/Models/Entity.cs |  5 ++
 .../MyExtensionsApp/MyExtensionsApp.csproj    | 38 +++++++------
 .../Presentation/MainViewModel.cs             | 53 +++++++++++++++++--
 .../Presentation/SecondViewModel.cs           |  6 +--
 12 files changed, 125 insertions(+), 35 deletions(-)
 create mode 100644 src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Business/Models/Entity.cs

diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/dotnetcli.host.json b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/dotnetcli.host.json
index a61b636854..8bc7cfdf8b 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/dotnetcli.host.json
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/dotnetcli.host.json
@@ -41,6 +41,10 @@
       "longName": "tests",
       "shortName": "tests"
     },
+    "reactive": {
+      "longName": "reactive",
+      "shortName": "reactive"
+    },
     "wasm-pwa-manifest": {
       "longName": "wasm-pwa-manifest",
       "shortName": "pwa"
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/ide.host.json b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/ide.host.json
index 42c6aaa91f..3a17188114 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/ide.host.json
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/ide.host.json
@@ -129,6 +129,18 @@
       },
       "defaultValue": "true",
       "isVisible":true
+    },
+    {
+      "id": "Reactive",
+      "name": {
+        "text": "Data - Reactive Extensions"
+      },
+      "description":
+      {
+        "text": "Uses Reactive extensions"
+      },
+      "defaultValue": "true",
+      "isVisible":true
     }
   ]
 }
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/template.json b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/template.json
index 7629c833a8..5f2e935f1d 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/template.json
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/.template.config/template.json
@@ -106,6 +106,12 @@
       "defaultValue": "true",
       "description": "Includes unit and UI test projects"
     },
+    "reactive": {
+      "type": "parameter",
+      "dataType": "bool",
+      "defaultValue": "true",
+      "description": "Whether or not to use Reactive extensions"
+    },
     "mobile": {
       "type": "computed",
       "dataType": "bool",
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Build.props b/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Build.props
index 8b16110a09..ae49deea27 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Build.props
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Build.props
@@ -14,6 +14,7 @@
 
 		<TreatWarningsAsErrors>True</TreatWarningsAsErrors>
 	  	<NoWarn>$(NoWarn);Uno0001;CS1998;CA1416;NU1507</NoWarn>
+		<DefineConstants>$(DefineConstants);REACTIVE</DefineConstants>
 	</PropertyGroup>
 
 	<ItemGroup>
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Packages.props b/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Packages.props
index 7ba4c8fbbd..ade7c1c555 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Packages.props
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/Directory.Packages.props
@@ -1,5 +1,6 @@
 <Project ToolsVersion="15.0">
   <ItemGroup>
+    <PackageVersion Include="CommunityToolkit.Mvvm" Version="8.0.0" />
     <PackageVersion Include="coverlet.collector" Version="3.1.0" />
     <PackageVersion Include="FluentAssertions" Version="6.7.0" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="6.0.0" />
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/App.xaml.host.cs b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/App.xaml.host.cs
index 0544b1a514..86c2783680 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/App.xaml.host.cs
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/App.xaml.host.cs
@@ -1,5 +1,4 @@
 //-:cnd:noEmit
-
 namespace MyExtensionsApp;
 
 public sealed partial class App : Application
@@ -46,7 +45,13 @@ private static IHost BuildAppHost()
 
 
 				// Enable navigation, including registering views and viewmodels
-				.UseNavigation(ReactiveViewModelMappings.ViewModelMappings, RegisterRoutes)
+				.UseNavigation(
+//+:cnd:noEmit
+#if(reactive)
+			ReactiveViewModelMappings.ViewModelMappings,
+#endif
+//-:cnd:noEmit
+			RegisterRoutes)
 
 				// Add navigation support for toolkit controls such as TabBar and NavigationView
 				.UseToolkitNavigation()
@@ -59,7 +64,7 @@ private static void RegisterRoutes(IViewRegistry views, IRouteRegistry routes)
 		views.Register(
 			new ViewMap(ViewModel: typeof(ShellViewModel)),
 			new ViewMap<MainPage, MainViewModel>(),
-			new ViewMap<SecondPage, SecondViewModel>()
+			new DataViewMap<SecondPage, SecondViewModel, Entity>()
 			);
 
 		routes
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/MainPage.xaml b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/MainPage.xaml
index c55c72c705..1950cc9859 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/MainPage.xaml
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/MainPage.xaml
@@ -8,6 +8,7 @@
 	  xmlns:uen="using:Uno.Extensions.Navigation.UI"
 	  xmlns:utu="using:Uno.Toolkit.UI"
 	  xmlns:not_skia="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
+	  NavigationCacheMode="Required"
 	  Background="{ThemeResource BackgroundBrush}">
 
 	<Grid>
@@ -16,11 +17,16 @@
 			<RowDefinition />
 		</Grid.RowDefinitions>
 		<utu:NavigationBar Content="{Binding Title}" />
-		<Button Grid.Row="1"
-				HorizontalAlignment="Center"
-				VerticalAlignment="Center"
-				Content="Go to Second Page"
-				not_skia:AutomationProperties.AutomationId="SecondPageButton"
-				Command="{Binding GoToSecond}"/>
+
+		<StackPanel Grid.Row="1"
+					HorizontalAlignment="Center"
+					VerticalAlignment="Center">
+			<TextBox Text="{Binding Name, Mode=TwoWay}"
+					 PlaceholderText="Enter your name:"
+					 Margin="8" />
+			<Button 				Content="Go to Second Page"
+									not_skia:AutomationProperties.AutomationId="SecondPageButton"
+									Command="{Binding GoToSecond}" />
+		</StackPanel>
 	</Grid>
 </Page>
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/SecondPage.xaml b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/SecondPage.xaml
index 9f2a1cb94c..e2f1f71a51 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/SecondPage.xaml
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp.UI/Views/SecondPage.xaml
@@ -18,5 +18,10 @@
 				</AppBarButton>
 			</utu:NavigationBar.MainCommand>
 		</utu:NavigationBar>
+
+		<TextBlock Text="{Binding Entity.Name}"
+				   HorizontalAlignment="Center"
+				   VerticalAlignment="Center"
+				   />
 	</Grid>
 </Page>
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Business/Models/Entity.cs b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Business/Models/Entity.cs
new file mode 100644
index 0000000000..eddea85910
--- /dev/null
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Business/Models/Entity.cs
@@ -0,0 +1,5 @@
+//-:cnd:noEmit
+
+namespace MyExtensionsApp.Business.Models;
+
+public record Entity(string Name);
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/MyExtensionsApp.csproj b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/MyExtensionsApp.csproj
index 1ef6f01cde..e6431a0916 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/MyExtensionsApp.csproj
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/MyExtensionsApp.csproj
@@ -2,21 +2,25 @@
 	<PropertyGroup>
 		<TargetFramework>net6.0</TargetFramework>
 	</PropertyGroup>
-	
-  <ItemGroup>
-    <PackageReference Include="Uno.Extensions.Authentication" />
-    <PackageReference Include="Uno.Extensions.Configuration" />
-    <PackageReference Include="Uno.Extensions.Core" />
-    <PackageReference Include="Uno.Extensions.Hosting" />
-    <PackageReference Include="Uno.Extensions.Http" />
-    <PackageReference Include="Uno.Extensions.Http.Refit" />
-    <PackageReference Include="Uno.Extensions.Localization" />
-    <PackageReference Include="Uno.Extensions.Logging.Serilog" />
-    <PackageReference Include="Uno.Extensions.Navigation" />
-    <PackageReference Include="Uno.Extensions.Serialization" />
-    <PackageReference Include="Uno.Extensions.Serialization.Http" />
-    <PackageReference Include="Uno.Extensions.Serialization.Refit" />
-    <PackageReference Include="Uno.Extensions.Reactive" />
-    <PackageReference Include="Uno.Extensions.Reactive.WinUI" />
-  </ItemGroup>
+
+	<ItemGroup>
+		<!--#if (!reactive) -->
+		<PackageReference Include="CommunityToolkit.Mvvm" />
+		<!--#endif -->
+
+		<PackageReference Include="Uno.Extensions.Authentication" />
+		<PackageReference Include="Uno.Extensions.Configuration" />
+		<PackageReference Include="Uno.Extensions.Core" />
+		<PackageReference Include="Uno.Extensions.Hosting" />
+		<PackageReference Include="Uno.Extensions.Http" />
+		<PackageReference Include="Uno.Extensions.Http.Refit" />
+		<PackageReference Include="Uno.Extensions.Localization" />
+		<PackageReference Include="Uno.Extensions.Logging.Serilog" />
+		<PackageReference Include="Uno.Extensions.Navigation" />
+		<PackageReference Include="Uno.Extensions.Serialization" />
+		<PackageReference Include="Uno.Extensions.Serialization.Http" />
+		<PackageReference Include="Uno.Extensions.Serialization.Refit" />
+		<PackageReference Include="Uno.Extensions.Reactive" />
+		<PackageReference Include="Uno.Extensions.Reactive.WinUI" />
+	</ItemGroup>
 </Project>
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/MainViewModel.cs b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/MainViewModel.cs
index 7d182c72fe..311c9709ad 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/MainViewModel.cs
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/MainViewModel.cs
@@ -1,11 +1,50 @@
-//-:cnd:noEmit
+//+:cnd:noEmit
+#if(reactive)
+using Uno.Extensions.Reactive;
+#else
+using System.Windows.Input;
+using CommunityToolkit.Mvvm.ComponentModel;
+using CommunityToolkit.Mvvm.Input;
+#endif
 
 namespace MyExtensionsApp.Presentation;
 
-public partial class MainViewModel
+#if(reactive)
+public partial class MainViewModel 
+{
+	public string? Title { get; }
+
+	public IState<string> Name { get; }
+
+	public MainViewModel(
+		INavigator navigator,
+		IOptions<AppConfig> appInfo)
+	{
+
+		_navigator = navigator;
+		Title = $"Main - {appInfo?.Value?.Title}";
+
+		Name = State<string>.Value(this, ()=>"");
+	}
+
+	public async Task GoToSecond()
+	{
+		var name = await Name;
+		await _navigator.NavigateViewModelAsync<SecondViewModel>(this, data: new Entity(name!));
+	}
+
+	private INavigator _navigator;
+}
+#else
+public partial class MainViewModel:ObservableObject
 {
 	public string? Title { get; }
 
+	[ObservableProperty]
+	private string? name;
+
+	public ICommand GoToSecond { get; }
+
 	public MainViewModel(
 		INavigator navigator,
 		IOptions<AppConfig> appInfo)
@@ -13,12 +52,18 @@ public MainViewModel(
 	
 		_navigator = navigator;
 		Title = $"Main - {appInfo?.Value?.Title}";
+
+		GoToSecond = new AsyncRelayCommand(GoToSecondView);
 	}
 
-	public async Task GoToSecond(CancellationToken cancellation)
+	public async Task GoToSecondView()
 	{
-		await _navigator.NavigateViewModelAsync<SecondViewModel>(this, cancellation: cancellation);
+		await _navigator.NavigateViewModelAsync<SecondViewModel>(this, data: new Entity(Name!));
 	}
 
 	private INavigator _navigator;
 }
+#endif
+
+//-:cnd:noEmit
+
diff --git a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/SecondViewModel.cs b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/SecondViewModel.cs
index 655f0ddc00..aa94071611 100644
--- a/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/SecondViewModel.cs
+++ b/src/Uno.Extensions.Templates/content/unoapp-extensions/MyExtensionsApp/Presentation/SecondViewModel.cs
@@ -2,10 +2,6 @@
 
 namespace MyExtensionsApp.Presentation;
 
-public partial class SecondViewModel
+public partial record SecondViewModel (Entity Entity)
 {
-	
-	public SecondViewModel()
-	{
-	}
 }