[wp-trac] [WordPress Trac] #12130: Add Donate URI plugin header

WordPress Trac wp-trac at lists.automattic.com
Sat Feb 6 20:39:12 UTC 2010


#12130: Add Donate URI plugin header
-------------------------+--------------------------------------------------
 Reporter:  Otto42       |       Owner:  westi                       
     Type:  enhancement  |      Status:  accepted                    
 Priority:  low          |   Milestone:  3.0                         
Component:  Plugins      |     Version:                              
 Severity:  trivial      |    Keywords:  has-patch, reporter-feedback
-------------------------+--------------------------------------------------

Comment(by chipbennett):

 Replying to [comment:18 otto42]:
 > Why so complicated, and for so little benefit? I mean, all that just to
 avoid putting one extra line in a file?
 >
 > The current code in core does not look at the readme file at all. Adding
 a parsing mechanism just to avoid having an extra line is overkill.
 >
 > Plus, having a separate link makes more sense for tracking reasons as
 well as for ppl who may not want links in certain locations. It's the
 simplest and most straightforward approach. A whole complicated set of
 rules is not necessary.

 hat rules, and how is it complicated?

 The vast majority of plugin authors would add PluginURI to their plugin
 header and be done with it.

 For your two use cases, you can a) add a separate Donate link to
 readme.txt, or b) add Donate link to readme.txt and leave the plugin
 header PluginURI undefined.

 Extend does all the rest, by looking at readme.txt and/or the plugin
 header.

 Of course, I'm with Westi's original suggestion: just parse readme.txt for
 the existing Donate link. I'm just trying to suggest the most simple
 compromise, that will be easiest/most convenient for the vast majority of
 plugin authors.

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


More information about the wp-trac mailing list