Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0026043 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | have not tried | 2014-03-20 13:06 | 2014-03-22 04:14 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | 3.0PR14Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 06c118995adc | ||||
Projection | none | ETA | none | Target Version | 3.0PR14Q2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | AugustoMauch | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | Pre packaging ( pi ) | |||||||
Regression date | 2013-02-26 | |||||||
Regression introduced in release | ||||||||
Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/20eb3426d990 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0026043: blur triggers selector ds request without pagination | |||||||
Description | Having focus in a selector field, blur causes a request to datasource without any pagination. In case of big amount of data this can cause the oom error in server. This request shouldn't be triggered at all (it was not before SC upgrade) | |||||||
Steps To Reproduce | 1. Create a new sales order 2. Add a new line 3. Open product drop down -> see there is a request to ProductByPriceAndWarehouse OK (it includes pagination) -Select a product in the drop down list -> see there is an additional request: INCORRECT * This second request shouldn't be launched at all * It has no pagination info When the selector popup is opened the second request is also performed: - Instead of step 3, open product selector popup and see 2nd request By only blurring it is also performede: 1. Open an existing sales order 2. Go to lines and edit 3. Focus on product 4. Move to any other field -> Request is done | |||||||
Tags | Performance | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Notes | |
(0065340) alostale (manager) 2014-03-21 09:55 |
Executed test plan: 1. None of the 3 ways of reproducing this issue reported in steps to reproduce triggers request 2. Issues 0018076 and 0018075 are reproducible |
(0065341) hgbot (developer) 2014-03-21 09:58 |
Repository: erp/devel/pi Changeset: 06c118995adc10ec206f039743ec34585bb34ac6 Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Fri Mar 21 09:58:08 2014 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/06c118995adc10ec206f039743ec34585bb34ac6 [^] fixed bug 26043: blur triggers selector ds request without pagination addUnknownValues property should be false to prevent non-paginated request: http://www.smartclient.com/docs/9.1/a/b/c/go.html#attr..ComboBoxItem.addUnknownValues [^] --- M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js --- |
(0065353) AugustoMauch (administrator) 2014-03-21 11:16 |
Code reviewed and verified in pi@617ac4497332 |
(0065372) hudsonbot (developer) 2014-03-22 04: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/b70224bef848 [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2014-03-20 13:06 | alostale | New Issue | |
2014-03-20 13:06 | alostale | Assigned To | => AugustoMauch |
2014-03-20 13:06 | alostale | Modules | => Core |
2014-03-20 13:06 | alostale | Triggers an Emergency Pack | => No |
2014-03-20 13:07 | alostale | Regression level | => Pre packaging ( pi ) |
2014-03-20 13:07 | alostale | Regression date | => 2013-02-26 |
2014-03-20 13:07 | alostale | Regression introduced by commit | => https://code.openbravo.com/erp/devel/pi/rev/20eb3426d990 [^] |
2014-03-20 13:07 | alostale | Assigned To | AugustoMauch => alostale |
2014-03-20 13:07 | alostale | Target Version | => PR14Q2 |
2014-03-20 13:10 | alostale | Relationship added | related to 0025574 |
2014-03-20 13:14 | alostale | Steps to Reproduce Updated | View Revisions |
2014-03-21 08:47 | alostale | Priority | normal => immediate |
2014-03-21 08:47 | alostale | Review Assigned To | => AugustoMauch |
2014-03-21 09:02 | alostale | Tag Attached: Performance | |
2014-03-21 09:43 | alostale | Relationship added | has duplicate 0018075 |
2014-03-21 09:52 | alostale | Relationship added | related to 0018076 |
2014-03-21 09:54 | alostale | Summary | double selector ds request after selection without pagination => blur triggers selector ds request without pagination |
2014-03-21 09:54 | alostale | Description Updated | View Revisions |
2014-03-21 09:54 | alostale | Steps to Reproduce Updated | View Revisions |
2014-03-21 09:55 | alostale | Note Added: 0065340 | |
2014-03-21 09:58 | hgbot | Checkin | |
2014-03-21 09:58 | hgbot | Note Added: 0065341 | |
2014-03-21 09:59 | hgbot | Status | new => resolved |
2014-03-21 09:59 | hgbot | Resolution | open => fixed |
2014-03-21 09:59 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/06c118995adc10ec206f039743ec34585bb34ac6 [^] |
2014-03-21 11:16 | AugustoMauch | Note Added: 0065353 | |
2014-03-21 11:16 | AugustoMauch | Status | resolved => closed |
2014-03-21 11:16 | AugustoMauch | Fixed in Version | => PR14Q2 |
2014-03-22 04:14 | hudsonbot | Checkin | |
2014-03-22 04:14 | hudsonbot | Note Added: 0065372 | |
2014-04-08 08:50 | alostale | Relationship added | related to 0026189 |
2014-04-10 14:15 | alostale | Relationship added | related to 0026238 |
2014-04-30 15:45 | alostale | Relationship added | related to 0026432 |
2014-07-16 11:19 | alostale | Relationship added | related to 0027088 |
2014-12-24 14:16 | AugustoMauch | Relationship added | related to 0027619 |
2014-12-24 16:02 | AugustoMauch | Relationship replaced | causes 0027619 |
Copyright © 2000 - 2009 MantisBT Group |