Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

#7812 closed defect (bug) (fixed)

Cannot redeclare function issue

Reported by: dunhakdis's profile dunhakdis Owned by: imath's profile imath
Milestone: 3.0 Priority: normal
Severity: blocker Version:
Component: Settings Keywords: has-screenshots has-patch commit
Cc:

Description

I'm working on a new version of my theme which will be based on 'Nouveau'. I have BP 3.0-beta-2 along with TwentySixteen wp theme. No plugins are active aside from buddypress itself.

Clicking on the "Save Changes" button under "Settings" > "General" when you are in the user's settings (e.g. http://buddysite.local/members/<username>/settings/) page returns an error.

Additionally, going to http://buddysite.local/members/<username>/settings/general produces the same issue:

Here is the screenshot:

https://i.imgur.com/TX5cwqH.png

Thanks!

Attachments (2)

Screen Shot 2018-05-09 at 8.27.19 AM.png (34.3 KB) - added by dunhakdis 7 years ago.
screenshot
7812.patch (705 bytes) - added by imath 7 years ago.

Download all attachments as: .zip

Change History (6)

@imath
7 years ago

#1 @imath
7 years ago

  • Keywords has-patch 2nd-opinion added

Hi @dunhakdis

Thanks a lot for your feedback, @r-a-y i think it's a typo between screens & actions, could you check my patch please :)

#2 @dunhakdis
7 years ago

@imath Thanks for looking into it.. I just tested the patch and now the settings > general works great again :)

#3 @r-a-y
7 years ago

  • Keywords commit added; 2nd-opinion removed

Ugh, good catch everyone.

For some reason, I wasn't getting the fatal error in my development environment, but the patch by @imath is absolutely correct!

Commit when you have a chance, @imath.

Thanks again.

#4 @imath
7 years ago

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

In 12072:

Use the right path when including Settings component action handlers

Fixes #7812

Note: See TracTickets for help on using tickets.