Changeset 5134

Timestamp:

10/21/2013 08:30:12 PM (12 years ago)

Author:

johnjamesjacoby

Message:

Use rewrite ID helper functions instead of hardcoded strings.

Location:

trunk

Files:

• bbpress.php (modified) (1 diff)
• includes/common/functions.php (modified) (1 diff)
• includes/common/shortcodes.php (modified) (1 diff)
• includes/common/template.php (modified) (6 diffs)
• includes/core/theme-compat.php (modified) (2 diffs)
• includes/search/template.php (modified) (3 diffs)
• templates/default/bbpress/content-search.php (modified) (1 diff)
• templates/default/bbpress/content-single-view.php (modified) (1 diff)

trunk/bbpress.php

@@ -781 +781 @@
         // Unique rewrite ID's
         $feed_id            = 'feed';
-        $edit_id            = 'edit';
+        $edit_id            = bbp_get_edit_rewrite_id();
         $view_id            = bbp_get_view_rewrite_id();
         $paged_id           = bbp_get_paged_rewrite_id();

trunk/includes/common/functions.php

@@ -1714 +1714 @@
 
         // Single Topic Vview
-        } elseif ( isset( $query_vars['bbp_view'] ) ) {
+        } elseif ( isset( $query_vars[ bbp_get_view_rewrite_id() ] ) ) {
 
             // Get the view
-            $view = $query_vars['bbp_view'];
+            $view = $query_vars[ bbp_get_view_rewrite_id() ];
 
             // We have a view to display a feed

trunk/includes/common/shortcodes.php

@@ -699 +699 @@
 
         // Start output buffer
-        $this->start( 'bbp_search' );
+        $this->start( bbp_get_search_rewrite_id() );
 
         // Output template

trunk/includes/common/template.php

@@ -807 +807 @@
 
     // Check query name
-    if ( empty( $retval ) && bbp_is_query_name( 'bbp_search' ) )
+    if ( empty( $retval ) && bbp_is_query_name( bbp_get_search_rewrite_id() ) )
         $retval = true;
 
@@ -845 +845 @@
 
     // Check $_REQUEST
-    if ( empty( $retval ) && !empty( $_REQUEST[bbp_get_search_rewrite_id()] ) )
+    if ( empty( $retval ) && !empty( $_REQUEST[ bbp_get_search_rewrite_id() ] ) )
         $retval = true;
 
@@ -1889 +1889 @@
      * Get the view id
      *
-     * If a view id is supplied, that is used. Otherwise the 'bbp_view'
-     * query var is checked for.
+     * Use view id if supplied, otherwise bbp_get_view_rewrite_id() query var.
      *
      * @since bbPress (r2789)
@@ -1896 +1895 @@
      * @param string $view Optional. View id.
      * @uses sanitize_title() To sanitize the view id
-     * @uses get_query_var() To get the view id from query var 'bbp_view'
+     * @uses get_query_var() To get the view id query variable
+     * @uses bbp_get_view_rewrite_id() To get the view rewrite ID
      * @return bool|string ID on success, false on failure
      */
@@ -1902 +1902 @@
         $bbp = bbpress();
 
-        $view = !empty( $view ) ? sanitize_title( $view ) : get_query_var( 'bbp_view' );
+        $view = !empty( $view ) ? sanitize_title( $view ) : get_query_var( bbp_get_view_rewrite_id() );
 
         if ( array_key_exists( $view, $bbp->views ) )
@@ -1982 +1982 @@
         // Unpretty permalinks
         } else {
-            $url = add_query_arg( array( 'bbp_view' => $view ), home_url( '/' ) );
+            $url = add_query_arg( array( bbp_get_view_rewrite_id() => $view ), home_url( '/' ) );
         }
 

trunk/includes/core/theme-compat.php

@@ -691 +691 @@
             'post_author'    => 0,
             'post_date'      => 0,
-            'post_content'   => $bbp_shortcodes->display_view( array( 'id' => get_query_var( 'bbp_view' ) ) ),
+            'post_content'   => $bbp_shortcodes->display_view( array( 'id' => get_query_var( bbp_get_view_rewrite_id() ) ) ),
             'post_type'      => '',
             'post_status'    => bbp_get_public_status_id(),
@@ -707 +707 @@
             'post_author'    => 0,
             'post_date'      => 0,
-            'post_content'   => $bbp_shortcodes->display_search( array( 'search' => get_query_var( 'bbp_search' ) ) ),
+            'post_content'   => $bbp_shortcodes->display_search( array( 'search' => get_query_var( bbp_get_search_rewrite_id() ) ) ),
             'post_type'      => '',
             'post_status'    => bbp_get_public_status_id(),

trunk/includes/search/template.php

@@ -271 +271 @@
         // Unpretty permalinks
         } else {
-            $url = add_query_arg( array( 'bbp_search' => '' ), home_url( '/' ) );
+            $url = add_query_arg( array( bbp_get_search_rewrite_id() => '' ), home_url( '/' ) );
         }
 
@@ -321 +321 @@
         // Unpretty permalinks
         } else {
-            $url = add_query_arg( array( 'bbp_search' => urlencode( $search_terms ) ), home_url( '/' ) );
+            $url = add_query_arg( array( bbp_get_search_rewrite_id() => urlencode( $search_terms ) ), home_url( '/' ) );
         }
 
@@ -349 +349 @@
      * @param string $passed_terms Optional. Search terms
      * @uses sanitize_title() To sanitize the search terms
-     * @uses get_query_var*( To get the search terms from query var 'bbp_search'
+     * @uses get_query_var() To get the search terms from query variable
      * @return bool|string Search terms on success, false on failure
      */

trunk/templates/default/bbpress/content-search.php

@@ -14 +14 @@
     <?php bbp_breadcrumb(); ?>
 
-    <?php bbp_set_query_name( 'bbp_search' ); ?>
+    <?php bbp_set_query_name( bbp_get_search_rewrite_id() ); ?>
 
     <?php do_action( 'bbp_template_before_search' ); ?>

trunk/templates/default/bbpress/content-single-view.php

@@ -14 +14 @@
     <?php bbp_breadcrumb(); ?>
 
-    <?php bbp_set_query_name( 'bbp_view' ); ?>
+    <?php bbp_set_query_name( bbp_get_view_rewrite_id() ); ?>
 
     <?php if ( bbp_view_query() ) : ?>