[IDEA] Deprecate document.page.history and use base_revision
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 24/100
Research direction
The issue identifies document_page and base_revision but gives no files, tests, or implementation entry point. Start by reading both modules’ revision models and the history logic, then assess how inheritance and existing document data would be handled. Done means document.page uses base_revision and the old history model is removed, with revision tracking covering document context as well as content.
Written by the indexing model from the issue text.
Description
Is your feature request related to a problem?
- Document_page_history handles only changes in document content.
- In ISO (and most other certification schemes, I guess) changes in the context of the document (aka scope, use, applicability, responsibility) should also create a new version.
- Document history has been problematic working ON / OFF between Odoo versions
Describe the solution you'd like
Remove document.page.history completely and replace, inheriting from base_revision.
Using base_revision as the foundation for document revisions in the document_page module would simplify the architecture considerably while also making revision tracking more complete and consistent.
Today, document.page.history duplicates part of the data model and requires explicit logic to decide which fields are copied into history records. This creates several limitations:
- only selected fields are revisioned (content)
- schema changes must be mirrored manually
- custom modules adding fields are not automatically revision-aware
- history synchronization logic becomes harder to maintain
- revision consistency depends on custom implementation details
By refactoring document_page to inherit from base_revision, revisions would become model-native instead of relying on a parallel history model.
Key advantages of this approach:
- Automatic revisioning of all fields
Any stored field added by OCA or third-party modules automatically becomes part of the revision system without additional code. - Elimination of duplicated models
Removing document.page.history reduces maintenance burden and avoids keeping two partially synchronized representations of the same document. - Better extensibility for OCA ecosystem modules
Modules extending document.page would gain revision support automatically, including custom metadata, categorization, workflow fields, attachments, references, etc. - Consistent behavior with other OCA modules
base_revision already provides a standardized revision workflow used across the OCA, improving predictability and reducing custom code. - Reduced upgrade complexity
Future field additions or refactors would not require updating history-copying logic or migration scripts for duplicated fields. - Improved auditability
Since revisions are based on the actual business record structure, the revision history becomes more complete and reliable, thus you get a better point-in-time view of the documents. - Cleaner ORM integration
Access rights, computed fields, tracking behavior, and inheritance patterns remain centered on a single model architecture instead of a shadow history model. - Easier maintenance long-term
Less custom revision logic means fewer edge cases and lower risk of history inconsistencies (issues such as #565).
This refactoring would effectively shift the module from a “manual snapshot/history table” approach to a generic ORM-level revision mechanism, making the revision system both more powerful and easier to maintain.
Furthermore any logic that needs to be used for documents (such as summary of changes) can be added to base_revision so all modules that inherit from it can use it if it makes sense for them.
Describe alternatives you've considered
Duplicate all fields of document.page to document.page history and keep track of changes.
Additional context
This is just a use case investigation and I haven't reached deep in implementation details.
What do you think @etobella @pedrobaeza @victoralmau
- Dominant language
- HTML
- Stars
- 256
- Forks
- 405
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from OCA/knowledge
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 94/100
Maintainers usually reply within 1 day
-
help wanted no stale work in progress
Difficulty 4/5 3-5 days Newbie friendliness 30/100
Maintainers usually reply within 1 day
-
meaning of SummaryOpen
Difficulty 4/5 3-5 days Newbie friendliness 43/100
OCA/knowledge#616 · 8 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
Migration to version 19.0Possibly taken A pull request linked to this issue is open or already merged. Openhelp wanted no stale work in progress
Difficulty 4/5 3-5 days Newbie friendliness 35/100
OCA/knowledge#559 · 4 comments ·
Maintainers usually reply within 1 day
-
Migration to version 18.0Possibly taken A pull request linked to this issue is open or already merged. Openhelp wanted no stale work in progress
Difficulty 4/5 3-5 days Newbie friendliness 25/100
OCA/knowledge#499 · 8 comments ·
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
OHDSI/Data2Evidence#3693 ·
Maintainers usually reply within 3 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Project-N-E-K-O/N.E.K.O#3725 ·
Maintainers usually reply within 1 day
-
[Bug] Sending a Telegram message chain twice duplicates mentionsPossibly taken @123XieC claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
AstrBotDevs/AstrBot#11125 ·
Maintainers usually reply within 1 day
-
backend bug priority: high
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
nooriegarg/ContribIQ#73 ·
Maintainers usually reply within 1 day
-
backend enhancement
Difficulty 2/5 Half a day Newbie friendliness 70/100
damodharg6/payledger#16 ·