Skip to:
Content

BuddyPress.org

Opened 6 weeks ago

Closed 6 weeks ago

Last modified 6 weeks ago

#9143 closed defect (bug) (fixed)

Permalink corrupted in BuddyPress Add-on for Membership Pro

Reported by: johndawson155's profile johndawson155 Owned by: imath's profile imath
Milestone: 12.5.0 Priority: normal
Severity: normal Version: 12.4.1
Component: Groups Keywords: has-patch
Cc:

Description

We're using BuddyPress with Membership Pro (v3.0.3); the main reason is to avail of BuddyPress Groups, lacking in Membership Pro

Whenever we save a Group, the Permalink for the Group is corrupted; specifically, it prepends an additional "http-" to the Permalink on each Save; see screenshot attached

This has no effect on the Menu system but broke our Member Import, which relies on the column bp_group to effect assigning the Member to their Group(s)

Attachments (1)

Screenshot.png (46.1 KB) - added by johndawson155 6 weeks ago.
Screenshot showing manifestation of bug

Download all attachments as: .zip

Change History (6)

@johndawson155
6 weeks ago

Screenshot showing manifestation of bug

#1 @imath
6 weeks ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 12.5.0
  • Version changed from 12.4.0 to 12.4.1

Thanks for your report. We will fix this asap.

This ticket was mentioned in PR #277 on buddypress/buddypress by imath.


6 weeks ago
#2

  • Keywords has-patch added; needs-patch removed

Using esc_url() is a mistake (my bad) as it prepends the http protocol.

Trac ticket: https://buddypress.trac.wordpress.org/ticket/9143

#3 @imath
6 weeks ago

In 13844:

Use a more appropriate function to escape slugs

Props johndawson155

See #9143 (trunk)
Closes https://github.com/buddypress/buddypress/pull/277

#4 @imath
6 weeks ago

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

In 13845:

Use a more appropriate function to escape slugs

Props johndawson155

Fixes #9143 (branch 12.0)

This ticket was mentioned in Slack in #buddypress by imath. View the logs.


6 weeks ago

Note: See TracTickets for help on using tickets.