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

Issue 46531002: purchase: Use single report for purchase

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 8 months ago by pokoli
Modified:
4 years, 8 months ago
Reviewers:
ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -24 lines) Patch
M purchase.fodt View 8 chunks +21 lines, -22 lines 0 comments Download
M purchase.py View 2 chunks +8 lines, -2 lines 1 comment Download
M purchase.xml View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3
pokoli
4 years, 8 months ago (2018-05-24 10:56:26 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/46531002
4 years, 8 months ago (2018-05-24 11:25:37 UTC) #2
ced
4 years, 8 months ago (2018-05-31 15:59:35 UTC) #3
https://codereview.tryton.org/46531002/diff/1/purchase.py
File purchase.py (right):

https://codereview.tryton.org/46531002/diff/1/purchase.py#newcode1645
purchase.py:1645: context['purchase'] = context['record']
I do not think it is a good idea to customize the name of the record. This makes
template less reusable.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld d9ca037-tainted