dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #38641
[Branch ~dhis2-devs-core/dhis2/trunk] Rev 19678: Nasty fix for masking bug in Jexl, will fix later
------------------------------------------------------------
revno: 19678
committer: Lars Helge Overland <larshelge@xxxxxxxxx>
branch nick: dhis2
timestamp: Sun 2015-07-19 18:36:56 +0200
message:
Nasty fix for masking bug in Jexl, will fix later
modified:
dhis-2/dhis-support/dhis-support-commons/src/main/java/org/hisp/dhis/commons/util/ExpressionUtils.java
dhis-2/dhis-support/dhis-support-system/src/test/java/org/hisp/dhis/system/util/ExpressionUtilsTest.java
--
lp:dhis2
https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk
Your team DHIS 2 developers is subscribed to branch lp:dhis2.
To unsubscribe from this branch go to https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk/+edit-subscription
=== modified file 'dhis-2/dhis-support/dhis-support-commons/src/main/java/org/hisp/dhis/commons/util/ExpressionUtils.java'
--- dhis-2/dhis-support/dhis-support-commons/src/main/java/org/hisp/dhis/commons/util/ExpressionUtils.java 2015-07-19 15:50:46 +0000
+++ dhis-2/dhis-support/dhis-support-commons/src/main/java/org/hisp/dhis/commons/util/ExpressionUtils.java 2015-07-19 16:36:56 +0000
@@ -162,6 +162,11 @@
}
catch ( JexlException ex )
{
+ if ( ex.getMessage().contains( "divide error" ) )
+ {
+ return true; //TODO Masking bug in Jexl, fix
+ }
+
return false;
}
}
=== modified file 'dhis-2/dhis-support/dhis-support-system/src/test/java/org/hisp/dhis/system/util/ExpressionUtilsTest.java'
--- dhis-2/dhis-support/dhis-support-system/src/test/java/org/hisp/dhis/system/util/ExpressionUtilsTest.java 2015-07-19 15:50:46 +0000
+++ dhis-2/dhis-support/dhis-support-system/src/test/java/org/hisp/dhis/system/util/ExpressionUtilsTest.java 2015-07-19 16:36:56 +0000
@@ -156,6 +156,7 @@
assertTrue( ExpressionUtils.isValid( "3 - v1", vars ) );
assertTrue( ExpressionUtils.isValid( "d2:zing(1)", null ) );
assertTrue( ExpressionUtils.isValid( "(d2:zing(1)+d2:zing(1))*50/1", null ) );
+ assertTrue( ExpressionUtils.isValid( "1/(1/100)", null ) );
assertFalse( ExpressionUtils.isValid( "2 a 3", null ) );
assertFalse( ExpressionUtils.isValid( "v2 + 3", vars ) );