Fix export (to clipboard) for inspection results and todo items

Author: MDoerner

Rubberduck.Core/Formatters/InspectionResultFormatter.cs

@@ -1,6 +1,5 @@
 using Rubberduck.Common;
 using Rubberduck.Parsing.Inspections.Abstract;
-using System.IO;
 using Rubberduck.Resources.Inspections;
 
 namespace Rubberduck.Formatters

Rubberduck.Core/UI/Inspections/InspectionResultsViewModel.cs

@@ -5,6 +5,7 @@
 using System.ComponentModel;
 using System.Diagnostics;
 using System.Globalization;
+using System.IO;
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
@@ -13,6 +14,7 @@
 using System.Windows.Input;
 using NLog;
 using Rubberduck.Common;
+using Rubberduck.Formatters;
 using Rubberduck.Inspections.Abstract;
 using Rubberduck.Interaction.Navigation;
 using Rubberduck.JunkDrawer.Extensions;
@@ -666,15 +668,24 @@ private void ExecuteCopyResultsCommand(object parameter)
                 return;
             }
 
-            var resultArray = Results.OfType<IExportable>().Select(result => result.ToArray()).ToArray();
+            var resultArray = Results
+                .OfType<IInspectionResult>()
+                .Select(result => new InspectionResultFormatter(result, DocumentName(result)))
+                .Select(formattedResult => formattedResult.ToArray())
+                .ToArray();
 
             var resource = resultArray.Length == 1
                 ? Resources.RubberduckUI.CodeInspections_NumberOfIssuesFound_Singular
                 : Resources.RubberduckUI.CodeInspections_NumberOfIssuesFound_Plural;
 
             var title = string.Format(resource, DateTime.Now.ToString(CultureInfo.InvariantCulture), resultArray.Count());
 
-            var textResults = title + Environment.NewLine + string.Join(string.Empty, Results.OfType<IExportable>().Select(result => result.ToClipboardString() + Environment.NewLine).ToArray());
+            var resultTexts = Results
+                .OfType<IInspectionResult>()
+                .Select(result => new InspectionResultFormatter(result, DocumentName(result)))
+                .Select(formattedResult => $"{formattedResult.ToClipboardString()}{Environment.NewLine}")
+                .ToArray();
+            var textResults = $"{title}{Environment.NewLine}{string.Join(string.Empty, resultTexts)}";
             var csvResults = ExportFormatter.Csv(resultArray, title, ColumnInformation);
             var htmlResults = ExportFormatter.HtmlClipboardFragment(resultArray, title, ColumnInformation);
             var rtfResults = ExportFormatter.RTF(resultArray, title);
@@ -691,6 +702,20 @@ private void ExecuteCopyResultsCommand(object parameter)
             _clipboard.Flush();
         }
 
+        private string DocumentName(IInspectionResult result)
+        {
+            var module = result.QualifiedSelection.QualifiedName;
+            var projectId = module.ProjectId;
+            var project = _state.ProjectsProvider.Project(projectId);
+
+            if (project == null)
+            {
+                return Path.GetFileName(module.ProjectPath);
+            }
+
+            return project.ProjectDisplayName;
+        }
+
         private bool CanExecuteCopyResultsCommand(object parameter)
         {
             return !IsBusy && _results != null && _results.Any();

Rubberduck.Core/UI/ToDoItems/ToDoExplorerViewModel.cs

@@ -249,15 +249,21 @@ public void ExecuteCopyResultsCommand(object obj)
 
             ColumnInfo[] columnInfos = { new ColumnInfo("Type"), new ColumnInfo("Description"), new ColumnInfo("Project"), new ColumnInfo("Component"), new ColumnInfo("Line", hAlignment.Right), new ColumnInfo("Column", hAlignment.Right) };
 
-            var resultArray = _items.OfType<IExportable>().Select(result => result.ToArray()).ToArray();
+            var resultArray = _items
+                .Select(item => new ToDoItemFormatter(item))
+                .Select(formattedItem => formattedItem.ToArray()).ToArray();
 
             var resource = _items.Count == 1
                 ? ToDoExplorerUI.ToDoExplorer_NumberOfIssuesFound_Singular
                 : ToDoExplorerUI.ToDoExplorer_NumberOfIssuesFound_Plural;
 
             var title = string.Format(resource, DateTime.Now.ToString(CultureInfo.InvariantCulture), _items.Count);
 
-            var textResults = title + Environment.NewLine + string.Join(string.Empty, _items.OfType<IExportable>().Select(result => result.ToClipboardString() + Environment.NewLine).ToArray());
+            var itemTexts = _items
+                .Select(item => new ToDoItemFormatter(item))
+                .Select(formattedItem => $"{formattedItem.ToClipboardString()}{Environment.NewLine}")
+                .ToArray();
+            var textResults = $"{title}{Environment.NewLine}{string.Join(string.Empty, itemTexts)}";
             var csvResults = ExportFormatter.Csv(resultArray, title, columnInfos);
             var htmlResults = ExportFormatter.HtmlClipboardFragment(resultArray, title, columnInfos);
             var rtfResults = ExportFormatter.RTF(resultArray, title);