Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0020874Openbravo ERPA. Platformpublic2012-06-26 16:582012-06-28 08:34
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0MP13Fixed in Version3.0MP13 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2012-05-03
Regression introduced in release
Regression introduced by commit
Triggers an Emergency Packyes
Summary0020874: Down key navigation does not work in grid view if not all records have been loaded.
DescriptionIt is not possible to use the down key to move to the next record in the grid until all the records have been loaded from the datasource
Steps To Reproduce- Open the Country and Region window
- In the grid view, notice that you can move to the previous record by pressing the up arrow key, but you can not move to the next record by pressing the down arrow key.
- Scroll down until the last record is loaded.
- Notice that now you can use up and down arrow keys to move among the records
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to design defect 0017910 closed migueldejuana In grid of generated windows you can scroll out of the range of existing rows using cursor keys 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-06-26 16:58AugustoMauchNew Issue
2012-06-26 16:58AugustoMauchAssigned To => AugustoMauch
2012-06-26 16:58AugustoMauchModules => Core
2012-06-26 16:58AugustoMauchOBNetwork customer => No
2012-06-26 17:05AugustoMauchNote Added: 0050098
2012-06-26 17:06AugustoMauchNote Edited: 0050098bug_revision_view_page.php?bugnote_id=0050098#r3576
2012-06-26 17:13AugustoMauchRegression level => Production - QA Approved
2012-06-26 17:18AugustoMauchNote Added: 0050100
2012-06-26 19:36shuehnerIssue Monitored: shuehner
2012-06-27 12:51AugustoMauchRegression date => 2012-05-03
2012-06-27 12:51AugustoMauchRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/d0ca61f83c8f11b99f93ba31eafa30919d52ea48 [^]
2012-06-27 12:51AugustoMauchRelationship addedrelated to 0017910
2012-06-27 13:01hgbotCheckin
2012-06-27 13:01hgbotNote Added: 0050188
2012-06-27 13:01hgbotStatusnew => resolved
2012-06-27 13:01hgbotResolutionopen => fixed
2012-06-27 13:01hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7ada3b41a87a90d3b3f30ee3359c925e13db152d [^]
2012-06-28 07:59hudsonbotCheckin
2012-06-28 07:59hudsonbotNote Added: 0050220
2012-06-28 08:33alostaleNote Added: 0050242
2012-06-28 08:34alostaleStatusresolved => closed
2012-06-28 08:34alostaleFixed in Version => 3.0MP13

Notes
(0050098)
AugustoMauch   
2012-06-26 17:05   
(edited on: 2012-06-26 17:06)
This issue was not reproducible in MP7, thus it is a regression.
(0050100)
AugustoMauch   
2012-06-26 17:18   
This issue is reproducible in MP12 but not in MP11.
(0050188)
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
---
(0050220)
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
(0050242)
alostale   
2012-06-28 08:33   
Code reviewed and verified in pi@db25f47ad0a4

Verified also 0017910 is still fixed