From 9c05d467ea521c16c9e36b94e5862c05bf9e4dd7 Mon Sep 17 00:00:00 2001
From: Gustavo Adolfo Cruz Laura <gustavo@colosa.com>
Date: Fri, 7 Jan 2011 20:02:40 +0000
Subject: [PATCH] Fixed the jump-to feature in the cases List

---
 workflow/engine/templates/cases/casesList.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/workflow/engine/templates/cases/casesList.js b/workflow/engine/templates/cases/casesList.js
index ba760d8af..054338f00 100755
--- a/workflow/engine/templates/cases/casesList.js
+++ b/workflow/engine/templates/cases/casesList.js
@@ -799,13 +799,15 @@ Ext.onReady ( function() {
 
   textJump = {
     xtype: 'numberfield',
+    id   : 'textJump',
     allowBlank: true,
     width: 50,
     emptyText: TRANSLATIONS.ID_CASESLIST_APP_UID,
     listeners: {
       specialkey: function(f,e){
         if (e.getKey() == e.ENTER) {
-          caseNumber = parseFloat(Ext.util.Format.trim(textJump.getValue()));
+          // defining an id and using the Ext.getCmp method improves the accesibility of Ext components
+          caseNumber = parseFloat(Ext.util.Format.trim(Ext.getCmp('textJump').getValue()));
           if( caseNumber )
             jumpToCase(caseNumber);
           else
@@ -818,13 +820,13 @@ Ext.onReady ( function() {
   var btnJump = new Ext.Button ({
     text: TRANSLATIONS.LABEL_OPT_JUMP,
     handler: function(){
-      caseNumber = parseFloat(Ext.util.Format.trim(textJump.getValue()));
-       caseNumber = parseFloat(Ext.util.Format.trim(textJump.getValue()));
-          if( caseNumber )
+      var caseNumber = parseFloat(Ext.util.Format.trim(Ext.getCmp('textJump').getValue()));
+      if (caseNumber){
         jumpToCase(caseNumber);
-      else
+      } else {
             msgBox('Input Error', 'You have set a invalid Application Number', 'error');
     }
+    }
   });
 
   /*** menu and toolbars **/