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

Timestamp:

01/15/2019 11:02:54 AM (6 years ago)

Author:

vedjain

Message:

WCPT: Do not send notifications when Old and New status are same

We would have already notified when Meetup or WordCamp application would have been accepted for first time. So no need to do it again.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-admin.php (modified) (1 diff)

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

@@ -725 +725 @@
             }
 
-            if ( 'wcpt-scheduled' === $new_status ) {
+            if ( 'wcpt-scheduled' === $new_status && 'wcpt-scheduled' !== $old_status ) {
                 return $this->notify_new_wordcamp_in_slack( $wordcamp );
-            } elseif ( 'wcpt-rejected' === $new_status ) {
+            } elseif ( 'wcpt-rejected' === $new_status && 'wcpt-rejected' !== $old_status ) {
                 $location = get_post_meta( $wordcamp->ID, 'Location', true );
                 return $this->schedule_decline_notification( $wordcamp, 'WordCamp', $location );