Support » Plugin: Relevanssi - A Better Search » Redefining the global $post object

  • Did you really mean to do this:

    function relevanssi_index_doc($indexpost, $remove_first = false, $custom_fields = false) {
    	global $wpdb, $relevanssi_table, $post;
        $post = $indexpost;

    You pull the global post object in and then redefine it with the contents of $indexpost.

    I only found this because I’ve spent 3 days troubleshooting a very obscure problem which was only being experienced by one user of my plugin and we couldn’t reproduce it on my site. But he narrowed it down to your plugin and one specific call and then I narrowed it down to the above bit of code.

    Removing the $post from the list of globals fixes the issue

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author Mikko Saari


    This isn’t my work, but added by a contributor. Looks like there’s some point to it, however, as some shortcodes need the global $post object to match the indexed post to make them work.

    I’ll have to investigate this a bit, but I think the correct solution is to replace the global $post while indexing the document and then restore the old value.

    Meanwhile, removing the global $post solves the issue, but may cause problems when indexing certain shortcodes.

    All I know is that changing the global post object does some very funky things – I have a hook on the transition_post and it updates its own post_meta and replacing the global post object seems to trash that change in the post meta. I’m not sure why it does it- its only in one specific set of circumstances but….

    Actually looking at the code you set it to $indexpost and then you do this:

    $post = $wpdb->get_row("SELECT *,parent.post_status
    			FROM $wpdb->posts parent, $wpdb->posts post WHERE
                (parent.post_status='publish' OR parent.post_status

    So you completely replace a global object with a call from the Database. So if WordPress change the structure of the global post object (like adding new fields) you completely destroy that.

    Actually you only do that if $post is no longer an object.

    But even then I don’t think you should be using the global object.

    Plugin Author Mikko Saari


    Yes, there are cases where the $indexpost can be a post ID, in which case it’s necessary to fetch the post object. In other cases it’s already a $post object.

    I understand the global $post object equals a row in the _posts database, so whatever fetched from the database will always match the $post object.

    Again, I’m not 100% sure about this, as this is contributed code, but I understand changing the global $post object is necessary in order to make sure that shortcode plugins that rely on the global object know which post is calling them.

    Try replacing the relevanssi_index_doc() function with this and let me know if it works:

    That version stores the global $post object and replaces it after the function is done. That would be a simple solution.

    No that doesn’t work.

    You’ve admitted that the global post and the indexpost aren’t always the same but simply destroying the current global post object and replacing it with something else is something that I really think you need to reconsider.

    Plugin Author Mikko Saari


    Well, that’s part of the thing – I really don’t know why they would be different. As far as I can tell they should be the same.

    Plugin Author Mikko Saari


    I finally had time to start figuring this one out.

    When Relevanssi indexes documents from the Relevanssi settings page, global $post is null. Relevanssi doesn’t destroy global $post object when indexing, because there’s no global $post object set when Relevanssi indexes documents.

    When Relevanssi indexes document after an update, the global $post is set and it and Relevanssi $post don’t match. In that case Relevanssi points to the latest revision of the edited post, while the global $post is the public version.

    So, looks like the correct behaviour is actually to check if the global $post is set and if that’s the case, use it. If no global $post is set, then set it from the $indexpost, so the shortcode plugins that rely on global $post know what we’re dealing with.

    Steve, if you’re still following this thread, could you please try the follow version of relevanssi_index_doc() and let me know if it works correctly for you:


    Plugin Author Mikko Saari


    The above code has one more problem, this should be correct:

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Redefining the global $post object’ is closed to new replies.