[wp-trac] [WordPress Trac] #10653: Update comment_author when display_name changes
WordPress Trac
wp-trac at lists.automattic.com
Tue Dec 8 20:15:40 UTC 2009
#10653: Update comment_author when display_name changes
----------------------+-----------------------------------------------------
Reporter: mptre | Type: enhancement
Status: closed | Priority: normal
Milestone: | Component: Comments
Version: | Severity: normal
Resolution: wontfix | Keywords: has-patch close
----------------------+-----------------------------------------------------
Comment(by apljdi):
Replying to [comment:5 Denis-de-Bernardy]:
These objections seem mostly inapplicable here.
> http://core.trac.wordpress.org/ticket/5787#comment:3
>
> potential problems:
>
> - Previously approved comments are changed without admin
notification/approval.
In what way? mpretty's patch doesn't update anything it checks for an
existing user and uses the display_name if a user is found. Nothing is
changed.
> - Discussions on a post are often are in the context of "@display_name"
This was my concern but it is not an issue if changes are flagged as I
suggested.
> - An upset or spammy person could abuse it.
This is also not an issue if changes are flagged as I suggested.
Actually, the current setup is prone to spammy abuse. I can comment under
one name. Change my display_name and comment again. Because of the
'nickname' field I can do this as many times as I like. This wouldn't be
addressed by anything suggested on this thread but it does make the
'spammy abuse' objection ring a little hollow.
Part of the issue here an underlying problem with the display_name
feature-- it is inconsistently applied. Leaving things alone perpetuates
that inconsistency. That is a mistake.
> = Isn't compatible with WP Mu.
Valid point. I don't WPMU enough to know what it would take to make it
compatible.
Suggesting this be re-opened and solved, possibly (preferably) rolling
that solution into a general solution, or set of solutions, to the
display_name inconsistency problem.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/10653#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list