Changeset 9957 for trunk/tests/phpunit/testcases/core/caps.php

Timestamp:

06/20/2015 01:54:56 PM (11 years ago)

Author:

boonebgorges

Message:

Convert second parameter of bp_current_user_can() to an $args array.

This makes the syntax simpler for the vast majority of use cases, where an
explicit blog_id doesn't need to be specified. It also makes it possible to
pass additional arguments, such as an item_id, to bp_current_user_can().

An integer passed as the second parameter will trigger the backward-
compatibility layer.

Props thebrandonallen, r-a-y.
See #5121. Fixes #6501.

File:

• trunk/tests/phpunit/testcases/core/caps.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/caps.php

@@ -25 +25 @@
     }
 
+    /**
+     * @ticket BP6501
+     */
+    public function test_bp_current_user_can_should_respect_blog_id_passed_in_args_array() {
+        if ( ! is_multisite() ) {
+            $this->markTestSkipped( __METHOD__ . ' requires multisite.' );
+        }
+
+        $b = $this->factory->blog->create();
+        $u = $this->factory->user->create();
+
+        $this->set_current_user( $u );
+
+        add_filter( 'user_has_cap', array( $this, 'grant_cap_foo' ), 10, 2 );
+        $can  = bp_current_user_can( 'foo', array( 'blog_id' => bp_get_root_blog_id() ) );
+        $cant = bp_current_user_can( 'foo', array( 'blog_id' => $b ) );
+        remove_filter( 'user_has_cap', array( $this, 'grant_cap_foo' ), 10, 2 );
+
+        $this->assertTrue( $can );
+        $this->assertFalse( $cant );
+    }
+
     public function grant_cap_foo( $allcaps, $caps ) {
         if ( bp_is_root_blog() ) {