Openbravo Issue Tracking System - Openbravo ERP |
View Issue Details |
|
ID | Project | Category | View Status | Date Submitted | Last Update |
0030505 | Openbravo ERP | 00. Application dictionary | public | 2015-08-06 09:52 | 2015-08-20 23:16 |
|
Reporter | gorkaion | |
Assigned To | alostale | |
Priority | immediate | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | |
Platform | | OS | 5 | OS Version | |
Product Version | | |
Target Version | | Fixed in Version | | |
Merge Request Status | |
Review Assigned To | gorkaion |
OBNetwork customer | |
Web browser | |
Modules | Core |
Support ticket | |
Regression level | Automated tests |
Regression date | 2015-08-05 |
Regression introduced in release | pi |
Regression introduced by commit | http://code.openbravo.com/erp/devel/pi/rev/1f572104e426366ac67e9efc7bd74040f8142ac3 [^] |
Triggers an Emergency Pack | No |
|
Summary | 0030505: Compilation in 2.50 mode fails for Navigation Rules tab of Window, Tabs and Fields |
Description | There is a compilation error in 2.50 mode.
https://ci.openbravo.com/job/int-full-pgsql-compile-all/601/consoleFull [^]
The issue is in "Window, Tabs and Fields" window when computing the xsql file for the new Navigation Rules tab. In this file it is not generating properly the query to load the identifier of the parent field (AD_Field_id) |
Steps To Reproduce | On a pi on changeset 0c08e36aad01. Compile the application in 2.50 mode:
ant compile.complete -Dwad.generateAllClassic250Windows=true
There is a query that it is not correctly generated:
SELECT (TO_CHAR(COALESCE(TO_CHAR((CASE WHEN tableTRL1.Name IS NULL THEN TO_CHAR(table1.Name) ELSE TO_CHAR(tableTRL1.Name) END)), '')) || ' - ' || list1.name) AS NAME
FROM AD_Field left join (select AD_Field_ID, Name, AD_Tab_ID from AD_Field) table1 on (AD_Field.AD_Field_ID = table1.AD_Field_ID)
left join (select AD_Field_ID,AD_Language, Name from AD_Field_TRL) tableTRL1 on (table1.AD_Field_ID = tableTRL1.AD_Field_ID and tableTRL1.AD_Language = ?)
left join ad_ref_list_v list1 on (table2.AD_Tab_ID = list1.value and list1.ad_reference_id = '' and list1.ad_language = ?)
WHERE AD_Field.AD_Field_ID = ?
|
Proposed Solution | |
Additional Information | |
Tags | No tags attached. |
Relationships | caused by | feature request | 0015379 | | closed | gorkaion | Extend navigation model |
|
Attached Files | |
|
Issue History |
Date Modified | Username | Field | Change |
2015-08-06 09:52 | gorkaion | New Issue | |
2015-08-06 09:52 | gorkaion | Assigned To | => alostale |
2015-08-06 09:52 | gorkaion | Modules | => Core |
2015-08-06 09:52 | gorkaion | Triggers an Emergency Pack | => No |
2015-08-06 09:54 | alostale | Relationship added | caused by 0030204 |
2015-08-06 09:55 | alostale | Regression level | => Automated tests |
2015-08-06 09:55 | alostale | Regression date | => 2015-08-05 |
2015-08-06 09:55 | alostale | Regression introduced in release | => pi |
2015-08-06 09:55 | alostale | Regression introduced by commit | => http://code.openbravo.com/erp/devel/pi/rev/1f572104e426366ac67e9efc7bd74040f8142ac3 [^] |
2015-08-06 09:55 | alostale | Priority | high => immediate |
2015-08-06 09:55 | alostale | Severity | major => minor |
2015-08-06 09:55 | alostale | Status | new => scheduled |
2015-08-06 09:55 | alostale | Relationship deleted | caused by 0030204 |
2015-08-06 09:55 | alostale | Relationship added | caused by 0015379 |
2015-08-06 09:56 | alostale | Review Assigned To | => gorkaion |
2015-08-06 15:04 | shuehner | Issue Monitored: shuehner | |
2015-08-06 16:26 | alostale | Note Added: 0079225 | |
2015-08-06 16:27 | alostale | Note Edited: 0079225 | bug_revision_view_page.php?bugnote_id=0079225#r9168 |
2015-08-06 16:41 | hgbot | Checkin | |
2015-08-06 16:41 | hgbot | Note Added: 0079226 | |
2015-08-06 16:41 | hgbot | Checkin | |
2015-08-06 16:41 | hgbot | Note Added: 0079227 | |
2015-08-06 16:41 | hgbot | Status | scheduled => resolved |
2015-08-06 16:41 | hgbot | Resolution | open => fixed |
2015-08-06 16:41 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/f5cac263bc766a60806d547e06317e1a54adcfa7 [^] |
2015-08-06 16:47 | alostale | Note Added: 0079228 | |
2015-08-07 10:21 | gorkaion | Note Added: 0079244 | |
2015-08-07 10:21 | gorkaion | Status | resolved => closed |
2015-08-20 23:16 | hudsonbot | Checkin | |
2015-08-20 23:16 | hudsonbot | Note Added: 0079530 | |
2015-08-20 23:16 | hudsonbot | Checkin | |
2015-08-20 23:16 | hudsonbot | Note Added: 0079531 | |
Notes |
|
(0079225)
|
alostale
|
2015-08-06 16:26
(edited on: 2015-08-06 16:27) |
|
There are two different issues affecting this tab:
1. When the record identifier of the parent tab is compound by more than one field, being one of them a TableDir and the other translatable, the generated query is fails to execute.
In fact TableDir fields participating in identifiers are always incorrectly generated because they are joining ID with ad_ref_list_value which is incorrect, when compiling a clean Openbravo 3 without forcing 2.50 generation the only affected file is org/openbravo/erpWindows/ProcessPlan/IOProducts_data.xsql with a query:
select ...
(CASE WHEN MA_SequenceProduct.M_Product_Uom_Id IS NULL THEN '' ELSE ( COALESCE(TO_CHAR(list2.name),'') ) END) AS M_Product_Uom_IdR,
...
from SequenceProduct
left join (select M_Product_Uom_Id, C_UOM_ID from M_Product_UOM) table5
on (MA_SequenceProduct.M_Product_Uom_Id = table5.M_Product_Uom_Id)
left join ad_ref_list_v list2
on (table5.C_UOM_ID = list2.value
and list2.ad_reference_id = ''
and list2.ad_language = ?)
...
which is incorrect.
2. When there is in a parent tab an auxiliary input with a parameter based in a field stored in session, the java generated is incorrect. The only occurrence of this case is the new Navigation Rules tab.
Correct implementation is not going to be done for none of these 2 cases because this code is planned to be retired soon, meanwhile a workaround to make it compile will be implemented.
|
|
|
(0079226)
|
hgbot
|
2015-08-06 16:41
|
|
Repository: erp/devel/pi
Changeset: 9d134526fceeb95e137a2863e59bce834ba20736
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Aug 06 16:38:01 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/9d134526fceeb95e137a2863e59bce834ba20736 [^]
related to issue 30505: 2.50 compilation mode fails
TableDir fields participating in identifiers are incorrectly generated because
they are joined by ID with ad_ref_list_value which is incorrect.
This is not a fix but just a workarround to make the code to compile.
---
M src-wad/src/org/openbravo/wad/controls/WADTableDir.java
---
|
|
|
(0079227)
|
hgbot
|
2015-08-06 16:41
|
|
Repository: erp/devel/pi
Changeset: f5cac263bc766a60806d547e06317e1a54adcfa7
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Aug 06 16:40:39 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f5cac263bc766a60806d547e06317e1a54adcfa7 [^]
fixed issue 30505: 2.50 compilation mode fails
When there is in a parent tab an auxiliary input with a parameter based in a
field stored in session, the java generated is incorrect.
This is not a fix but a workarround: values are not retrieved by using the java
field but using the FieldProvider.getField method instead.
---
M src-wad/src/org/openbravo/wad/WadUtility.java
---
|
|
|
|
Tested:
*changeset 1: http://code.openbravo.com/erp/devel/pi/rev/9d13452 [^]
-Comparing wad generated code after the fix and before the only modified file is ProcessPlan/IOProducts_data.xsql if 2.50 mode is not forced.
-There are few other xsql files modified when forcing 2.50 mode.
-New query does not implement the identifier replacing it with xxx string (note before was not correctly implemented)
*changeset 2: http://code.openbravo.com/erp/devel/pi/rev/f5cac263b [^]
-There are many java files affected in the way auxiliary inputs are retrieved, but the old and new ways are completely interchangeable. |
|
|
|
checked that window is now properly created and compilation does not fail. |
|
|
|
|
|
|
|