Xavier Morel
ac3cb21836
The SSF had pretty much punted on nested o2m as that's... not usually a concern because few people are insane enough to put o2ms in their o2ms in their views. And also because even if somebody did, sometimes nothing would break because it turns out to be pretty difficult to actually get into *using* those things. MRP managed to do it though, and repro-ing required copying over parts of mrp.production and then understanding why it didn't fail: * obviously needs an o2m (f1) which contains an o2m (f2), in the view (an invisible tree inside a visible tree) * needs an onchange which somehow updates the sub-o2m * needs to actually trigger an onchange on the root form for f1, meaning f1 must be a dependency of a compute field or something (here I just marked every damn field as on_change as the optimisation of "don't call onchange when there's no need to" doesn't matter) Also needs to be working on an existing record with existing lines *and sublines* as the issue occurs with records to update. The issue here is that `_onchange_values` would clean up f1 e.g. send nothing for unmodified entries, and only send modified fields otherwise, but it would only do so for the toplevel, meaning the sub-level would not go through this step, and could send UPDATE commands with an `id` field (set to the original value but still). This would then proceed to blow up while loading the record, as id fields are not writeable. The fix is to perform `_onchange_values` recursively. Do that using a separate helper in order to avoid blowing up on override and whatnot, or faffling about with weird branching to get the "default" values in case they're not provided, the root function can get all the relevant bits and call the helper with them, then the helper does that setup internally and calls itself directly. An other issue I stumbled upon when investigating is a similar problem on *save*, due to an implementation detail of the SSF: UPDATE commands are fetched lazily. `_values_to_save` took care of "hydrating" all update commands (and validating and filtering them) of modified o2m fields, but as it would not do so recursively a modified f2 would not get properly hydrated and filtered, and could try to write `None` onto existing records. closes odoo/odoo#51350 X-original-commit: 3dfb4cfad849936748cc6a5b8f712e100461a86f Signed-off-by: Xavier Morel (xmo) <xmo@odoo.com>
3.1 KiB
3.1 KiB
| 1 | id | name | model_id:id | group_id:id | perm_read | perm_write | perm_create | perm_unlink |
|---|---|---|---|---|---|---|---|---|
| 2 | access_test_testing_utilities_a | access_test_testing_utilities_a | model_test_testing_utilities_a | 1 | 0 | 0 | 0 | |
| 3 | access_test_testing_utilities_readonly | access_test_testing_utilities_readonly | model_test_testing_utilities_readonly | 1 | 0 | 0 | 0 | |
| 4 | access_test_testing_utilities_c | access_test_testing_utilities_c | model_test_testing_utilities_c | 1 | 0 | 0 | 0 | |
| 5 | access_test_testing_utilities_d | access_test_testing_utilities_d | model_test_testing_utilities_d | 1 | 0 | 0 | 0 | |
| 6 | access_test_testing_utilities_e | access_test_testing_utilities_e | model_test_testing_utilities_e | 1 | 0 | 0 | 0 | |
| 7 | access_test_testing_utilities_f | access_test_testing_utilities_f | model_test_testing_utilities_f | 1 | 0 | 0 | 0 | |
| 8 | access_test_testing_utilities_g | access_test_testing_utilities_g | model_test_testing_utilities_g | 1 | 0 | 0 | 0 | |
| 9 | access_test_testing_utilities_m2o | access_test_testing_utilities_m2o | model_test_testing_utilities_m2o | 1 | 0 | 0 | 0 | |
| 10 | access_test_testing_utilities_default | access_test_testing_utilities_default | model_test_testing_utilities_default | 1 | 0 | 0 | 0 | |
| 11 | access_test_testing_utilities_parent | access_test_testing_utilities_parent | model_test_testing_utilities_parent | 1 | 0 | 0 | 0 | |
| 12 | access_test_testing_utilities_sub | access_test_testing_utilities_sub | model_test_testing_utilities_sub | 1 | 0 | 0 | 0 | |
| 13 | access_test_testing_utilities_sub2 | access_test_testing_utilities_sub2 | model_test_testing_utilities_sub2 | 1 | 0 | 0 | 0 | |
| 14 | access_test_testing_utilities_sub3 | access_test_testing_utilities_sub3 | model_test_testing_utilities_sub3 | 1 | 0 | 0 | 0 | |
| 15 | access_test_testing_utilities_recursive | access_test_testing_utilities_recursive | model_test_testing_utilities_recursive | 1 | 0 | 0 | 0 | |
| 16 | access_test_testing_utilities_onchange_parent | access_test_testing_utilities_onchange_parent | model_test_testing_utilities_onchange_parent | 1 | 0 | 0 | 0 | |
| 17 | access_test_testing_utilities_onchange_line | access_test_testing_utilities_onchange_line | model_test_testing_utilities_onchange_line | 1 | 0 | 0 | 0 | |
| 18 | access_model_test_testing_utilities_onchange_count | access_model_test_testing_utilities_onchange_count | model_test_testing_utilities_onchange_count | 1 | 1 | 1 | 1 | |
| 19 | access_test_testing_utilities_onchange_count_sub | access_test_testing_utilities_onchange_count_sub | model_test_testing_utilities_onchange_count_sub | 1 | 1 | 1 | 1 | |
| 20 | access_o2m_readonly_subfield_parent | access_o2m_readonly_subfield_parent | model_o2m_readonly_subfield_parent | 1 | 1 | 1 | 1 | |
| 21 | access_o2m_readonly_subfield_child | access_o2m_readonly_subfield_child | model_o2m_readonly_subfield_child | 1 | 1 | 1 | 1 | |
| 22 | access_test_testing_utilities_req_bool | access_test_testing_utilities_req_bool | model_test_testing_utilities_req_bool | 1 | 0 | 0 | 0 | |
| 23 | access_o2m_changes_children | access_o2m_changes_children | model_o2m_changes_children | base.group_user | 1 | 0 | 0 | 0 |
| 24 | access_o2m_changes_children_lines | access_o2m_changes_children_lines | model_o2m_changes_children_lines | base.group_user | 1 | 0 | 0 | 0 |
| 25 | access_ttu_product | access_ttu_product | model_ttu_product | base.group_user | 1 | 0 | 0 | 0 |
| 26 | access_ttu_root | access_ttu_root | model_ttu_root | base.group_user | 1 | 0 | 0 | 0 |
| 27 | access_ttu_child | access_ttu_child | model_ttu_child | base.group_user | 1 | 0 | 0 | 0 |
| 28 | access_ttu_grandchild | access_ttu_grandchild | model_ttu_grandchild | base.group_user | 1 | 0 | 0 | 0 |