Feature(s) - Editpost, approvepost, deletepost
The code for these 3 functions were spread out over too many files to reliably manage. I've now made all the editors and positions of the buttons consistent.
A ton of code cleanup was done ( cleaned out a ton of obsolete code ). Security holes were fixed and common functions ( validation / permissioning ) were used to ensure consistency and non-corrupt databases.
The number of things that could potentially go wrong are too long to list here, but we now have a more solid foundation.
To take just edit post for an example: the total code size in 756 was 29K.. now with the combined / cleaner code, it is 17K and does fewer unnecessary queries as well.
Smaller things, like LOCKING THE TOPIC that is about to be deleted have also been added. Before, there was a chance that a use could be replying to a topic that was about to go gonzo and produce a post that was an orphan.
( the same idea / cleanup has been applied to deleting a user -- 756 leaves stray unhooked PMs from users -- spammers -- who get deleted ) This happens on this forum all the time and is very annoying!
Anyway, for many forums, there is a desire to only make the topic subject be the subject for all replies after that. Some people change the subject and especially in flat mode, it makes no sense at all.
There is now a way to configure the board to disallow this:
When a user edits any reply, they just see the subject and cannot change it.
All of the save buttons go thru a common fadeout message as well..
I really should post another 10 screens here to show all the possibilities, but this should just be considered a template for all editors now with the buttons right below the editor.
Additionally all templates are chock full of CSS goodness for the stylers to go crazy with..