Changeset 3139326

Timestamp:

08/21/2024 10:58:34 PM (19 months ago)

Author:

mdawaffe

Message:

Shift the check for whether the feature is enabled into the post_updated hook rather than the constructor, which fires on init.

Helpful for multisites that may call switch_to_blog() after init.

File:

• email-post-changes/trunk/class.email-post-changes.php (modified) (2 diffs)

email-post-changes/trunk/class.email-post-changes.php

@@ -33 +33 @@
         ) );
 
-        $options = $this->get_options();
-
-        if ( $options['enable'] ) {
-            add_action( 'post_updated', array( $this, 'post_updated' ), 10, 3 );
-            add_action( 'epc_new_bbpress_item', array( $this, 'post_updated' ), 10, 3 );  // Support for bbPress 2
-        }
+        add_action( 'post_updated', array( $this, 'post_updated' ), 10, 3 );
+        add_action( 'epc_new_bbpress_item', array( $this, 'post_updated' ), 10, 3 );  // Support for bbPress 2
 
         if ( current_user_can( 'manage_options' ) ) {
@@ -73 +69 @@
 
         $options = $this->get_options();
+        if ( ! $options['enable'] ) {
+            return;
+        }
 
         // Transitioning from an Auto Draft to Published shouldn't result in a notification.