Issue 9820

Title
Unable to create inventory moves
Priority
bug
Status
resolved
Superseder
The depends of depends are not added to the view (issue 9828)
Nosy list
ced, pokoli, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2020-11-11.18:07:16 by pokoli, last changed 3 days ago by roundup-bot.

Messages

New changeset 405f0ae5b65f by Cédric Krier in branch '5.8':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/modules/stock/rev/405f0ae5b65f

New changeset 0d34482d1be2 by Cédric Krier in branch '5.6':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/modules/stock/rev/0d34482d1be2

New changeset c0ea635e6329 by Cédric Krier in branch '5.4':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/modules/stock/rev/c0ea635e6329

New changeset 79042b153b75 by Cédric Krier in branch '5.0':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/modules/stock/rev/79042b153b75
New changeset 3283445ab457 by Cédric Krier in branch 'default':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/tryton-env/rev/3283445ab457
New changeset b7473de967d6 by Cédric Krier in branch 'default':
Add missing inventory to depends of inventory_state
https://hg.tryton.org/modules/stock/rev/b7473de967d6
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-11-13.13:06:58

Indeed the problem comes from issue9828 but also the inventory_state must depend on inventory field.
Here is review300911002 that adds the depends.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-11-13.12:16:24

Unlike msg61833, it does not work neither on tryton.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-11-12.10:29:18

If it works on tryton then it is probably a bug in sao.
Or it is something similar to issue9630.

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-11-12.10:06:36

This is strange, because lines only depends on inventory_state which is an on change_field that depends on inventory. But if the inventory is not present on the view the on_change_with should return the draft value.

So for me the depends is correct

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-11-12.09:52:24

But it is not the proper fix because depends should solve this.

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-11-12.09:33:23

You are right. Including the inventory on the view fixes the issue. Thanks

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-11-12.00:38:22

I think it is linked to the fact that the inventory is not in the editable list view. So the inventory_state can not be computed and then all fields are readonly.
It starts to work if the form view is opened and new lines are added because now the inventory field is set.

Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-11-11.18:07:16

Using sao on version 5.8 it is not possible to create a new inventory move because the line is readonly.

Everything works well using tryton.

History
Date User Action Args
2020-11-29 17:43:51roundup-botsetkeyword: - backport
messages: + msg62292
2020-11-23 23:36:50roundup-botsetmessages: + msg62155
2020-11-23 23:36:33roundup-botsetmessages: + msg62152
nosy: + roundup-bot
status: testing -> resolved
2020-11-13 13:24:21reviewbotsetmessages: + msg61898
2020-11-13 13:06:58cedsetmessages: + msg61895
reviews: 295051002 -> 295051002,300911002
status: in-progress -> testing
superseder: + The depends of depends are not added to the view
2020-11-13 12:16:24cedsetassignedto: ced
messages: + msg61892
status: chatting -> in-progress
2020-11-12 10:30:18pokolisetstatus: testing -> chatting
2020-11-12 10:30:08pokolisetassignedto: pokoli -> (no value)
2020-11-12 10:29:18cedsetmessages: + msg61857
2020-11-12 10:06:36pokolisetmessages: + msg61854

Showing 10 items. Show all history (warning: this could be VERY long)