Fixed issue with error message when opening VSTA. Name property not implemented, so don't call it.

Author: sqlgreen

SSIS/AutoSortProjectFilesPlugin.cs

@@ -66,6 +66,7 @@ private void SolutionOpened()
             try
             {
                 var solExplorer = ApplicationObject.ToolWindows.SolutionExplorer;
+
                 if (solExplorer.UIHierarchyItems.Count != 1)
                 {
                     return;
@@ -88,13 +89,17 @@ private void SolutionOpened()
             }
             catch (Exception ex)
             {
-                MessageBox.Show(ex.ToString());
+                ShowErrorMessageBox(ex.ToString(), "AutoSortProjectFilesPlugin.SolutionOpened");
             }
         }
 
         private void ProcessHierarchyItem(UIHierarchyItem hierarchyItem)
         {
-            System.Diagnostics.Debug.WriteLine(hierarchyItem.Name);
+            if (hierarchyItem == null)
+                return;
+
+            // This may be helpfull for testing, but VSTA (e.g. Script Task) will still invoke this and the Name property is not implemented, so don't leave in for release
+            //System.Diagnostics.Debug.WriteLine(hierarchyItem.Name);
 
             // Check if it is a Project, skip if not (Solution folders are a type of project too)
             var project = hierarchyItem.Object as Project;

core/BIDSHelperPluginBase.cs

@@ -593,6 +593,22 @@ protected Microsoft.DataWarehouse.VsIntegration.Shell.Project.Extensibility.Proj
             }
             return proj;
         }
+
+        protected void ShowErrorMessageBox(string message)
+        {
+            ShowErrorMessageBox(message, null);
+        }
+
+        protected void ShowErrorMessageBox(string message, string captionSuffix)
+        {
+            string caption = DefaultMessageBoxCaption;
+            if (string.IsNullOrEmpty(captionSuffix))
+            {
+                caption = string.Format("{0} - {1}", DefaultMessageBoxCaption, captionSuffix);
+            }
+
+            MessageBox.Show(message, caption, MessageBoxButtons.OK, MessageBoxIcon.Error);
+        }
     }
 
 }