Take the 2-minute tour ×
WordPress Development Stack Exchange is a question and answer site for WordPress developers and administrators. It's 100% free, no registration required.

I am trying to create a rewrite rule for one of my custom page and here is the rewrite rule

function add_my_rule() {
    global $wp;
    $wp->add_query_var('name_state');
    $wp->add_query_var('arg_second');
    add_rewrite_rule('state/([^/]+)/([^/]+)/page/([^/]+)','index.php?pagename=places-to-visit&name_state=$matches[1]&arg_second=$matches[2]&paged=$matches[3]','top');
    add_rewrite_rule('state/([^/]+)/([^/]+)','index.php?pagename=places-to-visit&name_state=$matches[1]&arg_second=$matches[2]','top');

    global $wp_rewrite;
    $wp_rewrite->flush_rules();
}

add_action('init', 'add_my_rule');

Here is what i am trying to do. I have following URL

www.mydomain.com/state/state-name/places-to-visit

I have already created a custom page with name places-to-visit.php.My intentions are to redirect all such URL's to this custom page and want following information as query variable from the URL

share|improve this question
1  
Looks like you missunderstand how Rewrite API works in WordPress. I would like to recommend you to read this article: The Rewrite API: The Basics –  Eugene Manuilov Aug 15 '13 at 18:14
    
@EugeneManuilov: i have gone through the post but not sure about my issue.Though i am not sure about my regular expression but the logic seems correct to me. –  Umesh Awasthi Aug 16 '13 at 9:35
2  
is places-to-visit the slug of a page you've created? you can't direct requests to physical files, you have to load a WordPress object- a page, a category, etc.. –  Milo Aug 16 '13 at 15:12
    
@Milo: I am not sure what you mean by slug created.I have created a php page with name places-to-visit.php and have set Template Name as places-to-visit –  Umesh Awasthi Aug 16 '13 at 15:32
    
@EugeneManuilov:Thanks for the link and yes you are correct, i misunderstood how rewrite rule working in wordpress. –  Umesh Awasthi Aug 18 '13 at 10:51
show 1 more comment

2 Answers

In the tags of your answer there is <plugins> so I think you are trying to add custom rewrite rules from a plugin.

In your question you say something about page template that is (generally) a theme-related matter. Of course you can create rewrite rules from a plugin and use them for a page template defined in a theme, but probably is not a good idea: your plugin functionalities will depend on theme.

Moreover, if you create a page template than you have to create a page, assign the page template and use a defined slug or your plugin rewrite rules will not work. So your plugin also depend from content, and this, again, is not a good idea.

So, how make your plugin functionalities indipendent from page templates in theme and from content but mantaining wordpress standards and functionalities?

A method is create a file in the plugin folder and redirect the rewrite rule to the url of this file instead to index.php. Doing so, if you need wordpress environment you need to manual require the file wp_load.php that resides on the root of worpress installation. But this path must be hardcoded and a lot of times this is not a good idea and can bring problems when migrating server. In addition this is a very bad idea if you plan to distribuite your plugin.

There is another solution?

For a plugin I developed I've used a sort of 'virtual page' where redirect the requests.

First of all, in the main plugin file, we define the virtual slug for the page. Is a good idea create a setting page for the plugin where this slug can be changed, but for semplicity I will put it in a constant:

define('MY_VIRTUAL_PAGENAME', 'a-strange-slug-for-places-to-visit-fake-page');

After that, in the same file, we define the custom rewrite rules and register the custom query vars:

add_action('init', 'add_my_rules');

function add_my_rules() {
  add_rewrite_rule('state/([^/]+)/([^/]+)/page/([^/]+)', 'index.php?pagename=' . MY_VIRTUAL_PAGENAME . '&name_state=$matches[1]&arg_second=$matches[2]&paged=$matches[3]', 'top');
  add_rewrite_rule('state/([^/]+)/([^/]+)', 'index.php?pagename=' . MY_VIRTUAL_PAGENAME . '&name_state=$matches[1]&arg_second=$matches[2]', 'top');
}

add_filter( 'query_vars', 'add_my_vars' );

function add_my_vars( $vars ) {
  $vars[] = 'name_state';
  $vars[] = 'arg_second';
  return $vars;
}

Now we have to flush rewrite rules. In your code you do this on init, so rules are flushed for every page request: this is absolutely not a good idea. We use the registration hook to run this only one time:

register_activation_hook( __FILE__, 'my_flush_rewrite_rules' );

function my_flush_rewrite_rules() {
  add_my_rules();
  flush_rewrite_rules();
}

At this moment when a particular url is required we tell wordpress to load a page that have a slug we saved on the constant MY_VIRTUAL_PAGENAME, but this page does not exist.

So now we cheat wordpress and make it shows our virtual page.

add_action( 'pre_get_posts', 'show_places_to_visit', 1 );

function show_places_to_visit( $wp_query ) {
  $pagename = isset($wp_query->query['pagename']) ? $wp_query->query['pagename'] : NULL;
  if ( ! is_admin() && is_main_query() && $pagename == MY_VIRTUAL_PAGENAME ) {

    // get our custom vars
    $name_state = $wp_query->get('name_state');
    $arg_second = $wp_query->get('arg_second');
    $paged = $wp_query->get('paged') ? : 1;

    // now I require the file that contain our business logic
    require( plugin_dir_path(__FILE__) . 'places-to-visit.php' );

    exit(); // stop default wordpress behavior
  }
}

With the above code we run a function before the wp main query is fired. In this function we check if the pagename requested (if exists) is the one we saved in the constant and if so we require a file called places-to-visit.php that is located in the same folder of main plugin file.

Now we have to just create file places-to-visit.php and there we have all wordpress environment and we can use as we like.

If you plan to run a WP_Query in this file, probably this is one of the rare case in which is better make use of query_posts instead of using a WP_Query instance: in this way you will replace the original main query that is not useful because it is just a fake query.

The following is just an example for places-to-visit.php file, I bet you can do something more useful in it:

/**
* in places-to-visit.php file I can simply use the variables defined in
* show_places_to_visit() function in main plugin file
*/
$args = array(
  'post_type' => 'places',
  'posts_per_page' => $arg_second,
  'category_name' => $name_state,
  'paged' => $paged
);
query_posts($args);

/**
* If in your theme is present a file called places-to-visit.php or a file called 
* archive-places it will be used to handle the query.
* If not you have to output the page content from plugin.
*/

if ( locate_template( array('places-to-visit.php','archive-places.php'), true ) == '' ) {

  /**
  * Code that generate page content goes here.
  * If you want, you can use template functions like
  * get_header(); get_footer(); get_template_part(); and so on.
  */

}
share|improve this answer
    
No, i am not creating any plugin, i am trying to use rewrite rule in my theme only. –  Umesh Awasthi Aug 18 '13 at 10:43
add comment
up vote 0 down vote accepted

I am able to solve this problem. Honestly speaking till now i was not able to fully understand how WordPress rewrite rule work and how WordPress able to pick my custom page.

I was doing everything right except one thing, i was not associating my custom template with any slug. Moment i assigned that from WordPress dashboard, i was able to solve my issue

Thanks everyone for your valuable inputs.

share|improve this answer
add comment

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.