Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 6 years ago

#7875 closed defect (bug) (fixed)

BP Nouveau - Group Invites UI, the optional inviter message is never sent to invited users

Reported by: imath Owned by: imath
Priority: normal Milestone: 6.3.0
Component: Templates Version: 3.0.0
Severity: normal Keywords: needs-patch
Cc:

Description

As This feature was introduced before BuddyPress 2.5 in BP Nouveau, the 'groups_notification_group_invites_message' filter is deprecated. I suggest the attached patch to actually append the user message to the group-invites BP email.

Attachments (2)

7875.patch​ (2.7 KB ) - added by imath 8 years ago.
7875.2.patch​ (1.8 KB ) - added by imath 6 years ago.

Download all attachments as: .zip

Change History (9)

@imath
8 years ago

#1 @DJPaul
8 years ago

  • Keywords needs-patch added; has-patch 2nd-opinion removed
  • Milestone Awaiting Review → 4.0

The idea is fine, the patch needs some work because changing the function signature like this can be dangerous. I don't think you can type-hint the first parameter, and you need to handle that possibly being passed a string for backwards compatibility (in case someone is calling the function manually, or has hooked it somewhere else, etc).

(On commit, change the modified since version to 4.0.)

#2 @boonebgorges
8 years ago

  • Milestone 4.0 → Awaiting Contributions

#3 @imath
6 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone Awaiting Contributions → 7.0.0

This bug has been fixed during 6.0.0 release cycle with the introduction of the BP Invitations feature. We now just need to deprecate the code that is no more used.

@imath
6 years ago

#4 @imath
6 years ago

  • Owner set to imath
  • Resolution → fixed
  • Status new → closed

In 12714:

BP Nouveau: remove unnecessary code & deprecate a function

The introduction of the BP Invitations feature in BuddyPress 6.0.0 is now making sure the optional Group's inviter message is included into the BP Email sent to the user.

Let's simply remove unnecessary code & deprecate bp_nouveau_groups_invites_custom_message().

Props DJPaul

Fixes #7875

#5 @imath
6 years ago

  • Keywords needs-patch added; has-patch removed
  • Milestone 7.0.0 → 6.3.0
  • Resolution fixed
  • Status closed → reopened

This should actually have been fixed into the 6.0 branch. Reopening this ticket to do so.

#6 @imath
6 years ago

In 12715:

BP Nouveau: move r12714 fix into 6.0 branch

See #7875 (trunk)

#7 @imath
6 years ago

  • Resolution → fixed
  • Status reopened → closed

In 12716:

BP Nouveau: remove unnecessary code & deprecate a function

The introduction of the BP Invitations feature in BuddyPress 6.0.0 is now making sure the optional Group's inviter message is included into the BP Email sent to the user.

Let's simply remove unnecessary code & deprecate bp_nouveau_groups_invites_custom_message().

Props DJPaul

Fixes #7875 (6.0 branch)

Note: See TracTickets for help on using tickets.