Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Support insert for keyword named columns, such as 'User' #74

Closed
wants to merge 1 commit into from

2 participants

Chris Constantin Rob Conery
Chris Constantin

Wrapped the column names in brackets to allow for columns sharing a name with a T-SQL keyword.

Cheers,
Chris

Rob Conery
Owner

That shouldn't be a column name.

Rob Conery robconery closed this August 08, 2011
Chris Constantin

That's a matter of taste, and sometimes a matter of necessity, when working with legacy databases. At the same time it is an easy change for anyone to figure out/make.

Cheers,
Chris

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Aug 05, 2011
proper Insert support for keyword named columns 3edda5f
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  Massive.cs
2  Massive.cs
@@ -315,7 +315,7 @@ public class DynamicModel : DynamicObject {
315 315
             result = CreateCommand(stub, null);
316 316
             int counter = 0;
317 317
             foreach (var item in settings) {
318  
-                sbKeys.AppendFormat("{0},", item.Key);
  318
+                sbKeys.AppendFormat("[{0}],", item.Key);
319 319
                 sbVals.AppendFormat("@{0},", counter.ToString());
320 320
                 result.AddParam(item.Value);
321 321
                 counter++;
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.