Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0026085 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] B. User interface | major | have not tried | 2014-03-25 17:27 | 2014-05-12 09:18 | |||
| Reporter | jonalegriaesarte | View Status | public | |||||
| Assigned To | guillermogil | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR14Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | e16d7577a075 | ||||
| Projection | none | ETA | none | Target Version | 3.0PR14Q3 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | pi | SCM revision | ||||||
| Merge Request Status | ||||||||
| Review Assigned To | alostale | |||||||
| OBNetwork customer | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 27475 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0026085: Location is not correctly filtered in grid view, if the country does not have region | |||||||
| Description | Location is not correctly filtered in grid view, if the country does not have region | |||||||
| Steps To Reproduce | - Warehouse and bins - Filter by location, it works fine - Select one and change the country of the location, select one without region (Anguilla for example). Save it - Try to filter, notice that the location does not appear as an option | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0066820) alostale (viewer) 2014-05-05 17:00 |
Problem is in nullable properties that are part of the identifier of a FK. In this case the query is from Warehouse as e left join e.locationAddress as join_0 where ( upper((COALESCE(to_char(join_0.addressLine1),'') || ' - ' || COALESCE(to_char(join_0.addressLine2),'') || ' - ' || COALESCE(to_char(join_0.postalCode),'') || ' - ' || COALESCE(to_char(join_0.cityName),'') || ' - ' || (COALESCE(to_char(join_0.region.name),'')) || ' - ' || (COALESCE(to_char((select name from CountryTrl as t where t.country = join_0.country and t.language.language='en_US')), to_char(join_0.country.name), '')))) like upper(:alias_0) escape '|' ) order by e.name,e.id this part: join_0.region.name creates an inner join with region table which results in no row in case of empty region Left join should be used instead: from Warehouse as e left join e.locationAddress as join_0 left join join_0.region r where ( upper((COALESCE(to_char(join_0.addressLine1),'') || ' - ' || COALESCE(to_char(join_0.addressLine2),'') || ' - ' || COALESCE(to_char(join_0.postalCode),'') || ' - ' || COALESCE(to_char(join_0.cityName),'') || ' - ' || (COALESCE(to_char(r.name),'')) || ' - ' || (COALESCE(to_char((select name from CountryTrl as t where t.country = join_0.country and t.language.language='en_US')), to_char(join_0.country.name), '')))) like upper('%pam%') escape '|' ) order by e.name,e.id |
|
(0067020) hgbot (developer) 2014-05-08 11:41 |
Repository: erp/devel/pi Changeset: e16d7577a0754fa07c6d3883d77eaa5f2c0754c1 Author: Guillermo Gil <guillermo.gil <at> openbravo.com> Date: Thu May 08 11:09:49 2014 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/e16d7577a0754fa07c6d3883d77eaa5f2c0754c1 [^] Fixed issue 26085:Location is not correctly filtered in grid view If the country does not have a region it was not filtered. Added new joins if the property is referenced and it can allow null values. Prefix modified to take into account the alias of the joins. --- M modules/org.openbravo.service.json/src/org/openbravo/service/json/AdvancedQueryBuilder.java --- |
|
(0067072) hudsonbot (viewer) 2014-05-09 16:21 |
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/e8c0af2263a0 [^] Maturity status: Test |
|
(0067083) alostale (viewer) 2014-05-12 09:18 |
code reviewed verified new left join is only added for nullable properties in identifier and in other cases same query as before is generated |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2014-03-25 17:27 | jonalegriaesarte | New Issue | |
| 2014-03-25 17:27 | jonalegriaesarte | Assigned To | => AugustoMauch |
| 2014-03-25 17:27 | jonalegriaesarte | Modules | => Core |
| 2014-03-25 17:27 | jonalegriaesarte | OBNetwork customer | => Yes |
| 2014-03-25 17:27 | jonalegriaesarte | Support ticket | => 27475 |
| 2014-03-25 17:27 | jonalegriaesarte | Resolution time | => 1398808800 |
| 2014-03-25 17:27 | jonalegriaesarte | Triggers an Emergency Pack | => No |
| 2014-03-25 17:52 | daniOpenbravo | Issue Monitored: daniOpenbravo | |
| 2014-05-05 17:00 | alostale | Note Added: 0066820 | |
| 2014-05-05 17:00 | alostale | Assigned To | AugustoMauch => guilleaer |
| 2014-05-06 08:23 | guillermogil | Assigned To | guilleaer => guillermogil |
| 2014-05-08 11:04 | guillermogil | Issue Monitored: alostale | |
| 2014-05-08 11:04 | guillermogil | Review Assigned To | => alostale |
| 2014-05-08 11:41 | hgbot | Checkin | |
| 2014-05-08 11:41 | hgbot | Note Added: 0067020 | |
| 2014-05-08 11:41 | hgbot | Status | new => resolved |
| 2014-05-08 11:41 | hgbot | Resolution | open => fixed |
| 2014-05-08 11:41 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/e16d7577a0754fa07c6d3883d77eaa5f2c0754c1 [^] |
| 2014-05-09 16:21 | hudsonbot | Checkin | |
| 2014-05-09 16:21 | hudsonbot | Note Added: 0067072 | |
| 2014-05-12 09:18 | alostale | Note Added: 0067083 | |
| 2014-05-12 09:18 | alostale | Status | resolved => closed |
| 2014-05-12 09:18 | alostale | Fixed in Version | => PR14Q3 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |