Changeset 7888

Timestamp:

11/23/2018 01:10:37 AM (7 years ago)

Author:

iandunn

Message:

WordCamp Post Site: Return early to avoid PHP notice.

We don't create sites for meetups, so this function should only run for the WordCamp post type.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-wordcamp/wordcamp-new-site.php (modified) (2 diffs)

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

@@ -116 +116 @@
      */
     public function maybe_create_new_site( $wordcamp_id ) {
+        $wordcamp = get_post( $wordcamp_id );
+
+        if ( ! $wordcamp instanceof WP_Post || WCPT_POST_TYPE_ID !== $wordcamp->post_type ) {
+            return;
+        }
+
         /*
          * If this were to be called again before it had finished -- e.g., when `WCORMailer::replace_placeholders()`
@@ -155 +161 @@
 
         $path              = isset( $url_components['path'] ) ? $url_components['path'] : '';
-        $wordcamp          = get_post( $wordcamp_id );
         $wordcamp_meta     = get_post_custom( $wordcamp_id );
         $lead_organizer    = $this->get_user_or_current_user( $wordcamp_meta['WordPress.org Username'][0] );