Changeset 2741

Timestamp:

03/11/2016 11:25:16 PM (10 years ago)

Author:

iandunn

Message:

WordCamp Participation Notifier: Log errors instead of triggering them.

Triggering them breaks the POST/Redirect/GET process in development environments, because it prints the error.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php

@@ -368 +368 @@
                                 );
                         } elseif ( 200 != $response['response']['code'] || 1 != (int) $response['body'] ) {
-                                // trigger_error() has a message limit of 1024 bytes, so we truncate $response['body'] to make sure that $body doesn't get truncated.
+                                // error_log() has a message limit of 1024 bytes, so we truncate $response['body'] to make sure that $body doesn't get truncated.
                                 
                                 $error = sprintf(
@@ -379 +379 @@
                         
                         if ( $error ) {
-                                trigger_error( sprintf( '%s error for %s: %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ), sanitize_text_field( $error ) ), E_USER_WARNING );
+                                error_log( sprintf( '%s error for %s: %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ), sanitize_text_field( $error ) ), E_USER_WARNING );
                                 
                                 if ( $to = apply_filters( 'wpn_error_email_addresses', array() ) ) {