trytond: Add support for single record report

A single report render each record separately and return them in a zip file.
To ensure the uniqueness of the file in the zip, it uses the id of the record
but also the record name to be user-friendly.

issue6640

[reviewbot, 2017-07-13 14:12:59]

flake8 OK
URL: https://codereview.tryton.org/37581002

[pokoli, 2017-07-27 13:30:27]

Maybe it's worth to add a test about calling single with multiple records.

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
File doc/topics/reports/index.rst (right):

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
doc/topics/reports/index.rst:53: * ``single``: `True` if the template works for
only one record.
i think correct wording is: "the template works only for one record".

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
doc/topics/reports/index.rst:53: * ``single``: `True` if the template works for
only one record.
We should probably explain that a zip file with all the reports are returned in
case of calling the action with multiple records.

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/trytond/ir/action.py
File trytond/ir/action.py (right):

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/trytond/ir/action.py#...
trytond/ir/action.py:413: help="Check if the template works for only one
record.")
idem

[ced, 2017-07-28 07:33:48]

On 2017/07/27 13:30:27, pokoli wrote:
> Maybe it's worth to add a test about calling single with multiple records.

We have no report test in trytond because it will required to create one etc.
I think before adding such custom code, we should reorganise the trytond tests
to be more manageable.
Also the test is in the account_invoice report :-)

[ced, 2017-07-28 07:34:00]

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
File doc/topics/reports/index.rst (right):

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
doc/topics/reports/index.rst:53: * ``single``: `True` if the template works for
only one record.
On 2017/07/27 13:30:27, pokoli wrote:
> We should probably explain that a zip file with all the reports are returned
in
> case of calling the action with multiple records. 

Done.

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/doc/topics/reports/in...
doc/topics/reports/index.rst:53: * ``single``: `True` if the template works for
only one record.
On 2017/07/27 13:30:26, pokoli wrote:
> i think correct wording is: "the template works only for one record". 

Acknowledged.

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/trytond/ir/action.py
File trytond/ir/action.py (right):

https://tryton-rietveld-hrd.appspot.com/37581002/diff/1/trytond/ir/action.py#...
trytond/ir/action.py:413: help="Check if the template works for only one
record.")
On 2017/07/27 13:30:27, pokoli wrote:
> idem

Done.

[ced, 2017-07-28 07:34:02]

Fix wording

[reviewbot, 2017-07-28 07:40:13]

flake8 OK
URL: https://codereview.tryton.org/37581002

[pokoli, 2017-08-01 09:20:39]

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report.py
File trytond/report/report.py (right):

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report...
trytond/report/report.py:150: records = None
probably better to make records = [] as default as len(None) will fail

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report...
trytond/report/report.py:213: report_context['record'] = records[0]
what if there is no records?

[ced, 2017-08-01 14:01:51]

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report.py
File trytond/report/report.py (right):

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report...
trytond/report/report.py:150: records = None
On 2017/08/01 09:20:39, pokoli wrote:
> probably better to make records = [] as default as len(None) will fail

Done.

https://tryton-rietveld.appspot.com/37581002/diff/20001/trytond/report/report...
trytond/report/report.py:213: report_context['record'] = records[0]
On 2017/08/01 09:20:39, pokoli wrote:
> what if there is no records?

Done.

[ced, 2017-08-01 14:02:11]

Allow to execute report without records

[reviewbot, 2017-08-01 14:05:59]

flake8 OK
URL: https://codereview.tryton.org/37581002

[pokoli, 2017-08-01 15:20:30]

LGTM

[ced, 2017-09-03 09:08:29]

Update to tip

[rietveld-bot_tryton.org, 2017-09-03 09:14:17]

New changeset 0bd088a8474b by Cédric Krier in branch 'default':
Add support for single record report
http://hg.tryton.org/trytond/rev/0bd088a8474b