! the wrong replacement value was being used for the message body
! some minor refactoring

Signed-off-by: Spuds <spuds@spudsdesign.com>
! remove some overuse of superG's

Signed-off-by: Spuds <spuds@spudsdesign.com>
! prevent array to string conversion error in news template

Signed-off-by: Spuds <spuds@spudsdesign.com>
! this todo was taken care of in the getAttachmentFromTopic function

Signed-off-by: Spuds <spuds@spudsdesign.com>
! old todo that does not mean a thing anymore

Signed-off-by: Spuds <spuds@spudsdesign.com>
! per the todo, log the error, if we find it triggers a lot it can be removed
! remove commented out code
! remove error suppression on var check

Signed-off-by: Spuds <spuds@spudsdesign.com>
! this todo was discussed on git

Signed-off-by: Spuds <spuds@spudsdesign.com>
! that was some 2.0 or before hack that should have been handled in the template

Signed-off-by: Spuds <spuds@spudsdesign.com>
! that is the correct fix, so the todo is done

Signed-off-by: Spuds <spuds@spudsdesign.com>
! tested imagick on a variety of images, no issues found

Signed-off-by: Spuds <spuds@spudsdesign.com>
! per the todo, convert this over to our standard subAction setup

Signed-off-by: Spuds <spuds@spudsdesign.com>
! unnecessary br in the template

Signed-off-by: Spuds <spuds@spudsdesign.com>
! wrong class used on the link button

Signed-off-by: Spuds <spuds@spudsdesign.com>
! improve the error text just a bit

Signed-off-by: Spuds <spuds@spudsdesign.com>
! use errorbox on our fatal error screen

Signed-off-by: Spuds <spuds@spudsdesign.com>
! bounce guests to the login screen instead of just a fatal error

Signed-off-by: Spuds <spuds@spudsdesign.com>
! move docblock, pedantic change

Signed-off-by: Spuds <spuds@spudsdesign.com>
! the depreciated function did not even work, so now it does :P

Signed-off-by: Spuds <spuds@spudsdesign.com>
Merge pull request #1037 from Spuds/notific

Culling the todo list a bit