[wp-trac] Re: [WordPress Trac] #2831: Gzip support in class-snoopy.php

WordPress Trac wp-trac at lists.automattic.com
Sun Jun 25 15:27:33 GMT 2006


#2831: Gzip support in class-snoopy.php
---------------------------------------+------------------------------------
 Reporter:  Phantagom                  |        Owner:  anonymous
     Type:  enhancement                |       Status:  new      
 Priority:  normal                     |    Milestone:           
Component:  General                    |      Version:  2.0.4    
 Severity:  normal                     |   Resolution:           
 Keywords:  RSS gzip snoopy has-patch  |  
---------------------------------------+------------------------------------
Comment (by random):

 masquerade, I don't understand the comment about undefined constants. The
 only one I see in the patch is E_USER_NOTICE, which is PHP core.

 Disagree about triggering the error, since reporting of E_NOTICE level
 errors is disabled by default (and not intended for production
 environments anyway). IIRC WordPress explicitly disables E_NOTICE, so the
 user will have already had to modify wp-settings.php in order to turn it
 on.

 +1 technical merits, -1 disapproval that it looks like the modifications
 have been taken without credit (comments and all) from Kellan Elliot-
 McCrea's MagpieRSS copy. Not good form.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/2831>
WordPress Trac <http://wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list