Anonymous | Login
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Openbravo ERP] A. Platformminoralways2015-03-12 09:562015-03-13 20:09
ReportercaristuView Statuspublic 
Assigned ToAugustoMauch 
PriorityimmediateResolutionfixedFixed in Version3.0PR14Q3.8
StatusclosedFix in branchFixed in SCM revisionea610afa4225
ProjectionnoneETAnoneTarget Version3.0PR14Q3.8
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned Toalostale
Web browser
Regression levelProduction - Confirmed Stable
Regression date2014-05-20
Regression introduced in release3.0PR14Q3
Regression introduced by commit [^]
Triggers an Emergency PackNo

0029254: Can't filter in numeric fields using the value 0

DescriptionIs not possible to filter numeric fields using the value 0, the following error appears in the javascript console:

  "Uncaught TypeError: Cannot read property 'operator' of undefined"
Steps To Reproduce1) Go to the Sales Order window
2) Try to filter by Delivery Status or Invoice Status fields , using the value 0
3) Notice that the filtering is not done and the error in description appears
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
caused by defect 0026305 closedguillermogil [PR14Q2]In column filter, when i enter ^511 and press tab, filter value gets deleted 
blocks defect 0029248 closedAugustoMauch Can't filter in numeric fields using the value 0 

-  Notes
hgbot (developer)
2015-03-12 17:08

Repository: erp/backports/3.0PR14Q3.8
Changeset: ea610afa4225516668fe5f6c86c9232ff8d12013
Author: Augusto Mauch <augusto.mauch <at>>
Date: Thu Mar 12 12:00:30 2015 +0100
URL: [^]

Fixes issue 29254: 0 can be used to filter numeric columns

In this changeset [1] this code was added to the OBNumberItem.parseValueExpressions function:

     ret = this.Super('parseValueExpressions', [value, fieldName, operator]);
+ // if operator is not supported remove it
+ if (!this.validOperators.contains(ret.operator)) {
+ ret.operator = '';
+ ret.value = '';
+ this.setValue('');
+ }
     if (ret && ret.start) {
       ret.start = this.convertToTypedValue(ret.start);

The problem is that if the provided value is 0, this.Super('parseValueExpressions', [value, fieldName, operator]) will return undefined. This happens because of a bug in smartclient's implementation of the
 parseValueExpressions function. This code is placed at the beginning of that function:

        if (!value) value = this.getValue();
        if (!value) return;
        if (!isc.isA.String(value)) value += "";

If the provided value is 0, the value will be taken from this.getValue(). The returned value will again be evaluated to false, so the function will not return any value. Right after that, there is a comman
d to convert the provided value to String.

To fix this, in the call to this.Super('parseValueExpressions'), the string representation of the number will be passed instead of its numerical value. This way smartclient will accept the value and will n
ot return undefined. Smartclient was already converting the passed values to String, so there is no risk there.

The value passed to the OBNumberItem.parseValueExpressions is not modified to prevent unexpected consequences. A copy of it is converted to string and passed to the this.Super('parseValueExpressions') func

[1] [^]

M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js
alostale (developer)
2015-03-13 11:57

code reviewed

tested in Sales Order delivery status (computed column) and total gross amount (regular column) with values:
-^3 (as reported in 0026305, in this case the filter is removed as it is not supported textual filters in numeric columns)

- Issue History
Date Modified Username Field Change
2015-03-12 11:33 AugustoMauch Type defect => backport
2015-03-12 11:33 AugustoMauch Target Version => 3.0PR14Q3.8
2015-03-12 17:07 AugustoMauch Issue Monitored: alostale
2015-03-12 17:08 AugustoMauch Review Assigned To => alostale
2015-03-12 17:08 hgbot Checkin
2015-03-12 17:08 hgbot Note Added: 0075496
2015-03-12 17:08 hgbot Status scheduled => resolved
2015-03-12 17:08 hgbot Resolution open => fixed
2015-03-12 17:08 hgbot Fixed in SCM revision => [^]
2015-03-13 11:06 alostale Relationship added caused by 0026305
2015-03-13 11:57 alostale Note Added: 0075539
2015-03-13 11:57 alostale Status resolved => closed
2015-03-13 11:57 alostale Fixed in Version => 3.0PR14Q3.8
2015-03-13 20:09 dmitry_mezentsev Tag Attached: Approved

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker