Skip to:
Content

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#3087 closed defect (bug) (fixed)

avatars cant use CAPITAL fileext like JPG

Reported by: Ghost_ufa Owned by:
Milestone: 1.5 Priority: normal
Severity: Version: 1.2.8
Component: Core Keywords: has-patch
Cc:

Description

bp-core-avatars.php, function bp_core_check_avatar_type should use case-independent regexp patterns:

if ( ( !empty( $filefile?type? ) && !preg_match(’/(jpe?g|gif|png)$/i’, $filefile?type? ) )
!preg_match( ’/(jpe?g|gif|png)$/i’, $filefile?name? ) )

Attachments (2)

3087.01.patch (2.1 KB) - added by r-a-y 3 years ago.
Patch for BP 1.3-bleeding
3087.02.patch (936 bytes) - added by r-a-y 3 years ago.
Patch for BP 1.2-branch

Download all attachments as: .zip

Change History (5)

r-a-y3 years ago

Patch for BP 1.3-bleeding

r-a-y3 years ago

Patch for BP 1.2-branch

comment:1 r-a-y3 years ago

  • Keywords has-patch added; avatars removed

comment:2 djpaul3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4169]) Update regex to match case insensitive file extensions when checking avatar uploads. Fixes #3087, props r-a-y

comment:3 DJPaul3 years ago

  • Milestone changed from Awaiting Review to 1.3
Note: See TracTickets for help on using tickets.