Search - Relevant Post Links Broken

[hpguru]

Search has a bug. Some posts links broken! http://bbpress.org/forums/topic/search-relevant-post-links-broken

[matveb]

To replicate this bug:

1. On fresh installation create a new topic. Title being "Lorem ipsum", and content being "Debug search".
2. Then search for "debug". The created topic should appear both under "Recent Posts" and "Relevant posts".

Result: the topic link for the second result --under "Relevant posts"-- should be set incorrectly without any id.

The bug, I believe, comes form search.php and only occurs for topics under "Relevant posts" when they have already been displayed on "Recent posts". If you erase the $bb_query_form->BB_Query_Form in search.php for $recent = $bb_query_form->results; the topic link under "Relevant posts" works fine.

So there is some overlapping in the queries when fetching the same post both times which renders the second link without an id.

[Gautam Gupta]

Error verified. Attaching patch, which can be improved upon.

[Gautam Gupta]

Basic patch, can be improved upon.

[GautamGupta]

Do not limit posts nor order by time

[GautamGupta]

I think the search.php.2.diff would work good.

[chrishajer]

(In [2414]) Fix Relevant Post links in search results. Fixes #1222. Props GautamGupta

[mr_pelle]

I found a new bug, consequence of this patch:

To replicate this bug: 1. Make a search. 2. Make the same search.

Result: no results shown.

This happens because search form id changed from topic-search-form to post-search-form.

[Markus Pezold]

I can't replicate this bug on my own customized theme (Scoun).
After the first test I go around to my test-forum with many other themes - For me it seems the bug is only in themes with two different search-forms!

The second search changes the url-param:

1. http://bbpress.org/forums/search.php?q=topic
2. http://bbpress.org/forums/search.php?post_text=topic&forum_id=0

[mr_pelle]

Yes, the url-param changes, and that's what breaks the search.
I've also found out that a third search completes correctly... o_0"

[mr_pelle]

I've experienced the bug with Kakumei too; the search form also loses its styling, since its id changes.

[GautamGupta]

A simple patch for the second problem

[GautamGupta]

The problem was that when we changed the relevant search's type to post, $bb_query_form->type was also changed to it and the bb_topic_search_form() function uses that to determine what name should be given to the form, which created the problem.

I have attached the patch which would fix this problem.

[mr_pelle]

Works perfectly, great work!
We can close this ticket.

[chrishajer]

(In [2429]) Reported by mr_pelle. Patch in 2414 broke relevant search results. Fixes #1222. Props GautamGupta

[mr_pelle]

Last patch broke something huge: now every search I make returns me every post in the forum! =P
I didn't know if to open a new ticket, because this new bug is caused by this ticket's last patch...

[mr_pelle]

Little correction: if you select a forum to search into, search returns all the posts in that forum. If not (i.e. Any is selected), search returns all the posts in your forum.

[GautamGupta]

Yes, you're right. I've opened a new ticket for this - #1274