Changeset 10144

Timestamp:

09/25/2015 06:16:05 PM (9 years ago)

Author:

imath

Message:

Make sure the thumb size is used to crop the thumb avatar

2.3.0 introduced the BP_Attachment_Avatar class to manage avatar uploads. Unfortunately the full avatar size was mistakenly used to set full and thumb avatars.
This commit makes sure the right size will be used to set the avatars.

Props damland

See #6634 (branch 2.3)

File:

• branches/2.3/src/bp-core/classes/class-bp-attachment-avatar.php (modified) (1 diff)

branches/2.3/src/bp-core/classes/class-bp-attachment-avatar.php

@@ -259 +259 @@
 
         foreach ( $avatar_types as $key_type => $type ) {
-            $args['dst_w']    = bp_core_avatar_full_width();
-            $args['dst_h']    = bp_core_avatar_full_height();
+            if ( 'thumb' === $key_type ) {
+                $args['dst_w'] = bp_core_avatar_thumb_width();
+                $args['dst_h'] = bp_core_avatar_thumb_height();
+            } else {
+                $args['dst_w'] = bp_core_avatar_full_width();
+                $args['dst_h'] = bp_core_avatar_full_height();
+            }
+
             $args['dst_file'] = $avatar_folder_dir . '/' . wp_hash( $absolute_path . time() ) . '-bp' . $key_type . '.' . $ext;