[FIXED-598][3.x/2.x] isSpecialKey() does not include the DELETE key

[kairinsama]

The following mask was preventing the DELETE key from executing normally:

maskRe: /[a-zA-Z\'\- ]/

Overrode the isSpecialKey function to include DELETE.

Ext.EventObjectImpl.prototype.isSpecialKey = function() {
  var k = this.keyCode;
  k = Ext.isSafari ? (safariKeys[k] || k) : k;
  return (this.type == 'keypress' && this.ctrlKey) ||
    this.isNavKeyPress() ||
    (k == this.BACKSPACE) || // Backspace
    (k == this.DELETE)    || // Delete -- Currently not implemented, can also change 45 to 46 below to include it as well.
    (k >= 16 && k <= 20)  || // Shift, Ctrl, Alt, Pause, Caps Lock
    (k >= 44 && k <= 45);    // Print Screen, Insert
};

[mystix]

checking for keycode 46 is definitely more efficient

Ext.override(Ext.EventObjectImpl, function() {
    var safariKeys = {
        3 : 13, // enter
        63234 : 37, // left
        63235 : 39, // right
        63232 : 38, // up
        63233 : 40, // down
        63276 : 33, // page up
        63277 : 34, // page down
        63272 : 46, // delete
        63273 : 36, // home
        63275 : 35  // end
    };

    return {
        normalizeKey : function(k) {
            return Ext.isSafari ? (safariKeys[k] || k) : k;
        },

        isSpecialKey : function(){
            var k = this.normalizeKey(this.keyCode);
            return (this.type == 'keypress' && this.ctrlKey) ||
            this.isNavKeyPress() ||
            (k == this.BACKSPACE) || // Backspace
            (k >= 16 && k <= 20) || // Shift, Ctrl, Alt, Pause, Caps Lock
            (k >= 44 && k <= 46);   // Print Screen, Insert, Delete
        }
    }
}());

(look ma, no change in filesize!)

[kairinsama]

Using the Ext.override() instead of overwriting the prototype.isSpecialKey function yields the following result on the key events in a masked field:

// this.normalizeKey is not a function

var k = this.normalizeKey(this.keyCode);\n

[mystix]

hmmm.. odd... try the updated override.

[kairinsama]

Appears to function correctly.

Results, however, in the following "warnings" in Firefox (3.5.3)

The 'charCode' property of a keydown event should not be used. The value is meaningless.
The 'charCode' property of a keyup event should not be used. The value is meaningless.

[mystix]

Appears to function correctly.

Results, however, in the following "warnings" in Firefox (3.5.3)

The 'charCode' property of a keydown event should not be used. The value is meaningless.
The 'charCode' property of a keyup event should not be used. The value is meaningless.

you can ignore those for now. isSpecialKey() was designed to work on either the keydown / keypress / keyup event.

[CableDawg]

Has this fix been committed for the next release?

[mystix]

it's definitely not in yet - i inspected the latest 3.x (and 2.x) codebase from SVN and it still says

// ... [ SNIP ] ...

(k >= 44 && k <= 45);   // Print Screen, Insert


// ... [ SNIP ] ...

[Jamie Avins]

This is in 2.x and 3.2.x SVN. Due the nature of this change, it will not be in 3.1.x.