[wp-trac] [WordPress Trac] #12392: Move comment fields to meta data instead

WordPress Trac wp-trac at lists.automattic.com
Sun Jun 13 03:23:14 UTC 2010


#12392: Move comment fields to meta data instead
-------------------------+--------------------------------------------------
 Reporter:  Otto42       |       Owner:     
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.1
Component:  Comments     |     Version:     
 Severity:  normal       |    Keywords:     
-------------------------+--------------------------------------------------

Comment(by shidouhikari):

 comment_author_IP - I have a plugin that uses it to add country flags on
 author name

 comment_agent - I've seen sites that parse user-agent to add browser and
 OS info, I just didn't add this to my plugin because it's too much theme
 related and plugin already has needed code to help any theme that would
 wanna do that (I just didn't implemented on my theme because I couldn't
 find best place for it :D )

 comment_karma - I've read it was created to support a variety of voting
 features for comments and left unused by core so that plugins could use
 the field to implement their specific features, a field that remains
 unused so that plugins can use it without conflicting with other features'
 data - now that we have comment metadata it's pratically useless and
 plugins should create their own metadata tag for their features, remaining
 used only by legacy plugins (is there any really?)


 The problem of those 2 is that they ARE storing data, so it would remain
 taking database space. If you don't want them just develop a plugin to
 delete them upon comment creating and anywhere they could be being edited.
 The extra load to query them as metadata when they are used doesn't
 justify the little and imperceptable performance gain we'd get when they
 aren't.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/12392#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list