Changeset 5512

Timestamp:

05/24/2017 10:50:47 PM (7 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_added_to_pending_schedule' => array(
                 'name'     => 'WordCamp added to pending schedule',
@@ -629 +641 @@
 
     /**
+
+
+
+
+
+
+
+
+
+
+
      * Sends e-mails hooked to the wcor_added_to_pending_schedule trigger.
      *