elkarte/Elkarte — pull request #1147 by Spuds (2013-12-11 04:45:41+00:00)
Title: 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 !