Skip to:
Content

Changeset 8214


Ignore:
Timestamp:
03/30/14 18:57:53 (4 years ago)
Author:
johnjamesjacoby
Message:

Prevent outputting visibility settings table for empty profile field groups. See #5352.

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-templates/bp-legacy/buddypress/members/single/settings/profile.php

    r8210 r8214  
    77        <?php while ( bp_profile_groups() ) : bp_the_profile_group(); ?> 
    88 
    9             <table class="profile-settings" id="xprofile-settings-<?php bp_the_profile_group_slug(); ?>"> 
    10                 <thead> 
    11                     <tr> 
    12                         <th class="title field-group-name"><?php bp_the_profile_group_name(); ?></th> 
    13                         <th class="title"><?php _e( 'Visibility', 'buddypress' ); ?></th> 
    14                     </tr> 
    15                 </thead> 
     9            <?php if ( bp_profile_fields() ) : ?> 
    1610 
    17                 <tbody> 
     11                <table class="profile-settings" id="xprofile-settings-<?php bp_the_profile_group_slug(); ?>"> 
     12                    <thead> 
     13                        <tr> 
     14                            <th class="title field-group-name"><?php bp_the_profile_group_name(); ?></th> 
     15                            <th class="title"><?php _e( 'Visibility', 'buddypress' ); ?></th> 
     16                        </tr> 
     17                    </thead> 
    1818 
    19                     <?php while ( bp_profile_fields() ) : bp_the_profile_field(); ?> 
     19                    <tbody> 
    2020 
    21                         <tr <?php bp_field_css_class(); ?>> 
    22                             <td class="field-name"><?php bp_the_profile_field_name(); ?></td> 
    23                             <td class="field-visibility"><?php bp_profile_settings_visibility_select(); ?></td> 
    24                         </tr> 
     21                        <?php while ( bp_profile_fields() ) : bp_the_profile_field(); ?> 
    2522 
    26                     <?php endwhile; ?> 
     23                            <tr <?php bp_field_css_class(); ?>> 
     24                                <td class="field-name"><?php bp_the_profile_field_name(); ?></td> 
     25                                <td class="field-visibility"><?php bp_profile_settings_visibility_select(); ?></td> 
     26                            </tr> 
    2727 
    28                 </tbody> 
    29             </table> 
     28                        <?php endwhile; ?> 
     29 
     30                    </tbody> 
     31                </table> 
     32 
     33            <?php endif; ?> 
    3034 
    3135        <?php endwhile; ?> 
  • trunk/bp-themes/bp-default/members/single/settings/profile.php

    r8210 r8214  
    5757                        <?php while ( bp_profile_groups() ) : bp_the_profile_group(); ?> 
    5858 
    59                             <table class="profile-settings" id="xprofile-settings-<?php bp_the_profile_group_slug(); ?>"> 
    60                                 <thead> 
    61                                     <tr> 
    62                                         <th class="title field-group-name"><?php bp_the_profile_group_name(); ?></th> 
    63                                         <th class="title"><?php _e( 'Visibility', 'buddypress' ); ?></th> 
    64                                     </tr> 
    65                                 </thead> 
     59                            <?php if ( bp_profile_fields() ) : ?> 
    6660 
    67                                 <tbody> 
     61                                <table class="profile-settings" id="xprofile-settings-<?php bp_the_profile_group_slug(); ?>"> 
     62                                    <thead> 
     63                                        <tr> 
     64                                            <th class="title field-group-name"><?php bp_the_profile_group_name(); ?></th> 
     65                                            <th class="title"><?php _e( 'Visibility', 'buddypress' ); ?></th> 
     66                                        </tr> 
     67                                    </thead> 
    6868 
    69                                     <?php while ( bp_profile_fields() ) : bp_the_profile_field(); ?> 
     69                                    <tbody> 
    7070 
    71                                         <tr <?php bp_field_css_class(); ?>> 
    72                                             <td class="field-name"><?php bp_the_profile_field_name(); ?></td> 
    73                                             <td class="field-visibility"><?php bp_profile_settings_visibility_select(); ?></td> 
    74                                         </tr> 
     71                                        <?php while ( bp_profile_fields() ) : bp_the_profile_field(); ?> 
    7572 
    76                                     <?php endwhile; ?> 
     73                                            <tr <?php bp_field_css_class(); ?>> 
     74                                                <td class="field-name"><?php bp_the_profile_field_name(); ?></td> 
     75                                                <td class="field-visibility"><?php bp_profile_settings_visibility_select(); ?></td> 
     76                                            </tr> 
    7777 
    78                                 </tbody> 
    79                             </table> 
     78                                        <?php endwhile; ?> 
     79 
     80                                    </tbody> 
     81                                </table> 
     82 
     83                            <?php endif; ?> 
    8084 
    8185                        <?php endwhile; ?> 
  • trunk/bp-xprofile/bp-xprofile-template.php

    r8210 r8214  
    290290 
    291291        $field_ids = ''; 
    292         foreach ( (array) $group->fields as $field ) 
    293             $field_ids .= $field->id . ','; 
     292 
     293        if ( !empty( $group->fields ) ) { 
     294            foreach ( (array) $group->fields as $field ) { 
     295                $field_ids .= $field->id . ','; 
     296            } 
     297        } 
    294298 
    295299        return substr( $field_ids, 0, -1 ); 
Note: See TracChangeset for help on using the changeset viewer.