On Input - Avoiding Infinite Loop

On Input - Avoiding Infinite Loop

I have a nice routine to validate a UK phone number, but I want to update the field with the reformatted number (spaces removed) as part of the routine, which just sends me in an 'on input' infinite loop ?

Any suggestions ?

Thanks

Neil

  2. // Set RegEx Variable Match Strings
  3. UK_Phone_RX = ("^(((\+44\s?\d{4}|\(?0\d{4}\)?)\s?\d{3}\s?\d{3})|((\+44\s?\d{3}|\(?0\d{3}\)?)\s?\d{3}\s?\d{4})|((\+44\s?\d{2}|\(?0\d{2}\)?)\s?\d{4}\s?\d{4}))(\s?\#(\d{4}|\d{3}))?$");
  4. UK_Mobile_RX = ("^(\+44\s?7\d{3}|\(?07\d{3}\)?)\s?\d{3}\s?\d{3}$");
  5. // Strip out ALL spaces from the number
  6. e = input.Your_Phone_Number.replaceAll(" ","",false);
  7. Error = 0;
  8. // Check if Valid UK Landline
  9. if (!e.matches(UK_Phone_RX))
  10. {
  11.     Error = 1;
  12. }
  13. // Check if Valid UK Mobile - only check if it failed the Landline test 
  14. if (Error  =  1)
  15. {
  16.     if (!e.matches(UK_Mobile_RX))
  17.     {
  18.         Error = 1;
  19.     }
  20.     else
  21.     {
  22.         Error = 0;
  23.     }
  24. }
  25. // Alert if number not valid
  26. if (Error  =  1)
  27. {
  28.     alert("Sorry " + e + " does not appear to be a valid phone number");
  29. }
  30. else
  31. {
  32.     input.Valid_Number = true;
  33. }