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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 2 days ago by albert
Modified:
1 week, 2 days ago
Reviewers:
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: 2
albert
1 week, 2 days ago (2018-04-11 00:02:07 UTC) #1
reviewbot
1 week, 2 days ago (2018-04-11 00:05:37 UTC) #2
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 not cond:'

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1136
move.py:1136:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1146
move.py:1146:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1167
move.py:1167:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1208
move.py:1208:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1225
move.py:1225:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44961003/diff/1/move.py#newcode1272
move.py:1272:
E731 do not assign a lambda expression, use a def

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

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