Unfortunately, that only sets $file to the second argument passed if $self->{file} isn't already set (to a true value, but I assume any filename passed will evaluate to true), which doesn't seem to be what princepawn wants (which is to set $file to be the second value passed if passed, or to be the value of $self->{file} otherwise).