Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010133Openbravo ERPA. Platformpublic2009-07-30 15:392009-08-19 00:00
Reportershuehner 
Assigned Toshuehner 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.40 
Target VersionFixed in Versionpi 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0010133: Generated windows still execute code for old keyboard shortcut system which is then later not used.
DescriptionThe generated servlets for edit- & grid-view still run code code (including a database query) for some old keyboard shortcut system (class KeyMap.java). However the result of this code is then discarded when building the html with xmlEngine.

To improve performance the code should be removed to not execute this unused database query.
Steps To Reproduce
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0010134 closed shuehner Generated windows still execute code for old keyboard shortcut system which is then later not used. 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-07-30 15:39shuehnerNew Issue
2009-07-30 15:39shuehnerAssigned To => shuehner
2009-07-30 15:39shuehnerStatusnew => scheduled
2009-07-30 15:39shuehnerfix_in_branch => pi
2009-07-30 16:39hgbotCheckin
2009-07-30 16:39hgbotNote Added: 0018726
2009-07-30 16:39hgbotStatusscheduled => resolved
2009-07-30 16:39hgbotResolutionopen => fixed
2009-07-30 16:39hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/eb8040ff6595d122b4beaf7e9fc80dcd9f34a0a6 [^]
2009-07-30 18:20shuehnerNote Added: 0018730
2009-08-18 10:57sureshbabuStatusresolved => closed
2009-08-18 10:57sureshbabuNote Added: 0018997
2009-08-18 10:57sureshbabuFixed in Version => pi
2009-08-19 00:00anonymoussf_bug_id0 => 2839893

Notes
(0018726)
hgbot   
2009-07-30 16:39   
Repository: erp/devel/pi
Changeset: eb8040ff6595d122b4beaf7e9fc80dcd9f34a0a6
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Thu Jul 30 16:39:29 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/eb8040ff6595d122b4beaf7e9fc80dcd9f34a0a6 [^]

Fixed 10133: Remove unused code for old KeyMap keyboard shortcut system
- Removes one unneeded database query from each generated window

---
M src-wad/src/org/openbravo/wad/Configuration_Edition.xmlxml
M src-wad/src/org/openbravo/wad/Configuration_Relation.xmlxml
M src-wad/src/org/openbravo/wad/Template_Edition.html
M src-wad/src/org/openbravo/wad/Template_Relation.html
M src-wad/src/org/openbravo/wad/javasource.javaxml
---
(0018730)
shuehner   
2009-07-30 18:20   
@qa: This should introduce not user visible change in behavior.

All keyboard shortcuts which did work before should still work:

List of shortcuts can be found at [1]:

[1] http://wiki.openbravo.com/wiki/Projects/KeyboardOperation/FunctionalSpecifications [^]
(0018997)
sureshbabu   
2009-08-18 10:57   
Checked the hot keys listed under http://wiki.openbravo.com/wiki/Projects/KeyboardOperation/FunctionalSpecifications [^] [^]