Skip to content
New issue

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

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

Already on GitHub? # to your account

documents: fix fulltext search #967

Closed
wants to merge 1 commit into from
Closed

Conversation

jma
Copy link
Contributor

@jma jma commented Dec 21, 2023

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

@jma jma force-pushed the maj-fix-fulltext branch 3 times, most recently from 0b53b1c to 9d4d138 Compare December 21, 2023 13:54
@jma jma force-pushed the maj-fix-fulltext branch 3 times, most recently from befb473 to 56ea39f Compare December 21, 2023 15:06
@jma jma marked this pull request as ready for review December 21, 2023 15:18
return config
# list of existing fields without the boosting factor
existing_fields = [re.sub(r'\^\d+$', '', field) for field in config]
mapping_path = current_search.mappings['documents-document-v1.0.0']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

index name in config ?

@jma jma force-pushed the maj-fix-fulltext branch from 56ea39f to b58a77d Compare January 8, 2024 07:16
* Excludes the fulltext field when the search query is not a "fulltext" search.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma force-pushed the maj-fix-fulltext branch from b58a77d to dbcc3e2 Compare January 8, 2024 07:26
@jma
Copy link
Contributor Author

jma commented Jan 8, 2024

Integrated

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The button "search in full-text" doesn't work
3 participants