Avatar filename could generate: "There was a problem cropping your avatar"

[SGr33n]

Hi people :)

If you try to reupload an avatar previously downloaded from BP it returns error: "There was a problem cropping your avatar".

Despite of the generic message error, it's related to the filename, looks like returns this when the filename ends by -bpfull and -bpthumb.

[DJPaul]

This sounds familiar. There might be some old ticket, or perhaps the issue has been reported/investigated before somewhere.

@SGr33n are you sure it's to do with the filename extension? Did you test?

[SGr33n]

Hi DjPaul,

I know there was an old issue about the filename lenght (I suppose fixed now), but this looks never reported. I also tried with -bpcustom.jpg and the upload works good, the only not working two I found are -bpthumb.* and -bpfull.* (the suffix used by BP). I tested it removing the suffix and the upload works, nothing related to the lenght, because i tried also to replace the -bpfull with 1234567 and it still work.

Example of filename not working 12345-bpfull.jpg
Example of filename working 123456789012.jpg

[imath]

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.

It's not something that happened since 2.4, so i think we can work on it for 2.5.

[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.

[SGr33n]

That's absolutely a solution, but I think the best way may be to load the temporary image on the browser cache via javascript and then upload the final image once cropped on the server. Infact could happen that the user doesn't crop the image after he uploaded it, and that temporary file will not deleted. This solution could also optimize the server usage, I made this before on a BP Cover Plugin.