Changeset 1683

Timestamp:

06/22/2015 02:16:57 PM (10 years ago)

Author:

iandunn

Message:

WordCamp Post Type: Switch to wpmu_create_blog() to fire wpmu_new_blog.

We want wpmu_new_blog to fire so that Jetpack will automatically connect new sites to WordPress.com, but create_empty_blog() doesn't fire it.

We no longer need to explicity add the lead organizer to the new site, because wpmu_create_blog() does that for us.

File:

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

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

@@ -100 +100 @@
         $path = isset( $url['path'] ) ? $url['path'] : '';
 
-        $this->new_site_id = create_empty_blog( $url['host'], $path, 'WordCamp Event', 1 );
+        $current_blog_id_before_new_blog = get_current_blog_id();
+
+        $wordcamp_meta     = get_post_custom( $wordcamp_id );
+        $lead_organizer    = $this->get_user_or_current_user( $wordcamp_meta['WordPress.org Username'][0]  );
+        $this->new_site_id = wpmu_create_blog( $url['host'], $path, 'WordCamp Event', $lead_organizer->ID );
+
+        /*
+         * Work around for https://github.com/Automattic/jetpack/issues/2280
+         * @todo - remove this after that is fixed
+         */
+        if ( get_current_blog_id() != $current_blog_id_before_new_blog ) {
+            restore_current_blog();
+        }
+
         if ( is_int( $this->new_site_id ) ) {
             update_post_meta( $wordcamp_id, '_site_id', $this->new_site_id );    // this is used in other plugins to map the `wordcamp` post to it's corresponding site
@@ -111 +124 @@
 
     /**
+     * Get the requested user, but fall back to the current user
+     *
+     * @param string $username
+     *
+     * @return WP_User
+     */
+    protected function get_user_or_current_user( $username ) {
+        $lead_organizer = get_user_by( 'login', $username );
+
+        if ( ! $lead_organizer ) {
+            $lead_organizer = wp_get_current_user();
+        }
+
+        return $lead_organizer;
+    }
+
+    /**
      * Configure a new site and populate it with default content
      *
@@ -129 +159 @@
         switch_to_blog( $this->new_site_id );
 
-        $lead_organizer = get_user_by( 'login', $meta['WordPress.org Username'][0] );
-        if ( ! $lead_organizer ) {
-            $lead_organizer = wp_get_current_user();
-        }
-        add_user_to_blog( get_current_blog_id(), $lead_organizer->ID, 'administrator' );
+        $lead_organizer = $this->get_user_or_current_user( $meta['WordPress.org Username'][0] );
 
         activate_plugins( array(