Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
11/03/2024 10:01:19 PM (20 months ago)
Author:
espellcaste
Message:

WPCS - Part I: miscellaneous fixes for some of the files of the groups component.

See #9173
See #9174

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-groups/classes/class-bp-group-extension.php

    r13878 r14076  
    1111defined( 'ABSPATH' ) || exit;
    1212
    13 if ( ! class_exists( 'BP_Group_Extension', false ) ) :
     13if ( class_exists( 'BP_Group_Extension', false ) ) {
     14    return;
     15}
     16
    1417/**
    1518 * API for creating group extensions without having to hardcode the content into
     
    767770
    768771        foreach ( $screens as $context => &$screen ) {
    769             $screen['enabled']     = true;
    770             $screen['name']        = $this->name;
    771             $screen['slug']        = $this->slug;
    772 
    773             $screen['screen_callback']      = $this->get_screen_callback( $context, 'screen'      );
     772            $screen['enabled'] = true;
     773            $screen['name']    = $this->name;
     774            $screen['slug']    = $this->slug;
     775
     776            $screen['screen_callback']      = $this->get_screen_callback( $context, 'screen' );
    774777            $screen['screen_save_callback'] = $this->get_screen_callback( $context, 'screen_save' );
    775778        }
     
    887890                    $this->params['show_tab'] = 'anyone';
    888891                }
    889 
    890892            } else {
    891893                /*
     
    927929
    928930        switch ( $access_condition ) {
    929             case 'admin' :
     931            case 'admin':
    930932                $meets_condition = groups_is_user_admin( bp_loggedin_user_id(), $this->group_id );
    931933                break;
    932934
    933             case 'mod' :
     935            case 'mod':
    934936                $meets_condition = groups_is_user_mod( bp_loggedin_user_id(), $this->group_id );
    935937                break;
    936938
    937             case 'member' :
     939            case 'member':
    938940                $meets_condition = groups_is_user_member( bp_loggedin_user_id(), $this->group_id );
    939941                break;
    940942
    941             case 'loggedin' :
     943            case 'loggedin':
    942944                $meets_condition = is_user_logged_in();
    943945                break;
    944946
    945             case 'noone' :
     947            case 'noone':
    946948                $meets_condition = false;
    947949                break;
    948950
    949             case 'anyone' :
    950             default :
     951            case 'anyone':
     952            default:
    951953                $meets_condition = true;
    952954                break;
     
    10351037            // When we are viewing the extension display page, set the title and options title.
    10361038            if ( bp_is_current_action( $this->slug ) ) {
    1037                 add_filter( 'bp_group_user_has_access',   array( $this, 'group_access_protection' ), 10, 2 );
     1039                add_filter( 'bp_group_user_has_access', array( $this, 'group_access_protection' ), 10, 2 );
    10381040
    10391041                $extension_name = $this->name;
    10401042                add_action(
    10411043                    'bp_template_content_header',
    1042                     function() use ( $extension_name ) {
     1044                    function () use ( $extension_name ) {
    10431045                        echo esc_attr( $extension_name );
    10441046                    }
     
    10461048                add_action(
    10471049                    'bp_template_title',
    1048                     function() use ( $extension_name ) {
     1050                    function () use ( $extension_name ) {
    10491051                        echo esc_attr( $extension_name );
    10501052                    }
     
    12921294                $this->edit_screen_template = '/groups/single/home';
    12931295            } else {
    1294                 add_action( 'bp_template_content_header', function () {
    1295                     echo '<ul class="content-header-nav">';
    1296                     bp_group_admin_tabs();
    1297                     echo '</ul>';
    1298                 } );
     1296                add_action(
     1297                    'bp_template_content_header',
     1298                    function () {
     1299                        echo '<ul class="content-header-nav">';
     1300                        bp_group_admin_tabs();
     1301                        echo '</ul>';
     1302                    }
     1303                );
    12991304                add_action( 'bp_template_content', array( &$this, 'call_edit_screen' ) );
    13001305                $this->edit_screen_template = '/groups/single/plugins';
     
    13801385             * @param string $value URL to redirect to.
    13811386             */
    1382             $redirect_to = apply_filters( 'bp_group_extension_edit_screen_save_redirect', bp_get_requested_url( ) );
     1387            $redirect_to = apply_filters( 'bp_group_extension_edit_screen_save_redirect', bp_get_requested_url() );
    13831388
    13841389            bp_core_redirect( $redirect_to );
     
    13901395     * Load the template that houses the Edit screen.
    13911396     *
    1392      * Separated out into a callback so that it can run after all other
     1397     * Separated out into a callback so that it can run after all others
    13931398     * Group Extensions have had a chance to register their navigation, to
    13941399     * avoid missing tabs.
     
    14011406     */
    14021407    public function call_edit_screen_template_loader() {
    1403         bp_core_load_template( $this->edit_screen_template );
     1408        bp_core_load_template( (array) $this->edit_screen_template );
    14041409    }
    14051410
     
    17731778
    17741779        switch ( $key ) {
    1775             case 'enable_create_step' :
     1780            case 'enable_create_step':
    17761781                $this->screens['create']['enabled'] = $value;
    17771782                break;
    17781783
    1779             case 'enable_edit_item' :
     1784            case 'enable_edit_item':
    17801785                $this->screens['edit']['enabled'] = $value;
    17811786                break;
    17821787
    1783             case 'enable_admin_item' :
     1788            case 'enable_admin_item':
    17841789                $this->screens['admin']['enabled'] = $value;
    17851790                break;
    17861791
    1787             case 'create_step_position' :
     1792            case 'create_step_position':
    17881793                $this->screens['create']['position'] = $value;
    17891794                break;
    17901795
    17911796            // Note: 'admin' becomes 'edit' to distinguish from Dashboard 'admin'.
    1792             case 'admin_name' :
     1797            case 'admin_name':
    17931798                $this->screens['edit']['name'] = $value;
    17941799                break;
    17951800
    1796             case 'admin_slug' :
     1801            case 'admin_slug':
    17971802                $this->screens['edit']['slug'] = $value;
    17981803                break;
    17991804
    1800             case 'create_name' :
     1805            case 'create_name':
    18011806                $this->screens['create']['name'] = $value;
    18021807                break;
    18031808
    1804             case 'create_slug' :
     1809            case 'create_slug':
    18051810                $this->screens['create']['slug'] = $value;
    18061811                break;
    18071812
    1808             case 'admin_metabox_context' :
     1813            case 'admin_metabox_context':
    18091814                $this->screens['admin']['metabox_context'] = $value;
    18101815                break;
    18111816
    1812             case 'admin_metabox_priority' :
     1817            case 'admin_metabox_priority':
    18131818                $this->screens['admin']['metabox_priority'] = $value;
    18141819                break;
    18151820
    1816             default :
     1821            default:
    18171822                $this->data[ $key ] = $value;
    18181823                break;
     
    18871892
    18881893            switch ( $property ) {
    1889                 case 'enable_create_step' :
     1894                case 'enable_create_step':
    18901895                    $lpc['screens']['create']['enabled'] = (bool) $value;
    18911896                    break;
    18921897
    1893                 case 'enable_edit_item' :
     1898                case 'enable_edit_item':
    18941899                    $lpc['screens']['edit']['enabled'] = (bool) $value;
    18951900                    break;
    18961901
    1897                 case 'enable_admin_item' :
     1902                case 'enable_admin_item':
    18981903                    $lpc['screens']['admin']['enabled'] = (bool) $value;
    18991904                    break;
    19001905
    1901                 case 'create_step_position' :
     1906                case 'create_step_position':
    19021907                    $lpc['screens']['create']['position'] = $value;
    19031908                    break;
    19041909
    19051910                // Note: 'admin' becomes 'edit' to distinguish from Dashboard 'admin'.
    1906                 case 'admin_name' :
     1911                case 'admin_name':
    19071912                    $lpc['screens']['edit']['name'] = $value;
    19081913                    break;
    19091914
    1910                 case 'admin_slug' :
     1915                case 'admin_slug':
    19111916                    $lpc['screens']['edit']['slug'] = $value;
    19121917                    break;
    19131918
    1914                 case 'create_name' :
     1919                case 'create_name':
    19151920                    $lpc['screens']['create']['name'] = $value;
    19161921                    break;
    19171922
    1918                 case 'create_slug' :
     1923                case 'create_slug':
    19191924                    $lpc['screens']['create']['slug'] = $value;
    19201925                    break;
    19211926
    1922                 case 'admin_metabox_context' :
     1927                case 'admin_metabox_context':
    19231928                    $lpc['screens']['admin']['metabox_context'] = $value;
    19241929                    break;
    19251930
    1926                 case 'admin_metabox_priority' :
     1931                case 'admin_metabox_priority':
    19271932                    $lpc['screens']['admin']['metabox_priority'] = $value;
    19281933                    break;
    19291934
    1930                 default :
     1935                default:
    19311936                    $lpc[ $property ] = $value;
    19321937                    break;
     
    19601965        foreach ( $properties as $property ) {
    19611966            switch ( $property ) {
    1962                 case 'enable_create_step' :
     1967                case 'enable_create_step':
    19631968                    $lp['enable_create_step'] = $params['screens']['create']['enabled'];
    19641969                    break;
    19651970
    1966                 case 'enable_edit_item' :
     1971                case 'enable_edit_item':
    19671972                    $lp['enable_edit_item'] = $params['screens']['edit']['enabled'];
    19681973                    break;
    19691974
    1970                 case 'enable_admin_item' :
     1975                case 'enable_admin_item':
    19711976                    $lp['enable_admin_item'] = $params['screens']['admin']['enabled'];
    19721977                    break;
    19731978
    1974                 case 'create_step_position' :
     1979                case 'create_step_position':
    19751980                    $lp['create_step_position'] = $params['screens']['create']['position'];
    19761981                    break;
    19771982
    19781983                // Note: 'admin' becomes 'edit' to distinguish from Dashboard 'admin'.
    1979                 case 'admin_name' :
     1984                case 'admin_name':
    19801985                    $lp['admin_name'] = $params['screens']['edit']['name'];
    19811986                    break;
    19821987
    1983                 case 'admin_slug' :
     1988                case 'admin_slug':
    19841989                    $lp['admin_slug'] = $params['screens']['edit']['slug'];
    19851990                    break;
    19861991
    1987                 case 'create_name' :
     1992                case 'create_name':
    19881993                    $lp['create_name'] = $params['screens']['create']['name'];
    19891994                    break;
    19901995
    1991                 case 'create_slug' :
     1996                case 'create_slug':
    19921997                    $lp['create_slug'] = $params['screens']['create']['slug'];
    19931998                    break;
    19941999
    1995                 case 'admin_metabox_context' :
     2000                case 'admin_metabox_context':
    19962001                    $lp['admin_metabox_context'] = $params['screens']['admin']['metabox_context'];
    19972002                    break;
    19982003
    1999                 case 'admin_metabox_priority' :
     2004                case 'admin_metabox_priority':
    20002005                    $lp['admin_metabox_priority'] = $params['screens']['admin']['metabox_priority'];
    20012006                    break;
    20022007
    2003                 default :
     2008                default:
    20042009                    // All other items get moved over.
    20052010                    $lp[ $property ] = $params[ $property ];
     
    20072012                    // Also reapply to the object, for backpat.
    20082013                    $this->{$property} = $params[ $property ];
    2009 
    20102014                    break;
    20112015            }
     
    20132017    }
    20142018}
    2015 endif; // End class_exists check.
Note: See TracChangeset for help on using the changeset viewer.