details: https://code.tryton.org/tryton/commit/fcfd1fe3255c
branch: 7.8
user: Cédric Krier <[email protected]>
date: Thu Oct 01 09:54:29 2026 +0200
description:
Show price and currency fields on stock move when they are not required
but still filled
This is to let the user clear the invalid fields from past moves.
Closes #14919
(grafted from d5ace112f8a9315a6740ef75c2bda4cc8b94b90e)
diffstat:
modules/stock/move.py | 14 +++++++++-----
1 files changed, 9 insertions(+), 5 deletions(-)
diffs (38 lines):
diff -r a8dd7c6d5b12 -r fcfd1fe3255c modules/stock/move.py
--- a/modules/stock/move.py Thu Oct 01 15:30:57 2026 +0200
+++ b/modules/stock/move.py Thu Oct 01 09:54:29 2026 +0200
@@ -324,8 +324,10 @@
()),
],
states={
- 'invisible': ~Eval('unit_price_required'),
- 'required': Bool(Eval('unit_price_required')),
+ 'invisible': (
+ ~Eval('unit_price_required', False)
+ & (Eval('unit_price', None) == None)), # noqa: E711
+ 'required': Eval('unit_price_required', False),
'readonly': Eval('state') != 'draft',
})
unit_price_company = fields.Function(
@@ -348,7 +350,7 @@
()),
],
states={
- 'invisible': ~Eval('cost_price_required'),
+ 'invisible': ~Eval('cost_price_required', False),
'required': (
(Eval('state') == 'done')
& Eval('cost_price_required', False)),
@@ -372,8 +374,10 @@
()),
],
states={
- 'invisible': ~Eval('unit_price_required'),
- 'required': Bool(Eval('unit_price_required')),
+ 'invisible': (
+ ~Eval('unit_price_required')
+ & (Eval('currency', None) == None)), # noqa: E711
+ 'required': Eval('unit_price_required', False),
'readonly': Eval('state') != 'draft',
},
help="The currency in which the unit price is specified.")