[wp-trac] [WordPress Trac] #41217: WP_Community_Events Disable Event Logs

WordPress Trac noreply at wordpress.org
Thu Jul 27 13:39:24 UTC 2017


#41217: WP_Community_Events Disable Event Logs
-------------------------+-----------------------------
 Reporter:  Howdy_McGee  |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  4.9
Component:  Widgets      |     Version:  4.8
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:  administration
-------------------------+-----------------------------

Comment (by charlestonsw):

 My 2-cents -

 You're proposing adding another filter which is a lot of extra overhead.
 You're adding data to the memory stack and forcing WP Core to pull through
 the filter list and run callbacks.

 IMO it would be a better solution overall to force the Community Events
 developers to add a define to wp-config.php if they DO want to execute
 this logging.   This seems like a far simpler solution and one that is
 specific to the class at hand versus hijacking the default DEBUG_LOG
 behavior for a corner case.

                 if ( ! WP_COMMUNITY_EVENTS_LOG ) {
                         return;
                 }

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41217#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list