Whenever I see this I usually see this right afterwards:

Where they should probably be using:

Which would make the || "" in the assignment unnecessary.

Yes, I understand sometimes zero and the empty string are distinct values from undef in many business rules, but I don't think that applies as aften as many coders believe.