[Tickets #8248] Undefined Object Line 79 Mailbox.php IE7

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8248
------------------------------------------------------------------------------
  Ticket             | 8248
  Created By         | jacob at mediashaker.com
  Summary            | Undefined Object Line 79 Mailbox.php IE7
  Queue              | IMP
  Version            | 4.1.6
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


jacob at mediashaker.com (2009-05-06 15:43) wrote:

IE 7 Bug. Occurs near setKeyBindings inside of javascript.inc and the  
corresponding function inside of keybindings.js.

I noticed inside of keybindings.js that you're using an array to hash  
keycodes.

In IE 7 (and possibly other browsers) if you want to do hashing it's  
better to use an object because:

var a = [];
a[0] = 'foo';
a[100] = 'bar';

Will result in the array being populated with nulls from 0->100.

Of course, maybe you're modifying the array prototype or something  
like that and are working around this.

I tested the following change:

var _keyMap = new Array();

to

var _keyMap = {};

With improved results.