Skip to:

Opened 15 years ago

Closed 15 years ago

#950 closed defect (bug) (fixed)

bp-sn-framework shows the most recent posts on all 'home blog slug' pages

Reported by: junsuijin's profile junsuijin Owned by: junsuijin's profile junsuijin
Milestone: 1.1 Priority: major
Severity: Version:
Component: Keywords: has-patch, tested


The first part of this patch corrects the bp_dtheme_show_home_blog function so it will both handle page/*, and direct the user to a 404 for improperly formatted /blog/* urls (for SEO avoiding duplicate content, and avoiding otherwise-confusing situations).

The added bp_dtheme_filter_home_query function prevents double-querying of the database when viewing the home blog slug pages (including its index), by returning null to the first query statement.

Attachments (1)

fix-bp_dtheme_show_home_blog.patch (1.6 KB) - added by junsuijin 15 years ago.

Download all attachments as: .zip

Change History (4)

#1 @junsuijin
15 years ago

updated the patch to fix some simple errors. Verified that the patch reduces queries by at least 1 for home_blog_slug and home_blog_slug/page/*(some server setups should notice drastic loading time improvements due to this).

#2 @junsuijin
15 years ago

updated the patch again to make use of locate_template for 404.php instead of load_template.

#3 @apeatling
15 years ago

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

This is fixed in trunk, can you open a separate ticket for the multi query bug if it is still an issue?

Note: See TracTickets for help on using tickets.