Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#3087 closed defect (bug) (fixed)

avatars cant use CAPITAL fileext like JPG

Reported by: Ghost_ufa Owned by:
Priority: normal Milestone: 1.5
Component: Core Version: 1.2.8
Severity: 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 2 years ago.
Patch for BP 1.3-bleeding
3087.02.patch​ (936 bytes) - added by r-a-y 2 years ago.
Patch for BP 1.2-branch

Download all attachments as: .zip

Change History (5)

r-a-y — 2 years ago

Patch for BP 1.3-bleeding

r-a-y — 2 years ago

Patch for BP 1.2-branch

comment:1   r-a-y — 2 years ago

  • Keywords has-patch added; avatars removed

comment:2   djpaul — 2 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   DJPaul — 2 years ago

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