Help with annoying error...

[sciguyryan]

Hi evryone,

Its been quite a while since I last used Javascript so, I'm a bit rusty

I am using the following code here: http://s9.invisionfree.com/FS_Code_T...hp?act=Members

I am previewing the code in IE 6.0.

var MemberTable = document.getElementsByTagName("table")[2];
var MemberListTDs = MemberTable.getElementsByTagName("td");
var PageURL = window.location.href.split("?")[0]
for (var i = 0; i <= MemberListTDs.length; i++){
if ((window.location.href.indexOf("act=Members") > 0) && (MemberListTDs[i].innerHTML.indexOf("showuser=") > 0)){
var UserID = MemberListTDs[i].getElementsByTagName("a")[0].href.split("=")[1];
var NewText = ' (<a href="' + PageURL + '?act=Msg&CODE=04&MID=' + UserID + '">PM ' + MemberListTDs[i].getElementsByTagName("a")[0].innerText + '</a>)';
MemberListTDs[i].innerHTML += NewText;
}}

Now, I know the code works as it adds the text it is supposed to but I am getting an error: "MemberListTDs[i].innerHTML is null or not an object."


Does anyone know why this error is occuring even though the code works and a way to fix this?

Thanks for the help,

RyanJ

[Psyrus]

Try changing this line:

for (var i = 0; i <= MemberListTDs.length; i++)

to this:

for (var i = 0; i <= MemberListTDs.length - 1; i++)

You might be exceeding the size of the array/collection. If that doesn't work try:

for (var i = 1; i <= MemberListTDs.length; i++)

I can't remember if the count starts at 0 or 1.