* Monodoc/man-provider.cs: NEVER return a non-null string from
authorJonathan Pryor <jpryor@novell.com>
Thu, 16 Apr 2009 20:09:40 +0000 (20:09 -0000)
committerJonathan Pryor <jpryor@novell.com>
Thu, 16 Apr 2009 20:09:40 +0000 (20:09 -0000)
  HelpSource.GetText() unless *useful* content is being returned, as
  when non-null is returned RootTree.RenderUrl() assumes that the url
  was handled and won't try any other HelpSources.  In this case, we
  can (and do) have multiple ManHelpSources present, but since
  ManHelpSource.GetText() was returning "<html>url was null</html>",
  RootTree thought it was handled...when it wasn't.  Fix.

svn path=/trunk/mcs/; revision=131934

mcs/tools/monodoc/ChangeLog
mcs/tools/monodoc/Monodoc/man-provider.cs

index d15c6ddfd7b25d4ca0cf459886ee5ae7eecea9c5..2ee427ade03c3897bd94bf7e8982af3eb1411b6f 100644 (file)
@@ -1,3 +1,13 @@
+2009-04-16  Jonathan Pryor  <jpryor@novell.com>
+
+       * Monodoc/man-provider.cs: NEVER return a non-null string from
+         HelpSource.GetText() unless *useful* content is being returned, as
+         when non-null is returned RootTree.RenderUrl() assumes that the url
+         was handled and won't try any other HelpSources.  In this case, we
+         can (and do) have multiple ManHelpSources present, but since
+         ManHelpSource.GetText() was returning "<html>url was null</html>",
+         RootTree thought it was handled...when it wasn't.  Fix.
+
 2009-04-16  Jonathan Pryor  <jpryor@novell.com>
 
        * Monodoc/provider.cs: Overview: MonoDevelop loads monodoc.dll into
index a970e583da52c571612e99f74df000f54e10a80b..9f82d2c9d7a6aecfe173e702fb46d9bebe11623f 100755 (executable)
@@ -110,7 +110,7 @@ public class ManHelpSource : HelpSource {
                if (url == null || url.Length == 0)
                {
                        Message (TraceLevel.Warning, "Warning, NULL url!");
-                       return "<html>url was null</html>";
+                       return null;
                }
 
                Stream stream = GetHelpStream (url);
@@ -120,7 +120,7 @@ public class ManHelpSource : HelpSource {
        public static string GetTextFromStream (Stream stream)
        {
                if (stream == null)
-                       return "<html>url was null</html>";
+                       return null;
                StreamReader file = new StreamReader(stream);
 
                string line;