Connector/NET commit: r1152 - in branches/5.1: . Driver/Source

[rburnett]

Modified:
   branches/5.1/CHANGES
   branches/5.1/Driver/Source/Connection.cs
Log:
- Fixed problem where connection state reported through the state change handler was not
  showing Open (bug #34082).
This was a result of the code we added to internally set the state but not send out notifications.  There was a line of code that short circuited the SetState method if the  old and new state was the same.  The problem is that the Open method sets the state to open with no broadcast and then sets the state to open with broadcast.  The short circuits keeps the second call from notifying clients of the change.

Modified: branches/5.1/CHANGES
===================================================================
--- branches/5.1/CHANGES	2008-01-24 16:31:45 UTC (rev 1151)
+++ branches/5.1/CHANGES	2008-01-29 15:41:41 UTC (rev 1152)
@@ -3,6 +3,8 @@
     a second parameter with the same name as the first (bug #33347)
   - Fixed long standing problem with compression over a network.  It's now fast again. (bug #27865)    
   - Fixed nant compilation problem on mono (bug #33508)
+  - Fixed problem where connection state reported through the state change handler was not
+    showing Open (bug #34082)   
    
 Version 5.1.4 - 11/12/2007
   - Fixed issue where column name metadata was not using the charset given on the connection string

Modified: branches/5.1/Driver/Source/Connection.cs
===================================================================
--- branches/5.1/Driver/Source/Connection.cs	2008-01-24 16:31:45 UTC (rev 1151)
+++ branches/5.1/Driver/Source/Connection.cs	2008-01-29 15:41:41 UTC (rev 1152)
@@ -404,7 +404,7 @@
 
         internal void SetState(ConnectionState newConnectionState, bool broadcast)
         {
-            if (newConnectionState == connectionState)
+            if (newConnectionState == connectionState && !broadcast)
                 return;
             ConnectionState oldConnectionState = connectionState;
             connectionState = newConnectionState;