bp_core_avatar_handle_crop() needs to save extensions correctly

[voldomazta]

this is how this function sets the thumb filenames:

$full_filename = wp_hash( $original_file . time() ) . '-bpfull.jpg';
$thumb_filename = wp_hash( $original_file . time() ) . '-bpthumb.jpg';

it should instead check whether the filename was a png or a gif using the getimagesize() php func so that it can choose a correct extension for the filename instead of just jpg all the time:

$data = @getimagesize($original_file);
$ext = $data['mime'] == 'image/png' ? 'png' : 'jpg';

// Set the full and thumb filenames
$full_filename = wp_hash( $original_file . time() ) . '-bpfull.' . $ext;
$thumb_filename = wp_hash( $original_file . time() ) . '-bpthumb.' . $ext;

[johnjamesjacoby]

(In [6600]) Avatars:

• Clean up avatar code.
• When uploading a png, use the correct file extension.
• Fixes #4705.
• Enqueue Jcrop styling.