Changeset 9839

Timestamp:

05/06/2015 04:36:24 AM (10 years ago)

Author:

r-a-y

Message:

Blogs: Delete blogmeta cache when editing a site on the "Network Admin > Sites" page.

Fixes an issue with the blog domain / path not being updated to reflect the
new domain / path for activity action links related to the updated site.

Fixes #6365.

Location:

trunk

Files:

• src/bp-blogs/bp-blogs-functions.php (modified) (1 diff)
• tests/phpunit/testcases/blogs/cache.php (modified) (1 diff)

trunk/src/bp-blogs/bp-blogs-functions.php

@@ -410 +410 @@
 
 /**
+ * Deletes the 'url' blogmeta for a site.
+ *
+ * Hooked to 'refresh_blog_details', which is notably used when editing a site
+ * under "Network Admin > Sites".
+ *
+ * @since BuddyPress (2.3.0)
+ *
+ * @param int $site_id The site ID
+ */
+function bp_blogs_delete_url_blogmeta( $site_id = 0 ) {
+    bp_blogs_delete_blogmeta( (int) $site_id, 'url' );
+}
+add_action( 'refresh_blog_details', 'bp_blogs_delete_url_blogmeta' );
+
+/**
  * Record activity metadata about a published blog post.
  *

trunk/tests/phpunit/testcases/blogs/cache.php

@@ -289 +289 @@
         $this->assertEquals( 1, bp_blogs_total_blogs() );
     }
+
+    /**
+     * @group update_blog_details
+     */
+    public function test_update_blog_details_should_purge_blogmeta_cache() {
+        if ( ! is_multisite() ) {
+            return;
+        }
+
+        $u = $this->factory->user->create();
+
+        $b1 = $this->factory->blog->create();
+        bp_blogs_record_blog( $b1, $u, true );
+
+        // prime cache
+        bp_blogs_get_blogmeta( $b1, 'url' );
+        $this->assertNotEmpty( wp_cache_get( $b1, 'blog_meta' ) );
+
+        // updating blog details should purge cache
+        update_blog_details( $b1, array(
+            'domain' => 'awesome.com'
+        ) );
+
+        // assert cache is purged
+        $this->assertEmpty( wp_cache_get( $b1, 'blog_meta' ) );
+    }
 }