“BuddyPress Global Unified search” plugin error messages

[kraigg]

“BuddyPress Global Unified Search" plugin causes error message when browsing to non-BuddyPress pages (ie. pages created in WordPress).

Error message is ("Warning: strstr() [function.strstr]: Empty delimiter in ../wp-content/plugins/buddypress/bp-core/bp-core-template.php on line 864")

[r-a-y]

Please contact the plugin author about this issue.

If the plugin author deems this to be a BP problem, feel free to reopen this ticket with some steps to reproduce.

Thanks!

[r-a-y]

Sorry... just read the issue in further detail.

That portion of code is a little weird. Not sure why we're doing a check on the WP page template.

Reopening for further investigation.

[sbrajesh]

Just had a look.
The problem is caused by empty component slug being checked using the bp_is_current_component. The empty slug is an issue with global unified search.

But should not the function bp_is_current_component( $component ) return false if $component is empty or false.

I guess, we should really put a check for this.

[sbrajesh]

kraigg,
I have just pushed an update to the global unified plugin. Please use the latest version from here. It should fix the issue
​https://github.com/sbrajesh/bp-global-unified-search

For Dev Team, We should put an if(empty($component)) return false; as the first line of bp_is_current_component to avoid this issue. But again, If it is there, It may not help developers to locate the empty component issue. So, I will take my last request back and will suggest to avoid putting anything like this and close this ticket.

[kraigg]

Thanks Brajesh for the quick update.

The errors are gone, but now the global unified search doesn't work. It is reverting back to the default BuddyPress search, with the dropdown list for members, groups, and posts.

Should I close this ticket and create a new ticket elsewhere?

[sbrajesh]

Hi Kraigg,
Let us close this ticket as the discussion is no more relevant to the BuddyPress but to that particular plugin.

I am looking at it and will post an update soon(Just testing at the moment). Can you please open a ticket on the github page ?

[sbrajesh]

Ok, just the final comment. The plugin will work if you followed the installation instructions and updated functions.php.

[r-a-y]

Saw your commit on Github, Brajesh.

Closing this ticket. Feel free to reopen if needed.

---

This ticket did open my eyes to a weird bit of code though. For those that want to follow, here's the bit in question:
​https://buddypress.trac.wordpress.org/browser/tags/1.6.4/bp-core/bp-core-template.php#L859

Not sure why we're checking for the WP page template when we're on a regular WP page. This might have something to do with BP's auto-created WP pages like "Register" and "Activate". Just thinking out loud.