Support # and $ in PL/SQL named parameters

Fixes #822
sql-formatter-org · Jan 25, 2025 · 4eb0193 · 4eb0193
1 parent ac92853
commit 4eb0193
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/languages/plsql/plsql.formatter.ts b/src/languages/plsql/plsql.formatter.ts
@@ -107,7 +107,6 @@ export const plsql: DialectOptions = {
     identChars: { rest: '$#' },
     variableTypes: [{ regex: '&{1,2}[A-Za-z][A-Za-z0-9_$#]*' }],
     paramTypes: { numbered: [':'], named: [':'] },
-    paramChars: {}, // Empty object used on purpose to not allow $ and # chars as specified in identChars
     operators: [
       '**',
       ':=',

diff --git a/test/plsql.test.ts b/test/plsql.test.ts
@@ -84,11 +84,13 @@ describe('PlSqlFormatter', () => {
     `);
   });
 
-  // Parameters don't allow the same characters as identifiers
-  it('does not support #, $ in named parameters', () => {
-    expect(() => format('SELECT :col$foo')).toThrowError(`Parse error: Unexpected "$foo"`);
-
-    expect(() => format('SELECT :col#foo')).toThrowError(`Parse error: Unexpected "#foo"`);
+  // Issue #822
+  it('supports #, $ in named parameters', () => {
+    expect(format('SELECT :col$foo, :col#foo')).toBe(dedent`
+      SELECT
+        :col$foo,
+        :col#foo
+    `);
   });
 
   it('supports &name substitution variables', () => {