JavaScript not working in Firefox

Question

Hi ' im having a problem with my script in firefox, i'm getting the fault , " window.event is undefined " , where seems to be the problem?

<script language="JavaScript1.2"> 

function formattel(){

 var value = document.formkkush.telefoon.value; 
 var valueLen = document.formkkush.telefoon.value.length; 

 if(valueLen>2 && valueLen<4) 
 { 
 if(window.event.keyCode==8) 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value.remove(valueLen-1,1); 
 else 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value+"/"; 
 }
 if(valueLen>5 && valueLen<7) 
 { 
 if(window.event.keyCode==8) 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value.remove(valueLen-1,1); 
 else 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value+"."; 
 }
 if(valueLen>8 && valueLen<10) 
 { 
 if(window.event.keyCode==8) 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value.remove(valueLen-1,1); 
 else 
 document.formkkush.telefoon.value=document.formkkush.telefoon.value+"."; 
 }
} 

Answer 1

The global event object isn't created in Firefox. It's instead passed as the first argument in the event handler's function.

function formattel(eventObj){ 
  var value = document.formkkush.telefoon.value;  
  var valueLen = document.formkkush.telefoon.value.length;  

  // Make sure we have the even object no matter which browser we're running on
  eventObj = eventObj || window.event;

  if(valueLen>2 && valueLen<4)  
  {  
    if(eventObj.keyCode==8)  
      document.formkkush.telefoon.value=document.formkkush.telefoon.value.remove(valueLen-1,1);  
    else  
      document.formkkush.telefoon.value=document.formkkush.telefoon.value+"/";  
  } 
  // etc...
}  

See the Mozilla documentation for event.

As a completely unrelated suggestion, you should try and lessen the number of DOM node lookups by setting repeated lookups to a variable. It makes your code faster and much more presentable:

function formattel(eventObj){ 

  var telefoon = document.formkkush.telefoon; 
  var value = telefoon.value;  
  var valueLen = value.length;  

  // Make sure we have the even object no matter which browser we're running on
  eventObj = eventObj || window.event;

  if(valueLen>2 && valueLen<4)  
  {  
    if(eventObj.keyCode==8)  
      telefoon.value = telefoon.value.remove(valueLen-1,1);  
    else  
      telefoon.value += "/";  
  } 
  // etc...
}  

Don't forget about assignment operators (such as +=) too!

Answer 2

Seems like many faced this issue. Did you check this post?