Skip to:
Content

BuddyPress.org

Ticket #6605: 6605-02.patch

File 6605-02.patch, 1.3 KB (added by hnla, 10 years ago)

Re-factor checked attr function to prevent echoing to screen. Add 'for' attr to label.

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

     
    49534953                        $invites = groups_get_invites_for_group( $r['user_id'], $r['group_id'] );
    49544954
    49554955                        for ( $i = 0, $count = count( $friends ); $i < $count; ++$i ) {
    4956                                 $checked = in_array( (int) $friends[ $i ]['id'], (array) $invites );
    4957                                 $items[] = '<' . $r['separator'] . '><input' . checked( $checked ) . ' type="checkbox" name="friends[]" id="f-' . esc_attr( $friends[ $i ]['id'] ) . '" value="' . esc_attr( $friends[ $i ]['id'] ) . '" /> ' . esc_html( $friends[ $i ]['full_name'] ) . '</' . $r['separator'] . '>';
     4956
     4957                                if( $checked = in_array( (int) $friends[ $i ]['id'], (array) $invites ) ) :
     4958                                        $is_checked = ' checked="checked"';
     4959                                else :
     4960                                        $is_checked = '';
     4961                                endif;
     4962
     4963                                $items[] = '<' . $r['separator'] . '><label for="f-' . esc_attr( $friends[ $i ]['id'] ) . '"><input' . $is_checked . ' type="checkbox" name="friends[]" id="f-' . esc_attr( $friends[ $i ]['id'] ) . '" value="' . esc_attr( $friends[ $i ]['id'] ) . '" /> ' . esc_html( $friends[ $i ]['full_name'] ) . '</label></' . $r['separator'] . '>';
    49584964                        }
    49594965                }
    49604966