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

Issue 50771002: sao: Set only fields in _loaded (Closed)

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

Description

_loaded is compared with the model.fields so it should only contains existing field and no pseudo-field like _timestamp. issue8120

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M src/model.js View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6
ced
2 months ago (2019-02-15 15:05:02 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/50771002
2 months ago (2019-02-15 15:12:21 UTC) #2
pokoli
LGTM
1 month, 4 weeks ago (2019-02-19 08:31:20 UTC) #3
rietveld-bot_tryton.org
New changeset 50a80eac8876 by Cédric Krier in branch 'default': Set only fields in _loaded https://hg.tryton.org/sao/rev/50a80eac8876
1 month, 3 weeks ago (2019-02-26 14:58:49 UTC) #4
rietveld-bot_tryton.org
New changeset 7481deddecad by Cédric Krier in branch 'default': Set only fields in _loaded https://hg.tryton.org/tryton-env/rev/7481deddecad
1 month, 3 weeks ago (2019-02-26 14:58:55 UTC) #5
rietveld-bot_tryton.org
1 month, 1 week ago (2019-03-06 23:06:28 UTC) #6
New changeset 16c54e1871cb by Cédric Krier in branch '5.0':
Set only fields in _loaded
https://hg.tryton.org/sao/rev/16c54e1871cb


New changeset d3b6ae39c137 by Cédric Krier in branch '4.8':
Set only fields in _loaded
https://hg.tryton.org/sao/rev/d3b6ae39c137


New changeset b561aa4422e9 by Cédric Krier in branch '4.6':
Set only fields in _loaded
https://hg.tryton.org/sao/rev/b561aa4422e9


New changeset 88762da11d1d by Cédric Krier in branch '4.4':
Set only fields in _loaded
https://hg.tryton.org/sao/rev/88762da11d1d
Sign in to reply to this message.

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