Merge pull request #54 from deadlydog/CopyPathsInSameOrderAsGrid

feat: Copy Paths to the clipboard in the same order they appear in th…

Changelog.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v1.9.0 - December 26, 2020
+
+Features:
+
+- Copy to clipboard functions copy paths in the same order they appear in the UI grid.
+
 ## v1.8.9 - December 16, 2020
 
 Fixes:

build/azure-pipelines.yml

@@ -15,7 +15,7 @@ variables:
   buildConfiguration: 'Release'
   appBinariesDirectory: 'src\PathLengthCheckerGUI\bin\$(buildConfiguration)'
 
-  version.MajorMinor: '1.8' # Manually adjust the version number as needed for semantic versioning. Revision is auto-incremented.
+  version.MajorMinor: '1.9' # Manually adjust the version number as needed for semantic versioning. Revision is auto-incremented.
   version.Revision: $[counter(variables['version.MajorMinor'], 0)]
   versionNumber: '$(version.MajorMinor).$(version.Revision)'
 

src/PathLengthCheckerGUI/MainWindow.xaml.cs

@@ -1,5 +1,6 @@
 using PathLengthChecker;
 using System;
+using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.ComponentModel;
 using System.Diagnostics;
@@ -66,6 +67,11 @@ public ObservableCollection<PathInfo> Paths
 		}
 		private ObservableCollection<PathInfo> _paths = new ObservableCollection<PathInfo>();
 
+		/// <summary>
+		/// This returns the same items as the Paths property, but sorted however the UI happens to be sorted.
+		/// </summary>
+		private IEnumerable<PathInfo> PathsFromUiDataGrid => dgPaths.Items.Cast<PathInfo>();
+
 		public PathInfo SelectedPath
 		{
 			get { return (PathInfo)GetValue(SelectedPathProperty); }
@@ -228,27 +234,27 @@ private void DisplayResultsMetadata()
 
 		private void splitbtnCopyToClipboard_Click(object sender, RoutedEventArgs e)
 		{
-			var text = GetPathsAsString(includeLength: true);
+			var text = GetPathsFromUiDataGridAsString(includeLength: true);
 			SetClipboardText(text);
 		}
 
 		private void btnCopyToClipboardWithoutLengths_Click(object sender, RoutedEventArgs e)
 		{
-			var text = GetPathsAsString(includeLength: false);
+			var text = GetPathsFromUiDataGridAsString(includeLength: false);
 			SetClipboardText(text);
 			CloseCopyToClipboardSplitButtonDropDown();
 		}
 
 		private void btnCopyToClipboardAsCsv_Click(object sender, RoutedEventArgs e)
 		{
-			var text = GetPathsAsCsvString(includeLength: true);
+			var text = GetPathsFromUiDataGridAsCsvString(includeLength: true);
 			SetClipboardText(text);
 			CloseCopyToClipboardSplitButtonDropDown();
 		}
 
 		private void btnCopyToClipboardWithoutLengthsAsCsv_Click(object sender, RoutedEventArgs e)
 		{
-			var text = GetPathsAsCsvString(includeLength: false);
+			var text = GetPathsFromUiDataGridAsCsvString(includeLength: false);
 			SetClipboardText(text);
 			CloseCopyToClipboardSplitButtonDropDown();
 		}
@@ -258,18 +264,18 @@ private void CloseCopyToClipboardSplitButtonDropDown()
 			splitbtnCopyToClipboard.IsOpen = false;
 		}
 
-		private string GetPathsAsString(bool includeLength)
+		private string GetPathsFromUiDataGridAsString(bool includeLength)
 		{
 			var text = new StringBuilder();
-			foreach (var path in Paths)
+			foreach (var path in PathsFromUiDataGrid)
 			{
 				var item = includeLength ? $"{path.Length}: {path.Path}" : path.Path;
 				text.AppendLine(item);
 			}
 			return text.ToString().Trim();
 		}
 
-		private string GetPathsAsCsvString(bool includeLength)
+		private string GetPathsFromUiDataGridAsCsvString(bool includeLength)
 		{
 			var text = new StringBuilder();
 
@@ -278,7 +284,7 @@ private string GetPathsAsCsvString(bool includeLength)
 					"\"Path\"";
 			text.AppendLine(header);
 
-			foreach (var path in Paths)
+			foreach (var path in PathsFromUiDataGrid)
 			{
 				var item = includeLength ?
 					$"{path.Length},\"{path.Path}\"" :