WordPress.org

Ready to get started?Download WordPress

Forums

Multisite Global Search
[resolved] Thank you + possible bug (2 posts)

  1. ScreenfeedFr
    Member
    Posted 6 months ago #

    Hi @aliciagh.

    First, thank you.
    I won't use your plugin as it is right now but it will be a very useful basis to me for a quite similar need (I only need specific post types and post metas, not only for search but also displaying posts on the front page). Also thank you because I wasn't aware of VIEW in sql, it seems to be a good solution for the performance problem of this case.

    Maybe I'm wrong, but here is something that can cause troubles imho:
    In your function ms_global_search_init() you use the actions wpmu_new_blog, delete_blog, archive_blog, etc. While digging into wp-includes/ms-blogs.php => update_blog_status() I see there are more available actions not listed in your function: make_spam_blog, make_ham_blog, mature_blog, unmature_blog, etc. Shouldn't they be used to update your VIEW?

    A simple (and maybe stupid) though: in your shortcode you use get_blog_permalink(), and it uses switch_to_blog(), which seems to add many things in cache. Wouldn't that be an issue? I mean, as far as I understand, VIEW allow us to reduce the number of requests to the database and the cache size, so, wouldn't that be counter-productive to use switch_to_blog()?

    Thanks again :)
    Greg

    https://wordpress.org/plugins/multisite-global-search/

  2. aliciagh
    Member
    Plugin Author

    Posted 1 month ago #

    Hi @ScreenfeedFr,
    Thank you very much for your suggestions. I will include hooks for spam but not ofr mature or unmature because need version 3.1 and I prefer support from 3.0 version a bit more time.

    Regarding the get_blog_permalink() function, the other option to avoid switch_to_blog() is rewriting the whole get_permalink() function because permalink structure is different in each blog, but I prefer not rewrite core functions.

    Thanks again :)
    Alicia

Reply

You must log in to post.

About this Plugin

About this Topic