Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0023890Openbravo ERPA. Platformpublic2013-05-22 23:572013-06-02 09:43
Reporteregoitz 
Assigned ToAugustoMauch 
PriorityurgentSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0MP24Fixed in Version3.0MP24 
Merge Request Status
Review Assigned Toshankarb
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0023890: Bad behaviour on grid view when searching on a selector field if the selector has defined a diplayed field
DescriptionBad behaviour on grid view when searching on a selector field if the selector has defined a diplayed field.

The query executed on grid view is different than the one executed on form view
Steps To Reproduce-As system administrador define a template
-Then go to the product selector reference.
-On the defined selector tab add a displayed field. For example name
Enable all the queries on the postgresql log
Logout and login
-go to sales order window
-go to lines window
-insert a new row on form view
-insert a letter on the product selector field, for example a.
*Check the sql that appears on the log
-repeat the same on grid view
*Check that the sql is different than the one that you get on form view
Proposed Solution
Additional Information
TagsPerformance
Relationships
related to defect 00238893.0MP24 closed AugustoMauch When searching on the field of a new selector the request to the datasource is done twice 
related to defect 0030403 closed caristu Unexpected behavior in grid view with OBUISEL_Selector reference when the Display Field is set 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-05-22 23:57egoitzNew Issue
2013-05-22 23:57egoitzAssigned To => AugustoMauch
2013-05-22 23:57egoitzModules => Core
2013-05-22 23:57egoitzTriggers an Emergency Pack => No
2013-05-22 23:58egoitzRelationship addedrelated to 0023889
2013-05-22 23:58egoitzTag Attached: Performance
2013-05-23 02:06eintelauIssue Monitored: eintelau
2013-05-28 17:49AugustoMauchIssue Monitored: shankarb
2013-05-28 17:50AugustoMauchReview Assigned To => shankarb
2013-05-28 17:50hgbotCheckin
2013-05-28 17:50hgbotNote Added: 0059000
2013-05-28 17:50hgbotStatusnew => resolved
2013-05-28 17:50hgbotResolutionopen => fixed
2013-05-28 17:50hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3b9604eee5d7b16109e339e75f17b5f8be9b4467 [^]
2013-06-01 02:14hudsonbotCheckin
2013-06-01 02:14hudsonbotNote Added: 0059097
2013-06-02 09:43shankarbNote Added: 0059116
2013-06-02 09:43shankarbStatusresolved => closed
2013-06-02 09:43shankarbFixed in Version => 3.0MP24
2015-07-23 11:20alostaleRelationship addedrelated to 0030403

Notes
(0059000)
hgbot   
2013-05-28 17:50   
Repository: erp/devel/pi
Changeset: 3b9604eee5d7b16109e339e75f17b5f8be9b4467
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Tue May 28 17:49:38 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/3b9604eee5d7b16109e339e75f17b5f8be9b4467 [^]

Fixes issue 23890: The same hql is generated in the grid and in form view

The problem was that for that particular selector (Product selector), the generated displayfield string was 'product$product$name'. This is not correct, because the Product entity does not have a product property, the correct displayfield string is product$name. This has been fixed in the getDisplayFielName method of the FKSelectorUIDefinition class. A check has been added to make sure that the property used in the display field is part of the referenced entity. If it is not, then the displayFieldName is returned directly, instead of trying to append it to the referenced property.

---
M modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/reference/FKSelectorUIDefinition.java
---
(0059097)
hudsonbot   
2013-06-01 02:14   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/31f9b82dde77 [^]

Maturity status: Test
(0059116)
shankarb   
2013-06-02 09:43   
Code reviewed and tested in pi changeset f0878dbca750