Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 8 years ago

Last modified 5 years ago

#6962 closed defect (bug) (fixed)

Profile Cover not working when we define a custom BP_XPROFILE_SLUG

Reported by: maniou's profile Maniou Owned by: r-a-y's profile r-a-y
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.4.0
Component: Media Keywords: has-patch commit
Cc:

Description

Hello,

When we define a custom BP_XPROFILE_SLUG for example : 

define ( 'BP_XPROFILE_SLUG', 'profil' );

The profil cover is not working because of this condition in /buddypress/bp-core/bp-core-attachments.php line 1036

if ( 'profile' === $current_component ) {

Can we compare with the name of the custom profil slug instead of the hardcoded profile text ?

Thank you !

Attachments (1)

6962.01.patch​ (446 bytes) - added by r-a-y 8 years ago.

Download all attachments as: .zip

Change History (10)

#1 @r-a-y
8 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 2.6
  • Version changed from 2.5.0 to 2.4.0

Thanks for the report. I can confirm the issue.

I've attached a patch that should fix this.

Last edited 8 years ago by r-a-y (previous) (diff)

@r-a-y
8 years ago

#2 @r-a-y
8 years ago

  • Priority changed from high to normal

This ticket was mentioned in ​Slack in #buddypress by djpaul. ​View the logs.
8 years ago

This ticket was mentioned in ​Slack in #buddypress by dcavins. ​View the logs.
8 years ago

#5 @imath
8 years ago

  • Keywords commit added

01.patch looks good to me, thanks a lot @r-a-y.

Last edited 8 years ago by imath (previous) (diff)

This ticket was mentioned in ​Slack in #buddypress by imath. ​View the logs.
8 years ago

#7 @r-a-y
8 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 10707:

Cover Image: Fix issue when a custom BP_XPROFILE_SLUG is in use.

Fixes #6962.

#8 @DJPaul
8 years ago

  • Component changed from Component - Attachments to Media

#9 @raruto
5 years ago

For some strange reason this patch, in BP v5.1.2, has not yet been applied ...

Version 0, edited 5 years ago by raruto (next)
Note: See TracTickets for help on using tickets.