Fixed quotation of numbers inside prepared statements

lib/Client.js

@@ -652,6 +652,8 @@ Client.prototype._format_value = function(v) {
     for (var i = 0, len = v.length; i < len; ++i)
       r.push(this._format_value(v[i]));
     return r.join(',');
+  } else if ((v ^ 0) === v) {
+    return v;
   } else if (v !== null && v !== undefined)
     return "'" + Client.escape(v + '') + "'";
 

test/test.js

@@ -35,22 +35,40 @@ var tests = [
       var client = new Client();
       var fn;
       fn = client.prepare("SELECT * FROM foo WHERE id = '123'");
-      assert.strictEqual(fn({ id: 456 }),
+      assert.strictEqual(fn({ id: '456' }),
                          "SELECT * FROM foo WHERE id = '123'");
       fn = client.prepare("SELECT * FROM foo WHERE id = :id");
-      assert.strictEqual(fn({ id: 123 }),
+      assert.strictEqual(fn({ id: '123' }),
                          "SELECT * FROM foo WHERE id = '123'");
-      assert.strictEqual(fn({ id: 456 }),
+      assert.strictEqual(fn({ id: '456' }),
                          "SELECT * FROM foo WHERE id = '456'");
       fn = client.prepare("SELECT * FROM foo WHERE id = ?");
-      assert.strictEqual(fn([123]),
+      assert.strictEqual(fn(['123']),
                          "SELECT * FROM foo WHERE id = '123'");
-      assert.strictEqual(fn([456]),
+      assert.strictEqual(fn(['456']),
                          "SELECT * FROM foo WHERE id = '456'");
       fn = client.prepare("SELECT * FROM foo WHERE id = :0 AND name = :1");
       assert.strictEqual(fn(['123', 'baz']),
                          "SELECT * FROM foo WHERE id = '123' AND name = 'baz'");
 
+      // Integer parsing cases
+      fn = client.prepare("SELECT * FROM foo WHERE id = 123");
+      assert.strictEqual(fn({ id: 456 }),
+                         "SELECT * FROM foo WHERE id = 123");
+      fn = client.prepare("SELECT * FROM foo WHERE id = :id");
+      assert.strictEqual(fn({ id: 123 }),
+                         "SELECT * FROM foo WHERE id = 123");
+      assert.strictEqual(fn({ id: 456 }),
+                         "SELECT * FROM foo WHERE id = 456");
+      fn = client.prepare("SELECT * FROM foo WHERE id = ?");
+      assert.strictEqual(fn([123]),
+                         "SELECT * FROM foo WHERE id = 123");
+      assert.strictEqual(fn([456]),
+                         "SELECT * FROM foo WHERE id = 456");
+      fn = client.prepare("SELECT * FROM foo WHERE id = :0 AND name = :1");
+      assert.strictEqual(fn([123, 'baz']),
+                         "SELECT * FROM foo WHERE id = 123 AND name = 'baz'");
+
       // Edge cases
       fn = client.prepare("SELECT * FROM foo WHERE id = :id"
                           + " AND first = ? AND last = ? AND middle = '?'");
@@ -76,6 +94,32 @@ var tests = [
                          "SELECT * FROM foo WHERE id = '123'"
                          + " AND first = 'foo' AND last = '?'"
                          + " AND middle = '?'");
+
+      // Edge cases with integers
+      fn = client.prepare("SELECT * FROM foo WHERE id = :id"
+                          + " AND first = ? AND last = ? AND middle = '?'");
+      assert.strictEqual(fn(appendProps(['foo', 'bar', 'baz'], { id: 123 })),
+                         "SELECT * FROM foo WHERE id = 123"
+                         + " AND first = 'foo' AND last = 'bar'"
+                         + " AND middle = '?'");
+      fn = client.prepare("SELECT * FROM foo WHERE id = :id"
+                          + " AND first = ? AND last = '?' AND middle = ?");
+      assert.strictEqual(fn(appendProps(['foo', 'bar', 'baz'], { id: 123 })),
+                         "SELECT * FROM foo WHERE id = 123"
+                         + " AND first = 'foo' AND last = '?'"
+                         + " AND middle = 'bar'");
+      fn = client.prepare("SELECT * FROM foo WHERE id = :id"
+                          + " AND first = '?' AND last = '?' AND middle = ?");
+      assert.strictEqual(fn(appendProps(['foo', 'bar', 'baz'], { id: 123 })),
+                         "SELECT * FROM foo WHERE id = 123"
+                         + " AND first = '?' AND last = '?'"
+                         + " AND middle = 'foo'");
+      fn = client.prepare("SELECT * FROM foo WHERE id = :id"
+                          + " AND first = ? AND last = '?' AND middle = '?'");
+      assert.strictEqual(fn(appendProps(['foo', 'bar', 'baz'], { id: 123 })),
+                         "SELECT * FROM foo WHERE id = 123"
+                         + " AND first = 'foo' AND last = '?'"
+                         + " AND middle = '?'");
       next();
     }
   },
@@ -1035,7 +1079,7 @@ function makeFooTable(client, colDefs, tableOpts) {
       opts.push(format('%s=%s', key, tableOpts[key]));
     });
     tableOpts = opts.join(' ');
-  } else 
+  } else
     tableOpts = '';
 
   client.query('CREATE DATABASE IF NOT EXISTS `foo`', NOOP);
@@ -1045,7 +1089,7 @@ function makeFooTable(client, colDefs, tableOpts) {
                  cols.join(', '),
                  tableOpts);
   client.query(query, NOOP);
-  
+
 }
 
 function next() {