add no changes state to diff viewer (#11001)

SHKnudsen · web-flow · commit ee7a504aae09 · 2020-08-17T09:53:12.000-04:00
* add has changes state to diff viewer

* update text + change engine on accept

* typo
diff --git a/src/PythonMigrationViewExtension/Controls/BaseDiffViewer.xaml b/src/PythonMigrationViewExtension/Controls/BaseDiffViewer.xaml
@@ -29,6 +29,42 @@
             <DataTemplate DataType="{x:Type viewModels:SideBySideViewModel}">
                 <local:SideBySideControl DataContext="{Binding}" />
             </DataTemplate>
+            <Style x:Key="HasDifferenceController"
+                   TargetType="{x:Type ContentControl}">
+                <Style.Triggers>
+                    <DataTrigger Binding="{Binding Path=CurrentViewModel.HasChanges}"
+                                 Value="True">
+                        <Setter Property="Content"
+                                Value="{Binding Path=CurrentViewModel}">
+                        </Setter>
+                    </DataTrigger>
+                    <DataTrigger Binding="{Binding Path=CurrentViewModel.HasChanges}"
+                                 Value="False">
+                        <Setter Property="ContentTemplate">
+                            <Setter.Value>
+                                <DataTemplate>
+                                    <StackPanel Orientation="Vertical"
+                                                VerticalAlignment="Center"
+                                                Width="400">
+                                        <TextBlock Text="{x:Static p:Resources.MigrationAssistantNoChangesStateHeader}"
+                                                   TextAlignment="Center"
+                                                   HorizontalAlignment="Center"
+                                                   FontSize="24"
+                                                   Foreground="{StaticResource WorkspaceTabHeaderActiveTextBrush}"
+                                                   TextWrapping="Wrap" />
+                                        <TextBlock Text="{x:Static p:Resources.MigrationAssistantNoChangesStateMessage}"
+                                                   TextAlignment="Center"
+                                                   HorizontalAlignment="Center"
+                                                   FontSize="14"
+                                                   Foreground="{StaticResource WorkspaceTabHeaderActiveTextBrush}"
+                                                   TextWrapping="Wrap" />
+                                    </StackPanel>
+                                </DataTemplate>
+                            </Setter.Value>
+                        </Setter>
+                    </DataTrigger>
+                </Style.Triggers>
+            </Style>
         </ResourceDictionary>
     </Window.Resources>
 
@@ -56,8 +92,9 @@
             </Style>
         </Grid.Resources>
         
-        <ContentControl Content="{Binding CurrentViewModel}"
+        <ContentControl Style="{StaticResource HasDifferenceController}"
                         Grid.Row="0" HorizontalAlignment="Stretch" VerticalAlignment="Stretch"/>
+        
 
         <!-- Buttons row -->
         <Grid HorizontalAlignment="Stretch"
diff --git a/src/PythonMigrationViewExtension/Differ/IDiffViewViewModel.cs b/src/PythonMigrationViewExtension/Differ/IDiffViewViewModel.cs
@@ -3,6 +3,7 @@
     public interface IDiffViewViewModel
     {
         ViewMode ViewMode { get; }
+        bool HasChanges { get; }
     }
 
     public enum ViewMode
diff --git a/src/PythonMigrationViewExtension/Differ/InLineViewModel.cs b/src/PythonMigrationViewExtension/Differ/InLineViewModel.cs
@@ -8,6 +8,7 @@ public class InLineViewModel : IDiffViewViewModel
     {
         public ViewMode ViewMode { get; set; }
         public DiffPaneModel DiffModel { get; set; }
+        public bool HasChanges { get { return DiffModel.HasDifferences; } } 
 
         public InLineViewModel(SideBySideDiffModel diffModel)
         {
diff --git a/src/PythonMigrationViewExtension/Differ/SideBySideViewModel.cs b/src/PythonMigrationViewExtension/Differ/SideBySideViewModel.cs
@@ -8,6 +8,7 @@ public class SideBySideViewModel : IDiffViewViewModel
         public SideBySideDiffModel DiffModel { get; set; }
         public DiffPaneModel AfterPane { get { return DiffModel.NewText; } }
         public DiffPaneModel BeforePane { get { return DiffModel.OldText; } }
+        public bool HasChanges { get { return DiffModel.NewText.HasDifferences | DiffModel.OldText.HasDifferences; } }
 
         public SideBySideViewModel(SideBySideDiffModel diffModel)
         {
diff --git a/src/PythonMigrationViewExtension/MigrationAssistant/PythonMigrationAssistantViewModel.cs b/src/PythonMigrationViewExtension/MigrationAssistant/PythonMigrationAssistantViewModel.cs
@@ -68,6 +68,14 @@ private void MigrateCode()
         /// </summary>
         public void ChangeCode()
         {
+            if (!this.CurrentViewModel.HasChanges)
+            {
+                if (this.PythonNode.Engine == PythonEngineVersion.CPython3)
+                    return;
+                this.PythonNode.Engine = PythonEngineVersion.CPython3;
+                return;
+            }
+
             if (!Models.DynamoModel.IsTestMode && !File.Exists(GetMigrationAssistantDisclaimerDismissFile()))
             {
                 var warningMessage = new MigrationAssistantDisclaimer(this);
diff --git a/src/PythonMigrationViewExtension/Properties/Resources.Designer.cs b/src/PythonMigrationViewExtension/Properties/Resources.Designer.cs
diff --git a/src/PythonMigrationViewExtension/Properties/Resources.en-US.resx b/src/PythonMigrationViewExtension/Properties/Resources.en-US.resx
@@ -164,6 +164,12 @@ By clicking “Continue”, you are agreeing to implement the syntax changes to
   <data name="MigrationAssistantDisclaimerWindowTitle" xml:space="preserve">
     <value>Migration Assistant Disclaimer</value>
   </data>
+  <data name="MigrationAssistantNoChangesStateHeader" xml:space="preserve">
+    <value>Nothing to see here!</value>
+  </data>
+  <data name="MigrationAssistantNoChangesStateMessage" xml:space="preserve">
+    <value>Your code is ready to run with the Python 3 engine. Click accept and we'll switch to the Python 3 engine for you.</value>
+  </data>
   <data name="PackagedCustomNodesHeader" xml:space="preserve">
     <value>Packaged</value>
   </data>
diff --git a/src/PythonMigrationViewExtension/Properties/Resources.resx b/src/PythonMigrationViewExtension/Properties/Resources.resx
@@ -164,6 +164,12 @@ By clicking “Continue”, you are agreeing to implement the syntax changes to
   <data name="MigrationAssistantDisclaimerWindowTitle" xml:space="preserve">
     <value>Migration Assistant Disclaimer</value>
   </data>
+  <data name="MigrationAssistantNoChangesStateHeader" xml:space="preserve">
+    <value>Nothing to see here!</value>
+  </data>
+  <data name="MigrationAssistantNoChangesStateMessage" xml:space="preserve">
+    <value>Your code is ready to run with the Python 3 engine. Click accept and we'll switch to the Python 3 engine for you.</value>
+  </data>
   <data name="PackagedCustomNodesHeader" xml:space="preserve">
     <value>Packaged</value>
   </data>

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`public interface IDiffViewViewModel`
`4`	`4`	`{`
`5`	`5`	`ViewMode ViewMode { get; }`
	`6`	`+ bool HasChanges { get; }`
`6`	`7`	`}`
`7`	`8`
`8`	`9`	`public enum ViewMode`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@ public class InLineViewModel : IDiffViewViewModel`
`8`	`8`	`{`
`9`	`9`	`public ViewMode ViewMode { get; set; }`
`10`	`10`	`public DiffPaneModel DiffModel { get; set; }`
	`11`	`+ public bool HasChanges { get { return DiffModel.HasDifferences; } }`
`11`	`12`
`12`	`13`	`public InLineViewModel(SideBySideDiffModel diffModel)`
`13`	`14`	`{`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@ public class SideBySideViewModel : IDiffViewViewModel`
`8`	`8`	`public SideBySideDiffModel DiffModel { get; set; }`
`9`	`9`	`public DiffPaneModel AfterPane { get { return DiffModel.NewText; } }`
`10`	`10`	`public DiffPaneModel BeforePane { get { return DiffModel.OldText; } }`
	`11`	`+ public bool HasChanges { get { return DiffModel.NewText.HasDifferences \| DiffModel.OldText.HasDifferences; } }`
`11`	`12`
`12`	`13`	`public SideBySideViewModel(SideBySideDiffModel diffModel)`
`13`	`14`	`{`