From: Date: February 28 2008 10:31pm
Subject: Connector/NET commit: r1199 - in branches/5.2: . Driver/Source
List-Archive: http://lists.mysql.com/commits/43183
X-Bug: 34940
Message-Id: <200802282131.m1SLVgXq021679@bk-internal.mysql.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Modified:
   branches/5.2/CHANGES
   branches/5.2/Driver/Source/Statement.cs
Log:
- Fixed problem where new parameter code prevented stored procedures from being
  altered in Visual Studio (bug #34940)

We needed to add a check that any parameters that come through with a ` or ' after the @ should not be considered parameters.

Modified: branches/5.2/CHANGES
===================================================================
--- branches/5.2/CHANGES	2008-02-28 20:18:31 UTC (rev 1198)
+++ branches/5.2/CHANGES	2008-02-28 21:31:42 UTC (rev 1199)
@@ -1,6 +1,8 @@
 Version 5.2.2 - 
 - Fixed profile provider that would throw an exception if you were updating
   a profile that already existed.
+- Fixed problem where new parameter code prevented stored procedures from being
+  altered in Visual Studio (bug #34940)
   
 Version 5.2.1 - 2/27/2008
 - Tons of fixes in providers.  The actually work now.  :)

Modified: branches/5.2/Driver/Source/Statement.cs
===================================================================
--- branches/5.2/Driver/Source/Statement.cs	2008-02-28 20:18:31 UTC (rev 1198)
+++ branches/5.2/Driver/Source/Statement.cs	2008-02-28 21:31:42 UTC (rev 1199)
@@ -208,7 +208,9 @@
 
         protected virtual bool ShouldIgnoreMissingParameter(string parameterName)
         {
-            return Connection.Settings.AllowUserVariables;
+            return Connection.Settings.AllowUserVariables ||
+                (parameterName.Length > 1 &&
+                (parameterName[1] == '`' || parameterName[1] == '\''));
         }
 
         /// <summary>