Culling the todo list a bit

Started to go through the todo's in the code, this is the result along with some collateral changes.

@emaneule45 https://github.com/Spuds/Elkarte/commit/7ac3b4406d6b7569b7230bce48eb4b825332c611 I think the only permission you can give a guest is profile_view_any so if they don't have that, we bounce them early (and to the log in screen) .. I think that fixes the IRC todo ?

fixes #1015 

fixed the travis issue, it was due to the sa buddy being replaced with addbuddy and removebuddy, so the test script was calling an invalid sa=