[Mono-bugs] [Bug 78420][Nor] Changed - DataGrid throws 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 miguel at ximian.com.

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

--- shadow/78420	2006-05-16 12:39:52.000000000 -0400
+++ shadow/78420.tmp.31208	2006-05-16 19:09:24.000000000 -0400
@@ -1,14 +1,14 @@
 Bug#: 78420
 Product: Mono: Class Libraries
 Version: 1.1
-OS: 
+OS: unknown
 OS Details: Centos 4.2
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: Windows.Forms
 AssignedTo: peter at novonyx.com                            
 ReportedBy: les42_1 at hotmail.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
@@ -61,6 +61,38 @@
 Using the latest from svn. The code runs on windows.
 
 ------- Additional Comments From les42_1 at hotmail.com  2006-05-16 12:39 -------
 Created an attachment (id=17039)
 source file
 
+
+------- Additional Comments From miguel at ximian.com  2006-05-16 19:09 -------
+Jordi, in revision 53626 you added the following code, which is
+causing the problem:
+
+                public void SetDataBinding (object dataSource, string
+dataMember)
+                {
+-                       bool source = SetDataSource (dataSource);
+-                       bool member = SetDataMember (dataMember);
+-
+-                       if (source == false  && member == false) {
+-                               return;
+-                       }
+-
++                       dataMember = null;
++                       SetDataSource (dataSource);
++                       SetDataMember (dataMember);
+                        SetNewDataSource ();
+                }
+
+
+The dataMember initial value should be String.Empty, instead of the null.
+
+But am wondering about this routine, as its supposed to set the new
+dataset, wasnt the original code correct?
+
+Because this seems to completely ignore the value passed for
+dataMember (its set to null right away).
+
+This was supposed to fix this bug:
+http://bugzilla.ximian.com/show_bug.cgi?id=76770