Update: I thought it was precedence, but now I think it's a function argument vs precednce problem. Solution should still work. Try:

if((not defined $username) || ($username eq ""))
[download]

Just guessing, but I think perl is seeing:

if(not defined($username || ($username eq "")))
[download]

It is usually a good idea to explicitly put brackets around everything you want to logically group. Old programmers are religious about this because each language has a different set of operator precedences. After you program in a few, they all mush together and you have no chance of remembering which ones work in the language you are currently programming in.

____________________
Jeremy
I didn't believe in evil until I dated it.