[18.0][FIX] document_page_approval: Avoid errors when creating a page history and send mail

[victoralmau]

Avoid errors when creating a page history and end mail

Example use case:

• Modify the Odoo Features category, check Require approval, and set Approver group > Administration / Access Rights
• Modify the Marc Demo user and define Document Knowledge > Editor permissions
• Go to the Odoo Features category and, taking into account the ID, delete the mail_message record table with the same ID from the database
• With the Marc Demo user, go to the Knowledge > Odoo Features menu and enter the Odoo 15.0 Functional Demo page (it is important to follow these specific steps)
• Modify the page and save

Before this change, an error could occur because the action https://github.com/OCA/knowledge/blob/0881d75a35e7fc2f814110df4878177e805e589b/document_page/views/document_page_category. xml#L83C17-L83C42 defined a default_parent_id=x (page category ID) if there was no mail_message linked to that category ID; now the value of default_parent_id is removed to avoid this specific error.

Please @pedrobaeza and @pilarvargas-tecnativa can you review it?

@Tecnativa TT61290

[christian-ramos-tecnativa]

Can you be a bit more specific about the third step?

Go to the Odoo Features category and, taking into account the ID, delete the mail_message record table with the same ID from the database

I'm not able to reproduce the error and assuming is because of this step. I went to mail_message and deleted the one linked to the Category Odoo Features the one saying "Document Page created", is that correct?

[victoralmau]

Can you be a bit more specific about the third step?

Go to the Odoo Features category and, taking into account the ID, delete the mail_message record table with the same ID from the database

I'm not able to reproduce the error and assuming is because of this step. I went to mail_message and deleted the one linked to the Category Odoo Features the one saying "Document Page created", is that correct?

I think an example is the best way to explain it.

Before:

After:

[christian-ramos-tecnativa]

Thanks a lot for the Gif, I was able to replicate and understand the error

[pedrobaeza]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-584-by-pedrobaeza-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at f747aa3. Thanks a lot for contributing to OCA. ❤️