From 55d30d5440abb63920ae3fe54af685695f43710a Mon Sep 17 00:00:00 2001
From: andrea4ever <4everamd@gmail.com>
Date: Thu, 28 Apr 2016 10:37:12 +0200
Subject: [PATCH] [IMP] Fill report open invoice with first info's

---
 account_financial_report_qweb/reports.xml     |  5 +--
 .../wizard/open_invoice_wizard.py             | 36 ++++++++-----------
 .../wizard/open_invoice_wizard_view.xml       |  1 -
 3 files changed, 17 insertions(+), 25 deletions(-)

diff --git a/account_financial_report_qweb/reports.xml b/account_financial_report_qweb/reports.xml
index f2f78dfd..e56ae668 100644
--- a/account_financial_report_qweb/reports.xml
+++ b/account_financial_report_qweb/reports.xml
@@ -25,12 +25,13 @@
 
         <report
             string="Open Invoice Report"
-            id="action_open_invoice_report"
+            id="report_open_invoice_report"
             model="open.invoice.wizard"
             report_type="qweb-pdf"
             name="account_financial_report_qweb.open_invoice_report_qweb"
             file="account_financial_report_qweb.open_invoice_report_qweb"/>
-        <record id="action_open_invoice_report" model="ir.actions.report.xml">
+
+        <record id="report_open_invoice_report" model="ir.actions.report.xml">
             <field name="paperformat_id" ref="account_financial_report_qweb.open_invoice_report_format_a4_horizontal"/>
         </record>
 
diff --git a/account_financial_report_qweb/wizard/open_invoice_wizard.py b/account_financial_report_qweb/wizard/open_invoice_wizard.py
index bc42ef3e..a0770c8c 100644
--- a/account_financial_report_qweb/wizard/open_invoice_wizard.py
+++ b/account_financial_report_qweb/wizard/open_invoice_wizard.py
@@ -5,7 +5,7 @@
 # Copyright 2016 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from openerp import models, fields, api
+from openerp import models, fields, api, _
 from openerp.exceptions import Warning as UserError
 from datetime import datetime
 
@@ -36,7 +36,7 @@ class OpenInvoiceWizard(models.TransientModel):
         ('all', 'All Entries')], 'Target Moves',
         required=True, default='all')
     until_date = fields.Date(
-        "Clearance date", required=True,
+        "Clearance date",
         help="""The clearance date is essentially a tool used for debtors
         provisionning calculation.
         By default, this date is equal to the the end date (
@@ -67,8 +67,7 @@ class OpenInvoiceWizard(models.TransientModel):
             account_type = ('payable', )
         domain = [
             ('company_id', '=', data['company_id'].id),
-            ('move_id.date', '>=', data['at_date']),
-            ('move_id.date', '<=', data['until_date']),
+            ('move_id.date', '<=', data['at_date']),
             ('account_id.user_type_id.type', 'in', account_type)
             ]
         if data['target_move'] != 'all':
@@ -81,25 +80,18 @@ class OpenInvoiceWizard(models.TransientModel):
 
     @staticmethod
     def _get_moves_data(move):
-        # return {
-        #     'date': data.date,
-        #     'period': data.invoice_id.period_id.name,
-        #     'journal': data.move_id.journal_id.name,
-        #     'reference': data.,
-        #     '': data.,
-        #     '': data.,
-        #     }
         return {
-            'date': '',
+            'date': move.date,
             'period': '',
-            'entry': '',
-            'journal': '',
-            'reference': '',
-            'label': '',
-            'rec': '',
-            'due_date': '',
-            'debit': '',
-            'credit': '',
+            'entry': move.move_id.name,
+            'journal': move.move_id.journal_id.code,
+            'reference': move.ref,
+            'label': '{move_line_name} ({move_ref})'.format(
+                move_line_name=move.name, move_ref=move.move_id.ref),
+            'rec': move.full_reconcile_id.name,
+            'due_date': move.date_maturity,
+            'debit': move.debit or '',
+            'credit': move.credit or '',
             'balance': '',
             }
 
@@ -117,6 +109,6 @@ class OpenInvoiceWizard(models.TransientModel):
                 datas[move.account_id.name][move.partner_id.name] = []
             datas[move.account_id.name][move.partner_id.name].append(
                 self._get_moves_data(move))
-        return self.env['report'].get_action(
+        return self.env['report'].with_context(landscape=True).get_action(
             self, 'account_financial_report_qweb.open_invoice_report_qweb',
             data={'data': datas})
diff --git a/account_financial_report_qweb/wizard/open_invoice_wizard_view.xml b/account_financial_report_qweb/wizard/open_invoice_wizard_view.xml
index 8a3870f1..12115de8 100644
--- a/account_financial_report_qweb/wizard/open_invoice_wizard_view.xml
+++ b/account_financial_report_qweb/wizard/open_invoice_wizard_view.xml
@@ -12,7 +12,6 @@
                     </group>
                     <group name="date_info">
                         <field name="at_date"/>
-                        <field name="until_date" />
                     </group>
                     <group name="partner_info">
                         <field name="partner_ids" />