Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0038066 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | always | 2018-03-07 13:52 | 2018-03-13 17:23 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | 3.0PR18Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 253508d68cbf | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | caristu | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | 2016-11-08 | |||||||
Regression introduced in release | 3.0PR17Q1 | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/7acec5407733 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0038066: incorrect view cache in browser for windows with server display logic | |||||||
Description | View definition for windows that have at least a field with server side display logic are not correctly cached in browser if there is no preference configuring that field. The affected windows are: select distinct w.name from ad_field f, ad_tab t, ad_window w where displaylogic_server is not null and f.ad_Tab_id = t.ad_tab_id and w.ad_window_id = t.ad_window_id order by 1; name ----------------------------- Create Purchase Order Lines Goods Movements Goods Receipt Goods Shipment Manage Requisitions Purchase Invoice Purchase Order Referenced Inventory Requisition Return from Customer Return Material Receipt Return to Vendor Return to Vendor Shipment Sales Invoice Sales Order Sales Quotation (16 rows) | |||||||
Steps To Reproduce | 1. Log in as Openbravo 2. Open dev tools to check requests 3. Open Sales Order window -> OK: a request to View?viewId=_143 is triggered with response code 200 (success) 4. Close Sales Order window and reopen it in the same session -> OK: no request is perfomed 5. Close session and log in again with Openbravo user in the same browser 6. Open Sales Order window -> GOT: a request to View?viewId=_143 is triggered with response code 200 EXPECTED: a request to View?viewId=_143 with response code 304 (not modified) | |||||||
Tags | Performance | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||||||||||||||||||||||||||||||
|
Notes | |
(0103075) hgbot (developer) 2018-03-08 10:11 |
Repository: erp/devel/pi Changeset: 253508d68cbfc9cf0bf3c1a493ac101e27f5a4dd Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Wed Mar 07 14:26:09 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/253508d68cbfc9cf0bf3c1a493ac101e27f5a4dd [^] fixed bug 38066: no view cache in browser for windows with server display logic Window having at least a field with server side display logic were not cached in browser if there was no preference configuring that display logic. The problem was cuased by an incorrect eTag calculation that if not preference was set a date in future was set including current hour. The fix: * Simplifies eTag calculation for this case: if no preference defined, no need to add any date. * Uses for StringBuilder for String concatenation to slightly improve performance --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/ViewComponent.java --- |
(0103076) hgbot (developer) 2018-03-08 10:11 |
Repository: erp/devel/pi Changeset: 92b12ce052c6dffbf6c6a573ae32757b08babd6d Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Thu Mar 08 10:10:30 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/92b12ce052c6dffbf6c6a573ae32757b08babd6d [^] related to bug 38066: added test case --- M src-test/src/org/openbravo/test/AllWebserviceTests.java M src-test/src/org/openbravo/test/datasource/BaseDataSourceTestDal.java M src-test/src/org/openbravo/test/datasource/DatasourceTestUtil.java A src-test/src/org/openbravo/test/views/ETagGeneration.java --- |
(0103077) hgbot (developer) 2018-03-08 10:18 |
Repository: erp/devel/pi Changeset: 0c460868d807e1d7ad12b5c1ed5893d0a5cc894a Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Thu Mar 08 10:17:46 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/0c460868d807e1d7ad12b5c1ed5893d0a5cc894a [^] related to bug 38066: updated copyright --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/ViewComponent.java --- |
(0103096) hudsonbot (developer) 2018-03-08 16:51 |
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/bce7d98857dd [^] Maturity status: Test |
(0103097) hudsonbot (developer) 2018-03-08 16:51 |
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/bce7d98857dd [^] Maturity status: Test |
(0103105) hudsonbot (developer) 2018-03-09 04:28 |
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/0d8dac9d2802 [^] Maturity status: Test |
(0103151) hgbot (developer) 2018-03-12 12:42 |
Repository: erp/devel/pi Changeset: aa980425cd68a277618322131c6c8301075a1eb1 Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Mon Mar 12 12:42:12 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/aa980425cd68a277618322131c6c8301075a1eb1 [^] related to issue 38066: fix typo --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/ViewComponent.java --- |
(0103153) hgbot (developer) 2018-03-12 12:55 |
Repository: erp/devel/pi Changeset: b87b8a66ba6b528bd6c2c8b4091fbf778cf9184b Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Mon Mar 12 12:55:00 2018 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/b87b8a66ba6b528bd6c2c8b4091fbf778cf9184b [^] related to issue 38066: javadoc improvements --- M src-test/src/org/openbravo/test/datasource/BaseDataSourceTestDal.java --- |
(0103154) caristu (developer) 2018-03-12 13:03 |
Code reviewed + tested OK. |
(0103210) hudsonbot (developer) 2018-03-13 17:23 |
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/1d3ccc92fab6 [^] Maturity status: Test |
(0103211) hudsonbot (developer) 2018-03-13 17:23 |
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/1d3ccc92fab6 [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2018-03-07 13:52 | alostale | New Issue | |
2018-03-07 13:52 | alostale | Assigned To | => platform |
2018-03-07 13:52 | alostale | Modules | => Core |
2018-03-07 13:52 | alostale | Triggers an Emergency Pack | => No |
2018-03-07 13:54 | alostale | Tag Attached: Performance | |
2018-03-07 13:55 | alostale | Review Assigned To | => caristu |
2018-03-07 13:55 | alostale | Regression level | => Production - Confirmed Stable |
2018-03-07 13:55 | alostale | Regression date | => 2016-11-08 |
2018-03-07 13:55 | alostale | Regression introduced in release | => 3.0PR17Q1 |
2018-03-07 13:55 | alostale | Regression introduced by commit | => https://code.openbravo.com/erp/devel/pi/rev/7acec5407733 [^] |
2018-03-07 13:55 | alostale | Priority | normal => immediate |
2018-03-07 13:55 | alostale | Relationship added | caused by 0034406 |
2018-03-07 14:13 | alostale | Assigned To | platform => alostale |
2018-03-07 14:18 | alostale | Status | new => scheduled |
2018-03-07 14:22 | alostale | Summary | incorrect view cache in browse for windows with server display logic => incorrect view cache in browser for windows with server display logic |
2018-03-07 18:09 | egoitz | Relationship added | related to 0037999 |
2018-03-07 18:10 | egoitz | Issue Monitored: egoitz | |
2018-03-08 10:11 | hgbot | Checkin | |
2018-03-08 10:11 | hgbot | Note Added: 0103075 | |
2018-03-08 10:11 | hgbot | Status | scheduled => resolved |
2018-03-08 10:11 | hgbot | Resolution | open => fixed |
2018-03-08 10:11 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/253508d68cbfc9cf0bf3c1a493ac101e27f5a4dd [^] |
2018-03-08 10:11 | hgbot | Checkin | |
2018-03-08 10:11 | hgbot | Note Added: 0103076 | |
2018-03-08 10:18 | hgbot | Checkin | |
2018-03-08 10:18 | hgbot | Note Added: 0103077 | |
2018-03-08 16:51 | hudsonbot | Checkin | |
2018-03-08 16:51 | hudsonbot | Note Added: 0103096 | |
2018-03-08 16:51 | hudsonbot | Checkin | |
2018-03-08 16:51 | hudsonbot | Note Added: 0103097 | |
2018-03-09 04:28 | hudsonbot | Checkin | |
2018-03-09 04:28 | hudsonbot | Note Added: 0103105 | |
2018-03-12 12:42 | hgbot | Checkin | |
2018-03-12 12:42 | hgbot | Note Added: 0103151 | |
2018-03-12 12:55 | hgbot | Checkin | |
2018-03-12 12:55 | hgbot | Note Added: 0103153 | |
2018-03-12 13:03 | caristu | Note Added: 0103154 | |
2018-03-12 13:03 | caristu | Status | resolved => closed |
2018-03-12 13:03 | caristu | Fixed in Version | => 3.0PR18Q2 |
2018-03-13 17:23 | hudsonbot | Checkin | |
2018-03-13 17:23 | hudsonbot | Note Added: 0103210 | |
2018-03-13 17:23 | hudsonbot | Checkin | |
2018-03-13 17:23 | hudsonbot | Note Added: 0103211 | |
2018-11-22 09:13 | alostale | Relationship added | related to 0039670 |
Copyright © 2000 - 2009 MantisBT Group |