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

Issue 40271002: trytond: Ensure all _parent_ values are set on fields.depends (Closed)

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

Description

If the client does not sent all the values of the parent model fields, we must ensure that the empty ones have None value issue6450

Patch Set 1 #

Total comments: 1

Patch Set 2 : Make recursive and add unittest #

Patch Set 3 : Remove uneeded join and improve test #

Total comments: 4

Patch Set 4 : Fix comments #

Total comments: 8

Patch Set 5 : Fix ced's comments #

Total comments: 1

Patch Set 6 : Remove undeed parent depends #

Unified diffs Side-by-side diffs Delta from patch set Stats (+98 lines, -5 lines) Patch
M trytond/model/fields/field.py View 1 2 3 4 2 chunks +16 lines, -5 lines 0 comments Download
A trytond/tests/test_field_depends.py View 1 2 3 4 5 1 chunk +82 lines, -0 lines 0 comments Download

Messages

Total messages: 21
pokoli
3 months, 1 week ago (2017-04-19 12:30:43 UTC) #1
reviewbot
https://codereview.tryton.org/40271002/diff/1/trytond/model/fields/field.py#newcode375 trytond/model/fields/field.py:375: E712 comparison to False should be 'if cond is False:' or 'if not ...
3 months, 1 week ago (2017-04-19 12:32:52 UTC) #2
ced
I think it deserves its own issue. We should have unittest for this decorator. https://tryton-rietveld.appspot.com/40271002/diff/1/trytond/model/fields/field.py ...
3 months, 1 week ago (2017-04-19 12:56:54 UTC) #3
pokoli
Make recursive and add unittest
3 months, 1 week ago (2017-04-19 15:34:45 UTC) #4
reviewbot
https://codereview.tryton.org/40271002/diff/20001/trytond/tests/__init__.py#newcode5 trytond/tests/__init__.py:5: F403 'from test import *' used; unable to detect undefined names https://codereview.tryton.org/40271002/diff/20001/trytond/tests/__init__.py#newcode6 trytond/tests/__init__.py:6: ...
3 months, 1 week ago (2017-04-19 15:41:21 UTC) #5
pokoli
Remove uneeded join and improve test
3 months, 1 week ago (2017-04-19 15:41:45 UTC) #6
reviewbot
https://codereview.tryton.org/40271002/diff/20002/trytond/tests/__init__.py#newcode5 trytond/tests/__init__.py:5: F403 'from test import *' used; unable to detect undefined names https://codereview.tryton.org/40271002/diff/20002/trytond/tests/__init__.py#newcode6 trytond/tests/__init__.py:6: ...
3 months, 1 week ago (2017-04-19 16:02:52 UTC) #7
ced
https://tryton-rietveld.appspot.com/40271002/diff/20002/trytond/model/fields/field.py File trytond/model/fields/field.py (right): https://tryton-rietveld.appspot.com/40271002/diff/20002/trytond/model/fields/field.py#newcode87 trytond/model/fields/field.py:87: def set_parent_field(record, field): is it not better name: set_value ...
3 months, 1 week ago (2017-04-19 17:10:26 UTC) #8
pokoli
Fix comments
3 months ago (2017-04-20 14:02:14 UTC) #9
reviewbot
https://codereview.tryton.org/40271002/diff/50001/trytond/model/fields/field.py#newcode376 trytond/model/fields/field.py:376: E712 comparison to False should be 'if cond is False:' or 'if not ...
3 months ago (2017-04-20 14:38:48 UTC) #10
ced
https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/model/fields/field.py File trytond/model/fields/field.py (right): https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/model/fields/field.py#newcode87 trytond/model/fields/field.py:87: def set_value(record, field): Maybe the func could be defined ...
3 months ago (2017-04-20 15:35:31 UTC) #11
pokoli
Fix ced's comments
3 months ago (2017-04-20 15:59:32 UTC) #12
pokoli
https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/tests/test_field_depends.py File trytond/tests/test_field_depends.py (right): https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/tests/test_field_depends.py#newcode18 trytond/tests/test_field_depends.py:18: record = Model() On 2017/04/20 15:35:31, ced wrote: > ...
3 months ago (2017-04-20 15:59:42 UTC) #13
reviewbot
https://codereview.tryton.org/40271002/diff/70001/trytond/model/fields/field.py#newcode377 trytond/model/fields/field.py:377: E712 comparison to False should be 'if cond is False:' or 'if not ...
3 months ago (2017-04-20 16:01:40 UTC) #14
ced
https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/tests/test_field_depends.py File trytond/tests/test_field_depends.py (right): https://tryton-rietveld.appspot.com/40271002/diff/50001/trytond/tests/test_field_depends.py#newcode20 trytond/tests/test_field_depends.py:20: self.assertIsNone(record.name) On 2017/04/20 15:59:42, pokoli wrote: > On 2017/04/20 ...
3 months ago (2017-04-20 16:28:16 UTC) #15
ced
https://tryton-rietveld.appspot.com/40271002/diff/70001/trytond/tests/test_field_depends.py File trytond/tests/test_field_depends.py (right): https://tryton-rietveld.appspot.com/40271002/diff/70001/trytond/tests/test_field_depends.py#newcode61 trytond/tests/test_field_depends.py:61: '_parent_parent.description', '_parent_parent.parent', is '_parent_parent.parent' really needed?
3 months ago (2017-04-20 20:57:52 UTC) #16
pokoli
Remove undeed parent depends
3 months ago (2017-04-26 11:02:26 UTC) #17
ced
LGTM: commit test separately for backport.
3 months ago (2017-04-26 11:16:25 UTC) #18
reviewbot
https://codereview.tryton.org/40271002/diff/90001/trytond/model/fields/field.py#newcode377 trytond/model/fields/field.py:377: E712 comparison to False should be 'if cond is False:' or 'if not ...
3 months ago (2017-04-26 11:41:01 UTC) #19
rietveld-bot_tryton.org
New changeset 6383539b232e by Sergi Almacellas Abellana in branch 'default': Ensure all _parent_ values are ...
3 months ago (2017-04-26 11:57:30 UTC) #20
rietveld-bot_tryton.org
2 months, 3 weeks ago (2017-05-03 16:48:58 UTC) #21
New changeset fe6b783fc23d by Sergi Almacellas Abellana in branch '4.2':
Ensure all _parent_ values are set on fields.depends
http://hg.tryton.org/trytond/rev/fe6b783fc23d


New changeset b4eebc2f2c96 by Sergi Almacellas Abellana in branch '4.0':
Ensure all _parent_ values are set on fields.depends
http://hg.tryton.org/trytond/rev/b4eebc2f2c96


New changeset 83b060c3e3e2 by Sergi Almacellas Abellana in branch '3.8':
Ensure all _parent_ values are set on fields.depends
http://hg.tryton.org/trytond/rev/83b060c3e3e2


New changeset 494e697f5940 by Sergi Almacellas Abellana in branch '3.6':
Ensure all _parent_ values are set on fields.depends
http://hg.tryton.org/trytond/rev/494e697f5940


New changeset 3b3d30c4daa1 by Sergi Almacellas Abellana in branch '3.4':
Ensure all _parent_ values are set on fields.depends
http://hg.tryton.org/trytond/rev/3b3d30c4daa1
Sign in to reply to this message.

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