Skip to content

[FIX] knowledge: avertissements relevés par la CI - #5

Merged
XtremXpert merged 1 commit into
20.0from
20.0-fix-ci-warnings
Oct 11, 2026
Merged

XtremXpert merged 1 commit into
20.0from
20.0-fix-ci-warnings

Conversation

@diverse-cite-claude

Copy link
Copy Markdown

Problème

La CI vient d’\être activée sur le fork. Les tests de 20.0, après #4, échouent sur des avertissements, que checklog-odoo traite comme des erreurs :

  • deux champs de document.page portent le même libellé « Template » : page_template_id (knowledge_templates) et template (document_page) ;
  • « Domains don't support NewId » sur page_id : document_page_approval cherche avec rec.id dans des champs calculés pendant l'édition d'un formulaire ;
  • une icône <i data-icon> sans title, dans les vues.

Changement

  • page_template_id s'appelle « Page template ». La traduction française reste « Modèle ».
  • La recherche passe par rec._origin.id, dans document_page_approval et document_page_history._compute_diff.
  • Les icônes <i data-icon> des vues de knowledge_page, knowledge_partner et knowledge_equipment ont un title.

Validation

Tests Odoo 20 de knowledge_*, document_page_approval et document_page_access_group, avec l'image de maison : 0 failed, 0 error(s) of 64 tests, et aucun avertissement de module dans le journal. pre-commit passe sur les fichiers modifiés.

🤖 Generated with Claude Code

- knowledge_templates : libellé « Page template » pour page_template_id,
  distinct du champ template de document_page (traduction : « Modèle »).
- document_page_approval, document_page : recherche par _origin.id dans
  les calculs, pour ne pas passer un NewId dans un domaine (formulaires).
- knowledge_page, knowledge_partner, knowledge_equipment : title sur les
  icônes <i data-icon> des vues.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@XtremXpert
XtremXpert merged commit fde2fdc into 20.0 Oct 11, 2026
4 checks passed
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.

1 participant