Changeset 5512

Timestamp:

05/24/2017 10:50:47 PM (6 years ago)

Author:

coreymckrill

Message:

WordCamp: Add organizer reminder trigger for pre-planning status

Fixes #2346
Props nvwd

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins

Files:

• wcpt/wcpt-wordcamp/wordcamp-admin.php (modified) (1 diff)
• wordcamp-organizer-reminders/wcor-mailer.php (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-admin.php

@@ -745 +745 @@
         }
 
-        if ( $old_status == 'wcpt-pre-planning' && $new_status == 'wcpt-pre-planning' ) {
-            // todo - bug: this won't ever run b/c of the early return above.
-            // what commit made this mistake, and why?
-            // what things are tied to this that haven't been running?
-            do_action( 'wcpt_added_to_planning_schedule', $post );
+        if ( 'wcpt-pre-planning' == $new_status ) {
+            do_action( 'wcpt_approved_for_pre_planning', $post );
         } elseif ( $old_status == 'wcpt-needs-schedule' && $new_status == 'wcpt-scheduled' ) {
             do_action( 'wcpt_added_to_final_schedule', $post );

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-organizer-reminders/wcor-mailer.php

@@ -13 +13 @@
     public function __construct() {
         $this->triggers = array(
+            'wcor_approved_for_pre_planning' => array(
+                'name'     => 'WordCamp approved for pre-planning',
+                'actions'  => array(
+                    array(
+                        'name'       => 'wcpt_approved_for_pre_planning',
+                        'callback'   => 'send_trigger_approved_for_pre_planning',
+                        'priority'   => 10,
+                        'parameters' => 1,
+                    ),
+                ),
+            ),
+
             'wcor_added_to_pending_schedule' => array(
                 'name'     => 'WordCamp added to pending schedule',
@@ -629 +641 @@
 
     /**
+     * Sends e-mails hooked to the wcor_approved_for_pre_planning trigger.
+     *
+     * This fires when a WordCamp has been approved for pre-planning.
+     *
+     * @param WP_Post $wordcamp
+     */
+    public function send_trigger_approved_for_pre_planning( $wordcamp ) {
+        $this->send_triggered_emails( $wordcamp, 'wcor_approved_for_pre_planning' );
+    }
+
+    /**
      * Sends e-mails hooked to the wcor_added_to_pending_schedule trigger.
      *