Skip to:
Content

BuddyPress.org

Ticket #4785: 4785.06.patch

File 4785.06.patch, 5.8 KB (added by dcavins, 9 years ago)

Include "noone" access and show_tab cases

  • src/bp-groups/bp-groups-classes.php 

    diff --git src/bp-groups/bp-groups-classes.php src/bp-groups/bp-groups-classes.php
index 2e96640..7bb8321 100644
    class BP_Group_Extension { 
    33653365         * Check whether the current user meets an access condition.
    33663366         *
    33673367         * @param string $access_condition 'anyone', 'loggedin', 'member',
    3368          *        'mod', 'admin'.
     3368         *        'mod', 'admin' or 'noone'.
    33693369         * @return bool
    33703370         */
    33713371        protected function user_meets_access_condition( $access_condition ) {

  • tests/phpunit/assets/group-extensions.php 

    diff --git tests/phpunit/assets/group-extensions.php tests/phpunit/assets/group-extensions.php
index 279c150..4f2ed87 100644
    class BPTest_Group_Extension_Access_Admin extends BP_Group_Extension { 
    303303        }
    304304}
    305305
     306class BPTest_Group_Extension_Access_Noone extends BP_Group_Extension {
     307        public function __construct() {
     308                $class_name = get_class( $this );
     309
     310                $args = array(
     311                        'name' => $class_name,
     312                        'slug' => sanitize_title( $class_name ),
     313                        'access' => 'noone',
     314                );
     315
     316                parent::init( $args );
     317        }
     318}
     319
    306320class BPTest_Group_Extension_ShowTab_Anyone extends BP_Group_Extension {
    307321        public function __construct() {
    308322                $class_name = get_class( $this );
    … … class BPTest_Group_Extension_ShowTab_Admin extends BP_Group_Extension { 
    380394                parent::init( $args );
    381395        }
    382396}
     397
     398class BPTest_Group_Extension_ShowTab_Noone extends BP_Group_Extension {
     399        public function __construct() {
     400                $class_name = get_class( $this );
     401
     402                $args = array(
     403                        'name' => $class_name,
     404                        'slug' => sanitize_title( $class_name ),
     405                        'access' => 'noone',
     406                        'show_tab' => 'noone',
     407                );
     408
     409                parent::init( $args );
     410        }
     411}

  • tests/phpunit/testcases/groups/class-bp-group-extension.php 

    diff --git tests/phpunit/testcases/groups/class-bp-group-extension.php tests/phpunit/testcases/groups/class-bp-group-extension.php
index eeda8a6..00cb10f 100644
    class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    405405                $e5->_register();
    406406                $this->assertFalse( $e5->user_can_visit() );
    407407
     408                $e6 = new BPTest_Group_Extension_Access_Noone();
     409                $e6->_register();
     410                $this->assertFalse( $e6->user_can_visit() );
     411
    408412                $this->set_current_user( $old_current_user );
    409413        }
    410414
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    443447                $e5->_register();
    444448                $this->assertFalse( $e5->user_can_visit() );
    445449
     450                $e6 = new BPTest_Group_Extension_Access_Noone();
     451                $e6->_register();
     452                $this->assertFalse( $e6->user_can_visit() );
     453
    446454                $this->set_current_user( $old_current_user );
    447455        }
    448456
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    483491                $e5->_register();
    484492                $this->assertFalse( $e5->user_can_visit() );
    485493
     494                $e6 = new BPTest_Group_Extension_Access_Noone();
     495                $e6->_register();
     496                $this->assertFalse( $e6->user_can_visit() );
     497
    486498                $this->set_current_user( $old_current_user );
    487499        }
    488500
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    525537                $e5->_register();
    526538                $this->assertFalse( $e5->user_can_visit() );
    527539
     540                $e6 = new BPTest_Group_Extension_Access_Noone();
     541                $e6->_register();
     542                $this->assertFalse( $e6->user_can_visit() );
     543
    528544                $this->set_current_user( $old_current_user );
    529545        }
    530546
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    567583                $e5->_register();
    568584                $this->assertTrue( $e5->user_can_visit() );
    569585
     586                $e6 = new BPTest_Group_Extension_Access_Noone();
     587                $e6->_register();
     588                $this->assertFalse( $e6->user_can_visit() );
     589
    570590                $this->set_current_user( $old_current_user );
    571591        }
    572592
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    604624                $e5->_register();
    605625                $this->assertFalse( $e5->user_can_see_nav_item() );
    606626
     627                $e6 = new BPTest_Group_Extension_Access_Noone();
     628                $e6->_register();
     629                $this->assertFalse( $e6->user_can_visit() );
     630
    607631                $this->set_current_user( $old_current_user );
    608632        }
    609633
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    641665                $e5->_register();
    642666                $this->assertFalse( $e5->user_can_see_nav_item() );
    643667
     668                $e6 = new BPTest_Group_Extension_ShowTab_Noone();
     669                $e6->_register();
     670                $this->assertFalse( $e6->user_can_see_nav_item() );
     671
    644672                $this->set_current_user( $old_current_user );
    645673        }
    646674
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    679707                $e5->_register();
    680708                $this->assertFalse( $e5->user_can_see_nav_item() );
    681709
     710                $e6 = new BPTest_Group_Extension_ShowTab_Noone();
     711                $e6->_register();
     712                $this->assertFalse( $e6->user_can_see_nav_item() );
     713
    682714                $this->set_current_user( $old_current_user );
    683715        }
    684716
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    718750                $e5 = new BPTest_Group_Extension_ShowTab_Admin();
    719751                $e5->_register();
    720752                $this->assertFalse( $e5->user_can_see_nav_item() );
     753               
     754                $e6 = new BPTest_Group_Extension_ShowTab_Noone();
     755                $e6->_register();
     756                $this->assertFalse( $e6->user_can_see_nav_item() );
    721757
    722758                $this->set_current_user( $old_current_user );
    723759        }
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    761797                $e5->_register();
    762798                $this->assertFalse( $e5->user_can_see_nav_item() );
    763799
     800                $e6 = new BPTest_Group_Extension_ShowTab_Noone();
     801                $e6->_register();
     802                $this->assertFalse( $e6->user_can_see_nav_item() );
     803
    764804                $this->set_current_user( $old_current_user );
    765805        }
    766806
    … … class BP_Tests_Group_Extension_TestCases extends BP_UnitTestCase { 
    802842                $e5 = new BPTest_Group_Extension_ShowTab_Admin();
    803843                $e5->_register();
    804844                $this->assertTrue( $e5->user_can_see_nav_item() );
     845               
     846                $e6 = new BPTest_Group_Extension_ShowTab_Noone();
     847                $e6->_register();
     848                $this->assertFalse( $e6->user_can_see_nav_item() );
    805849
    806850                $this->set_current_user( $old_current_user );
    807851        }