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

Issue 44961003: stock: Take advantage of planned_date and effective_date indexes

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 months, 1 week ago by albert
Modified:
3 months ago
Reviewers:
ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M move.py View 1 chunk +5 lines, -2 lines 0 comments Download

Messages

Total messages: 3
albert
3 months, 1 week ago (2018-04-11 00:02:07 UTC) #1
reviewbot
https://codereview.tryton.org/44961003/diff/1/move.py#newcode1089 move.py:1089: E712 comparison to True should be 'if cond is not True:' or 'if ...
3 months, 1 week ago (2018-04-11 00:05:37 UTC) #2
ced
3 months ago (2018-04-23 22:31:37 UTC) #3
NOT LGTM it makes SQL less readable.
Sign in to reply to this message.

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