[16.0][FIX] base_tier_validation: Field merge in view

[houzefa-abba]

Previous implementation was losing fields in some cases; new one properly merges dicts/tuples.

This fixes odd "Missing field string information" errors in form-embedded lists.

This cset also removes an obsolete base_model_name context key no longer used in Odoo 16.

This cset adds a test_get_view test which checks for that need_validation field; it was failing with the previous impl.

Fixes #825.

[OCA-git-bot]

Hi @LoisRForgeFlow,
some modules you are maintaining are being modified, check this out!

[rrebollo]

@houzefa-abba I was testing sale_tier_validation and experienced a bug where all reviews where accepted (with sequence approve enabled) but confirm action in sale order keep raising "The operation is under validation". After applied this PR to my code everything worked as expected. Any thoughts???

[houzefa-abba]

@houzefa-abba I was testing sale_tier_validation and experienced a bug where all reviews where accepted (with sequence approve enabled) but confirm action in sale order keep raising "The operation is under validation". After applied this PR to my code everything worked as expected. Any thoughts???

O that's good to know!

Well I haven't heavily configured / used tier validations so far so I'll take your word for it.
It doesn't surprise me though; with the previous implementation, some fields (such as those in test_get_view) could go missing, and since Odoo's view cache system is also involved here, results were not always predictable.

[AnizR]

Code review: LGTM
It solved an issue I had in a tree view of purchase orders due to purchase_validation_tier.

Thanks!