Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0030296Openbravo ERPA. Platformpublic2015-06-29 13:082015-07-07 09:44
egoitz 
caristu 
urgentminoralways
closedfixed 
5
 
3.0PR15Q2.33.0PR15Q2.3 
alostale
Core
No
0030296: In saved views with multiple criteria in one field, the uuid value is shown in the column header when filtering sometimes
When filtering on a FK column with two values, and then on a next column filtering a different FK for a unique value, the value shown on the last column is the uuid
Steps to reproduce:
1. Go to Sales Order window
2. In grid view, filter business partner form the drop-down by "==Hoteles Buenas Noches, S.A. or
==Restaurantes Luna Llena, S.A." and Payment method "==Transferencia"
3. Save view as "test"
4. Reopen sales order window and apply the saved view "test" -> payment method filter shows its
identifier ==A97CFD2AFC234B59BB0A72189BD8FC2A instead of ==Transferencia, making subsequent
filtering actions fail.


Notes:
- If you filter business partner by just one value, behavior is always ok
- If you move the column business partner after the column with a single filter, behavior seems to be OK
too, however if business partner (or the column with several values selected from the drop down) is
before payment method, behavior is wrong.
Approved
related to defect 0047320 closed alostale In saved views the uuid value is shown in the filter of some FK fields 
blocks defect 00302743.0PR15Q4 closed caristu In saved views with multiple criteria in one field, the uuid value is shown in the column header when filtering sometimes 
Issue History
2015-07-01 12:27caristuTypedefect => backport
2015-07-01 12:27caristuTarget Version => 3.0PR15Q2.3
2015-07-01 12:39caristuSummaryThe uuid value is shown in the column header when filtering. => In saved views with multiple criteria in one field, the uuid value is shown in the column header when filtering sometimes
2015-07-02 09:05alostaleTag Attached: Approved
2015-07-02 09:50caristuIssue Monitored: alostale
2015-07-02 09:50caristuReview Assigned To => alostale
2015-07-02 09:51hgbotCheckin
2015-07-02 09:51hgbotNote Added: 0078611
2015-07-02 09:51hgbotStatusscheduled => resolved
2015-07-02 09:51hgbotResolutionopen => fixed
2015-07-02 09:51hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR15Q2.3/rev/201080ae3edce57c5f2a2538aca07eadc7905a26 [^]
2015-07-07 09:37alostaleSeveritymajor => minor
2015-07-07 09:44alostaleNote Added: 0078696
2015-07-07 09:44alostaleStatusresolved => closed
2015-07-07 09:44alostaleFixed in Version => 3.0PR15Q2.3
2021-07-02 11:27caristuRelationship addedrelated to 0047320

Notes
(0078611)
hgbot   
2015-07-02 09:51   
Repository: erp/backports/3.0PR15Q2.3
Changeset: 201080ae3edce57c5f2a2538aca07eadc7905a26
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Jul 01 12:43:26 2015 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q2.3/rev/201080ae3edce57c5f2a2538aca07eadc7905a26 [^]

Fixes issue 30296: uuid appear in saved views with multiple criteria on a field

When a view is saved while any of its filters are currently using the 'id' filter type, the id and identifier of the selected options are stored in a cache. If we have multiple criteria on one field, an internal loop is executed to retrieve all the criterias in the getFKFilterAuxiliaryCache function.

The problem was that this internal loop was using the same variable as the external one to insert those criterias into the cache. This caused that the subsequent filters for next fields were not being handled properly as the condition of the external loop was not being fulfilled anymore due to the increment done in the internal loop.

This has been solved just using a new variable (j) in the internal loop.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
(0078696)
alostale   
2015-07-07 09:44   
code reviewed

tested:
* Saved view
* Default saved view
* Direct link