OeufMayo is exactly correct, but just to add a little detail regarding your logic: here's your code rearranged into a more traditional format:

if ($in{username} =~ /^(\[a-zA-Z]{2})/ || ' ') {
     $dirtype = $1 || 'other';
}
[download]

First of all, the "or" in the "if" statement will never trigger because "if" always tests for "truth." In other words, the "if" is saying "if the regular expression is true OR the string " " is true." Unfortunately, " "is NEVER true in Perl, so if your regex doesn't match the if is skipped entirely.

Second of all, IF the regular expression DOES match (triggering the code between the brackets and populating $1 at the same time) THEN in your case $1 is always guaranteed to be true (because it will contain two A-Za-z letters. Letters always evaluate to "true." Because the first expression in your short-circuit operation ($1) is always true, your program will never get to the 'other' value.

This is why OeufMayo resorted to a ternary operator, which is just a short-hand way to write another if/then statement. Here it is rewritten for clarity:

my $dirtype;
if ( $in{username} =~ /^([a-zA-Z]{2})$/ ) {
     $dirtype = $1;
} else {
      $dirtype = 'other';
}
[download]

This way the logic is all straigthened out.

And then there's the backslash in your regex. :-D

Gary Blackburn
Trained Killer