Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ repositories {
mavenCentral()
}

version = "1.4.1-SNAPSHOT"
version = "1.4.2-SNAPSHOT"
group = "org.hisp.dhis.lib.expression"

if (project.hasProperty("removeSnapshotSuffix")) {
Expand Down Expand Up @@ -107,4 +107,4 @@ sonarqube {

tasks.named("sonar").configure {
dependsOn(":koverXmlReport")
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ package org.hisp.dhis.lib.expression.eval

import kotlinx.datetime.LocalDate
import org.hisp.dhis.lib.expression.ast.*
import org.hisp.dhis.lib.expression.ast.Nodes.Utf8StringNode
import org.hisp.dhis.lib.expression.ast.UnaryOperator.Companion.negate
import org.hisp.dhis.lib.expression.spi.*

Expand Down Expand Up @@ -139,7 +140,7 @@ internal class Calculator(
evalToInteger(fn.child(2)))
NamedFunction.d2_validatePattern -> functions.d2_validatePattern(
evalToString(fn.child(0)),
evalToString(fn.child(1)))
evalToRawString(fn.child(1)))
NamedFunction.d2_weeksBetween -> functions.d2_weeksBetween(
evalToDate(fn.child(0)),
evalToDate(fn.child(1)))
Expand Down Expand Up @@ -301,6 +302,15 @@ internal class Calculator(
return eval(node, "String", Typed::toStringTypeCoercion)
}

private fun evalToRawString(node: Node<*>): String? {
return when (node.getType()) {
NodeType.STRING -> node.getRawValue()
NodeType.ARGUMENT -> evalToRawString(node.child(0))
NodeType.PAR -> evalToRawString(node.child(0))
else -> evalToString(node)
}
}

fun evalToBoolean(node: Node<*>): Boolean? {
return eval(node, "Boolean", Typed::toBooleanTypeCoercion)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ fun interface ExpressionFunctions {
}

fun d2_validatePattern(input: String?, regex: String?): Boolean {
return input != null && regex != null && input.matches(regex.toRegex())
return input != null && regex != null && matchesPattern(input, regex)
}

fun d2_weeksBetween(start: LocalDate?, end: LocalDate?): Int {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
package org.hisp.dhis.lib.expression.spi

internal expect fun matchesPattern(input: String, pattern: String): Boolean
Original file line number Diff line number Diff line change
Expand Up @@ -29,15 +29,17 @@ internal class ValidatePatternTest {
fun testValidatePattern_Match() {
assertTrue(evaluate("d2:validatePattern(\"124\", \"[0-9]+\")"))
assertTrue(evaluate("d2:validatePattern(\"12x4\", \"[0-9x]+\")"))
assertTrue(evaluate("d2:validatePattern(\"John\",(\"[a-zA-Z0-9À-ȕ\\'\\-\\‘\\`\\’\\ ]+\"))"))
}

@Test
fun testValidatePattern_NoMatch() {
assertFalse(evaluate("d2:validatePattern(\"12x4\", \"[0-9]+\")"))
assertFalse(evaluate("d2:validatePattern(\"ab0\", \"[0-9x]+\")"))
assertFalse(evaluate("d2:validatePattern(\"Иван\",(\"[a-zA-Z0-9À-ȕ\\'\\-\\‘\\`\\’\\ ]+\"))"))
}

private fun evaluate(expression: String): Boolean {
return Expression(expression, ExpressionMode.RULE_ENGINE_ACTION).evaluate() as Boolean
}
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
package org.hisp.dhis.lib.expression.spi

internal actual fun matchesPattern(input: String, pattern: String): Boolean {

Check warning on line 3 in src/jsMain/kotlin/org/hisp/dhis/lib/expression/spi/RegexMatch.kt

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Remove these unused function parameters.

See more on https://sonarcloud.io/project/issues?id=dhis2_expression-parser&issues=AZ3OBAJZ5cuuaAsLshsC&open=AZ3OBAJZ5cuuaAsLshsC&pullRequest=100
// Avoid Kotlin stdlib's Regex wrapper which may add the JS `u` flag and reject unknown backslash
// escapes. Use RegExp directly with ^(?:...)$ anchoring to replicate full-string matching.
return js("new RegExp('^(?:' + pattern + ')$').test(input)") as Boolean
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
package org.hisp.dhis.lib.expression.spi

internal actual fun matchesPattern(input: String, pattern: String): Boolean =
input.matches(pattern.toRegex())
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
package org.hisp.dhis.lib.expression.spi

internal actual fun matchesPattern(input: String, pattern: String): Boolean =
input.matches(pattern.toRegex())
Loading