You could run a simple check like !str_to_num() instead of having all of that ridiculous code. Also you do realise that || means or? Which means that your function would never reach the rest of the code as Backstabnoob said?
__________________
Quote:
vBulletin Tip #42: Not much would be accomplished by merging this item with itself.