Created copy of array

Fixed #179

Author: madskristensen

src/WebCompiler/Config/ConfigFileProcessor.cs

@@ -116,10 +116,12 @@ private IEnumerable<CompilerResult> SourceFileChanged(string configFile,
                 var dependencies = DependencyService.GetDependencies(projectPath, sourceFile);
                 if(dependencies != null)
                 {
-                    if(dependencies.ContainsKey(sourceFile.ToLowerInvariant()))
+                    string key = sourceFile.ToLowerInvariant();
+
+                    if (dependencies.ContainsKey(key))
                     {
                         //compile all files that have references to the compiled file
-                        foreach (var file in dependencies[sourceFile.ToLowerInvariant()].DependentFiles)
+                        foreach (var file in dependencies[key].DependentFiles.ToArray())
                         {
                             if (!compiledFiles.Contains(file.ToLowerInvariant()))
                                 SourceFileChanged(configFile, file, projectPath, compiledFiles);

src/WebCompiler/Dependencies/Dependencies.cs

@@ -11,40 +11,14 @@ namespace WebCompiler
     /// </summary>
     public class Dependencies
     {
-        private HashSet<string> _dependentOn;
-        private HashSet<string> _dependentFiles;
-
         /// <summary>
         /// Contains all files the current file is dependent ont
         /// </summary>
-        public HashSet<string> DependentOn
-        {
-            get
-            {
-                return _dependentOn;
-            }
-            set
-            {
-                _dependentOn = value;
-            }
-        }
+        public HashSet<string> DependentOn { get; set; } = new HashSet<string>();
 
         /// <summary>
         /// Contains all files that are dependent on this file
         /// </summary>
-        public HashSet<string> DependentFiles
-        {
-            get { return _dependentFiles; }
-            set { _dependentFiles = value; }
-        }
-
-        /// <summary>
-        /// Creates a new dependecies object
-        /// </summary>
-        public Dependencies()
-        {
-            _dependentOn = new HashSet<string>();
-            _dependentFiles = new HashSet<string>();
-        }
+        public HashSet<string> DependentFiles { get; set; } = new HashSet<string>();
     }
 }

src/WebCompiler/Dependencies/DependencyResolverBase.cs

@@ -30,6 +30,14 @@ public abstract string[] SearchPatterns
             get;
         }
 
+        /// <summary>
+        /// The file extension of files of this type
+        /// </summary>
+        public abstract string FileExtension
+        {
+            get;
+        }
+
         /// <summary>
         /// Gets the dependency tree
         /// </summary>

src/WebCompiler/Dependencies/LessDependencyResolver.cs

@@ -6,5 +6,14 @@ public override string[] SearchPatterns
         {
             get { return new string[] { "*.less" }; }
         }
+
+        public override string FileExtension
+        {
+            get
+            {
+                return ".less";
+            }
+        }
+
     }
 }

src/WebCompiler/Dependencies/SassDependencyResolver.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.IO;
 
 namespace WebCompiler
@@ -10,6 +11,15 @@ public override string[] SearchPatterns
             get { return new[] { "*.scss", "*.sass" }; }
         }
 
+        public override string FileExtension
+        {
+            get
+            {
+                return ".scss";
+            }
+        }
+
+
         /// <summary>
         /// Updates the dependencies of a single file
         /// </summary>
@@ -28,7 +38,7 @@ public override void UpdateFileDependencies(string path)
                 //remove the dependentfile registration of this file for all other files
                 foreach (var dependenciesPath in Dependencies.Keys)
                 {
-                    var lowerDependenciesPath = path.ToLowerInvariant();
+                    var lowerDependenciesPath = dependenciesPath.ToLowerInvariant();
                     if (Dependencies[lowerDependenciesPath].DependentFiles.Contains(path))
                     {
                         Dependencies[lowerDependenciesPath].DependentFiles.Remove(path);
@@ -43,6 +53,12 @@ public override void UpdateFileDependencies(string path)
                 foreach (System.Text.RegularExpressions.Match match in matches)
                 {
                     FileInfo importedfile = new FileInfo(Path.Combine(info.DirectoryName, match.Groups[3].Value));
+                    //if the file doesn't end with the correct extension, an import statement without extension is probably used, to re-add the extension (#175)
+                    if (string.Compare(importedfile.Extension, FileExtension, StringComparison.OrdinalIgnoreCase) != 0)
+                    {
+                        importedfile = new FileInfo(importedfile.FullName + this.FileExtension);
+                    }
+
                     var dependencyFilePath = importedfile.FullName.ToLowerInvariant();
 
                     if (!Dependencies[path].DependentOn.Contains(dependencyFilePath))

src/WebCompilerVsix/Helpers/ProjectHelpers.cs

@@ -58,7 +58,7 @@ public static IEnumerable<string> GetSelectedItemPaths()
 
         public static string GetRootFolder(this Project project)
         {
-            if (string.IsNullOrEmpty(project.FullName))
+            if (project == null || string.IsNullOrEmpty(project.FullName))
                 return null;
 
             string fullPath;
@@ -222,6 +222,16 @@ public static Project GetActiveProject()
 
                 if (activeSolutionProjects != null && activeSolutionProjects.Length > 0)
                     return activeSolutionProjects.GetValue(0) as Project;
+
+                var doc = _dte.ActiveDocument;
+
+                if (doc != null && !string.IsNullOrEmpty(doc.FullName))
+                {
+                    var item = _dte.Solution?.FindProjectItem(doc.FullName);
+
+                    if (item != null)
+                        return item.ContainingProject;
+                }
             }
             catch (Exception ex)
             {