Skip to:
Content

BuddyPress.org

Changeset 3270


Ignore:
Timestamp:
09/17/2010 06:07:07 PM (9 years ago)
Author:
johnjamesjacoby
Message:

Clean up HTML in xprofile admin area. Add maybe_unserialize to get_value_byid. Fixes #2637.

Location:
branches/1.2/bp-xprofile
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/1.2/bp-xprofile/bp-xprofile-classes.php

    r3143 r3270  
    483483        $input_types = array( 'checkbox', 'selectbox', 'multiselectbox', 'radio' );
    484484
    485         foreach ($input_types as $type) {
     485        foreach ( $input_types as $type ) {
    486486            $default_name = '';
    487487
    488             if ( 'multiselectbox' == $type || 'checkbox' == $type ) {
     488            if ( 'multiselectbox' == $type || 'checkbox' == $type )
    489489                $default_input = 'checkbox';
    490             } else {
    491                 $default_input = 'radio';
    492             }
    493         ?>
     490            else
     491                $default_input = 'radio'; ?>
     492
    494493            <div id="<?php echo $type ?>" class="options-box" style="<?php if ( $this->type != $type ) { ?>display: none;<?php } ?> margin-left: 15px;">
    495494                <h4><?php _e('Please enter options for this Field:', 'buddypress') ?></h4>
     
    520519                }
    521520
    522                 if ( !empty($options) ) {
    523                     for ( $i = 0; $i < count($options); $i++ ) {
     521                if ( !empty( $options ) ) {
     522                    for ( $i = 0; $i < count( $options ); $i++ ) {
    524523                        $j = $i + 1;
    525524
    526525                        if ( 'multiselectbox' == $type || 'checkbox' == $type )
    527                             $default_name = '[' . $j . ']';
    528                     ?>
    529                         <p><?php _e('Option', 'buddypress') ?> <?php echo $j ?>:
    530                            <input type="text" name="<?php echo $type ?>_option[<?php echo $j ?>]" id="<?php echo $type ?>_option<?php echo $j ?>" value="<?php echo esc_attr( $options[$i]->name ) ?>" />
    531                            <input type="<?php echo $default_input ?>" name="isDefault_<?php echo $type ?>_option<?php echo $default_name ?>" <?php if ( (int) $options[$i]->is_default_option ) {?> checked="checked"<?php } ?> " value="<?php echo $j ?>" /> <?php _e( 'Default Value', 'buddypress' ) ?>
    532                             <?php if ( $j != 1 &&
    533                                 $options[$i]->id != -1 ) : ?><a href="admin.php?page=bp-profile-setup&amp;mode=delete_option&amp;option_id=<?php echo $options[$i]->id ?>" class="ajax-option-delete" id="delete-<?php echo $options[$i]->id ?>">[x]</a><?php endif ?></p>
     526                            $default_name = '[' . $j . ']'; ?>
     527
     528                        <p>
     529                            <?php _e('Option', 'buddypress') ?> <?php echo $j ?>:
     530                            <input type="text" name="<?php echo $type ?>_option[<?php echo $j ?>]" id="<?php echo $type ?>_option<?php echo $j ?>" value="<?php echo esc_attr( $options[$i]->name ) ?>" />
     531                            <input type="<?php echo $default_input ?>" name="isDefault_<?php echo $type ?>_option<?php echo $default_name ?>" <?php if ( (int) $options[$i]->is_default_option ) {?> checked="checked"<?php } ?> value="<?php echo $j ?>" /> <?php _e( 'Default Value', 'buddypress' ) ?>
     532
     533                            <?php if ( $j != 1 && $options[$i]->id != -1 ) : ?>
     534
     535                                   <a href="admin.php?page=bp-profile-setup&amp;mode=delete_option&amp;option_id=<?php echo $options[$i]->id ?>" class="ajax-option-delete" id="delete-<?php echo $options[$i]->id ?>">[x]</a>
     536
     537                            <?php endif ?>
     538
    534539                        </p>
     540
    535541                    <?php } // end for ?>
     542
    536543                    <input type="hidden" name="<?php echo $type ?>_option_number" id="<?php echo $type ?>_option_number" value="<?php echo $j + 1 ?>" />
    537544
     
    539546                } else {
    540547                    if ( 'multiselectbox' == $type || 'checkbox' == $type )
    541                         $default_name = '[1]';
    542                 ?>
     548                        $default_name = '[1]'; ?>
    543549
    544550                    <p><?php _e('Option', 'buddypress') ?> 1: <input type="text" name="<?php echo $type ?>_option[1]" id="<?php echo $type ?>_option1" />
     
    547553
    548554                <?php } // end if ?>
     555
    549556                <div id="<?php echo $type ?>_more"></div>
    550557                <p><a href="javascript:add_option('<?php echo $type ?>')"><?php _e('Add Another Option', 'buddypress') ?></a></p>
     
    627634                </p>
    628635
     636            </div>
     637
    629638            <div class="clear"></div>
    630639
     
    800809            $user_ids = implode( ',', (array)$user_ids );
    801810            $data = $wpdb->get_results( $wpdb->prepare( "SELECT user_id, value FROM {$bp->profile->table_name_data} WHERE field_id = %d AND user_id IN ({$user_ids})", $field_id ) );
    802         } else
     811        } else {
    803812            $data = $wpdb->get_var( $wpdb->prepare( "SELECT value FROM {$bp->profile->table_name_data} WHERE field_id = %d AND user_id = %d", $field_id, $user_ids ) );
    804 
    805         return $data;
     813        }
     814
     815        return maybe_unserialize( $data );
    806816    }
    807817
  • branches/1.2/bp-xprofile/bp-xprofile-templatetags.php

    r3260 r3270  
    377377
    378378                for ( $k = 0; $k < count($options); $k++ ) {
    379                     $option_values = maybe_unserialize( BP_XProfile_ProfileData::get_value_byid( $options[$k]->parent_id ) );
     379                    $option_values = BP_XProfile_ProfileData::get_value_byid( $options[$k]->parent_id );
    380380                    $option_values = (array)$option_values;
    381381
Note: See TracChangeset for help on using the changeset viewer.