Merge pull request #697 from mono/develop

Fix bug [966887] : FrameworksIndex files not cleaned up (#696)

Author: huangmin-ms

Diff for: mdoc/Consts.cs

@@ -3,7 +3,7 @@ namespace Mono.Documentation
 {
     public static class Consts
     {
-        public static string MonoVersion = "5.9.3.1";
+        public static string MonoVersion = "5.9.3.2";
         public const string DocId = "DocId";
         public const string CppCli = "C++ CLI";
         public const string CppCx = "C++ CX";

Diff for: mdoc/Mono.Documentation/Updater/Frameworks/FrameworkIndex.cs

@@ -86,12 +86,12 @@ public void WriteToDisk (string path)
 			if (string.IsNullOrWhiteSpace (this.path))
 				return;
 
-      string outputPath = Path.Combine (path, Consts.FrameworksIndex);
+            string outputPath = Path.Combine (path, Consts.FrameworksIndex);
 
 			if (!Directory.Exists (outputPath))
 				Directory.CreateDirectory (outputPath);
 
-			foreach (var fx in this.frameworks)
+            foreach (var fx in this.frameworks)
 			{
 				XElement frameworkElement = new XElement("Framework", new XAttribute("Name", fx.Name));
 				XDocument doc = new XDocument(
@@ -139,6 +139,20 @@ public void WriteToDisk (string path)
 					doc.WriteTo (writer);
 				}
 			}
-		}
-	}
+            CleanupFrameworksIndex(outputPath);
+        }
+
+        private void CleanupFrameworksIndex(string outputPath)
+        {
+            var files = Directory.GetFiles(outputPath); 
+            var frameworkNames = frameworks.ToDictionary(item => item.Name, item => item);
+            foreach (var file in files)
+            {
+                if (!frameworkNames.ContainsKey(Path.GetFileNameWithoutExtension(file)))
+                {
+                    File.Delete(file);
+                }
+            }
+        }
+    }
 }

Diff for: mdoc/mdoc.nuspec

@@ -2,7 +2,7 @@
 <package >
   <metadata>
     <id>mdoc</id>
-    <version>5.9.3.1</version>
+    <version>5.9.3.2</version>
     <title>mdoc</title>
     <authors>Microsoft</authors>
     <owners>Microsoft</owners>