Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simple search should remove short words #217

Closed
wants to merge 1 commit into from

Conversation

@jesusbagpuss
Copy link
Contributor

@jesusbagpuss jesusbagpuss commented Mar 26, 2014

Indexing doesn't include words less than $freetext_min_word_size.
If these are not removed from the (non-Xapian) simple search, no results are found for searches than contain short words e.g. 'cake or death' would not find something called 'cake or death', whereas 'cake death' would.
This should make the simple search match the indexing pattern.

Indexing doesn't include words less than $freetext_min_word_size.
If these are not removed from the (non-Xapian) simple search, no results are found for searches than contain short words e.g. 'cake or death' would not find something called 'cake or death', whereas 'cake death' would.
This should make the simple search match the indexing pattern.
@jesusbagpuss
Copy link
Contributor Author

@jesusbagpuss jesusbagpuss commented Mar 27, 2014

Don't merge this quite yet - I think it should really be doing something more like:
my( $goodwords, $badwords ) =
$session->get_repository->call(
"extract_words" ,
$session,
$value );
[see: perl_lib/EPrints/MetaField/Text.pm]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant