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

Issue 56331002: sao: Fix domain inversion evaluation of constrained field (Closed)

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

Description

The evaluation to True of constrained clause should only be done for inversion. Also it should be valid even if a wrong value is defined in the context. issue7603

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -13 lines) Patch
M src/common.js View 4 chunks +23 lines, -12 lines 0 comments Download
M tests/sao.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
ced
4 months, 1 week ago (2018-08-09 22:11:28 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/56331002
4 months, 1 week ago (2018-08-09 22:33:45 UTC) #2
pokoli
LGTM
4 months, 1 week ago (2018-08-10 09:36:04 UTC) #3
rietveld-bot_tryton.org
New changeset 532976d3ab2d by Cédric Krier in branch 'default': Fix domain inversion evaluation of constrained ...
3 months, 4 weeks ago (2018-08-18 20:51:07 UTC) #4
rietveld-bot_tryton.org
New changeset e30446ac03ca by Cédric Krier in branch 'default': Fix domain inversion evaluation of constrained ...
3 months, 4 weeks ago (2018-08-18 20:51:11 UTC) #5
rietveld-bot_tryton.org
3 months, 3 weeks ago (2018-08-26 19:14:10 UTC) #6
New changeset ccb7109abf12 by Cédric Krier in branch '4.8':
Fix domain inversion evaluation of constrained field
https://hg.tryton.org/sao/rev/ccb7109abf12


New changeset d04a3ce20123 by Cédric Krier in branch '4.6':
Fix domain inversion evaluation of constrained field
https://hg.tryton.org/sao/rev/d04a3ce20123


New changeset 4bbcfd1a892c by Cédric Krier in branch '4.4':
Fix domain inversion evaluation of constrained field
https://hg.tryton.org/sao/rev/4bbcfd1a892c


New changeset f97a9d4be7b4 by Cédric Krier in branch '4.2':
Fix domain inversion evaluation of constrained field
https://hg.tryton.org/sao/rev/f97a9d4be7b4


New changeset c0367252b725 by Cédric Krier in branch '4.0':
Fix domain inversion evaluation of constrained field
https://hg.tryton.org/sao/rev/c0367252b725
Sign in to reply to this message.

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