[wp-trac] [WordPress Trac] #11426: Allow trash option in comment notification emails
WordPress Trac
wp-trac at lists.automattic.com
Tue Dec 15 04:10:58 UTC 2009
#11426: Allow trash option in comment notification emails
--------------------------+-------------------------------------------------
Reporter: nacin | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 2.9
Component: Trash | Version: 2.9
Severity: normal | Keywords: has-patch
--------------------------+-------------------------------------------------
Comment(by nacin):
Replying to [comment:11 caesarsgrunt]:
> Regarding the names of the actions, I don't see any point in renaming
them at all if the new names are still going to be different from the ones
used in other places. There's nothing really ''wrong'' with {{{cdc}}} and
{{{mac}}} - and if it's shortness you're going for, you can't beat them!
(Unless you were to have like {{{t}}} for trash, {{{s}}} for spam, and
{{{a}}} for approve... actually, there might be something to be said for
that...)
Here's my thought. It really isn't much to ask that the links should be
short, readable, and distinguishable. All three options are short enough.
However, I didn't have a clue what 'mac' or 'cdc' was until I looked back
in the changesets when they were shortened 3 years ago. They're not
understandable to an end user, but why shouldn't they be?
The one-character strings could work, but they're not readable or easily
distinguishable.
The point of renaming them is simple. We're already creating two new
actions, spam and trash, and getting rid of dt. So we might as well
standardize 'approve' and 'delete' as well. They're only different because
they perform a unique AYS function. And FWIW, adding "comment" to the
action generates the post-AYS action, so this standardizes it and makes
them alike, not different. And because of that, the code is easier, too.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/11426#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list