Skip to content

Localized fields support - #38

Closed
tomasc wants to merge 78 commits into
mongoid:masterfrom
tomasc:localized-fields
Closed

tomasc wants to merge 78 commits into
mongoid:masterfrom
tomasc:localized-fields

Conversation

@tomasc

@tomasc tomasc commented May 5, 2018

Copy link
Copy Markdown
Contributor

Here a PR with support for localized fields.

Basically, if there are multiple locales, and the document has any localized fields, an index is created per each locale, and then used when searching.

Built upon #37 , will rebase once merged.

@tomasc tomasc mentioned this pull request May 6, 2018
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock

dblock commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

AI posting on behalf of @dblock: adapted the localized-field proposal into #58, separately from the inherited criteria changes now in #57. @tomasc is credited in the changelog, PR description, and commit coauthor. The replacement keeps locale suffixes stable even with one configured locale, only localizes indexes that explicitly include a localized field, evaluates computed readers/filters/update_if in locale context, restores the caller locale on failures, and continues after empty translations. It also covers index creation/removal/rebuilding and includes migration instructions. Its full Mongoid 3–9 matrix is green: https://github.com/dblock/mongoid_fulltext/actions/runs/38996889008. Closing this older PR in favor of #58; thank you for the original implementation.

@dblock dblock closed this Oct 17, 2026
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dblock added a commit to dblock/mongoid_fulltext that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in mongoid#38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dblock added a commit that referenced this pull request Oct 17, 2026
Adapt the localized-fields proposal from @tomasc in #38. Keep locale suffixes stable, evaluate readers and filters within each locale, and update creation/removal/reindexing consistently.

Co-authored-by: Tomas Celizna <tomas.celizna@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants