[buddypress-trac] [BuddyPress Trac] #6746: Avatar filename could generate: "There was a problem cropping your avatar"

buddypress-trac noreply at wordpress.org
Sun Nov 22 04:42:47 UTC 2015


#6746: Avatar filename could generate: "There was a problem cropping your avatar"
---------------------------+------------------
 Reporter:  SGr33n         |       Owner:
     Type:  defect (bug)   |      Status:  new
 Priority:  normal         |   Milestone:  2.5
Component:  API - Avatars  |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  needs-patch    |
---------------------------+------------------

Comment (by boonebgorges):

 > That's probably due to the fact we need to remove the previously
 uploaded avatar when cropping. We are using
 bp_core_delete_existing_avatar() to do so which is removing files
 containing -bpthumb & -bpfull into their name. A possible way to fix this
 is to rename (stripping from its name -bpthumb & -bpfull) the file during
 the upload.

 This looks right to me, and I agree that renaming the file during the
 upload is probably the correct fix. We end up renaming the file anyway
 using `wp_hash()`, so there's no harm in renaming earlier too.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6746#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list