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

Issue 39021002: trytond: Improve validation of PYSON domain (Closed)

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

Description

Use a different strategy if we are close to one domain per record. It will use combined multiple domain at once instead of doing one query for almost each record validated. issue6983

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+78 lines, -1 line) Patch
M CHANGELOG View 1 chunk +1 line, -0 lines 0 comments Download
M trytond/model/modelstorage.py View 2 chunks +26 lines, -1 line 0 comments Download
M trytond/tests/__init__.py View 1 chunk +1 line, -0 lines 0 comments Download
M trytond/tests/model.py View 3 chunks +14 lines, -0 lines 0 comments Download
M trytond/tests/test_modelstorage.py View 1 chunk +36 lines, -0 lines 0 comments Download

Messages

Total messages: 3
ced
7 months, 3 weeks ago (2017-12-01 16:34:14 UTC) #1
reviewbot
https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newcode11 trytond/tests/__init__.py:11: F403 'from test import *' used; unable to detect undefined names https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newcode12 trytond/tests/__init__.py:12: ...
7 months, 3 weeks ago (2017-12-01 16:58:41 UTC) #2
rietveld-bot_tryton.org
6 months, 4 weeks ago (2017-12-26 16:28:46 UTC) #3
New changeset 27d2fcb4fc10 by Cédric Krier in branch 'default':
Improve validation of PYSON domain
http://hg.tryton.org/trytond/rev/27d2fcb4fc10
Sign in to reply to this message.

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