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

WordPress Trac wp-trac at lists.automattic.com
Sun Jun 25 15:49:57 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 masquerade):

 {{{                        if ( function_exists(gzinflate) ) { }}}
 Undefined constant.

 As for triggering errors, at this point, I believe it better that we
 should be using WP's built in error handling, which is there for this
 reason. I really don't understand why an error is being triggered at all,
 especially since changing the option to use gzip requires modifying a core
 file, which is a Bad Thing. If anything, we should silently fail and
 disable gzip, disable the checks to see if its enabled, and those with
 gzip will silently use it, those without it will silently not use it, no
 harm done either way.

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


More information about the wp-trac mailing list