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

Issue 44551003: trytond: Add help texts.

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

Description

Patch Set 1 #

Total comments: 57

Patch Set 2 : Fix Pokolis comments and general re-work. #

Patch Set 3 : Fix Pokolis comments 2 #

Patch Set 4 : Update tip #

Patch Set 5 : Fix typos #

Patch Set 6 : Update tip #

Patch Set 7 : Update tip #

Total comments: 108

Patch Set 8 : update tip #

Patch Set 9 : ceds comments #

Patch Set 10 : Update tip #

Patch Set 11 : Split review #

Patch Set 12 : Add help to ir.model.button instead of view. #

Patch Set 13 : Remove unrelated file. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+457 lines, -252 lines) Patch
M trytond/ir/action.py View 1 2 3 4 5 6 7 8 9 10 11 11 chunks +91 lines, -56 lines 0 comments Download
M trytond/ir/attachment.py View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +13 lines, -6 lines 0 comments Download
M trytond/ir/cron.py View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +21 lines, -14 lines 0 comments Download
M trytond/ir/cron.xml View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +1 line, -1 line 0 comments Download
M trytond/ir/export.py View 1 1 chunk +7 lines, -4 lines 0 comments Download
M trytond/ir/lang.py View 1 2 3 4 5 6 7 8 9 1 chunk +64 lines, -23 lines 0 comments Download
M trytond/ir/model.py View 1 2 3 4 5 6 7 8 9 10 11 10 chunks +90 lines, -53 lines 0 comments Download
M trytond/ir/model.xml View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +1 line, -1 line 0 comments Download
M trytond/ir/module.py View 1 2 3 4 5 6 7 8 9 6 chunks +33 lines, -18 lines 0 comments Download
M trytond/ir/module.xml View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +6 lines, -0 lines 0 comments Download
M trytond/ir/note.py View 1 2 chunks +7 lines, -5 lines 0 comments Download
M trytond/ir/resource.py View 1 1 chunk +3 lines, -3 lines 0 comments Download
M trytond/ir/rule.py View 1 2 3 4 5 6 7 8 9 10 11 2 chunks +26 lines, -15 lines 0 comments Download
M trytond/ir/sequence.py View 1 2 3 2 chunks +28 lines, -14 lines 0 comments Download
M trytond/ir/translation.py View 1 2 3 4 5 6 7 8 9 10 11 8 chunks +41 lines, -24 lines 0 comments Download
M trytond/ir/trigger.py View 1 2 3 4 5 6 7 8 9 10 11 1 chunk +25 lines, -15 lines 0 comments Download

Messages

Total messages: 34
udono
10 months, 3 weeks ago (2017-12-26 11:04:29 UTC) #1
reviewbot
https://codereview.tryton.org/44551003/diff/1/trytond/res/ir.py#newcode125 trytond/res/ir.py:125: F841 local variable 'database' is assigned to but never used https://codereview.tryton.org/44551003/diff/1/trytond/res/user.py#newcode548 trytond/res/user.py:548: E712 ...
10 months, 3 weeks ago (2017-12-26 11:33:42 UTC) #2
pokoli
Thanks for your work. I haven't review all the code but there are some guideliness ...
10 months, 1 week ago (2018-01-11 09:43:24 UTC) #3
ced
8 months, 2 weeks ago (2018-03-04 13:26:46 UTC) #4
udono
Fix Pokolis comments and general re-work.
8 months ago (2018-03-20 20:24:50 UTC) #5
udono
https://tryton-rietveld.appspot.com/44551003/diff/1/trytond/ir/action.py File trytond/ir/action.py (right): https://tryton-rietveld.appspot.com/44551003/diff/1/trytond/ir/action.py#newcode36 trytond/ir/action.py:36: help="The model which specifies the result of the action ...
8 months ago (2018-03-20 20:25:23 UTC) #6
reviewbot
https://codereview.tryton.org/44551003/diff/20001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/20001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
8 months ago (2018-03-20 20:53:27 UTC) #7
pokoli
https://codereview.tryton.org/44551003/diff/1/trytond/ir/action.py File trytond/ir/action.py (right): https://codereview.tryton.org/44551003/diff/1/trytond/ir/action.py#newcode36 trytond/ir/action.py:36: help="The model which specifies the result of the action ...
8 months ago (2018-03-21 10:13:27 UTC) #8
udono
Fix Pokolis comments 2
8 months ago (2018-03-21 11:18:42 UTC) #9
udono
https://codereview.tryton.org/44551003/diff/1/trytond/ir/action.py File trytond/ir/action.py (right): https://codereview.tryton.org/44551003/diff/1/trytond/ir/action.py#newcode36 trytond/ir/action.py:36: help="The model which specifies the result of the action ...
8 months ago (2018-03-21 11:19:50 UTC) #10
reviewbot
https://codereview.tryton.org/44551003/diff/40001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/40001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
8 months ago (2018-03-21 11:22:48 UTC) #11
udono
Update tip
7 months, 2 weeks ago (2018-04-04 08:14:18 UTC) #12
reviewbot
https://codereview.tryton.org/44551003/diff/60001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/60001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
7 months, 2 weeks ago (2018-04-04 08:27:53 UTC) #13
udono
Fix typos
7 months, 2 weeks ago (2018-04-04 12:32:20 UTC) #14
reviewbot
https://codereview.tryton.org/44551003/diff/80001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/80001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
7 months, 2 weeks ago (2018-04-04 12:58:30 UTC) #15
udono
Update tip
7 months, 2 weeks ago (2018-04-05 17:25:10 UTC) #16
reviewbot
https://codereview.tryton.org/44551003/diff/100001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/100001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
7 months, 2 weeks ago (2018-04-05 17:34:04 UTC) #17
udono
Update tip
6 months, 3 weeks ago (2018-04-25 06:48:48 UTC) #18
reviewbot
https://codereview.tryton.org/44551003/diff/120001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/120001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
6 months, 3 weeks ago (2018-04-25 07:03:31 UTC) #19
ced
I reviewed only the first file. I'm wondering if it will not be better to ...
6 months ago (2018-05-15 10:51:13 UTC) #20
udono
update tip
5 months ago (2018-06-20 15:59:28 UTC) #21
reviewbot
https://codereview.tryton.org/44551003/diff/140001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/140001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
5 months ago (2018-06-20 16:02:33 UTC) #22
udono
ceds comments
5 months ago (2018-06-20 16:51:40 UTC) #23
udono
https://tryton-rietveld.appspot.com/44551003/diff/120001/trytond/ir/action.py File trytond/ir/action.py (right): https://tryton-rietveld.appspot.com/44551003/diff/120001/trytond/ir/action.py#newcode36 trytond/ir/action.py:36: usage = fields.Char('Usage', help="Add the use of the action.") ...
5 months ago (2018-06-20 16:52:06 UTC) #24
reviewbot
https://codereview.tryton.org/44551003/diff/160001/trytond/tests/run-tests.py#newcode44 trytond/tests/run-tests.py:44: E402 module level import not at top of file https://codereview.tryton.org/44551003/diff/160001/trytond/model/model.py#newcode181 trytond/model/model.py:181: E127 continuation ...
5 months ago (2018-06-20 16:56:37 UTC) #25
ced
On 2018/06/20 16:51:40, udono wrote: > ceds comments I really encourage to split this review ...
5 months ago (2018-06-21 07:12:00 UTC) #26
udono
Update tip
4 months ago (2018-07-16 06:34:42 UTC) #27
udono
Split review
4 months ago (2018-07-16 06:36:47 UTC) #28
reviewbot
https://codereview.tryton.org/44551003/diff/200001/trytond/ir/action.py#newcode273 trytond/ir/action.py:273: E731 do not assign a lambda expression, use a def https://codereview.tryton.org/44551003/diff/200001/trytond/ir/action.py#newcode282 trytond/ir/action.py:282: E731 ...
4 months ago (2018-07-16 06:41:57 UTC) #29
udono
On 2018/06/21 07:12:00, ced wrote: > On 2018/06/20 16:51:40, udono wrote: > > ceds comments ...
4 months ago (2018-07-16 06:49:14 UTC) #30
udono
Add help to ir.model.button instead of view.
3 months, 1 week ago (2018-08-06 13:08:05 UTC) #31
reviewbot
https://codereview.tryton.org/44551003/diff/220001/trytond/ir/action.py#newcode272 trytond/ir/action.py:272: E731 do not assign a lambda expression, use a def https://codereview.tryton.org/44551003/diff/220001/trytond/ir/action.py#newcode281 trytond/ir/action.py:281: E731 ...
3 months, 1 week ago (2018-08-06 13:11:24 UTC) #32
udono
Remove unrelated file.
3 months, 1 week ago (2018-08-06 13:14:32 UTC) #33
reviewbot
3 months, 1 week ago (2018-08-06 13:41:35 UTC) #34
https://codereview.tryton.org/44551003/diff/240001/trytond/ir/action.py#newco...
trytond/ir/action.py:272:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/action.py#newco...
trytond/ir/action.py:281:
E731 do not assign a lambda expression, use a def

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/attachment.py#n...
trytond/ir/attachment.py:61:
E303 too many blank lines (2)

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode7
trytond/ir/lang.py:7:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode9
trytond/ir/lang.py:9:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode11
trytond/ir/lang.py:11:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode13
trytond/ir/lang.py:13:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode14
trytond/ir/lang.py:14:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode15
trytond/ir/lang.py:15:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode16
trytond/ir/lang.py:16:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode17
trytond/ir/lang.py:17:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode18
trytond/ir/lang.py:18:
E402 module level import not at top of file

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/lang.py#newcode49
trytond/ir/lang.py:49:
E302 expected 2 blank lines, found 1

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:100:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:279:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:517:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:710:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:710:
E203 whitespace before ','

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:1125:
E202 whitespace before ')'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/model.py#newcod...
trytond/ir/model.py:1164:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/rule.py#newcode52
trytond/ir/rule.py:52:
E712 comparison to False should be 'if cond is False:' or 'if not cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/rule.py#newcode202
trytond/ir/rule.py:202:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:233:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:252:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:272:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:306:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:333:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:379:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:466:
E712 comparison to False should be 'if cond is False:' or 'if not cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:667:
E712 comparison to False should be 'if cond is False:' or 'if not cond:'

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:1119:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/translation.py#...
trytond/ir/translation.py:1217:
E127 continuation line over-indented for visual indent

https://codereview.tryton.org/44551003/diff/240001/trytond/ir/trigger.py#newc...
trytond/ir/trigger.py:71:
E712 comparison to True should be 'if cond is True:' or 'if cond:'

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

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