Skip to:
Content

Changeset 8214


Ignore:
Timestamp:
03/30/2014 06:57:53 PM (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.