Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0017910Openbravo ERPA. Platformpublic2011-07-05 19:232012-06-28 07:59
Reportershuehner 
Assigned Tomigueldejuana 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0MP12 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0017910: In grid of generated windows you can scroll out of the range of existing rows using cursor keys
DescriptionSee steps to reproduce: scrolling behavior for cursor-up on first line or cursor down on last line strange .
Steps To ReproduceOpen sales order window
select a sales order having at least two lines
click on first sales order line to mark it as selected.
use cursor up -> notice marker dissappears
use cursor up again -> notice marker re-appears
same with cursor down + last line
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00208743.0MP13 closed AugustoMauch Down key navigation does not work in grid view if not all records have been loaded. 
Attached Files? CheckboxGrid.avi (2,092,574) 2012-04-25 22:49
https://issues.openbravo.com/file_download.php?file_id=5171&type=bug
Issue History
Date ModifiedUsernameFieldChange
2011-07-05 19:23shuehnerNew Issue
2011-07-05 19:23shuehnerAssigned To => alostale
2011-07-05 19:23shuehnerModules => Core
2011-07-05 19:23shuehnerOBNetwork customer => No
2012-04-24 15:00alostaleAssigned Toalostale => migueldejuana
2012-04-25 10:37hgbotCheckin
2012-04-25 10:37hgbotNote Added: 0047949
2012-04-25 10:39migueldejuanaNote Deleted: 0047949
2012-04-25 22:48mtaalNote Added: 0047985
2012-04-25 22:49mtaalFile Added: CheckboxGrid.avi
2012-04-26 12:14alostaleNote Added: 0048005
2012-04-26 12:14alostaleTypedefect => design defect
2012-04-26 19:32hudsonbotCheckin
2012-04-26 19:32hudsonbotNote Added: 0048094
2012-05-03 16:48hgbotCheckin
2012-05-03 16:48hgbotNote Added: 0048347
2012-05-03 16:48hgbotStatusnew => resolved
2012-05-03 16:48hgbotResolutionopen => fixed
2012-05-03 16:48hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d0ca61f83c8f11b99f93ba31eafa30919d52ea48 [^]
2012-05-03 16:50migueldejuanaNote Added: 0048350
2012-05-08 17:54AugustoMauchNote Added: 0048464
2012-05-08 17:54AugustoMauchStatusresolved => closed
2012-05-08 17:54AugustoMauchFixed in Version => 3.0MP12
2012-05-25 12:09hudsonbotCheckin
2012-05-25 12:09hudsonbotNote Added: 0049013
2012-06-27 12:51AugustoMauchRelationship addedrelated to 0020874
2012-06-27 13:01hgbotCheckin
2012-06-27 13:01hgbotNote Added: 0050187
2012-06-28 07:59hudsonbotCheckin
2012-06-28 07:59hudsonbotNote Added: 0050221

Notes
(0047985)
mtaal   
2012-04-25 22:48   
Asked on smartclient forum:
http://forums.smartclient.com/showthread.php?p=85406#post85406 [^]
(0048005)
alostale   
2012-04-26 12:14   
Setting as design defect because it depends on SmartClient to be fixed
(0048094)
hudsonbot   
2012-04-26 19:32   
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/70a717a9ffa1 [^]

Maturity status: Test
(0048347)
hgbot   
2012-05-03 16:48   
Repository: erp/devel/pi
Changeset: d0ca61f83c8f11b99f93ba31eafa30919d52ea48
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu May 03 16:46:24 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d0ca61f83c8f11b99f93ba31eafa30919d52ea48 [^]

Fixed issue 17910: Don't scroll out of range of existing rows using cursor keys

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
(0048350)
migueldejuana   
2012-05-03 16:50   
Read this post to know why it has been solved like this.
http://forums.smartclient.com/showthread.php?p=85406#post85406 [^]

Test plan:

-Follow steps to reproduce.


No risky.
(0048464)
AugustoMauch   
2012-05-08 17:54   
Code reviewed and verified in pi@5f0181ecc96c
(0049013)
hudsonbot   
2012-05-25 12:09   
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/5401e185a8b0 [^]

Maturity status: Test
(0050187)
hgbot   
2012-06-27 13:01   
Repository: erp/devel/pi
Changeset: 7ada3b41a87a90d3b3f30ee3359c925e13db152d
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Jun 27 13:00:29 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7ada3b41a87a90d3b3f30ee3359c925e13db152d [^]

Fixes issue 20874: Prevented exception when pushing the down arrow key

In the fix of issue 17910 some logic was introduced that checked the id of the last record of the grid. If that record was yet to be loaded, the exception 'Cannot read property from undefined' was raised.

Now, before executing that statement, it is checked if the last grid record is defined.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
(0050221)
hudsonbot   
2012-06-28 07:59   
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/c66e853c5cf4 [^]

Maturity status: Test