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

Issue 39021002: trytond: Improve validation of PYSON domain

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 weeks ago by ced
Modified:
2 weeks ago
Reviewers:
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: 2
ced
2 weeks ago (2017-12-01 16:34:14 UTC) #1
reviewbot
2 weeks ago (2017-12-01 16:58:41 UTC) #2
https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
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#newco...
trytond/tests/__init__.py:12:
F403 'from model import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:13:
F403 'from modelview import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:14:
F403 'from mptt import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:15:
F403 'from import_data import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:16:
F403 'from export_data import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:17:
F403 'from trigger import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:18:
F403 'from access import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:19:
F403 'from wizard import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:20:
F403 'from workflow import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:21:
F403 'from copy_ import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:22:
F403 'from history import *' used; unable to detect undefined names

https://codereview.tryton.org/39021002/diff/1/trytond/tests/__init__.py#newco...
trytond/tests/__init__.py:23:
F403 'from field_context import *' used; unable to detect undefined names

URL: https://codereview.tryton.org/39021002
Sign in to reply to this message.

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