Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
12/09/2011 04:22:44 AM (15 years ago)
Author:
johnjamesjacoby
Message:

Remove debug cruft from r5466.

In bp_core_fetch_avatar():

  • Use switch statements to set defaults
  • Use empty() to prevent notices
  • General code clean-up
  • See #3806
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/bp-core/bp-core-avatars.php

    r5466 r5468  
    111111        extract( $params, EXTR_SKIP );
    112112
    113         // Set item_id if not passed
    114         if ( !$item_id ) {
    115                 if ( 'user' == $object )
    116                         $item_id = bp_displayed_user_id();
    117                 else if ( bp_is_active( 'groups' ) && 'group' == $object )
    118                         $item_id = $bp->groups->current_group->id;
    119                 else if ( 'blog' == $object )
    120                         $item_id = $current_blog->id;
     113        /** Set item_id ***********************************************************/
     114       
     115        if ( empty( $item_id ) ) {
     116
     117                switch ( $object ) {
     118
     119                        case 'blog'  :
     120                                $item_id = $current_blog->id;
     121                                break;
     122
     123                        case 'group' :
     124                                if ( bp_is_active( 'groups' ) ) {
     125                                        $item_id = $bp->groups->current_group->id;
     126                                } else {
     127                                        $item_id = false;
     128                                }
     129
     130                                break;
     131                       
     132                        case 'user'  :
     133                        default      :
     134                                $item_id = bp_displayed_user_id();
     135                                break;
     136                }
    121137
    122138                $item_id = apply_filters( 'bp_core_avatar_item_id', $item_id, $object );
    123139
    124                 if ( !$item_id )
     140                if ( empty( $item_id ) ) {
    125141                        return false;
    126         }
    127 
    128         // Set avatar_dir if not passed (uses $object)
    129         if ( !$avatar_dir ) {
    130                 if ( 'user' == $object )
    131                         $avatar_dir = 'avatars';
    132                 else if ( bp_is_active( 'groups' ) && 'group' == $object )
    133                         $avatar_dir = 'group-avatars';
    134                 else if ( 'blog' == $object )
    135                         $avatar_dir = 'blog-avatars';
     142                }
     143        }
     144
     145        /** Set avatar_dir ********************************************************/
     146
     147        if ( empty( $avatar_dir ) ) {
     148
     149                switch ( $object ) {
     150
     151                        case 'blog'  :
     152                                $avatar_dir = 'blog-avatars';
     153                                break;
     154
     155                        case 'group' :
     156                                if ( bp_is_active( 'groups' ) ) {
     157                                        $avatar_dir = 'group-avatars';
     158                                } else {
     159                                        $avatar_dir = false;
     160                                }
     161
     162                                break;
     163                       
     164                        case 'user'  :
     165                        default      :
     166                                $avatar_dir = 'avatars';
     167                                break;
     168                }
    136169
    137170                $avatar_dir = apply_filters( 'bp_core_avatar_dir', $avatar_dir, $object );
    138171
    139                 if ( !$avatar_dir )
     172                if ( empty( $avatar_dir ) ) {
    140173                        return false;
    141         }
    142 
    143         // Add an identifying class to each item
    144         $class .= ' ' . $object . '-' . $item_id . '-avatar';
    145 
    146         // Get item name for alt/title tags
    147         $item_name = '';
    148        
    149         // Backpat for the 'alt' parameter, which used to encourage sprintf()-style values
     174                }
     175        }
     176
     177        /** <img> alt *************************************************************/
     178
    150179        if ( false !== strpos( $alt, '%s' ) || false !== strpos( $alt, '%1$s' ) ) {
    151                 var_dump( 'yes' );
    152                 if ( 'user' == $object )
    153                         $item_name = bp_core_get_user_displayname( $item_id );
    154                 elseif ( 'group' == $object )
    155                         $item_name = bp_get_group_name( groups_get_group( array( 'group_id' => $item_id ) ) );
    156                 elseif ( 'blog' == $object )
    157                         $item_name = get_blog_option( $item_id, 'blogname' );
    158180               
    159                 $alt = sprintf( $alt, apply_filters( 'bp_core_avatar_alt', $item_name, $item_id, $object ) );
    160         }
     181                // Get item name for alt/title tags
     182                $item_name = '';
     183
     184                switch ( $object ) {
     185
     186                        case 'blog'  :
     187                                $item_name = get_blog_option( $item_id, 'blogname' );
     188                                break;
     189
     190                        case 'group' :
     191                                $item_name = bp_get_group_name( groups_get_group( array( 'group_id' => $item_id ) ) );
     192                                break;
     193                       
     194                        case 'user'  :
     195                        default :
     196                                $item_name = bp_core_get_user_displayname( $item_id );
     197                                break;
     198                }
     199
     200                $item_name = apply_filters( 'bp_core_avatar_alt', $item_name, $item_id, $object );
     201                $alt       = sprintf( $alt, $item_name );
     202        }
     203
     204        /** Sanity Checks *********************************************************/
    161205
    162206        // Get a fallback for the 'alt' parameter       
    163         if ( !$alt ) {
     207        if ( empty( $alt ) )
    164208                $alt = __( 'Avatar Image', 'buddypress' );
    165         }
    166209
    167210        // Set title tag, if it's been provided
    168         if ( $title )
     211        if ( !empty( $title ) )
    169212                $title = " title='" . esc_attr( apply_filters( 'bp_core_avatar_title', $title, $item_id, $object ) ) . "'";
    170213
    171214        // Set CSS ID if passed
    172215        if ( !empty( $css_id ) )
    173                 $css_id = " id='{$css_id}'";
    174 
    175         // Set avatar width
    176         if ( $width )
    177                 $html_width = " width='{$width}'";
     216                $css_id = ' id="' . $css_id . '"';
     217
     218        // Set image width
     219        if ( false !== $width )
     220                $html_width = ' width="' . $width . '"';
    178221        else
    179222                $html_width = ( 'thumb' == $type ) ? ' width="' . bp_core_avatar_thumb_width() . '"' : ' width="' . bp_core_avatar_full_width() . '"';
    180223
    181         // Set avatar height
    182         if ( $height )
    183                 $html_height = " height='{$height}'";
     224        // Set image height
     225        if ( false !== $height )
     226                $html_height = ' height="' . $height . '"';
    184227        else
    185228                $html_height = ( 'thumb' == $type ) ? ' height="' . bp_core_avatar_thumb_height() . '"' : ' height="' . bp_core_avatar_full_height() . '"';
    186229
    187         // Set avatar URL and DIR based on prepopulated constants
    188         $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url', bp_core_avatar_url() . '/' . $avatar_dir . '/' . $item_id, $item_id, $object, $avatar_dir );
    189         $avatar_folder_dir = apply_filters( 'bp_core_avatar_folder_dir', bp_core_avatar_upload_path() . '/' . $avatar_dir . '/' . $item_id, $item_id, $object, $avatar_dir );
     230        // Set img URL and DIR based on prepopulated constants
     231        $avatar_loc        = array();
     232        $avatar_loc[]      = trailingslashit( bp_core_avatar_upload_path() );
     233        $avatar_loc[]      = trailingslashit( bp_core_avatar_url() );
     234        $avatar_loc[]      = trailingslashit( $avatar_dir );
     235        $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url', ( $avatar_loc[1] . $avatar_loc[2] . $item_id ), $item_id, $object, $avatar_dir );
     236        $avatar_folder_dir = apply_filters( 'bp_core_avatar_folder_dir', ( $avatar_loc[0] . $avatar_loc[2] . $item_id ), $item_id, $object, $avatar_dir );
     237        // Add an identifying class
     238        $class .= ' ' . $object . '-' . $item_id . '-avatar';
    190239
    191240        /****
    … …  
    206255                        // Stash files in an array once to check for one that matches
    207256                        $avatar_files = array();
    208                         while ( false !== ( $avatar_file = readdir($av_dir) ) ) {
     257                        while ( false !== ( $avatar_file = readdir( $av_dir ) ) ) {
    209258                                // Only add files to the array (skip directories)
    210                                 if ( 2 < strlen( $avatar_file ) )
     259                                if ( 2 < strlen( $avatar_file ) ) {
    211260                                        $avatar_files[] = $avatar_file;
     261                                }
    212262                        }
    213263
    … …  
    217267                                // Check for current avatar
    218268                                foreach( $avatar_files as $key => $value ) {
    219                                         if ( strpos ( $value, $avatar_size )!== false )
     269                                        if ( strpos ( $value, $avatar_size )!== false ) {
    220270                                                $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     271                                        }
    221272                                }
    222273
    … …  
    224275                                if ( !isset( $avatar_url ) ) {
    225276                                        foreach( $avatar_files as $key => $value ) {
    226                                                 if ( strpos ( $value, $legacy_user_avatar_name )!== false )
     277                                                if ( strpos ( $value, $legacy_user_avatar_name )!== false ) {
    227278                                                        $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     279                                                }
    228280                                        }
    229281
    … …  
    231283                                        if ( !isset( $avatar_url ) ) {
    232284                                                foreach( $avatar_files as $key => $value ) {
    233                                                         if ( strpos ( $value, $legacy_group_avatar_name )!== false )
     285                                                        if ( strpos ( $value, $legacy_group_avatar_name )!== false ) {
    234286                                                                $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     287                                                        }
    235288                                                }
    236289                                        }
    … …  
    262315
    263316                // Set gravatar size
    264                 if ( $width )
     317                if ( false !== $width ) {
    265318                        $grav_size = $width;
    266                 else if ( 'full' == $type )
     319                } else if ( 'full' == $type ) {
    267320                        $grav_size = bp_core_avatar_full_width();
    268                 else if ( 'thumb' == $type )
     321                } else if ( 'thumb' == $type ) {
    269322                        $grav_size = bp_core_avatar_thumb_width();
     323                }
    270324
    271325                // Set gravatar type
    272                 if ( empty( $bp->grav_default->{$object} ) )
     326                if ( empty( $bp->grav_default->{$object} ) ) {
    273327                        $default_grav = 'wavatar';
    274                 else if ( 'mystery' == $bp->grav_default->{$object} )
     328                } else if ( 'mystery' == $bp->grav_default->{$object} ) {
    275329                        $default_grav = apply_filters( 'bp_core_mysteryman_src', bp_core_avatar_default(), $grav_size );
    276                 else
     330                } else {
    277331                        $default_grav = $bp->grav_default->{$object};
     332                }
    278333
    279334                // Set gravatar object
    … …  
    287342
    288343                // Set host based on if using ssl
    289                 if ( is_ssl() )
     344                $host = 'http://www.gravatar.com/avatar/';
     345                if ( is_ssl() ) {
    290346                        $host = 'https://secure.gravatar.com/avatar/';
    291                 else
    292                         $host = 'http://www.gravatar.com/avatar/';
     347                }
    293348
    294349                // Filter gravatar vars
    … …  
    296351                $gravatar = apply_filters( 'bp_gravatar_url', $host ) . md5( strtolower( $email ) ) . '?d=' . $default_grav . '&amp;s=' . $grav_size;
    297352
     353        // No avatar was found, and we've been told not to use a gravatar.
    298354        } else {
    299                 // No avatar was found, and we've been told not to use a gravatar.
    300355                $gravatar = apply_filters( "bp_core_default_avatar_$object", BP_PLUGIN_URL . '/bp-core/images/mystery-man.jpg', $params );
    301356        }
Note: See TracChangeset for help on using the changeset viewer.