Added file processing flag. Fixed #70

Author: madskristensen

CHANGELOG.md

@@ -5,6 +5,7 @@
 - [x] LESS: Added relative URL compiler support (#63)
 - [x] Use the official CoffeeScript compiler (#68)
 - [x] Enable source maps for Iced CoffeeScript
+- [x] Compile up the LESS @import chain (#67)
 - [ ] Generate gulpfile.js from compilerconfig.json (#34)
 - [ ] Preview window (#6)
 - [ ] File globbing pattern support (#49)

src/WebCompiler/Config/ConfigFileProcessor.cs

@@ -11,25 +11,40 @@ namespace WebCompiler
     /// </summary>
     public class ConfigFileProcessor
     {
+        private static List<string> _processing = new List<string>();
+
         /// <summary>
         /// Parses a compiler config file and runs the configured compilers.
         /// </summary>
         /// <param name="configFile">The absolute or relative file path to compilerconfig.json</param>
         /// <returns>A list of compiler results.</returns>
         public IEnumerable<CompilerResult> Process(string configFile)
         {
-            FileInfo info = new FileInfo(configFile);
-            var configs = ConfigHandler.GetConfigs(configFile);
+            if (_processing.Contains(configFile))
+                return Enumerable.Empty<CompilerResult>();
+
+            _processing.Add(configFile);
             List<CompilerResult> list = new List<CompilerResult>();
 
-            if (configs.Any())
-                OnConfigProcessed(configs.First(), 0, configs.Count());
+            try
+            {
+                FileInfo info = new FileInfo(configFile);
+                var configs = ConfigHandler.GetConfigs(configFile);
+
+                if (configs.Any())
+                    OnConfigProcessed(configs.First(), 0, configs.Count());
 
-            foreach (Config config in configs)
+                foreach (Config config in configs)
+                {
+                    var result = ProcessConfig(info.Directory.FullName, config);
+                    list.Add(result);
+                    OnConfigProcessed(config, list.Count, configs.Count());
+                }
+            }
+            finally
             {
-                var result = ProcessConfig(info.Directory.FullName, config);
-                list.Add(result);
-                OnConfigProcessed(config, list.Count, configs.Count());
+                if (_processing.Contains(configFile))
+                    _processing.Remove(configFile);
             }
 
             return list;

src/WebCompilerVsix/CompilerService.cs

@@ -1,10 +1,9 @@
 using System;
 using System.IO;
-using System.Threading;
 using System.Linq;
+using System.Threading;
 using System.Windows.Forms;
 using System.Windows.Threading;
-using EnvDTE;
 using EnvDTE80;
 using WebCompiler;
 
@@ -58,8 +57,6 @@ public static void Process(string configFile)
             {
                 try
                 {
-                    _dte.StatusBar.Animate(true, vsStatusAnimation.vsStatusAnimationBuild);
-
                     var result = Processor.Process(configFile);
                     ErrorListService.ProcessCompilerResults(result, configFile);
 
@@ -85,7 +82,6 @@ public static void Process(string configFile)
                 }
                 finally
                 {
-                    _dte.StatusBar.Animate(false, vsStatusAnimation.vsStatusAnimationBuild);
                     _dte.StatusBar.Progress(false);
                 }
             });
@@ -97,7 +93,6 @@ public static void SourceFileChanged(string configFile, string sourceFile)
             {
                 try
                 {
-                    _dte.StatusBar.Animate(true, vsStatusAnimation.vsStatusAnimationBuild);
                     StatusText($"Compiling \"{Path.GetFileName(sourceFile)}\"...");
 
                     var result = Processor.SourceFileChanged(configFile, sourceFile);
@@ -113,10 +108,6 @@ public static void SourceFileChanged(string configFile, string sourceFile)
                     Logger.Log(ex);
                     ShowError(configFile);
                 }
-                finally
-                {
-                    _dte.StatusBar.Animate(false, vsStatusAnimation.vsStatusAnimationBuild);
-                }
             });
         }
 
@@ -130,7 +121,7 @@ private static void StatusText(string message)
 
         private static void ShowError(string configFile)
         {
-            MessageBox.Show($"There is an error in the {Constants.CONFIG_FILENAME} file. This could be due to a change in the format after this extension was updated.", Constants.VSIX_NAME, MessageBoxButtons.OK, MessageBoxIcon.Information);
+            MessageBox.Show($"There is an error in the {Constants.CONFIG_FILENAME} file. This could be due to a change in the format after this extension was updated.", Constants.VSIX_NAME, MessageBoxButtons.OK, MessageBoxIcon.Information, MessageBoxDefaultButton.Button1, MessageBoxOptions.ServiceNotification);
 
             if (File.Exists(configFile))
                 WebCompilerPackage._dte.ItemOperations.OpenFile(configFile);