[Revan]

It's good practice to always init your arrays like that, and if you are foreach'ing variables you didn't have complete control over, you should either typecast like in your second fix, or check "if (!is_array($arr)) { $arr = array(); }"
Reason is, if a database returns no rows, it doesn't create any arrays you manipulate inside a while() result iterator.