Merge branch 'master' of github.com:JavaScriptor/js-sql-parser into unicode-extended

Author: luizzappa

.gitignore

@@ -4,3 +4,4 @@ package-lock.json
 *.log
 *.swp
 dist/parser/sqlParser.js
+.vscode/

CHANGELOG

@@ -10,3 +10,4 @@
 1.3.0 fix tableFactor alias bug. AST changed in tableFactor. #34
 1.4.0 fix bug `using ' & " for column alias?` #40 #44
 1.4.1 hogfix "support quoted alias: multiple alias and orderby support"
+1.5.0 support feature placeholder.

Makefile

@@ -1,7 +1,10 @@
 publish: test
 	@npm publish
 
-test: 
+test:
 	@npm test
 
+test-with-log:
+	@DEBUG=js-sql-parser npm test
+
 .PHONY: publish test

README.md

@@ -10,6 +10,7 @@ sql grammar follows https://dev.mysql.com/doc/refman/5.7/en/select.html
 
 ## news
 
+- Support feature `PlaceHolder like ${param}` since v1.5.0 [#43](https://github.com/JavaScriptor/js-sql-parser/pull/43)
 - Fix bug `using ' & " for column alias?` since v1.4.1 [#40](https://github.com/JavaScriptor/js-sql-parser/issues/40), [#44](https://github.com/JavaScriptor/js-sql-parser/issues/44)
 - Fix bug tableFactor alias since v1.3.0 [#34](https://github.com/JavaScriptor/js-sql-parser/issues/34)
 - Add support for "`" quoted alias since v1.2.2. [#33](https://github.com/JavaScriptor/js-sql-parser/issues/33)
@@ -35,6 +36,18 @@ console.log(parser.stringify(ast));
 // SELECT foo FROM bar
 ```
 
+```js
+// placeholder test
+const parser = require('js-sql-parser');
+const ast = parser.parse('select ${a} as a');
+
+ast['value']['selectItems']['value'][0]['value'] = "'value'";
+console.log(parser.stringify(ast));
+// SELECT 'value' AS a
+```
+
+Note: PlaceHolder is an `literal` value but not an `identifier`. Table_name / column_name / function_name are `identifier` thus should NOT be placed with placeholder.
+
 ## script tag
 
 ```js
@@ -60,10 +73,6 @@ var sql = sqlParser.stringify(ast);
 - intervalexpr: Date INTERVAL keyword.   // to support
 - into outfile: INTO OUTFILE keyword.    // to support
 
-## TODO
-
-- ${value} like value place holder support.
-
 ## Build
 
 - Run `npm run build` to build the distributable.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "js-sql-parser",
-  "version": "1.4.1",
+  "version": "1.5.0",
   "description": "",
   "main": "./dist/parser/sqlParser.js",
   "scripts": {

src/stringify.js

@@ -552,18 +552,6 @@ Sql.prototype.travelSelectParenthesized = function(ast) {
   this.appendKeyword(')');
 };
 Sql.prototype.travelPlaceHolder = function (ast) {
-  if (ast.left) {
-    this.travel(ast.left);
-  }
-
-  if (ast.operator) {
-    this.append(ast.operator);
-  }
-  
-  if (ast.right) {
-    this.append(ast.right);
-  }
-
   if (ast.value) {
     this.travel(ast.value);
   }

test/main.test.js

@@ -2,6 +2,7 @@
 
 const debug = require('debug')('js-sql-parser');
 const parser = require('../');
+const assert = require('assert');
 
 const testParser = function (sql) {
   let firstAst = parser.parse(sql);
@@ -416,6 +417,19 @@ describe('select grammar support', function () {
     "select sum(quota_value) value, busi_col2 as sh, ${a} as a, YEAR(now()) from der_quota_summary where table_ename = 'gshmyyszje_derivedidx' and cd = (select id from t1 where a = ${t1})"
     )
   });
+  it('place holder support2', function() {
+    testParser(
+    "select sum(quota_value) b, busi_col2 as sh, '${a}' as a, YEAR(now()) from der_quota_summary where table_ename = 'gshmyyszje_derivedidx' and cd = (select id from t1 where a = '${t1}')"
+    )
+  });
+  it('place holder support3', function() {
+    let firstAst = parser.parse('select ${a} as a');
+    firstAst['value']['selectItems']['value'][0]['value'] = "'value'";
+    let firstSql = parser.stringify(firstAst);
+    debug(JSON.stringify(firstAst, null, 2));
+    assert.equal(firstSql.trim().toUpperCase(), "select 'value' as a".toUpperCase());
+    testParser(firstSql);
+  });
 
   it('support quoted alias: multiple alias and orderby support', function () {
     testParser('select a as `A A`, b as `B B` from z');