Last modified: 2013-12-05 21:39:46 UTC
IIRC, moderation notes should be saved in (a probably new column in) the revision table.
Define "moderation"? Standard for the moment is for rollback to be stored in rev_comment - an existing field - while revisiondelete and suppress are stored in logging. I'd actually advocate moving "hide" into the logging, rather than the revision table, unless there are factors I'm unaware of.
moderation = hide/delete/suppress/restore We already add an entry to logging, when a post is suppressed/deleted (not sure why hiding should not log either, or what log it should be in; perhaps file another bug if you believe it should write to log too?) This is more of a code thing. We currently "only" save the moderation reason to the log (which, for hiding, currently equals not at all). We'll need to have that information available (we'll probably need to display it in some places, why ask it otherwise) in the Flow revisions table as well. The only way to get the moderation reason right now would be to query the logging table. Not only can't that be done in an effective way on a per-revision basis, it'd get even more troublesome if/once Flow is on its separate database.
Oh also see follow-up bug https://bugzilla.wikimedia.org/show_bug.cgi?id=57056, something we can't accomplish until we also save the moderation reason in Flow's tables.
Makes sense!
The WMF core features team tracks this bug on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/461, but people from the community are welcome to contribute here and in Gerrit.
Change 95985 had a related patch set uploaded by Werdna: [SCHEMA CHANGE] Refactor of moderation logging. https://gerrit.wikimedia.org/r/95985
Change 95985 merged by jenkins-bot: [SCHEMA CHANGE] Refactor of moderation logging. https://gerrit.wikimedia.org/r/95985
