[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