Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

#6929 closed defect (bug) (fixed)

Wrong link to "reinstall emails" link

Reported by: nat0n's profile nat0n Owned by: boonebgorges's profile boonebgorges
Milestone: 2.5 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch
Cc:

Description

The URL to "reinstall emails" in the message box on the new Emails page says:

/wp-admin/network/tools.php?page=bp-tools (404 for me)

…but I guess it should be:

/wp-admin/network/admin.php?page=bp-tools (works for me)

Attachments (1)

6929.patch​ (802 bytes) - added by imath 9 years ago.

Download all attachments as: .zip

Change History (5)

#1 @imath
9 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 2.5

Thanks for your feedback @nat0n, i confirm the link is not right in case BuddyPress is network activated.

I suggest 6929.patch to fix this issue.

@imath
9 years ago

#2 @DJPaul
9 years ago

This looks OK to me (haven't tested it), let's get sign-off from at least more person prior to commit.

This ticket was mentioned in ​Slack in #buddypress by boone. ​View the logs.
9 years ago

#4 @boonebgorges
9 years ago

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

In 10623:

Use correct 'Tools' link for "reinstall emails" admin notice.

Previously, the link was incorrect for BP configs where administration takes
place in Network Admin.

Props imath.
Fixes #6929.

Note: See TracTickets for help on using tickets.