Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#7531 closed defect (bug) (fixed)

Variables conflict in messages_notification_new_message()

Reported by: idofri Owned by: r-a-y
Priority: normal Milestone: 2.9
Component: Messages Version: 2.5.0
Severity: normal Keywords: has-patch
Cc:

Description

I've noticed that the $args variable in the messages_notification_new_message() function, is being repeatedly overwritten inside the loop and then being passed to the 'bp_messages_sent_notification_email' action hook.

Attachments (2)

7531.patch​ (807 bytes ) - added by idofri 9 years ago.
7531.2.patch​ (838 bytes ) - added by r-a-y 9 years ago.

Download all attachments as: .zip

Change History (6)

@idofri
9 years ago

@r-a-y
9 years ago

#1 @r-a-y
9 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone Awaiting Review → 2.9
  • Version → 2.5.0

Nice catch, @idofri.

Looks like your first BuddyPress ticket, so thanks for contributing!

Instead of $_args, I think we can just get rid that variable since we don't need it. See 7531.2.patch.

#2 @idofri
9 years ago

Sounds good @r-a-y, Thanks!

#3 @r-a-y
9 years ago

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

In 11562:

Messages: Fix $args parameter in 'bp_messages_sent_notification_email' hook.

This was broken since we added BuddyPress HTML email support in v2.5 and
changed the $args variable when sending an email.

This commit removes the $args override, which fixes this problem.

Props idofri.

Fixes #7531.

#4 @r-a-y
9 years ago

In 11563:

Buttons: Ensure that certain BP_Button class properties are updated.

Specifically, the $parent_element and $parent_attr properties were not
being updated. Devs should now be able to filter the button and do checks
against these properties as expected.

Fixes #7531.

Version 1, edited 9 years ago by r-a-y (previous) (next) (diff)
Note: See TracTickets for help on using tickets.