Any reason this is not inside call_user_func?

There's quite a few global variables that get declared in these controllers and they have quite common variable names. Do they need to be global?  Could lead to some hard to debug issues

If not, I can put a pull request in that will wrap it in a call_user_func() to get it out of the global scope.

call_user_func(function() {
...<controller code>
});