Scope exclude to ignore admin searches

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Contributor pronskiy

    (@pronskiy)

    Hi!
    Good comment, thanks for sharing with me!
    I was not using search on admin side so never thought about this moment.
    Sure, I’m interested in merging your code to the repo. Actually there is a github mirror for the plugin code https://github.com/pronskiy/wp-search-exclude so you could send full PR there. Will this work for you?

    Also you can contact me by email ([email protected]) or skype (romanpronskiy).

    Plugin Contributor pronskiy

    (@pronskiy)

    Hey!
    Thanks for your your pull-request, I merged it and released version 1.0.5.
    Also added you to plugin contributors! ??

    Plugin Contributor stevelock

    (@stevelock)

    Hi there,

    The is_admin()-solution is a bit too drastic; because it also returns true when an Ajax-function is called. So themes querying a search using Ajax don’t benefit from your plugin. I’ve pulled a request solving this problem, while at the same time disabling excludes when in the admin panel.

    Plugin Contributor pronskiy

    (@pronskiy)

    Hi, Stevelock!

    Good catch! I will review your PR and merge it today. Thank you!

    Plugin Contributor pronskiy

    (@pronskiy)

    Hey, Stevelock!

    I merged your PR, thanks for contribution!

    Plugin Contributor stevelock

    (@stevelock)

    Superb! Thanks ??

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Scope exclude to ignore admin searches’ is closed to new replies.