[RESOLVED] Why does IE not like this java

[Dawg]

I am using a java script to rotate some ads. FF likes it just fine but IE does not seem to like line 66. I have highlighted it in red.

// Java Document
///////////////////////////////////////////////////////////
/// Magic Image Rotation ///
/// v 1.0.1 ///
/// Copyright 2007, Loyce Bradley Petrey ///
/// All Rights Reserved. ///
/// http://www.EchoChrist.com/MagicImage ///
/// [e-mail] ///
/// ///
/// This script is free to use as long as this notice ///
/// remains unchanged and intact. ///
/// ///
/// This program is free software: you can redistribute ///
/// it and/or modify it under the terms of the GNU ///
/// General Public License as published by the Free ///
/// Software Foundation. ///
/// ///
/// This program is distributed in the hope that it ///
/// will be useful, but WITHOUT ANY WARRANTY; without ///
/// even the implied warranty of MERCHANTABILITY or ///
/// FITNESS FOR A PARTICULAR PURPOSE. ///
/// ///
/// Be Imitators of God as dear children and walk ///
/// in love as Christ also loved us. ///
/// Ephesians 5:1-2 ///
///////////////////////////////////////////////////////////

var ImageLoad =
[
	['http://www.mauriprosailing.com/?gad=CJ-9pI4CEgi0_U8E8POhnhj79dL-AyCf1a8R','http://www.sailinganarchy.com/ADs/mauri pro/index_center.jpg',''],
	['http://www.predictwind.com/','http://www.sailinganarchy.com/ADs/predict_wind/index_center_predictwind.jpg','']
];
/*
['URL to linked page', 'URL to image', 'Caption under picture']
The caption is not required and may be left blank like this: ['URL to linked page', 'URL to image', '']
Add as many images as you like seperated by commmas
Almost ALL errors are caused by the url or path being wrong
The LAST image declaration does NOT have a comma after it
*/


var ImageCount = 2; // ***** Change this to the total number of images loaded above ***** //
var ImageDelay = 5000; // ***** Set this to the delay interval desired. 5000 = 5 seconds. //
var LinkTarget = "_blank" // ***** Defines where you want linked page to open. _self, _blank, _top, etc //
var ImageIndex = 0; // DO NOT ALTER //
var FirstLoad = 0; // DO NOT ALTER //
var QuickStartID = 0; // DO NOT ALTER //
var htmlString = "" // DO NOT ALTER //


// This function rotates the banner //
function ImageChange()
{

if (FirstLoad == 0) // Determins if this is the first time function has run. //
{
PickFirstRun();
}

htmlString = '<center>';
htmlString = htmlString + '<font face = "Verdana" size="2">'; // Font and Font Size for caption may be changed here //
htmlString = htmlString +'<a target="';
htmlString = htmlString + LinkTarget;
htmlString = htmlString + '" href="';
htmlString = htmlString + ImageLoad[ImageIndex][0];
[COLOR="Red"]htmlString = htmlString + '"><img border="0" src="'; // Image border size may be changed here //[/COLOR]
htmlString = htmlString + ImageLoad[ImageIndex][1];
htmlString = htmlString + '"></a><br>';
htmlString = htmlString + ImageLoad[ImageIndex][2];
htmlString = htmlString + '</font>';
htmlString = htmlString + '</center>';

document.getElementById('MagicImage').innerHTML = htmlString;

if (ImageIndex == ImageCount - 1) // This statement increments image displayed and resets if displaying last image //
{
ImageIndex= 0;
}
else
{
ImageIndex++;
}

if (FirstLoad == 0) // Determins if this is the first time function has run. //
{
SlowFinish();
}

}
// End Funtion //


// On first display select a random image of above array, then proceed in
// the order defined in above array

function PickFirstRun()
{
var first = Math.random() * ImageCount;
first = Math.round(first);
// alert (first); // Uncomment for testing (displays random ID)
ImageIndex = first;
}

// End function


// This function ensures first banner is displayted without a delay //
function QuickStart()
{
QuickStartID=setInterval("ImageChange()", 1000);
}
// End Funtion //

// This function sets display rate to user defined speed //
function SlowFinish()
{
clearInterval(QuickStartID);
FirstLoad = 1;
setInterval("ImageChange()", ImageDelay);
}
// End Funtion //

QuickStart();

Can anyone help with dealing with IE. This site has always been a great source of information.

Thanks
Dave

[Krik]

Have you tried it like this

htmlString = '<center>' + 
'<font face="Verdana" size="2">' +
'<a target="' + LinkTarget + '" href="' + ImageLoad[ImageIndex][0] + '"><img border="0" src="' + ImageLoad[ImageIndex][1] + '"></a><br>' +
ImageLoad[ImageIndex][2]+ 
'</font>' +
'</center>';

[Dawg]

Thanks for the help. A friend who is C teacher found something that was causeing the problem.

He suggested changing the line

first = Math.round(first);
to
first = Math.floor(first);