Skip to:

Opened 10 years ago

Closed 10 years ago

#2595 closed defect (bug) (fixed)

Invalid markup on BP group topic 404 page

Reported by: boonebgorges's profile boonebgorges Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version: trunk
Component: Extend - BuddyPress Keywords: has-patch


When viewing, and no topic by the name 'foo' exists, the HTML renderer is bailing without closing the 'bbpress-forums' div. The result is invalid markup that messes with some themes. Here's where the tag is opened: and here is where it should be closed but is not:

Patch attached.

Attachments (2)

2595.patch (592 bytes) - added by boonebgorges 10 years ago.
2595.2.patch (1.1 KB) - added by netweb 10 years ago.

Download all attachments as: .zip

Change History (5)

10 years ago

10 years ago

#1 @netweb
10 years ago

  • Keywords has-patch added
  • Milestone set to 2.6

Thanks Boone, patch 2595.2.patch adds the same same closing div for 'single reply'

This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.

10 years ago

#3 @johnjamesjacoby
10 years ago

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

In 5511:

In our BuddyPress Groups display_forums method, break out of switch statement rather than return. This prevents malformed markup caused by not allowing the method to continue on. Hat-tip boonebgorges, netweb. Fixes #2595.

Note: See TracTickets for help on using tickets.