[Mono-bugs] [Bug 77343][Wis] New - Help.ShowHelpIndex generates exception

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by don at edvalson.net.

http://bugzilla.ximian.com/show_bug.cgi?id=77343

--- shadow/77343	2006-01-24 09:46:05.000000000 -0500
+++ shadow/77343.tmp.16731	2006-01-24 09:46:05.000000000 -0500
@@ -0,0 +1,56 @@
+Bug#: 77343
+Product: Mono: Class Libraries
+Version: unspecified
+OS: 
+OS Details: Windows XP
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Wishlist
+Component: Windows.Forms
+AssignedTo: peter at novonyx.com                            
+ReportedBy: don at edvalson.net               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: Help.ShowHelpIndex generates exception
+
+Please fill in this template when reporting a bug, unless you know what 
+you are doing.
+Description of Problem:Help.ShowHelpIndex generates an exception instead 
+of just displaying not implemented yet.
+
+
+Steps to reproduce the problem:
+1. call Help.ShowHelpIndex
+2. 
+3. 
+
+Actual Results:exception
+
+
+Expected Results:Not Implemented MessageBox
+
+
+How often does this happen? always.
+
+
+Additional Information:
+This is because a null is being handed to ShowHelpIndex to ShowHelp and 
+ShowHelp is not prepared to deal with a null.
+
+I modified ShowHelp in Help.cs as shown and it fixed the problem.
+
+[MonoTODO("Create glue code to tie into libCHM")]
+public static void ShowHelp(Control parent, string url, HelpNavigator 
+command, object param) {
+    if (param != null)
+	MessageBox.Show(parent, (string)Locale.GetText("Help (" + command 
++ "(" + param.ToString() + ")) not yet implemented"), (string)
+Locale.GetText("Popup Help"), MessageBoxButtons.OK, MessageBoxIcon.Stop);
+    else
+	MessageBox.Show(parent, (string)Locale.GetText("Help (" + command 
++ ") not yet implemented"), (string)Locale.GetText("Popup Help"), 
+MessageBoxButtons.OK, MessageBoxIcon.Stop);
+}