Changeset 9194 for trunk/tests/phpunit/testcases/blogs/functions.php

Timestamp:

11/27/2014 05:13:29 PM (10 years ago)

Author:

imath

Message:

Post Type Activity Tracking feature

So far the tracking feature was requiring the Blogs component to be active. In 2.2 we are centralizing the majority of the tracking code into the Activity component. A new set of functions and hooks has been created to catch public post types supporting the feature 'buddypress-activity' and to automatically generate an activity when a new item is publicly published.

It's now possible to add this support to any public post type using one unique line of code, eg: add_post_type_support( 'page', 'buddypress-activity' ). In this case BuddyPress will use generic activity attributes.
Each activity attribute of the supported post type can be customized using a specific function (eg: set custom strings to describe the post type activity action).

When registering a post type in WordPress it's also possible to set the 'buddypress-activity' feature using the support parameter of the second argument of the register_post_type() function. Custom activity action strings can be defined within the labels parameter and activity attributes can be set using the new parameter 'bp_activity'.

When the Blogs component is active, the 'post' post type is automatically supporting the 'buddypress-activity' feature. The conditional logic (eg: blog_public option set to 1 ...) that occurs before a new activity is posted and the comments tracking remain unchanged.

props boonebgorges, DJPaul

Fixes #5669

File:

• trunk/tests/phpunit/testcases/blogs/functions.php (modified) (2 diffs)

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

@@ -299 +299 @@
 
         $this->assertTrue( $this->activity_exists_for_post( $post_id ), 'Published post should have activity (no change)' );
+    }
+
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_transition_post_status_password_publish() {
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'publish',
+            'post_type'     => 'post',
+            'post_password' => 'pass',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'new' => 'publish with password'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Published with password post should not have activity' );
+    }
+
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_transition_post_status_publish_update_password() {
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'publish',
+            'post_type'     => 'post',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'publish' => 'publish'
+        $this->assertTrue( $this->activity_exists_for_post( $post_id ), 'Published post should have activity' );
+
+        $post->post_content .= ' foo';
+        $post->post_password = 'pass';
+
+        wp_update_post( $post );
+
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Updated with password post should not have activity' );
+    }
+
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_transition_post_status_private_publish() {
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'private',
+            'post_type'     => 'post',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'new' => 'private'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Private post should not have activity' );
+
+        $post->post_status = 'publish';
+
+        wp_update_post( $post );
+
+        // 'private' => 'publish'
+        $this->assertTrue( $this->activity_exists_for_post( $post_id ), 'Published post should have activity' );
+    }
+
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_transition_post_status_publish_private() {
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'publish',
+            'post_type'     => 'post',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'new' => 'publish'
+        $this->assertTrue( $this->activity_exists_for_post( $post_id ), 'Published post should have activity' );
+
+        $post->post_status = 'private';
+
+        wp_update_post( $post );
+
+        // 'publish' => 'private'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Private post should not have activity' );
     }
 
@@ -478 +556 @@
     }
 
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_bp_blogs_is_blog_trackable_false_publish_post() {
+        add_filter( 'bp_blogs_is_blog_trackable', '__return_false' );
+
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'publish',
+            'post_type'     => 'post',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'new' => 'publish'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Not trackable blog post should not have activity' );
+
+        $post->post_content .= ' foo';
+
+        wp_update_post( $post );
+
+        // 'publish' => 'publish'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Not trackable blog post should not have activity' );
+
+        remove_filter( 'bp_blogs_is_blog_trackable', '__return_false' );
+    }
+
+    /**
+     * @group bp_blogs_catch_transition_post_status
+     */
+    public function test_bp_is_blog_public_zero_publish_post() {
+        if ( ! is_multisite() ) {
+            return;
+        }
+
+        add_filter( 'bp_is_blog_public', '__return_zero' );
+
+        $post_id = $this->factory->post->create( array(
+            'post_status'   => 'publish',
+            'post_type'     => 'post',
+        ) );
+        $post = get_post( $post_id );
+
+        // 'new' => 'publish'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Not public blog post should not have activity' );
+
+        $post->post_content .= ' foo';
+
+        wp_update_post( $post );
+
+        // 'publish' => 'publish'
+        $this->assertFalse( $this->activity_exists_for_post( $post_id ), 'Not public blog post should not have activity' );
+
+        remove_filter( 'bp_is_blog_public', '__return_zero' );
+    }
+
     protected function activity_exists_for_post( $post_id ) {
         $a = bp_activity_get( array(