feat(grammar): Improve support for TO_NUMBER syntax (#192)

zpa-checks/src/integrationTest/resources/expected/alexandria-plsql-utils/VariableInitializationWithFunctionCallCheck.json

@@ -13,12 +13,6 @@
   "extras/hash_util_pkg.pkb" : [
     19,
     20,
-    23,
-    24,
-    25,
-    28,
-    29,
-    30,
     192,
     193,
     194,

zpa-checks/src/integrationTest/resources/expected/antlr-grammars-v4/ParsingErrorCheck.json

@@ -413,9 +413,6 @@
   "examples/single_statement.sql" : [
     1
   ],
-  "examples/to_number.sql" : [
-    6
-  ],
   "examples/to_timestamp.sql" : [
     6
   ],

zpa-checks/src/integrationTest/resources/expected/oracle-database_23/ParsingErrorCheck.json

@@ -2858,9 +2858,6 @@
   "sqlrf/Syntax-for-Schema-Objects-and-Parts-in-SQL-Statements-4.sql" : [
     3
   ],
-  "sqlrf/TO_NUMBER-2.sql" : [
-    2
-  ],
   "sqlrf/TO_TIMESTAMP-1.sql" : [
     3
   ],

zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/PlSqlKeyword.kt

@@ -520,6 +520,7 @@ enum class PlSqlKeyword(override val value: String, val isReserved: Boolean = fa
     TO_BOOLEAN("to_boolean"),
     TO_DATE("to_date"),
     TO_DSINTERVAL("to_dsinterval"),
+    TO_NUMBER("to_number"),
     TRAILING("trailing"),
     TRANSACTION("transaction"),
     TREAT("treat"),

zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/SingleRowSqlFunctionsGrammar.kt

@@ -133,6 +133,7 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
     TO_BOOLEAN_EXPRESSION,
     TO_DATE_EXPRESSION,
     TO_DSINTERVAL_EXPRESSION,
+    TO_NUMBER_EXPRESSION,
     TRIM_EXPRESSION,
     TABLE_EXPRESSION,
     THE_EXPRESSION,
@@ -189,6 +190,7 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
                     TO_BOOLEAN_EXPRESSION,
                     TO_DATE_EXPRESSION,
                     TO_DSINTERVAL_EXPRESSION,
+                    TO_NUMBER_EXPRESSION,
                     TRIM_EXPRESSION,
                     TABLE_EXPRESSION,
                     THE_EXPRESSION,
@@ -266,6 +268,14 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
                 RPARENTHESIS
             )
 
+            b.rule(TO_NUMBER_EXPRESSION).define(
+                TO_NUMBER, LPARENTHESIS,
+                EXPRESSION,
+                b.optional(DEFAULT_ON_ERROR_CLAUSE),
+                b.optional(COMMA, EXPRESSION, b.optional(COMMA, EXPRESSION)),
+                RPARENTHESIS
+            )
+
             b.rule(TABLE_EXPRESSION).define(
                 TABLE, LPARENTHESIS,
                 b.firstOf(DmlGrammar.SELECT_EXPRESSION, EXPRESSION),

zpa-core/src/test/kotlin/org/sonar/plugins/plsqlopen/api/expressions/ToNumberExpressionTest.kt

@@ -0,0 +1,50 @@
+/**
+ * Z PL/SQL Analyzer
+ * Copyright (C) 2015-2024 Felipe Zorzo
+ * mailto:felipe AT felipezorzo DOT com DOT br
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+ */
+package org.sonar.plugins.plsqlopen.api.expressions
+
+import com.felipebz.flr.tests.Assertions.assertThat
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+import org.sonar.plugins.plsqlopen.api.PlSqlGrammar
+import org.sonar.plugins.plsqlopen.api.RuleTest
+
+class ToNumberExpressionTest : RuleTest() {
+
+    @BeforeEach
+    fun init() {
+        setRootRule(PlSqlGrammar.EXPRESSION)
+    }
+
+    @Test
+    fun matchesSimple() {
+        assertThat(p).matches("to_number(foo)")
+    }
+
+    @Test
+    fun matchesDefaultClause() {
+        assertThat(p).matches("to_number(foo default 0 on conversion error)")
+    }
+
+    @Test
+    fun matchesMultipleArguments() {
+        assertThat(p).matches("to_number(foo, '999G999', 'NLS_DATE_LANGUAGE = American')")
+    }
+
+}