Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2118)

Issue 275371002: tryton-env: Do not send value for readonly Many2Many

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 weeks, 3 days ago by ced
Modified:
2 weeks, 3 days ago
Reviewers:
rietveld-bot, nicoe, reviewbot
Visibility:
Public.

Description

tryton-env: Do not send value for readonly Many2Many In contrary to One2Many, Many2Many can not be modified when they are readonly. But Many2Many inherit from One2Many so we must exclude them explicitly from the test. issue8323

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M sao/src/model.js View 1 chunk +2 lines, -1 line 0 comments Download
M tryton/tryton/gui/window/view_form/model/record.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 6
ced
2 weeks, 3 days ago (2019-05-06 09:49:58 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/275371002
2 weeks, 3 days ago (2019-05-06 10:09:58 UTC) #2
nicoe
LGTM
2 weeks, 3 days ago (2019-05-06 10:21:10 UTC) #3
rietveld-bot_tryton.org
New changeset b1cdebcbb624 by Cédric Krier in branch 'default': Do not send value for readonly ...
2 weeks, 3 days ago (2019-05-06 11:37:33 UTC) #4
rietveld-bot_tryton.org
New changeset 14c98cd0ce6e by Cédric Krier in branch 'default': Do not send value for readonly ...
2 weeks, 3 days ago (2019-05-06 11:37:36 UTC) #5
rietveld-bot_tryton.org
2 weeks, 3 days ago (2019-05-06 11:37:36 UTC) #6
New changeset e45c12b690f8 by Cédric Krier in branch 'default':
Do not send value for readonly Many2Many
https://hg.tryton.org/tryton-env/rev/e45c12b690f8
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766