I'm designing a javascript clock, but for some reasons it doesn't work in IE


WORKS
http://www.fadetolife.be/Test/a.php
http://www.fadetolife.be/Test/a.php.txt

WORKS NOT
http://www.fadetolife.be/Test/b.php
http://www.fadetolife.be/Test/b.php.txt

Could someone tell me why this doesn't work? The only difference between the 2 pages are the last 4 lines... The rest is identical!

This should be in a JavaScript forum dude, you might get more response there.

But isn't it about the bloody thing working in a PHP function or not? Because the JS code works fine (at http://www.fadetolife.be/Test/a.php at least)

Oh dude i'm sorry it's me being a fool! Wasnt reading it properly. I think the problem is here:

A.

PHP Code:

if (document.layers){
                lyr=document.layers[''+ID+''].document;
                  lyr.open();
                  lyr.write(WriteThis);
                  lyr.close();
            } else if (document.all){
                document.all[''+ID+''].innerHTML = WriteThis;
               } else if (document.getElementById){
                range = document.createRange();
                element = document.getElementById(''+ID+'');
                range.setStartBefore(element);
                content = range.createContextualFragment(WriteThis)
                while(element.hasChildNodes()) element.removeChild(element.lastChild);
                element.appendChild(content);
               } 


B.

PHP Code:

   if (document.layers){
    lyr=document.layers[''+ID+''].document;
      lyr.open();
      lyr.write(WriteThis);
      lyr.close();
    }
   else if (document.all){
    document.all[''+ID+''].innerHTML = WriteThis;
   }    
   else if (document.getElementById){
    range = document.createRange();
    element = document.getElementById(''+ID+'');
    range.setStartBefore(element);
    content = range.createContextualFragment(WriteThis)
    while(element.hasChildNodes()) element.removeChild(element.lastChild);
    element.appendChild(content);
   }
   } 


In the B. part you need to more that last else if statement up so that it's next to the } of the previous else if.

Should sort it out

Excuse me, I don't quite got your explination right. Could you please specify?

Sure man, sorry im having a bit of an off day today.

Find this in your code:

PHP Code:

else if (document.all){ 
    document.all[''+ID+''].innerHTML = WriteThis; 
   }     
   else if (document.getElementById){ 
    range = document.createRange(); 


and change it to this:

PHP Code:

else if (document.all){ 
    document.all[''+ID+''].innerHTML = WriteThis; 
   } else if (document.getElementById){ 
    range = document.createRange(); 


The else if needed to be moved up a line, otherwise PHP won't understand it.

Frederic - I can't view your links, I get a 404. How does this relate to PHP? Are you using PHP to generate the javascript for the clock?