Changeset 11563 for sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-emails.php

Timestamp:

02/16/2022 05:57:57 AM (3 years ago)

Author:

dd32

Message:

Support Forums: Fix a logic order reversal in [11562] (Didn't affect email notifications), avoid some PHP Notices.

See #5505.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-emails.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-emails.php

@@ -58 +58 @@
         }
 
+        $no_reply    = apply_filters( 'bbp_subscription_to_email', bbp_get_do_not_reply_address() );
+        $dest_emails = is_array( $attrs['to'] ) ? $attrs['to'] : explode( ',', $attrs['to'] );
+
         // Pull out CC's and BCC's to a separate array.
         foreach ( $attrs['headers'] as $i => $header ) {
@@ -73 +76 @@
         }
 
-        $no_reply    = apply_filters( 'bbp_subscription_to_email', bbp_get_do_not_reply_address() );
-        $dest_emails = is_array( $attrs['to'] ) ? $attrs['to'] : explode( ',', $attrs['to'] );
         // Remove the noreply email, it was probably set as the $to address.
         $dest_emails = array_diff( $dest_emails, [ $no_reply ] );
-    
+
         // This shouldn't happen, but if it does, let wp_mail() do it's thing instead.
         // NOTE: We still process if there was only one to/cc/bcc address, to ensure the below custom filter gets run.