[patch] Small performance optimization, don't assign values to $states array in a loop

When building the menus for clients a local variable called $states which holds all states, plus all, all open, my open is assigned value in a while loop which is not necessary (statuses are global, not per individual client). Some variable values are fetched from the DB which causes unnecessary reads with lots of clients.

The same variable is assigned the same value (except the my open state) shortly after to build the menu for "ticket's assigned to user X".

Attached is patch against CVS HEAD which assigns the value of this variable only once thus slightly improving performance.