Last modified: 2011-03-13 18:05:59 UTC
the 'summary=' variable doesn't at the moment summary expansion of variables ($1 and $2 atm) for rollbacks.
Also, some more variables should be available imo:
$3: number of revisions reverted
$4: revid reverted to
$5: timestamp for revid reverted to
$6: revid reverted from
$7: timestamp for revid reverted from
$8: number of bytes reverted (±)
Surely the user can fill these in themselves. Why is this needed? We don't have expansion of variables in edit summaries, or anywhere else that's analogous to this.
(In reply to comment #1)
> Surely the user can fill these in themselves. Why is this needed? We don't
> have expansion of variables in edit summaries, or anywhere else that's
> analogous to this.
There is variable expansion in the default rollback summary. Why I ask for this, is that you probably havn't gathered information to what revision you are reverting to.
Though, $8 isn't necessary.
The default rollback summary is a system message. System messages need to have parameters, because they are not otherwise modifiable on a case-by-case basis. Custom messages of any kind do not need programmatic variables that can be obtained easily from other places. If a bot really thinks it's important to put in its custom message how many bytes were removed or whatever, it can calculate that itself. The requested feature would be completely pointless for anyone not using a script or other customization of some kind (since by default you can't add a rollback reason); anyone using a script or other customization would probably have no use for most of the requested variables; and even if they did (for instance $2, which might not be immediately available in all cases), they could easily get the variables themselves. Therefore this simply does not justify the required lines of code, in terms of code bloat, and does not justify breaking our practice in every other action summary of this type of printing the reason without any substitution of parameters.
I wasn't the one who closed this originally, but I'm reclosing it WONTFIX.