Array prototype extensions breaks plugin code that uses "for(var i in <Array object>)"

[Paul Vrugt]

Instead of creating a new array object, or expanding the static Array object, the fckeditor messes with the standard array prototype. This results in the following problem:

suppose i do this in a plugin:

var objArray = new Array();
objArray['key1'] = 'value1';
objArray['key2'] = 'value2';

for(var strKey in objArray)

alert(objArray[strKey]);

i wanted to get all values from the array, but instead, i get all values + 2 function objects, since the fckEditor added AddItem and IndexOf to the array prototype.

[Frederico Caldeira Knabben]

You are wrongly using an array in your sample. You should use and Object instead:

var objArray = new Object();
objArray['key1'] = 'value1';
objArray['key2'] = 'value2';

for(var strKey in objArray)
    alert(objArray[strKey]);

In this case you will not have problems.

In any case, it is already ​part of our plans to avoid touching native objects.