Removed some todo's by doing ?

This is the result of hitting a few, like a handful, of the todos (we haz like >300 !) 

Some were simply checked and removed since thats what the todo was, others led to a little code shuffling, or just verifying some things as you follow the todo rabbit hole :smiley_cat: 

So anyway a mixed bag here, one interesting find was the wrong array key was used in  PersonalMessage.controller in the subaction array, it had permissions and it should be permission, I think that was one of mine :innocent:

FWIW we may think about some various todo style things to use ... like @ enhancement, @ fixme, @ check, @  todo ... just something to think about and easy for most ide's to deal with.  Maybe there is a standard list of them that is common to use, or maybe I'm just tired now which is causing be to be overly anal !