? Pending

User tests: Successful: Unsuccessful:

avatar chrisdavenport
chrisdavenport
1 Feb 2019

It isn't necessary to add the values of all taxonomy (content map) node values as search terms in the index.

For example, all Joomla websites with regular content items will have the word "article" in the index and will bring back all articles if the word "article" is used as a search term, including any that don't actually contain the word "article". Similarly for "category". You'll even find all the author names have been indexed, including "super user"!

This is unnecessary for two reasons; firstly because any search term that brings back everything in the search results (like "article" does) is useless as a search term and secondly because these terms are often for internal use and shouldn't be exposed to users. The advanced search mode provides drop-downs so that users can search using the taxonomies (content maps) so it isn't necessary to have the taxonomy node values in the terms index too.

Summary of Changes

Removed code that adds taxonomy node values to the terms index.

Testing Instructions

Make sure Smart Search is set up and you've indexed the content. Now, on the front-end, use Smart Search (not the old search) to search for terms such as "Article", "Category" and "Super User".

[EDIT] After applying this PR you will need to do Clear Index and then Index (or equivalently do php cli/finder_indexer.php --purge on the command line) in order to completely rebuild the index.

Expected result

Search results should actually contain the terms/phrases that you entered.

Actual result

Depending on what content you have, each of these terms/phrases will return results which don't actually contain these terms/phrases (as well as results that do).

Documentation Changes Required

None.

avatar chrisdavenport chrisdavenport - open - 1 Feb 2019
avatar chrisdavenport chrisdavenport - change - 1 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Feb 2019
Category Administration com_finder
avatar Hackwar
Hackwar - comment - 1 Feb 2019

I agree with this. Good catch.

avatar HLeithner HLeithner - change - 1 Feb 2019
The description was changed
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-01 17:15:07
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 1 Feb 2019
avatar HLeithner HLeithner - merge - 1 Feb 2019
avatar HLeithner
HLeithner - comment - 1 Feb 2019

thx

Add a Comment

Login with GitHub to post a comment