Skip to:
Content

BuddyPress.org

Opened 6 months ago

Closed 4 weeks ago

#8396 closed defect (bug) (fixed)

Bug: BP Nouveau accept invitation error

Reported by: meijioro Owned by: imath
Milestone: 8.0.0 Priority: normal
Severity: normal Version: 6.3.0
Component: Templates Keywords: has-patch commit
Cc:

Description

I get a “There was a problem performing this action. Please try again.” notice error when I try to click the “Accept Invitation” button on one of the groups on the group page (mydomain.com/groups).

This seems to be only happening when using Nouveau. I tried removing all plugins and using Twenty Seventeen or Twenty Twenty theme. Both still give the error when I try to accept. When I try using Legacy instead, it works fine. It’s just Nouveau is broken. And it’s only where all groups are listed on the "/groups" page. Accepting in my manage profile area (the buttons are Accept & Reject are listed) works fine.

Attachments (1)

8396.patch​ (1.1 KB) - added by imath 4 weeks ago.

Download all attachments as: .zip

Change History (5)

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

#2 @imath
4 weeks ago

  • Component changed from Core to Templates
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 8.0.0

Thanks for reporting this issue. I confirm I was able to reproduce. I'm going to work on a fix.

@imath
4 weeks ago

#3 @imath
4 weeks ago

  • Keywords has-patch commit added; needs-patch removed

​8396.patch is fixing the issue. I'll commit it asap.

#4 @imath
4 weeks ago

  • Owner set to imath
  • Resolution set to fixed
  • Status changed from new to closed

In 12890:

BP Nouveau: improve the way we get the nonce in JavaScript main file

Splitting the data-bp-nonce attribute to get Ajax buttons nonce is not the right way to do it. There can be situation where the _wpnonce URL param is not the last one (eg: the data-bp-nonce attribute of the Accept Group Invite button contains a redirect_to param). Using the bp.Nouveau.getLinkParams() function is fixing the issue.

Prop meijioro

Fixes #8396

Note: See TracTickets for help on using tickets.