Changeset 3173

Timestamp:

05/19/2011 04:15:44 AM (13 years ago)

Author:

johnjamesjacoby

Message:

Switch bbp_template_include action into a filter, and move it down into the filters section of bbp-core-hooks.php. Also add more verbose descriptions to bbp_template_include and bbp_request_feed_trap filters. Fixes #1524. Props greenshady.

File:

• branches/plugin/bbp-includes/bbp-core-hooks.php (modified) (2 diffs)

branches/plugin/bbp-includes/bbp-core-hooks.php

@@ -36 +36 @@
 add_action( 'generate_rewrite_rules', 'bbp_generate_rewrite_rules', 12 );
 add_action( 'after_setup_theme',      'bbp_setup_theme_compat',     12 );
-add_action( 'template_include',       'bbp_template_include',       10 );
 
 /**
@@ -210 +209 @@
 /** FILTERS *******************************************************************/
 
-// Feeds
+/**
+ * Template Compatibility
+ *
+ * If you want to completely bypass this and manage your own custom bbPress
+ * template hierarchy, start here by removing this filter, then look at how
+ * bbp_template_include() works and do something similar. :)
+ */
+add_filter( 'template_include', 'bbp_template_include' );
+
+/**
+ * Feeds
+ *
+ * bbPress comes with a number of custom RSS2 feeds that get handled outside
+ * the normal scope of feeds that WordPress would normally serve. To do this,
+ * we filter every page request, listen for a feed request, and trap it.
+ */
 add_filter( 'request', 'bbp_request_feed_trap' );