so my PHP is operational.
The script I want to run is here:

<div style="">
 <table border="0" cellpadding="0" cellspacing="0" width="100%">
								<tr><td align=center><script language="JavaScript1.2">

/*
Left-Right image slideshow Script-
By Dynamic Drive (www.dynamicdrive.com)
For full source code, terms of use, and 100's more scripts, visit http://dynamicdrive.com
*/

///////configure the below four variables to change the style of the slider///////
//set the scrollerwidth and scrollerheight to the width/height of the LARGEST image in your slideshow!
var scrollerwidth='600px';
var scrollerheight='606px';
var scrollerbgcolor='white';
//3000 miliseconds=3 seconds//
var pausebetweenimages=3000;


//configure the below variable to change the images used in the slideshow. If you wish the images to be clickable, simply wrap the images with the appropriate <a> tag
var slideimages=new Array() slideimages[0]=<img src="../myadmin/stocklet/B3m2.jpg">;  slideimages[1]=<img src="../myadmin/stocklet/C3m3.jpg">;  slideimages[3]=<img src="../myadmin/stocklet/D3m4.jpg">; var ie=document.all;

var dom=document.getElementById;

if (slideimages.length>1)
i=2;
else
i=0;

function move1(whichlayer){
tlayer=eval(whichlayer);
if (tlayer.left>0&&tlayer.left<=5){
tlayer.left=0;
setTimeout("move1(tlayer)",pausebetweenimages);
setTimeout("move2(document.main.document.second)",pausebetweenimages);
return
}
if (tlayer.left>=tlayer.document.width*-1){
tlayer.left-=5;
setTimeout("move1(tlayer)",50);
}
else{
tlayer.left=parseInt(scrollerwidth)+5;
tlayer.document.write(slideimages[i]);
tlayer.document.close();
if (i==slideimages.length-1)
i=0;
else
i++;
}
}

function move2(whichlayer){
tlayer2=eval(whichlayer);
if (tlayer2.left>0&&tlayer2.left<=5){
tlayer2.left=0;
setTimeout("move2(tlayer2)",pausebetweenimages);
setTimeout("move1(document.main.document.first)",pausebetweenimages);
return
}
if (tlayer2.left>=tlayer2.document.width*-1){
tlayer2.left-=5;
setTimeout("move2(tlayer2)",50);
}
else{
tlayer2.left=parseInt(scrollerwidth)+5;
tlayer2.document.write(slideimages[i]);
tlayer2.document.close();
if (i==slideimages.length-1)
i=0;
else
i++;
}
}

function move3(whichdiv){
tdiv=eval(whichdiv);
if (parseInt(tdiv.style.left)>0&&parseInt(tdiv.style.left)<=5){
tdiv.style.left=0+"px";
setTimeout("move3(tdiv)",pausebetweenimages);
setTimeout("move4(scrollerdiv2)",pausebetweenimages);
return
}
if (parseInt(tdiv.style.left)>=tdiv.offsetWidth*-1){
tdiv.style.left=parseInt(tdiv.style.left)-5+"px";
setTimeout("move3(tdiv)",50);
}
else{
tdiv.style.left=scrollerwidth;
tdiv.innerHTML=slideimages[i];
if (i==slideimages.length-1)
i=0;
else
i++;
}
}

function move4(whichdiv){
tdiv2=eval(whichdiv);
if (parseInt(tdiv2.style.left)>0&&parseInt(tdiv2.style.left)<=5){
tdiv2.style.left=0+"px";
setTimeout("move4(tdiv2)",pausebetweenimages);
setTimeout("move3(scrollerdiv1)",pausebetweenimages);
return
}
if (parseInt(tdiv2.style.left)>=tdiv2.offsetWidth*-1){
tdiv2.style.left=parseInt(tdiv2.style.left)-5+"px";
setTimeout("move4(scrollerdiv2)",50);
}
else{
tdiv2.style.left=scrollerwidth;
tdiv2.innerHTML=slideimages[i];
if (i==slideimages.length-1)
i=0;
else
i++;
}
}

function startscroll(){
if (ie||dom){
scrollerdiv1=ie? first2 : document.getElementById("first2");
scrollerdiv2=ie? second2 : document.getElementById("second2");
move3(scrollerdiv1);
scrollerdiv2.style.left=scrollerwidth;
}
else if (document.layers){
document.main.visibility='show';
move1(document.main.document.first);
document.main.document.second.left=parseInt(scrollerwidth)+5;
document.main.document.second.visibility='show';
}
}

window.onload=startscroll;

</script><ilayer id="main" width=&{scrollerwidth}; height=&{scrollerheight}; bgColor=&{scrollerbgcolor}; visibility=hide>;
<layer id="first" left=1 top=0 width=&{scrollerwidth}; >
<script language="JavaScript1.2">
if (document.layers)
document.write(slideimages[0]);
</script>
</layer>
<layer id="second" left=0 top=0 width=&{scrollerwidth}; visibility=hide>
<script language="JavaScript1.2">
if (document.layers)
document.write(slideimages[1]);
</script>
</layer>
</ilayer>

<script language="JavaScript1.2">
if (ie||dom){
document.writeln('<div id="main2" style="position:relative;width:'+scrollerwidth+';height:'+scrollerheight+';overflow:hidden;background-color:'+scrollerbgcolor+'">');
document.writeln('<div style="position:absolute;width:'+scrollerwidth+';height:'+scrollerheight+';clip:rect(0 '+scrollerwidth+' '+scrollerheight+' 0);left:0px;top:0px">');
document.writeln('<div id="first2" style="position:absolute;width:'+scrollerwidth+';left:1px;top:0px;">');
document.write(slideimages[0]);
document.writeln('</div>');
document.writeln('<div id="second2" style="position:absolute;width:'+scrollerwidth+';left:0px;top:0px">');
document.write(slideimages[1]);
document.writeln('</div>');
document.writeln('</div>');
document.writeln('</div>');
}
</script></td></tr></table>

I am sure I am missing something here the sytax Webpage error details


Message: Expected ';'
Line: 150
Char: 29
Code: 0


Message: 'ie' is undefined
Line: 282
Char: 1
Code: 0

I cannot see a missing ; and I am not sure why it thinks ie is undefined. I am a php developer so this is newby territory for me

Affee,

It's this line :

var slideimages=new Array() slideimages[0]=<img src="../myadmin/stocklet/B3m2.jpg">;  slideimages[1]=<img src="../myadmin/stocklet/C3m3.jpg">;  slideimages[3]=<img src="../myadmin/stocklet/D3m4.jpg">; var ie=document.all;

Change to :

var slideimages=new Array();
slideimages[0]='<img src="../myadmin/stocklet/B3m2.jpg">';
slideimages[1]='<img src="../myadmin/stocklet/C3m3.jpg">';
slideimages[3]='<img src="../myadmin/stocklet/D3m4.jpg">';
var ie=document.all;

And I expect [3] should be [2] . If so then you can simplify to :

var slideimages = [
  '<img src="../myadmin/stocklet/B3m2.jpg">',
  '<img src="../myadmin/stocklet/C3m3.jpg">',
  '<img src="../myadmin/stocklet/D3m4.jpg">
];
var ie=document.all;

This will cure both JS errors on page load.

All the blocks that read :

if (i==slideimages.length-1)
i=0;
else
i++;
}

can be simplified to :

i = (i==slideimages.length-1) ? 0 : i+1;
//or
i = (++i)%slideimages.length;

Airshow

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.