This means if site_id is ever null, it would return in every query.  Is that what we want?

I agree with Jon on doing the null check first, I think it will show make your expectations clearer.