[19.0][MIG] attachment_unindex_content: Migration to 19.0#3521
Open
[19.0][MIG] attachment_unindex_content: Migration to 19.0#3521
Conversation
This module is intended to disable the indexation of data on the ir.attachment model. Attachment model has a field called 'index_content' where the content of the attachment is read and stored directly in the database. This field is useful in order to search content of a file. But most of cases it is not used, so, you can install this module in order to: - **Avoid Duplicating Data:** Because indexation extracts text content from files and put it on the database in order it could be searched, but this implies you have the file data in your `filestore` directory, and also part (or sometimes all) of that data in your database too. - **Improve Performance:** Since not all indexed files are plain text, they require extra process to read them. Maybe you could try to uninstall modules like `document` in order to disable its indexation features, but you could face the uninstallation of other modules that could be useful for you (e.g, `hr_recruitment` depends on that). But even if you don't have `document` installed, you'd still have plain text content indexation by default. Using this module you will not require to uninstall any module to disable the attachment content indexation, because we directly disable it at `ir.attachment` base.
Currently translated at 100.0% (4 of 4 strings) Translation: server-tools-14.0/server-tools-14.0-attachment_unindex_content Translate-URL: https://translation.odoo-community.org/projects/server-tools-14-0/server-tools-14-0-attachment_unindex_content/es_AR/
Odoo introduced a new param `checksum` to the method `_index` on the
attachment model [1]. Since that param is not supported on this model,
the following error is now being raised:
TypeError: _index() got an unexpected keyword argument 'checksum'
[1] odoo/odoo@3d59cc84
- Manifest's "Version" updated to "16.0.1.0.0" - Translation files updated to new version
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-tools-16.0/server-tools-16.0-attachment_unindex_content Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-attachment_unindex_content/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-tools-16.0/server-tools-16.0-attachment_unindex_content Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-attachment_unindex_content/
Currently translated at 100.0% (1 of 1 strings) Translation: server-tools-16.0/server-tools-16.0-attachment_unindex_content Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-attachment_unindex_content/es/
Currently translated at 100.0% (1 of 1 strings) Translation: server-tools-16.0/server-tools-16.0-attachment_unindex_content Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-attachment_unindex_content/it/
moylop260
force-pushed
the
19.0-attachment_unindex_content-moy
branch
from
c7a6a3a to
e194f13
Compare
Contributor
Author
|
/ocabot migration attachment_unindex_content |
hugho-ad
suggested changes
Feb 12, 2026
Contributor
|
Technically, LGTM 👍 |
|
I am just worried about no test, is this module needed? |
moylop260
force-pushed
the
19.0-attachment_unindex_content-moy
branch
10 times, most recently
from
1c6d599 to
78467ef
Compare
* Remove superfluous keys in the manifest * Remove financed migrations by other companies * Bump manifest version
… dependency Without declaring explicit dependency, Odoo did not guarantee its load order, resulting in attachment content still being indexed
…n about missing pdfminer.six The PDF indexation function is never called by the module `attachment_unindex_content`, so the package is not needed. Related to https://github.com/odoo/odoo/blob/d15685304f47/addons/attachment_indexation/models/ir_attachment.py#L17
moylop260
force-pushed
the
19.0-attachment_unindex_content-moy
branch
from
78467ef to
14f4266
Compare
Kiplangatdan
approved these changes
Feb 17, 2026
Member
Kiplangatdan
left a comment
Kiplangatdan
left a comment
There was a problem hiding this comment.
LGTM!
Tested on a v19 local instance.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
17 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Migration commits:
Other commits:
FUTURE TASKS AFTER MERGE