Skip to:
Content

BuddyPress.org

Changeset 13888


Ignore:
Timestamp:
06/01/2024 10:25:18 PM (4 months ago)
Author:
espellcaste
Message:

WPCS: Part IV: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], and [13887]

See #9164 and #7228

Location:
trunk/src/bp-core/classes
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/classes/class-bp-invitation-manager.php

    r13500 r13888  
    2626     *
    2727     * @since 5.0.0
    28      * @access public
     28     *
    2929     * @var string
    3030     */
    … …  
    3535     *
    3636     * @since 5.0.0
    37      *
    38      * @param array|string $args {
    39      * }
    40      */
    41     public function __construct( $args = array() ) {
     37     */
     38    public function __construct() {
    4239        $this->class_name = get_class( $this );
    4340    }
    … …  
    4744     *
    4845     * @since 5.0.0
    49      * @access public
     46     *
    5047     * @return string
    5148     */
    … …  
    6461     * @param array $args {
    6562     *     Array of arguments describing the invitation. All are optional.
    66      *     @type int    $user_id           ID of the invited user.
    67      *     @type int    $inviter_id        ID of the user who created the invitation.
    68      *     @type string $invitee_email     Email address of the invited user.
    69      *     @type int    $item_id           ID associated with the invitation and class.
    70      *     @type int    $secondary_item_id Secondary ID associated with the
    71      *                                     invitation and class.
    72      *     @type string $type              Type of record this is: 'invite' or 'request'.
    73      *     @type string $content           Extra information provided by the requester
    74      *                                     or inviter.
    75      *     @type string $date_modified     Date the invitation was last modified.
    76      *     @type int    $send_invite       Should the invitation also be sent, or is it a
    77      *                                     draft invite?
     63     *     @type int    $user_id           ID of the invited user.
     64     *     @type int    $inviter_id        ID of the user who created the invitation.
     65     *     @type string $invitee_email     Email address of the invited user.
     66     *     @type int    $item_id           ID associated with the invitation and class.
     67     *     @type int    $secondary_item_id Secondary ID associated with the
     68     *                                     invitation and class.
     69     *     @type string $type              Type of record this is: 'invite' or 'request'.
     70     *     @type string $content           Extra information provided by the requester
     71     *                                     or inviter.
     72     *     @type string $date_modified     Date the invitation was last modified.
     73     *     @type int    $send_invite       Should the invitation also be sent, or is it a
     74     *                                     draft invite?
    7875     * }
    7976     * @return int|bool ID of the newly created invitation on success, false
    80      *         on failure.
     77     *                  on failure.
    8178     */
    8279    public function add_invitation( $args = array() ) {
    83 
    8480        $r = bp_parse_args(
    8581            $args,
    … …  
    121117
    122118        // Avoid creating duplicate invitations.
    123         $invite_id = $this->invitation_exists( array(
    124             'user_id'           => $r['user_id'],
    125             'invitee_email'     => $r['invitee_email'],
    126             'inviter_id'        => $r['inviter_id'],
    127             'item_id'           => $r['item_id'],
    128             'secondary_item_id' => $r['secondary_item_id'],
    129         ) );
     119        $invite_id = $this->invitation_exists(
     120            array(
     121                'user_id'           => $r['user_id'],
     122                'invitee_email'     => $r['invitee_email'],
     123                'inviter_id'        => $r['inviter_id'],
     124                'item_id'           => $r['item_id'],
     125                'secondary_item_id' => $r['secondary_item_id'],
     126            )
     127        );
    130128
    131129        if ( ! $invite_id ) {
    132130            // Set up the new invitation as a draft.
    133             $invitation                    = new BP_Invitation;
     131            $invitation                    = new BP_Invitation();
    134132            $invitation->user_id           = $r['user_id'];
    135133            $invitation->inviter_id        = $r['inviter_id'];
    … …  
    162160     *
    163161     * @since 5.0.0
    164      * @access public
    165162     *
    166163     * @param int   $invitation_id ID of invitation to send.
    167164     * @param array $args          See BP_Invitation::mark_sent().
    168165     *
    169      * @return bool The result of `run_send_action()`.
     166     * @return bool
    170167     */
    171168    public function send_invitation_by_id( $invitation_id = 0, $args = array() ) {
    … …  
    195192            'secondary_item_id' => $invitation->secondary_item_id,
    196193        );
    197         $request = $this->request_exists( $request_args );
     194        $request      = $this->request_exists( $request_args );
    198195
    199196        if ( ! empty( $request ) ) {
    … …  
    220217     * @param array $args {
    221218     *     Array of arguments describing the invitation. All are optional.
    222      *     @type int    $user_id ID of the invited user.
    223      *     @type int    $inviter_id ID of the user who created the invitation.
    224      *     @type string $class Name of the invitations class.
    225      *     @type int    $item_id ID associated with the invitation and class.
    226      *     @type int    $secondary_item_id secondary ID associated with the
    227      *                  invitation and class.
    228      *     @type string $type    Type of record this is: 'invite' or 'request'.
    229      *     @type string $content Extra information provided by the requester
    230      *                  or inviter.
    231      *     @type string $date_modified Date the invitation was last modified.
    232      *     @type int    $invite_sent Has the invitation been sent, or is it a
    233      *          draft invite?
     219     *     @type int    $user_id ID of the invited user.
     220     *     @type int    $inviter_id ID of the user who created the invitation.
     221     *     @type string $class Name of the invitations class.
     222     *     @type int    $item_id ID associated with the invitation and class.
     223     *     @type int    $secondary_item_id secondary ID associated with the
     224     *                  invitation and class.
     225     *     @type string $type    Type of record this is: 'invite' or 'request'.
     226     *     @type string $content Extra information provided by the requester
     227     *                  or inviter.
     228     *     @type string $date_modified Date the invitation was last modified.
     229     *     @type int    $invite_sent Has the invitation been sent, or is it a
     230     *          draft invite?
    234231     * }
    235232     * @return int|bool ID of the newly created invitation on success, false
    236      *         on failure.
     233     *                  on failure.
    237234     */
    238235    public function add_request( $args = array() ) {
    239 
    240236        $r = bp_parse_args(
    241237            $args,
    … …  
    289285         */
    290286        $invite_args = array_merge( $base_args, array( 'invite_sent' => 'sent' ) );
    291         $invite = $this->invitation_exists( $invite_args );
     287        $invite      = $this->invitation_exists( $invite_args );
    292288
    293289        if ( $invite ) {
    … …  
    296292        } else {
    297293            // Set up the new request.
    298             $request                    = new BP_Invitation;
     294            $request                    = new BP_Invitation();
    299295            $request->user_id           = $r['user_id'];
    300296            $request->inviter_id        = $r['inviter_id'];
    … …  
    344340            'item_id'           => $request->item_id,
    345341            'secondary_item_id' => $request->secondary_item_id,
    346             'invite_sent'       => 'sent'
    347         );
    348         $invites = $this->invitation_exists( $invite_args );
     342            'invite_sent'       => 'sent',
     343        );
     344        $invites     = $this->invitation_exists( $invite_args );
    349345
    350346        if ( ! empty( $invites ) ) {
    … …  
    451447
    452448        $args['fields'] = 'ids';
    453         $invites = $this->get_invitations( $args );
     449        $invites        = $this->get_invitations( $args );
    454450        if ( $invites ) {
    455451            $is_invited = current( $invites );
    … …  
    471467
    472468        $args['fields'] = 'ids';
    473         $requests = $this->get_requests( $args );
     469        $requests       = $this->get_requests( $args );
    474470        if ( $requests ) {
    475471            $has_request = current( $requests );
    … …  
    608604     * @param array $update_args Associative array of fields to update,
    609605     *              and the values to update them to. Of the format
    610      *              array( 'user_id' => 4 )
     606     *              array( 'user_id' => 4 ).
    611607     * @param array $where_args Associative array of columns/values, to
    612608     *              determine which invitations should be updated. Formatted as
    613      *              array( 'item_id' => 7 )
     609     *              array( 'item_id' => 7 ).
    614610     * @return int|bool Number of rows updated on success, false on failure.
    615611     */
    … …  
    647643     *                        Special cases
    648644     *     @type string|array $invitee_email Email address of invited users
    649      *                        being queried. Can be an array of addresses.
     645     *                        being queried. Can be an array of addresses.
    650646     *     @type string|array $class Name of the class to
    651647     *                        filter by. Can be an array of class names.
    … …  
    667663     * @since 5.0.0
    668664     *
    669      * @param int $id The ID of the invitation to mark as sent.
    670      * @return bool True on success, false on failure.
    671      */
    672     abstract public function run_acceptance_action( $type, $r  );
     665     * @param string $type The type of record being accepted: 'invite' or 'request'.
     666     * @param array  $r    Associative array of arguments.
     667     * @return bool
     668     */
     669    abstract public function run_acceptance_action( $type, $r );
    673670
    674671    /**
    … …  
    679676     * @see BP_Invitation::mark_accepted()
    680677     *      for a description of arguments.
    681      * @return bool True on success, false on failure.
     678     *
     679     * @return bool
    682680     */
    683681    public function mark_accepted_by_id( $id, $args ) {
    … …  
    708706     *
    709707     * @see BP_Invitation::delete for a description of arguments.
     708     *
    710709     * @return int|bool Number of rows deleted on success, false on failure.
    711710     */
    … …  
    743742     */
    744743    public function delete_all() {
    745         return BP_Invitation::delete( array(
    746             'class' => $this->class_name,
    747         ) );
     744        return BP_Invitation::delete(
     745            array(
     746                'class' => $this->class_name,
     747            )
     748        );
    748749    }
    749750
    … …  
    772773     * @since 5.0.0
    773774     *
    774      * @param array $args The parameters describing the invitation.
    775      * @return bool True if allowed, false to end process.
    776      */
    777     public function allow_invitation( $args ) {
     775     * @return bool
     776     */
     777    public function allow_invitation() {
    778778        return true;
    779779    }
    … …  
    785785     * @since 5.0.0
    786786     *
    787      * @param array $args The parameters describing the request.
    788      * @return bool True if allowed, false to end process.
    789      */
    790     public function allow_request( $args ) {
     787     * @return bool
     788     */
     789    public function allow_request() {
    791790        return true;
    792791    }
    793 
    794792}

  • trunk/src/bp-core/classes/class-bp-invitation.php

    r13395 r13888  
    2525     *
    2626     * @since 5.0.0
    27      * @access public
     27     *
    2828     * @var int
    2929     */
    … …  
    3434     *
    3535     * @since 5.0.0
    36      * @access public
     36     *
    3737     * @var int
    3838     */
    … …  
    4343     *
    4444     * @since 5.0.0
    45      * @access public
     45     *
    4646     * @var int
    4747     */
    … …  
    5353     *
    5454     * @since 5.0.0
    55      * @access public
     55     *
    5656     * @var string
    5757     */
    … …  
    6262     *
    6363     * @since 5.0.0
    64      * @access public
     64     *
    6565     * @var string
    6666     */
    … …  
    7272     *
    7373     * @since 5.0.0
    74      * @access public
     74     *
    7575     * @var int
    7676     */
    … …  
    8282     *
    8383     * @since 5.0.0
    84      * @access public
    85      * @var int
     84     *
     85     * @var int|null
    8686     */
    8787    public $secondary_item_id = null;
    … …  
    9191     *
    9292     * @since 5.0.0
    93      * @access public
     93     *
    9494     * @var string
    9595     */
    … …  
    100100     *
    101101     * @since 5.0.0
    102      * @access public
     102     *
    103103     * @var string
    104104     */
    … …  
    109109     *
    110110     * @since 5.0.0
    111      * @access public
     111     *
    112112     * @var string
    113113     */
    … …  
    118118     *
    119119     * @since 5.0.0
    120      * @access public
     120     *
    121121     * @var bool
    122122     */
    … …  
    127127     *
    128128     * @since 5.0.0
    129      * @access public
     129     *
    130130     * @var bool
    131131     */
    … …  
    136136     *
    137137     * @since 9.0.0
    138      * @access public
     138     *
    139139     * @var array
    140140     */
    … …  
    151151        'date_modified',
    152152        'invite_sent',
    153         'accepted'
     153        'accepted',
    154154    );
    155155
    … …  
    161161     * @since 5.0.0
    162162     *
    163      * @param int $id Optional. Provide an ID to access an existing
    164      *        invitation item.
     163     * @param int $id Optional. Provide an ID to access an existing invitation item.
    165164     */
    166165    public function __construct( $id = 0 ) {
    … …  
    182181    public function save() {
    183182
    184         // Return value
     183        // Return value.
    185184        $retval = false;
    186185
    187         // Default data and format
    188         $data = array(
     186        // Default data and format.
     187        $data        = array(
    189188            'user_id'           => $this->user_id,
    190189            'inviter_id'        => $this->inviter_id,
    … …  
    213212        if ( ! empty( $this->id ) ) {
    214213            $result = self::_update( $data, array( 'ID' => $this->id ), $data_format, array( '%d' ) );
    215         // Insert.
     214            // Insert.
    216215        } else {
    217216            $result = self::_insert( $data, $data_format );
    … …  
    258257        if ( false === $invitation ) {
    259258            $invitation = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM {$invites_table_name} WHERE id = %d", $this->id ) );
    260             wp_cache_set( $this->id, $invitation,'bp_invitations' );
     259            wp_cache_set( $this->id, $invitation, 'bp_invitations' );
    261260        }
    262261
    … …  
    278277        $this->invite_sent       = (int) $invitation->invite_sent;
    279278        $this->accepted          = (int) $invitation->accepted;
    280 
    281279    }
    282280
    … …  
    287285     *
    288286     * @since 5.0.0
     287     *
     288     * @global wpdb $wpdb WordPress database object.
    289289     *
    290290     * @param array $data {
    291291     *     Array of invitation data, passed to {@link wpdb::insert()}.
    292      *     @type int    $user_id           ID of the invited user.
    293      *     @type int    $inviter_id        ID of the user who created the invitation.
    294      *     @type string $invitee_email     Email address of the invited user.
    295      *     @type string $class             Name of the related class.
    296      *     @type int    $item_id           ID associated with the invitation and component.
    297      *     @type int    $secondary_item_id Secondary ID associated with the invitation and
     292     *     @type int    $user_id           ID of the invited user.
     293     *     @type int    $inviter_id        ID of the user who created the invitation.
     294     *     @type string $invitee_email     Email address of the invited user.
     295     *     @type string $class             Name of the related class.
     296     *     @type int    $item_id           ID associated with the invitation and component.
     297     *     @type int    $secondary_item_id Secondary ID associated with the invitation and
    298298     *                                     component.
    299      *     @type string $content           Extra information provided by the requester
    300      *                                     or inviter.
    301      *     @type string $date_modified     Date the invitation was last modified.
    302      *     @type int    $invite_sent       Has the invitation been sent, or is it a draft
     299     *     @type string $content           Extra information provided by the requester
     300     *                                     or inviter.
     301     *     @type string $date_modified     Date the invitation was last modified.
     302     *     @type int    $invite_sent       Has the invitation been sent, or is it a draft
    303303     *                                     invite?
    304304     * }
    … …  
    315315     *
    316316     * @since 5.0.0
     317     *
     318     * @global wpdb $wpdb WordPress database object.
    317319     *
    318320     * @see wpdb::update() for further description of paramater formats.
    … …  
    338340     * @since 5.0.0
    339341     *
     342     * @global wpdb $wpdb WordPress database object.
     343     *
    340344     * @see wpdb::update() for further description of paramater formats.
    341345     *
    … …  
    359363     * @since 5.0.0
    360364     *
     365     * @global wpdb $wpdb WordPress database object.
     366     *
    361367     * @param array $args See {@link BP_Invitation::get()} for more details.
    362368     * @return string WHERE clause.
    … …  
    370376        // id.
    371377        if ( false !== $args['id'] ) {
    372             $id_in = implode( ',', wp_parse_id_list( $args['id'] ) );
     378            $id_in                  = implode( ',', wp_parse_id_list( $args['id'] ) );
    373379            $where_conditions['id'] = "id IN ({$id_in})";
    374380        }
    … …  
    376382        // user_id.
    377383        if ( ! empty( $args['user_id'] ) ) {
    378             $user_id_in = implode( ',', wp_parse_id_list( $args['user_id'] ) );
     384            $user_id_in                  = implode( ',', wp_parse_id_list( $args['user_id'] ) );
    379385            $where_conditions['user_id'] = "user_id IN ({$user_id_in})";
    380386        }
    … …  
    382388        // inviter_id. 0 can be meaningful, in the case of requests.
    383389        if ( ! empty( $args['inviter_id'] ) || 0 === $args['inviter_id'] ) {
    384             $inviter_id_in = implode( ',', wp_parse_id_list( $args['inviter_id'] ) );
     390            $inviter_id_in                  = implode( ',', wp_parse_id_list( $args['inviter_id'] ) );
    385391            $where_conditions['inviter_id'] = "inviter_id IN ({$inviter_id_in})";
    386392        }
    … …  
    399405            }
    400406
    401             $invitee_email_in = implode( ',', $email_clean );
     407            $invitee_email_in                  = implode( ',', $email_clean );
    402408            $where_conditions['invitee_email'] = "invitee_email IN ({$invitee_email_in})";
    403409        }
    … …  
    416422            }
    417423
    418             $cn_in = implode( ',', $cn_clean );
     424            $cn_in                     = implode( ',', $cn_clean );
    419425            $where_conditions['class'] = "class IN ({$cn_in})";
    420426        }
    … …  
    422428        // item_id.
    423429        if ( ! empty( $args['item_id'] ) ) {
    424             $item_id_in = implode( ',', wp_parse_id_list( $args['item_id'] ) );
     430            $item_id_in                  = implode( ',', wp_parse_id_list( $args['item_id'] ) );
    425431            $where_conditions['item_id'] = "item_id IN ({$item_id_in})";
    426432        }
    … …  
    428434        // secondary_item_id.
    429435        if ( ! empty( $args['secondary_item_id'] ) ) {
    430             $secondary_item_id_in = implode( ',', wp_parse_id_list( $args['secondary_item_id'] ) );
     436            $secondary_item_id_in                  = implode( ',', wp_parse_id_list( $args['secondary_item_id'] ) );
    431437            $where_conditions['secondary_item_id'] = "secondary_item_id IN ({$secondary_item_id_in})";
    432438        }
    … …  
    434440        // Type.
    435441        if ( ! empty( $args['type'] ) && 'all' !== $args['type'] ) {
    436             if ( 'invite' == $args['type'] || 'request' == $args['type'] ) {
    437                 $type_clean = $wpdb->prepare( '%s', $args['type'] );
     442            if ( 'invite' === $args['type'] || 'request' === $args['type'] ) {
     443                $type_clean               = $wpdb->prepare( '%s', $args['type'] );
    438444                $where_conditions['type'] = "type = {$type_clean}";
    439445            }
    … …  
    441447
    442448        /**
    443          * invite_sent
    444449         * Only create a where statement if something less than "all" has been
    445450         * specifically requested.
    446451         */
    447452        if ( ! empty( $args['invite_sent'] ) && 'all' !== $args['invite_sent'] ) {
    448             if ( $args['invite_sent'] == 'draft' ) {
    449                 $where_conditions['invite_sent'] = "invite_sent = 0";
    450             } else if ( $args['invite_sent'] == 'sent' ) {
    451                 $where_conditions['invite_sent'] = "invite_sent = 1";
     453            if ( $args['invite_sent'] === 'draft' ) {
     454                $where_conditions['invite_sent'] = 'invite_sent = 0';
     455            } elseif ( $args['invite_sent'] === 'sent' ) {
     456                $where_conditions['invite_sent'] = 'invite_sent = 1';
    452457            }
    453458        }
    … …  
    455460        // Accepted.
    456461        if ( ! empty( $args['accepted'] ) && 'all' !== $args['accepted'] ) {
    457             if ( $args['accepted'] == 'pending' ) {
    458                 $where_conditions['accepted'] = "accepted = 0";
    459             } else if ( $args['accepted'] == 'accepted' ) {
    460                 $where_conditions['accepted'] = "accepted = 1";
     462            if ( $args['accepted'] === 'pending' ) {
     463                $where_conditions['accepted'] = 'accepted = 0';
     464            } elseif ( $args['accepted'] === 'accepted' ) {
     465                $where_conditions['accepted'] = 'accepted = 1';
    461466            }
    462467        }
    … …  
    464469        // search_terms.
    465470        if ( ! empty( $args['search_terms'] ) ) {
    466             $search_terms_like = '%' . bp_esc_like( $args['search_terms'] ) . '%';
     471            $search_terms_like                = '%' . bp_esc_like( $args['search_terms'] ) . '%';
    467472            $where_conditions['search_terms'] = $wpdb->prepare( '( invitee_email LIKE %s OR content LIKE %s )', $search_terms_like, $search_terms_like );
    468473        }
    … …  
    539544        // Custom LIMIT.
    540545        if ( ! empty( $args['page'] ) && ! empty( $args['per_page'] ) ) {
    541             $page     = absint( $args['page']     );
     546            $page     = absint( $args['page'] );
    542547            $per_page = absint( $args['per_page'] );
    543548            $offset   = $per_page * ( $page - 1 );
    544             $retval   = $wpdb->prepare( "LIMIT %d, %d", $offset, $per_page );
     549            $retval   = $wpdb->prepare( 'LIMIT %d, %d', $offset, $per_page );
    545550        }
    546551
    … …  
    597602        if ( ! empty( $args['id'] ) ) {
    598603            $where_clauses['data']['id'] = absint( $args['id'] );
    599             $where_clauses['format'][] = '%d';
     604            $where_clauses['format'][]   = '%d';
    600605        }
    601606
    … …  
    603608        if ( ! empty( $args['user_id'] ) ) {
    604609            $where_clauses['data']['user_id'] = absint( $args['user_id'] );
    605             $where_clauses['format'][] = '%d';
     610            $where_clauses['format'][]        = '%d';
    606611        }
    607612
    … …  
    609614        if ( ! empty( $args['inviter_id'] ) ) {
    610615            $where_clauses['data']['inviter_id'] = absint( $args['inviter_id'] );
    611             $where_clauses['format'][] = '%d';
     616            $where_clauses['format'][]           = '%d';
    612617        }
    613618
    … …  
    615620        if ( ! empty( $args['invitee_email'] ) ) {
    616621            $where_clauses['data']['invitee_email'] = $args['invitee_email'];
    617             $where_clauses['format'][] = '%s';
     622            $where_clauses['format'][]              = '%s';
    618623        }
    619624
    … …  
    621626        if ( ! empty( $args['class'] ) ) {
    622627            $where_clauses['data']['class'] = $args['class'];
    623             $where_clauses['format'][] = '%s';
     628            $where_clauses['format'][]      = '%s';
    624629        }
    625630
    … …  
    627632        if ( ! empty( $args['item_id'] ) ) {
    628633            $where_clauses['data']['item_id'] = absint( $args['item_id'] );
    629             $where_clauses['format'][] = '%d';
     634            $where_clauses['format'][]        = '%d';
    630635        }
    631636
    … …  
    633638        if ( ! empty( $args['secondary_item_id'] ) ) {
    634639            $where_clauses['data']['secondary_item_id'] = absint( $args['secondary_item_id'] );
    635             $where_clauses['format'][] = '%d';
     640            $where_clauses['format'][]                  = '%d';
    636641        }
    637642
    638643        // type.
    639644        if ( ! empty( $args['type'] ) && 'all' !== $args['type'] ) {
    640             if ( 'invite' == $args['type'] || 'request' == $args['type'] ) {
     645            if ( 'invite' === $args['type'] || 'request' === $args['type'] ) {
    641646                $where_clauses['data']['type'] = $args['type'];
    642                 $where_clauses['format'][] = '%s';
     647                $where_clauses['format'][]     = '%s';
    643648            }
    644649        }
    645650
    646651        /**
    647          * invite_sent
    648652         * Only create a where statement if something less than "all" has been
    649653         * specifically requested.
    650654         */
    651655        if ( isset( $args['invite_sent'] ) && 'all' !== $args['invite_sent'] ) {
    652             if ( $args['invite_sent'] == 'draft' ) {
     656            if ( $args['invite_sent'] === 'draft' ) {
    653657                $where_clauses['data']['invite_sent'] = 0;
    654                 $where_clauses['format'][] = '%d';
    655             } else if ( $args['invite_sent'] == 'sent' ) {
     658                $where_clauses['format'][]            = '%d';
     659            } elseif ( $args['invite_sent'] === 'sent' ) {
    656660                $where_clauses['data']['invite_sent'] = 1;
    657                 $where_clauses['format'][] = '%d';
     661                $where_clauses['format'][]            = '%d';
    658662            }
    659663        }
    … …  
    661665        // accepted.
    662666        if ( ! empty( $args['accepted'] ) && 'all' !== $args['accepted'] ) {
    663             if ( $args['accepted'] == 'pending' ) {
     667            if ( $args['accepted'] === 'pending' ) {
    664668                $where_clauses['data']['accepted'] = 0;
    665                 $where_clauses['format'][] = '%d';
    666             } else if ( $args['accepted'] == 'accepted' ) {
     669                $where_clauses['format'][]         = '%d';
     670            } elseif ( $args['accepted'] === 'accepted' ) {
    667671                $where_clauses['data']['accepted'] = 1;
    668                 $where_clauses['format'][] = '%d';
    669             }
    670         }
    671 
    672         // date_modified
     672                $where_clauses['format'][]         = '%d';
     673            }
     674        }
     675
     676        // date_modified.
    673677        if ( ! empty( $args['date_modified'] ) ) {
    674678            $where_clauses['data']['date_modified'] = $args['date_modified'];
    675             $where_clauses['format'][] = '%s';
     679            $where_clauses['format'][]              = '%s';
    676680        }
    677681
    … …  
    685689     *
    686690     * @since 5.0.0
     691     *
     692     * @global wpdb $wpdb WordPress database object.
    687693     *
    688694     * @param array $args {
    … …  
    697703     *                                           invitation. Can be an array of IDs.
    698704     *     @type string|array $invitee_email     Email address of invited users
    699      *                                           being queried. Can be an array of
     705     *                                           being queried. Can be an array of
    700706     *                                           addresses.
    701707     *     @type string|array $class             Name of the class to filter by.
    … …  
    735741     * }
    736742     *
    737      * @return array BP_Invitation objects | IDs of found invite.
     743     * @return int[]|BP_Invitation[] BP_Invitation objects | IDs of found invite.
    738744     */
    739745    public static function get( $args = array() ) {
    … …  
    766772
    767773        $sql = array(
    768             'select'     => "SELECT",
     774            'select'     => 'SELECT',
    769775            'fields'     => '',
    770776            'from'       => "FROM {$invites_table_name} i",
    … …  
    775781
    776782        if ( 'item_ids' === $r['fields'] ) {
    777             $sql['fields'] = "DISTINCT i.item_id";
    778         } else if ( 'user_ids' === $r['fields'] ) {
    779             $sql['fields'] = "DISTINCT i.user_id";
    780         } else if ( 'inviter_ids' === $r['fields'] ) {
    781             $sql['fields'] = "DISTINCT i.inviter_id";
     783            $sql['fields'] = 'DISTINCT i.item_id';
     784        } elseif ( 'user_ids' === $r['fields'] ) {
     785            $sql['fields'] = 'DISTINCT i.user_id';
     786        } elseif ( 'inviter_ids' === $r['fields'] ) {
     787            $sql['fields'] = 'DISTINCT i.inviter_id';
    782788        } else {
    783789            $sql['fields'] = 'DISTINCT i.id';
    … …  
    785791
    786792        // WHERE.
    787         $sql['where'] = self::get_where_sql( array(
    788             'id'                => $r['id'],
    789             'user_id'           => $r['user_id'],
    790             'inviter_id'        => $r['inviter_id'],
    791             'invitee_email'     => $r['invitee_email'],
    792             'class'             => $r['class'],
    793             'item_id'           => $r['item_id'],
    794             'secondary_item_id' => $r['secondary_item_id'],
    795             'type'              => $r['type'],
    796             'invite_sent'       => $r['invite_sent'],
    797             'accepted'          => $r['accepted'],
    798             'search_terms'      => $r['search_terms'],
    799         ) );
     793        $sql['where'] = self::get_where_sql(
     794            array(
     795                'id'                => $r['id'],
     796                'user_id'           => $r['user_id'],
     797                'inviter_id'        => $r['inviter_id'],
     798                'invitee_email'     => $r['invitee_email'],
     799                'class'             => $r['class'],
     800                'item_id'           => $r['item_id'],
     801                'secondary_item_id' => $r['secondary_item_id'],
     802                'type'              => $r['type'],
     803                'invite_sent'       => $r['invite_sent'],
     804                'accepted'          => $r['accepted'],
     805                'search_terms'      => $r['search_terms'],
     806            )
     807        );
    800808
    801809        // ORDER BY.
    802         $sql['orderby'] = self::get_order_by_sql( array(
    803             'order_by'   => $r['order_by'],
    804             'sort_order' => $r['sort_order']
    805         ) );
     810        $sql['orderby'] = self::get_order_by_sql(
     811            array(
     812                'order_by'   => $r['order_by'],
     813                'sort_order' => $r['sort_order'],
     814            )
     815        );
    806816
    807817        // LIMIT %d, %d.
    808         $sql['pagination'] = self::get_paged_sql( array(
    809             'page'     => $r['page'],
    810             'per_page' => $r['per_page'],
    811         ) );
     818        $sql['pagination'] = self::get_paged_sql(
     819            array(
     820                'page'     => $r['page'],
     821                'per_page' => $r['per_page'],
     822            )
     823        );
    812824
    813825        $paged_invites_sql = "{$sql['select']} {$sql['fields']} {$sql['from']} {$sql['where']} {$sql['orderby']} {$sql['pagination']}";
    … …  
    840852        $uncached_ids = bp_get_non_cached_ids( $paged_invite_ids, 'bp_invitations' );
    841853        if ( $uncached_ids ) {
    842             $ids_sql = implode( ',', array_map( 'intval', $uncached_ids ) );
     854            $ids_sql      = implode( ',', array_map( 'intval', $uncached_ids ) );
    843855            $data_objects = $wpdb->get_results( "SELECT i.* FROM {$invites_table_name} i WHERE i.id IN ({$ids_sql})" );
    844856            foreach ( $data_objects as $data_object ) {
    … …  
    859871     *
    860872     * @since 5.0.0
     873     *
     874     * @global wpdb $wpdb WordPress database object.
    861875     *
    862876     * @see BP_Invitation::get() for a description of
    … …  
    893907        );
    894908
    895         // Build the query
    896         $select_sql = "SELECT COUNT(*)";
     909        // Build the query.
     910        $select_sql = 'SELECT COUNT(*)';
    897911        $from_sql   = "FROM {$invites_table_name}";
    898912        $where_sql  = self::get_where_sql( $r );
    899913        $sql        = "{$select_sql} {$from_sql} {$where_sql}";
    900914
    901         // Return the queried results
     915        // Return the queried results.
    902916        return $wpdb->get_var( $sql );
    903917    }
    … …  
    921935    public static function update( $update_args = array(), $where_args = array() ) {
    922936        $update = self::get_query_clauses( $update_args );
    923         $where  = self::get_query_clauses( $where_args  );
     937        $where  = self::get_query_clauses( $where_args );
    924938
    925939        /**
    … …  
    937951
    938952        // Clear matching items from the cache.
    939         $cache_args = $where_args;
     953        $cache_args           = $where_args;
    940954        $cache_args['fields'] = 'ids';
    941         $maybe_cached_ids = self::get( $cache_args );
     955        $maybe_cached_ids     = self::get( $cache_args );
    942956        foreach ( $maybe_cached_ids as $invite_id ) {
    943957            wp_cache_delete( $invite_id, 'bp_invitations' );
    … …  
    984998
    985999        // Clear matching items from the cache.
    986         $cache_args = $args;
     1000        $cache_args           = $args;
    9871001        $cache_args['fields'] = 'ids';
    988         $maybe_cached_ids = self::get( $cache_args );
     1002        $maybe_cached_ids     = self::get( $cache_args );
    9891003        foreach ( $maybe_cached_ids as $invite_id ) {
    9901004            wp_cache_delete( $invite_id, 'bp_invitations' );
    … …  
    10161030     *
    10171031     * @param int $id ID of the invitation item to be deleted.
    1018      * @return bool True on success, false on failure.
     1032     * @return bool|int Number of rows deleted on success, false on failure.
    10191033     */
    10201034    public static function delete_by_id( $id ) {
    1021         return self::delete( array(
    1022             'id' => $id,
    1023         ) );
     1035        return self::delete( array( 'id' => $id ) );
    10241036    }
    10251037
    … …  
    11531165        return self::update( $update_args, $args );
    11541166    }
    1155 
    11561167}

  • trunk/src/bp-core/classes/class-bp-members-suggestions.php

    r13372 r13888  
    3939    );
    4040
    41 
    4241    /**
    4342     * Validate and sanitise the parameters for the suggestion service query.
    … …  
    5857         * @param BP_Members_Suggestions $suggestions Current BP_Members_Suggestions instance.
    5958         */
    60         $this->args                 = apply_filters( 'bp_members_suggestions_args', $this->args, $this );
     59        $this->args = apply_filters( 'bp_members_suggestions_args', $this->args, $this );
    6160
    6261        // Check for invalid or missing mandatory parameters.
    … …  
    113112        }
    114113
    115 
    116114        $user_query = new BP_User_Query( $user_query );
    117115        $results    = array();
    … …  
    120118            $result          = new stdClass();
    121119            $result->ID      = $user->user_nicename;
    122             $result->image   = bp_core_fetch_avatar( array( 'html' => false, 'item_id' => $user->ID ) );
     120            $result->image   = bp_core_fetch_avatar(
     121                array(
     122                    'html'    => false,
     123                    'item_id' => $user->ID,
     124                )
     125            );
    123126            $result->name    = bp_core_get_user_displayname( $user->ID );
    124127            $result->user_id = $user->ID;

  • trunk/src/bp-core/classes/class-bp-optout.php

    r13395 r13888  
    2525     *
    2626     * @since 8.0.0
    27      * @access public
     27     *
    2828     * @var int
    2929     */
    … …  
    3535     *
    3636     * @since 8.0.0
    37      * @access public
     37     *
    3838     * @var string
    3939     */
    … …  
    4444     *
    4545     * @since 8.0.0
    46      * @access public
     46     *
    4747     * @var int
    4848     */
    … …  
    5454     *
    5555     * @since 8.0.0
    56      * @access public
     56     *
    5757     * @var string
    5858     */
    … …  
    6363     *
    6464     * @since 8.0.0
    65      * @access public
     65     *
    6666     * @var string
    6767     */
    … …  
    7575     * @since 8.0.0
    7676     *
    77      * @param int $id Optional. Provide an ID to access an existing
    78      *        optout item.
     77     * @param int $id Optional. Provide an ID to access an existing optout item.
    7978     */
    8079    public function __construct( $id = 0 ) {
    … …  
    8988     *
    9089     * @since 8.0.0
    91      * @access public
     90     *
    9291     * @return string
    9392     */
    … …  
    103102     * @global wpdb $wpdb WordPress database object.
    104103     *
    105      * @return bool True on success, false on failure.
     104     * @return bool
    106105     */
    107106    public function save() {
    108107
    109         // Return value
     108        // Return value.
    110109        $retval = false;
    111110
    112         // Default data and format
    113         $data = array(
     111        // Default data and format.
     112        $data        = array(
    114113            'email_address_hash' => $this->email_address,
    115114            'user_id'            => $this->user_id,
    … …  
    131130        if ( ! empty( $this->id ) ) {
    132131            $result = self::_update( $data, array( 'ID' => $this->id ), $data_format, array( '%d' ) );
    133         // Insert.
     132            // Insert.
    134133        } else {
    135134            $result = self::_insert( $data, $data_format );
    … …  
    151150         * @since 8.0.0
    152151         *
    153          * @param BP_optout object $this Characteristics of the opt-out just saved.
     152         * @param BP_optout $bp_optout Characteristics of the opt-out just saved.
    154153         */
    155154        do_action_ref_array( 'bp_optout_after_save', array( &$this ) );
    … …  
    189188        $this->email_type    = sanitize_key( $optout->email_type );
    190189        $this->date_modified = $optout->date_modified;
    191 
    192190    }
    193191
    … …  
    198196     *
    199197     * @since 8.0.0
     198     *
     199     * @global wpdb $wpdb WordPress database object.
    200200     *
    201201     * @param array $data {
    202202     *     Array of optout data, passed to {@link wpdb::insert()}.
    203      *     @type string $email_address     The hashed email address of the user that wishes to opt out of
     203     *     @type string $email_address     The hashed email address of the user that wishes to opt out of
    204204     *                                     communications from this site.
    205      *     @type int    $user_id           The ID of the user that generated the contact that resulted in the opt-out.
    206      *     @type string $email_type        The type of email contact that resulted in the opt-out.
    207      *     @type string $date_modified     Date the opt-out was last modified.
     205     *     @type int    $user_id           The ID of the user that generated the contact that resulted in the opt-out.
     206     *     @type string $email_type        The type of email contact that resulted in the opt-out.
     207     *     @type string $date_modified     Date the opt-out was last modified.
    208208     * }
    209209     * @param array $data_format See {@link wpdb::insert()}.
    … …  
    212212    protected static function _insert( $data = array(), $data_format = array() ) {
    213213        global $wpdb;
     214
    214215        // We must lowercase and hash the email address at insert.
    215216        $email                      = strtolower( $data['email_address_hash'] );
    216217        $data['email_address_hash'] = wp_hash( $email );
    217         return $wpdb->insert( BP_Optout::get_table_name(), $data, $data_format );
     218        return $wpdb->insert( self::get_table_name(), $data, $data_format );
    218219    }
    219220
    … …  
    222223     *
    223224     * @since 8.0.0
     225     *
     226     * @global wpdb $wpdb WordPress database object.
    224227     *
    225228     * @see wpdb::update() for further description of paramater formats.
    … …  
    244247        }
    245248
    246         return $wpdb->update( BP_Optout::get_table_name(), $data, $where, $data_format, $where_format );
     249        return $wpdb->update( self::get_table_name(), $data, $where, $data_format, $where_format );
    247250    }
    248251
    … …  
    251254     *
    252255     * @since 8.0.0
     256     *
     257     * @global wpdb $wpdb WordPress database object.
    253258     *
    254259     * @see wpdb::update() for further description of paramater formats.
    … …  
    262267    protected static function _delete( $where = array(), $where_format = array() ) {
    263268        global $wpdb;
    264         return $wpdb->delete( BP_Optout::get_table_name(), $where, $where_format );
     269        return $wpdb->delete( self::get_table_name(), $where, $where_format );
    265270    }
    266271
    … …  
    272277     *
    273278     * @since 8.0.0
     279     *
     280     * @global wpdb $wpdb WordPress database object.
    274281     *
    275282     * @param array $args See {@link BP_optout::get()} for more details.
    … …  
    358365     */
    359366    protected static function get_order_by_sql( $args = array() ) {
    360 
    361367        $conditions = array();
    362368        $retval     = '';
    … …  
    398404     * @since 8.0.0
    399405     *
     406     * @global wpdb $wpdb WordPress database object.
     407     *
    400408     * @param array $args See {@link BP_optout::get()} for more details.
    401409     * @return string LIMIT clause.
    … …  
    409417        // Custom LIMIT.
    410418        if ( ! empty( $args['page'] ) && ! empty( $args['per_page'] ) ) {
    411             $page     = absint( $args['page']     );
     419            $page     = absint( $args['page'] );
    412420            $per_page = absint( $args['per_page'] );
    413421            $offset   = $per_page * ( $page - 1 );
    414             $retval   = $wpdb->prepare( "LIMIT %d, %d", $offset, $per_page );
     422            $retval   = $wpdb->prepare( 'LIMIT %d, %d', $offset, $per_page );
    415423        }
    416424
    … …  
    498506     * @since 8.0.0
    499507     *
     508     * @global wpdb $wpdb WordPress database object.
     509     *
    500510     * @param array $args {
    501511     *     Associative array of arguments. All arguments but $page and
    … …  
    505515     *                                           Can be an array of IDs.
    506516     *     @type string|array $email_address     Email address of users who have opted out
    507      *                                           being queried. Can be an array of addresses.
     517     *                                           being queried. Can be an array of addresses.
    508518     *     @type int|array    $user_id           ID of user whose communication prompted the
    509519     *                                           opt-out. Can be an array of IDs.
    … …  
    519529     *                                           Default: false (no pagination,
    520530     *                                           all items).
    521      *     @type string       $fields            Which fields to return. Specify 'email_addresses' to
    522      *                                           fetch a list of opt-out email_addresses.
    523      *                                           Specify 'user_ids' to
    524      *                                           fetch a list of opt-out user_ids.
    525      *                                           Specify 'ids' to fetch a list of opt-out IDs.
    526      *                                           Default: 'all' (return BP_Optout objects).
     531     *     @type string       $fields            Which fields to return. Specify 'email_addresses' to
     532     *                                           fetch a list of opt-out email_addresses.
     533     *                                           Specify 'user_ids' to
     534     *                                           fetch a list of opt-out user_ids.
     535     *                                           Specify 'ids' to fetch a list of opt-out IDs.
     536     *                                           Default: 'all' (return BP_Optout objects).
    527537     * }
    528538     *
    … …  
    531541    public static function get( $args = array() ) {
    532542        global $wpdb;
    533         $optouts_table_name = BP_Optout::get_table_name();
     543        $optouts_table_name = self::get_table_name();
    534544
    535545        // Parse the arguments.
    … …  
    552562
    553563        $sql = array(
    554             'select'     => "SELECT",
     564            'select'     => 'SELECT',
    555565            'fields'     => '',
    556566            'from'       => "FROM {$optouts_table_name} o",
    … …  
    561571
    562572        if ( 'user_ids' === $r['fields'] ) {
    563             $sql['fields'] = "DISTINCT o.user_id";
    564         } else if ( 'email_addresses' === $r['fields'] ) {
    565             $sql['fields'] = "DISTINCT o.email_address_hash";
     573            $sql['fields'] = 'DISTINCT o.user_id';
     574        } elseif ( 'email_addresses' === $r['fields'] ) {
     575            $sql['fields'] = 'DISTINCT o.email_address_hash';
    566576        } else {
    567577            $sql['fields'] = 'DISTINCT o.id';
    … …  
    583593            array(
    584594                'order_by'   => $r['order_by'],
    585                 'sort_order' => $r['sort_order']
     595                'sort_order' => $r['sort_order'],
    586596            )
    587597        );
    … …  
    620630            // We only want the field that was found.
    621631            return array_map( 'intval', $paged_optout_ids );
    622         } else if ( 'email_addresses' === $r['fields'] ) {
     632        } elseif ( 'email_addresses' === $r['fields'] ) {
    623633            return $paged_optout_ids;
    624634        }
    … …  
    626636        $uncached_ids = bp_get_non_cached_ids( $paged_optout_ids, 'bp_optouts' );
    627637        if ( $uncached_ids ) {
    628             $ids_sql = implode( ',', array_map( 'intval', $uncached_ids ) );
     638            $ids_sql      = implode( ',', array_map( 'intval', $uncached_ids ) );
    629639            $data_objects = $wpdb->get_results( "SELECT o.* FROM {$optouts_table_name} o WHERE o.id IN ({$ids_sql})" );
    630640            foreach ( $data_objects as $data_object ) {
    … …  
    646656     * @since 8.0.0
    647657     *
     658     * @global wpdb $wpdb WordPress database object.
     659     *
    648660     * @see BP_optout::get() for a description of
    649661     *      arguments.
    … …  
    654666    public static function get_total_count( $args ) {
    655667        global $wpdb;
    656         $optouts_table_name = BP_Optout::get_table_name();
     668        $optouts_table_name = self::get_table_name();
    657669
    658670        // Parse the arguments.
    659         $r  = bp_parse_args(
     671        $r = bp_parse_args(
    660672            $args,
    661673            array(
    … …  
    674686        );
    675687
    676         // Build the query
    677         $select_sql = "SELECT COUNT(*)";
     688        // Build the query.
     689        $select_sql = 'SELECT COUNT(*)';
    678690        $from_sql   = "FROM {$optouts_table_name}";
    679691        $where_sql  = self::get_where_sql( $r );
    680692        $sql        = "{$select_sql} {$from_sql} {$where_sql}";
    681693
    682         // Return the queried results
     694        // Return the queried results.
    683695        return $wpdb->get_var( $sql );
    684696    }
    … …  
    702714    public static function update( $update_args = array(), $where_args = array() ) {
    703715        $update = self::get_query_clauses( $update_args );
    704         $where  = self::get_query_clauses( $where_args  );
     716        $where  = self::get_query_clauses( $where_args );
    705717
    706718        /**
    … …  
    736748        do_action( 'bp_optout_after_update', $where_args, $update_args );
    737749
    738         return $retval;
     750        return $retval;
    739751    }
    740752
    … …  
    795807     * @see BP_Optout::get() for a description of accepted parameters.
    796808     *
     809     * @param array $args Arguments to pass to BP_optout::get().
    797810     * @return int|bool ID of first found invitation or false if none found.
    798811     */
    … …  
    801814
    802815        $args['fields'] = 'ids';
    803         $optouts        = BP_Optout::get( $args );
     816        $optouts        = self::get( $args );
    804817        if ( $optouts ) {
    805818            $exists = current( $optouts );
    … …  
    818831     *
    819832     * @param int $id ID of the opt-out item to be deleted.
    820      * @return bool True on success, false on failure.
     833     * @return bool|int Number of rows deleted on success, false on failure.
    821834     */
    822835    public static function delete_by_id( $id ) {
    823         return self::delete( array(
    824             'id' => $id,
    825         ) );
     836        return self::delete( array( 'id' => $id ) );
    826837    }
    827838}

  • trunk/src/bp-core/classes/class-bp-optouts-list-table.php

    r13882 r13888  
    131131             * @param string $url_base Current URL base for view.
    132132             */
    133             do_action( 'bp_optouts_list_table_get_views', $url_base ); ?>
     133            do_action( 'bp_optouts_list_table_get_views', $url_base );
     134            ?>
    134135        </ul>
    135     <?php
     136        <?php
    136137    }
    137138
    … …  
    146147     * @param array $which Current table nav item.
    147148     */
    148     public function extra_tablenav( $which ) {
    149         return;
    150     }
     149    public function extra_tablenav( $which ) {}
    151150
    152151    /**
    … …  
    224223        $style = '';
    225224        foreach ( $this->items as $optout ) {
    226             $style = 'alt' == $style ? '' : 'alt';
     225            $style = 'alt' === $style ? '' : 'alt';
    227226
    228227            // Escapes are made into `self::single_row()`.
    … …  
    265264     */
    266265    public function column_cb( $optout = null ) {
    267     ?>
     266        ?>
    268267        <label class="screen-reader-text" for="optout_<?php echo intval( $optout->id ); ?>">
    269268            <?php
    … …  
    272271            ?>
    273272        </label>
    274         <input type="checkbox" id="optout_<?php echo intval( $optout->id ) ?>" name="optout_ids[]" value="<?php echo esc_attr( $optout->id ) ?>" />
     273        <input type="checkbox" id="optout_<?php echo intval( $optout->id ); ?>" name="optout_ids[]" value="<?php echo esc_attr( $optout->id ); ?>" />
    275274        <?php
    276275    }
    … …  
    281280     * @since 8.0.0
    282281     *
    283      * @param BP_Optout $optout BP_Optout object.
     282     * @param BP_Optout|null $optout BP_Optout object.
    284283     */
    285284    public function column_email_address( $optout = null ) {
    … …  
    295294
    296295        // Delete link.
    297         $delete_link = add_query_arg(
     296        $delete_link       = add_query_arg(
    298297            array(
    299298                'page'      => 'bp-optouts',
    … …  
    325324     * @since 8.0.0
    326325     *
    327      * @param BP_Optout $optout BP_Optout object.
     326     * @param BP_Optout|null $optout BP_Optout object.
    328327     */
    329328    public function column_username( $optout = null ) {
    … …  
    347346                    'height' => true,
    348347                    'width'  => true,
    349                 )
     348                ),
    350349            )
    351350        );
    … …  
    358357     * @since 8.0.0
    359358     *
    360      * @param BP_Optout $optout BP_Optout object.
     359     * @param BP_Optout|null $optout BP_Optout object.
    361360     */
    362361    public function column_user_registered( $optout = null ) {
    … …  
    367366        }
    368367
    369         echo esc_html( mysql2date( 'Y/m/d g:i:s a', $inviter->user_registered  ) );
     368        echo esc_html( mysql2date( 'Y/m/d g:i:s a', $inviter->user_registered ) );
    370369    }
    371370
    … …  
    375374     * @since 8.0.0
    376375     *
    377      * @param BP_Optout $optout BP_Optout object.
     376     * @param BP_Optout|null $optout BP_Optout object.
    378377     */
    379378    public function column_email_type( $optout = null ) {
    … …  
    386385     * @since 8.0.0
    387386     *
    388      * @param BP_Optout $optout BP_Optout object.
     387     * @param BP_Optout|null $optout BP_Optout object.
    389388     */
    390389    public function column_email_type_description( $optout = null ) {
    … …  
    394393            echo esc_html( $type_term->description );
    395394        }
    396 
    397395    }
    398396
    … …  
    402400     * @since 8.0.0
    403401     *
    404      * @param BP_Optout $optout BP_Optout object.
     402     * @param BP_Optout|null $optout BP_Optout object.
    405403     */
    406404    public function column_optout_date_modified( $optout = null ) {
    … …  
    413411     * @since 8.0.0
    414412     *
    415      * @param BP_Optout $optout      BP_Optout object.
    416      * @param string    $column_name The column name.
     413     * @param BP_Optout|null $optout      BP_Optout object.
     414     * @param string         $column_name The column name.
    417415     * @return string
    418416     */
    419     function column_default( $optout = null, $column_name = '' ) {
     417    public function column_default( $optout = null, $column_name = '' ) {
    420418
    421419        /**
    … …  
    424422         * @since 8.0.0
    425423         *
    426          * @param string    $column_name The column name.
    427          * @param BP_Optout $optout      BP_Optout object.
     424         * @param string         $column_name The column name.
     425         * @param BP_Optout|null $optout      BP_Optout object or null.
    428426         */
    429427        return apply_filters( 'bp_optouts_management_custom_column', '', $column_name, $optout );

  • trunk/src/bp-core/classes/class-bp-phpmailer.php

    r13297 r13888  
    6969         */
    7070        $phpmailer->Subject = $email->get_subject( 'replace-tokens' );
    71         $content_plaintext = PHPMailer\PHPMailer\PHPMailer::normalizeBreaks( $email->get_content_plaintext( 'replace-tokens' ) );
     71        $content_plaintext  = PHPMailer\PHPMailer\PHPMailer::normalizeBreaks( $email->get_content_plaintext( 'replace-tokens' ) );
    7272
    7373        if ( $email->get( 'content_type' ) === 'html' ) {
    … …  
    140140    }
    141141
    142     /*
    143      * Utility/helper functions.
    144      */
    145 
    146142    /**
    147143     * Get an appropriate hostname for the email. Varies depending on site configuration.

  • trunk/src/bp-core/classes/class-bp-recursive-query.php

    r13887 r13888  
    2222     *
    2323     * @since 2.2.0
     24     *
    2425     * @var array
    2526     */
Note: See TracChangeset for help on using the changeset viewer.