Skip to:
Content

BuddyPress.org

Opened 6 years ago

Closed 6 years ago

#8254 closed defect (bug) (fixed)

Undefined `$current_blog` in function `bp_core_delete_existing_avatar`

Reported by: itowhid06 Owned by: imath
Priority: normal Milestone: 6.0.0
Component: Core Version:
Severity: normal Keywords: has-patch commit
Cc:

Description

Undefined $current_blog in function bp_core_delete_existing_avatar

Attachments (2)

8254.diff​ (442 bytes ) - added by itowhid06 6 years ago.
8254.2.patch​ (1.1 KB ) - added by imath 6 years ago.

Download all attachments as: .zip

Change History (9)

@itowhid06
6 years ago

#1 @imath
6 years ago

  • Keywords reporter-feedback added
  • Milestone Awaiting Review → 6.0.0
  • Owner set to imath
  • Status new → assigned

Hi @itowhid06

Thanks for your feedback and patch. Out of curiosity how did you find the issue ?

#2 @itowhid06
6 years ago

Hi @imath

I am working on some BuddyPress integration code. I was looking into how BP is saving avatars and my Code Editor showed me this one. All credit goes to the CE :)

#3 @imath
6 years ago

  • Keywords reporter-feedback removed

Nice, thanks for catching it 👍

@imath
6 years ago

#4 follow-up: @imath
6 years ago

I believe it's best to use the function get_current_blog_id(). What do you think ?

#5 in reply to: ↑ 4 @itowhid06
6 years ago

Yes, that is better. Using $current_blog->id should be avoided in all possible cases, and I just created the patch to keep similarity with other usages. +1 to ​https://buddypress.trac.wordpress.org/attachment/ticket/8254/8254.2.patch.
Replying to imath:

I believe it's best to use the function get_current_blog_id(). What do you think ?

Last edited 6 years ago by itowhid06 (previous) (diff)

#6 @imath
6 years ago

  • Keywords commit added

Thanks for your feedback, let's have this in.

#7 @imath
6 years ago

  • Resolution → fixed
  • Status assigned → closed

In 12565:

Stop using the $current_blog global when fetching/deleting an avatar

Props itowhid06

Fixes #8254

Note: See TracTickets for help on using tickets.