details:   https://code.openbravo.com/erp/devel/pi/rev/75f848b61c97
changeset: 22321:75f848b61c97
user:      Shankar Balachandran <shankar.balachandran <at> openbravo.com>
date:      Thu Feb 06 14:23:45 2014 +0530
summary:   Fixes Issue 25447: isNull filter expression is not working properly 
for table reference

If the operator is isNull or notNull comparison should be done at the object 
level and
not at the field value level

diffstat:

 
modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java
 |  6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diffs (16 lines):

diff -r 47e4fc11f404 -r 75f848b61c97 
modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java
--- 
a/modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java
       Fri Feb 28 18:48:33 2014 +0000
+++ 
b/modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java
       Thu Feb 06 14:23:45 2014 +0530
@@ -610,6 +610,12 @@
     if (ignoreCase(properties, operator)) {
       clause = "upper(" + clause + ")";
     }
+
+    // if the operator is isNull or notNull comparison should be done at the 
object level and not at
+    // the field value level. Refer issue 
https://issues.openbravo.com/view.php?id=25447
+    if (tableReference && (operator.equals(OPERATOR_ISNULL) || 
operator.equals(OPERATOR_NOTNULL))) {
+      clause = clause.substring(0, clause.lastIndexOf(DalUtil.DOT));
+    }
     return clause;
   }
 

------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works. 
Faster operations. Version large binaries.  Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to