Merge lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina into lp:addons-vauxoo/7.0

[Sabrina Romero - http://www.vauxoo.com]

[FIX] Bug #918961 solved without l10n_ve_fiscal_requirements and test added.

The bug was happening in the module debit_credit_note, but not with l10n_ve_fiscal_requirements module installed.

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

review: Needs Fixing

[lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina]

961. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Supplier refund test added.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[ADD] Supplier refund test added and fields belonging to OVL were removed with their respective methods.

review: Needs Resubmitting

[lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina]

962. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Some dependencies added.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[ADD] Some dependencies added.

review: Needs Resubmitting

[lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina]

963. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] The total amount of the invoice is assigned to the payment.

964. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] id correct assignment.

965. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] amount_total correct assignment.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

corrected error in the allocation of payment amount.
Tested to review 965, working!

review: Approve

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

review: Needs Resubmitting

[lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina]

966. By hbto [Vauxoo] http://www.vauxoo.com on 2014-01-31

[CC][debit_credit_note] Getting Rid of Commented Code

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

In general It Looks Ok,

Some issues,

*) Why did you have to change the browse for a read method in this iteration:

  for form in self.read(cr, uid, ids, context=context):

  for form in self.browse(cr, uid, ids, context=context):

Which resulted in you to change a bit of code downstream.

browse is one method recommended by openerp to use within the code

*) When accessing the date field you made a split and after
that you try to look for periods following the code which you
assume will _always_ be dash (-) separated,

date = wzr_brw.date and wzr_brw.date.split('-')
        period = wzr_brw and wzr_brw.period and wzr_brw.period.id
        period_ids = date and len(date) == 3 and period_obj.search(cr,uid,[('code','=','%s/%s'%(date[1],date[0]))],context=context)

please use instead the same date without splitting it, and
when look for the periods use the date though this time instead
of the code use date_start>= date and date_end <= date and special=False

this way it will be Period Code independent

Best Regards

review: Needs Fixing

[lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina]

967. By Sabrina Romero - http://www.vauxoo.com on 2014-02-03

[FIX] Using browse method instead read and splitting date removed.

[Sabrina Romero - http://www.vauxoo.com (sabrina)]

[FIX] Using browse method instead read and splitting date removed.
Tested to review 967, working!

review: Needs Resubmitting

[Nhomar - Vauxoo (nhomar)]

7 Month it stoped....

We should delete or merge or test + merge, what is the task related?

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

My fault, I have been delaying it.
On Sep 23, 2014 1:54 AM, "Nhomar - Vauxoo" <email address hidden> wrote:

> 7 Month it stoped....
>
> We should delete or merge or test + merge, what is the task related?
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina/+merge/202786
> You are reviewing the proposed merge of
> lp:~vauxoo/addons-vauxoo/7.0_debit-credit-yml_dev-sabrina into
> lp:addons-vauxoo/7.0.
>

Unmerged revisions

967. By Sabrina Romero - http://www.vauxoo.com on 2014-02-03

[FIX] Using browse method instead read and splitting date removed.

966. By hbto [Vauxoo] http://www.vauxoo.com on 2014-01-31

[CC][debit_credit_note] Getting Rid of Commented Code

965. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] amount_total correct assignment.

964. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] id correct assignment.

963. By Sabrina Romero - http://www.vauxoo.com on 2014-01-30

[FIX] The total amount of the invoice is assigned to the payment.

962. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Some dependencies added.

961. By Sabrina Romero - http://www.vauxoo.com on 2014-01-27

[ADD] Supplier refund test added.

960. By Sabrina Romero - http://www.vauxoo.com on 2014-01-23

[FIX] Bug #918961 solved without l10n_ve_fiscal_requirements and test added.

=== modified file 'debit_credit_note/__openerp__.py'
--- debit_credit_note/__openerp__.py	2014-01-17 21:53:54 +0000
+++ debit_credit_note/__openerp__.py	2014-02-03 16:27:19 +0000
@@ -44,11 +44,16 @@
 
 ''',
     "init_xml": [],
-    "depends": ["base", "account"],
+    "depends": [
+        "base",
+        "account",
+        "account_voucher",
+        "account_cancel",
+    ],
     'data': [
         'wizard/account_invoice_debit_view.xml',
         'wizard/account_invoice_parent_view.xml',
-        'account_invoice_view.xml',
+        "account_invoice_view.xml",
     ],
     'js': [
     ],
@@ -59,6 +64,8 @@
     'demo': [
     ],
     'test': [
+		'test/account_supplier_refund.yml',
+		'test/account_customer_refund.yml',
     ],
     "installable": True,
     "active": False,
 
=== added directory 'debit_credit_note/test'
=== added file 'debit_credit_note/test/account_customer_refund.yml'
--- debit_credit_note/test/account_customer_refund.yml	1970-01-01 00:00:00 +0000
+++ debit_credit_note/test/account_customer_refund.yml	2014-02-03 16:27:19 +0000
@@ -0,0 +1,602 @@
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'out_invoice'
+-
+  1.- In order to test account refund with "Create a draft refund" method
+-
+  1.1.- I create a customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer0, view: account.invoice_form}: &customer_invoice
+    payment_term: account.account_payment_term_advance
+    journal_id: account.sales_journal
+    partner_id: base.res_partner_3
+    reference_type: none
+    name: 'Test Customer Invoice - debit_credit_note'
+    invoice_line:
+      - product_id: product.product_product_5
+        price_unit: 850.0
+        quantity: 1.0
+-       
+  1.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0}:
+    - state == 'draft'
+-
+  1.3.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  1.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer0}
+-
+  1.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer0}:
+    - state == 'open'
+-
+  1.6.- I create a customer refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer0')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'draft', 'The customer refund was not properly created, the state must be Draft.'
+-
+  1.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer0'))
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  2.- In order to test account refund with "Create refund and reconcile" method
+-
+  2.1.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer1, view: account.invoice_form }:
+    <<: *customer_invoice
+-
+  2.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1}:
+    - state == 'draft'
+-
+  2.3.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  2.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer1}
+-
+  2.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer1}:
+    - state == 'open'
+-
+  2.6.- I create a customer refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer1')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  2.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer1'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  3.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method 
+-  
+  3.1.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer2, view: account.invoice_form }:
+    <<: *customer_invoice
+-
+  3.2.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2}:
+    - state == 'draft'
+-
+  3.3.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  3.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer2}
+-
+  3.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer2}:
+    - state == 'open'
+-
+  3.6.- I create a customer refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer2')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  3.7.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer2'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  3.8.- I check that the new customer invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    customer_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert customer_inv_brw.state == 'draft', 'The new customer invoice was not properly created, the state must be Draft.'
+-
+  4.- In order to test account refund with "Create a draft refund" method in a "Paid" invoice
+-
+  4.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner8}:
+    name: Juan Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  4.2.- I create a Sale Journal.
+-
+  !record {model: account.journal, id: sale_journal_debit}:
+    code: sjou_test
+    company_id: base.main_company
+    name: Sale Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_sale_journal
+    type: sale
+-
+  4.3.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer3}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner8
+-
+  4.4.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'draft'
+-
+  4.5.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  4.6.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer3}
+-
+  4.7.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'open'
+-
+  4.8.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer3"))
+    assert(acc_id.move_id)
+-
+  4.9.- I create a Cash Journal.
+-
+  !record {model: account.journal, id: cash_journal_debit}:
+    code: cjou_test
+    company_id: base.main_company
+    name: Cash Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_cash_journal
+    type: cash
+-
+  4.10.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Juan Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer3')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner8"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner8'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  4.11.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer3}:
+    - state == 'paid'
+-
+  4.12.- I modified the sale journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: sale_journal_debit}:
+    update_posted: 1
+-
+  4.13.- I modified the cash journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: cash_journal_debit}:
+    update_posted: 1
+-
+  4.14.- I create a Sale Refund Journal.
+-
+  !record {model: account.journal, id: sale_refund_journal_debit}:
+    code: csrjou_test
+    company_id: base.main_company
+    name: Sale Refund Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_refund_sales_journal
+    type: sale_refund
+-
+  4.15.- I create a customer refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer3')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'draft', 'The customer refund was not properly created, the state must be Draft.'
+-
+  4.16.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer3'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  5.- In order to test account refund with "Create refund and reconcile" method in a "Paid" invoice
+-
+  5.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner7}:
+    name: Pedro Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  5.2.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer4}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner7
+-
+  5.3.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'draft'
+-
+  5.4.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  5.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer4}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'open'
+-
+  5.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer4"))
+    assert(acc_id.move_id)
+-
+  5.8.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Pedro Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer4')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner7"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner7'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  5.9.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer4}:
+    - state == 'paid'
+-
+  5.10.- I create a customer refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer4')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  5.11.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer4'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  6.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method in a "Paid" invoice
+-
+  6.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner9}:
+    name: Maria Perez
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+-
+  6.2.- I create a new customer invoice
+-
+  !record {model: account.invoice, id: account_invoice_customer5}:
+    account_id: account.a_recv
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_sale
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 450.0
+        quantity: 1.0
+        product_id: product.product_product_3
+        uos_id: product.product_uom_unit
+    journal_id: sale_journal_debit
+    partner_id: res_partner9
+-
+  6.3.- I check that Initially customer invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'draft'
+-
+  6.4.- I check that the customer invoice is a "Customer Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5, string: This is not a customer invoice}:
+    - type == 'out_invoice'
+-
+  6.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_customer5}
+-
+  6.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'open'
+-
+  6.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_customer5"))
+    assert(acc_id.move_id)
+-
+  6.8.- I will create and post an account voucher with the amount total 
+  of the customer invoice created for the partner Maria Perez.
+-
+  !python {model: account.voucher}: |
+    import netsvc
+    import time
+    vals = {}
+    amount_total = self.pool.get('account.invoice').browse(cr, uid, [ref('account_invoice_customer5')], context=None)[0].amount_total
+    journal_id = self.default_get(cr, uid, ['journal_id']).get('journal_id',None)
+    res = self.onchange_partner_id(cr, uid, [], ref("res_partner9"), journal_id, 0.0, 1, ttype='receipt', date=False)
+    vals = {
+        'account_id': ref('account.cash'),
+        'amount': amount_total,
+        'company_id': ref('base.main_company'),
+        'journal_id': ref('cash_journal_debit'),
+        'partner_id': ref('res_partner9'),
+        'period_id': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'type': 'receipt',
+    }
+    if not res['value']['line_cr_ids']:
+      res['value']['line_cr_ids'] = [{'type': 'cr', 'account_id': ref('account.a_recv'),}]
+    res['value']['line_cr_ids'][0]['amount'] = amount_total
+    vals['line_cr_ids'] = [(0,0,i) for i in res['value']['line_cr_ids']]
+    id = self.create(cr, uid, vals)
+    voucher_id = self.browse(cr, uid, id)
+    assert (voucher_id.state=='draft'), "Voucher is not in draft state"
+    wf_service = netsvc.LocalService("workflow")
+    wf_service.trg_validate(uid, 'account.voucher', voucher_id.id, 'proforma_voucher', cr)
+-
+  6.9.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_customer5}:
+    - state == 'paid'
+-
+  6.10.- I create a customer refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('sale_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_customer5')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_customer_inv_brw[0].state == 'paid', 'The customer refund was not properly created, the state must be Paid.'
+-
+  6.11.- I check the correct asigning of parent_id in the customer refund created, for that
+  the customer refund must have the customer invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_customer_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    customer_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_customer5'))    
+    assert (rfnd_customer_inv_brw[0].parent_id.id == customer_inv_brw.id), 'The customer refund dont have the correct parent_id.'
+-
+  6.12.- I check that the new customer invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    customer_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert customer_inv_brw.state == 'draft', 'The new customer invoice was not properly created, the state must be Draft.'
 
=== added file 'debit_credit_note/test/account_supplier_refund.yml'
--- debit_credit_note/test/account_supplier_refund.yml	1970-01-01 00:00:00 +0000
+++ debit_credit_note/test/account_supplier_refund.yml	2014-02-03 16:27:19 +0000
@@ -0,0 +1,616 @@
+-
+  I set the context that will be used for the encoding of all the invoices of this file
+-
+  !context
+    'type': 'in_invoice'
+-
+  1.- In order to test account refund with "Create a draft refund" method
+-
+  1.1.- I create a supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier0, view: account.invoice_supplier_form }: &supplier_invoice
+    account_id: account.a_pay
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_expense
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 300.0
+        product_id: product.product_product_3
+        quantity: 10.0
+        uos_id: product.product_uom_unit
+    journal_id: account.expenses_journal
+    partner_id: base.res_partner_12
+    reference_type: none
+    type: 'in_invoice'
+-
+  1.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0}:
+    - state == 'draft'
+-
+  1.3.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  1.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier0}
+-
+  1.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier0}:
+    - state == 'open'
+-
+  1.6.- I create a supplier refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier0')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'draft', 'The supplier refund was not properly created, the state must be Draft.'
+-
+  1.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier0'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  2.- In order to test account refund with "Create refund and reconcile" method
+-
+  2.1.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier1, view: account.invoice_supplier_form }:
+    <<: *supplier_invoice
+-
+  2.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1}:
+    - state == 'draft'
+-
+  2.3.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  2.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier1}
+-
+  2.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier1}:
+    - state == 'open'
+-
+  2.6.- I create a supplier refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier1')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  2.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier1'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  3.- In order to test account refund with "Create refund, reconcile and create a new draft invoice"
+  method
+-
+  3.1.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier2, view: account.invoice_supplier_form }:
+    <<: *supplier_invoice
+-
+  3.2.- I check that Initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2}:
+    - state == 'draft'
+-
+  3.3.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  3.4.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier2}
+-
+  3.5.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier2}:
+    - state == 'open'
+-
+  3.6.- I create a supplier refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('account.refund_expenses_journal'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier2')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])    
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  3.7.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier2'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  3.8.- I check that the new supplier invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    supplier_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert supplier_inv_brw.state == 'draft', 'The new supplier invoice was not properly created, the state must be Draft.'
+-
+  4.- In order to test account refund with "Create a draft refund" method in a "Paid" invoice
+-
+  4.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner10}:
+    name: Juan Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  4.2.- I create a Purchase Journal.
+-
+  !record {model: account.journal, id: purchase_journal_debit}:
+    code: pjou_test
+    company_id: base.main_company
+    name: Purchase Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_purchase_journal
+    type: purchase
+-
+  4.3.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier3, view: account.invoice_supplier_form }:
+    account_id: account.a_pay
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_expense
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 300.0
+        product_id: product.product_product_3
+        quantity: 10.0
+        uos_id: product.product_uom_unit
+    journal_id: purchase_journal_debit
+    partner_id: res_partner10
+    reference_type: none
+    type: 'in_invoice'
+-
+  4.4.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'draft'
+-
+  4.5.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  4.6.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier3}
+-
+  4.7.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'open'
+-
+  4.8.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier3"))
+    assert(acc_id.move_id)
+-
+  4.9.- I create a Bank Journal.
+-
+  !record {model: account.journal, id: bank_journal_debit}:
+    name: Bank Journal For Debit Credit Note Tests
+    code: bjou_test
+    type: bank
+    analytic_journal_id: account.sit
+    sequence_id: account.sequence_bank_journal
+    default_debit_account_id: account.cash
+    default_credit_account_id: account.cash
+    company_id: base.main_company
+-
+  4.10.- I will create and post an account voucher of amount 3450.00 for the partner Juan Garcia.
+-
+  !record {model: account.voucher, id: account_voucher0, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner10
+    reference: none
+-
+  4.11.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher0}:
+    - state == 'draft'
+-
+  4.12.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher0}
+-
+  4.13.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher0}:
+    - state == 'posted'
+-
+  4.14.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier3}:
+    - state == 'paid'
+-
+  4.15.- I modified the sale journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: purchase_journal_debit}:
+    update_posted: 1
+-
+  4.16.- I modified the cash journal record so it can cancel invoice
+-
+  !record {model: account.journal, id: bank_journal_debit}:
+    update_posted: 1
+-
+  4.17.- I create a Purchase Refund Journal.
+-
+  !record {model: account.journal, id: purchase_refund_journal_debit}:
+    code: sprjou_test
+    company_id: base.main_company
+    name: Purchase Refund Journal For Debit Credit Note Tests
+    sequence_id: account.sequence_refund_purchase_journal
+    type: purchase_refund
+-
+  4.18.- I create a supplier refund using the method "Create a draft refund" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'refund',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier3')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'draft', 'The supplier refund was not properly created, the state must be Draft.'
+-
+  4.19.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier3'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  5.- In order to test account refund with "Create refund and reconcile" method in a "Paid" invoice
+-
+  5.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner11}:
+    name: Maria Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  5.2.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier4, view: account.invoice_supplier_form }:
+    account_id: account.a_pay
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_expense
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 300.0
+        product_id: product.product_product_3
+        quantity: 10.0
+        uos_id: product.product_uom_unit
+    journal_id: purchase_journal_debit
+    partner_id: res_partner11
+    reference_type: none
+    type: 'in_invoice'
+-
+  5.3.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'draft'
+-
+  5.4.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  5.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier4}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'open'
+-
+  5.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier4"))
+    assert(acc_id.move_id)
+-
+  5.8.- I will create and post an account voucher of amount 3450.00 for the partner Maria Garcia.
+-
+  !record {model: account.voucher, id: account_voucher1, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner11
+    reference: none
+-
+  5.9.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher1}:
+    - state == 'draft'
+-
+  5.10.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher1}
+-
+  5.11.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher1}:
+    - state == 'posted'
+-
+  5.12.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier4}:
+    - state == 'paid'
+-
+  5.13.- I create a supplier refund using the method "Create refund and reconcile" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'cancel',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier4')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  5.14.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier4'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  6.- In order to test account refund with "Create refund, reconcile and create a new draft invoice" 
+  method in a "Paid" invoice
+-
+  6.1.- I create a new partner
+-
+  !record {model: 'res.partner', id: res_partner12}:
+    name: Pablo Garcia
+    use_parent_address: True
+    function: Tests
+    email: [e-mail]
+    supplier: True
+-
+  6.2.- I create a new supplier invoice
+-
+  !record {model: account.invoice, id: account_invoice_supplier5, view: account.invoice_supplier_form }:
+    account_id: account.a_pay
+    company_id: base.main_company
+    currency_id: base.EUR
+    invoice_line:
+      - account_id: account.a_expense
+        name: '[PCSC234] PC Assemble SC234'
+        price_unit: 300.0
+        product_id: product.product_product_3
+        quantity: 10.0
+        uos_id: product.product_uom_unit
+    journal_id: purchase_journal_debit
+    partner_id: res_partner12
+    reference_type: none
+    type: 'in_invoice'
+-
+  6.3.- I check that initially supplier invoice state is "Draft"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'draft'
+-
+  6.4.- I check that the supplier invoice is a "Supplier Invoice"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5, string: This is not a supplier invoice}:
+    - type == 'in_invoice'
+-
+  6.5.- I change the state of invoice to open by clicking Validate button
+-
+  !workflow {model: account.invoice, action: invoice_open, ref: account_invoice_supplier5}
+-
+  5.6.- I check that the invoice state is now "Open"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'open'
+-
+  6.7.- I check that a payment entry gets created in the account.move.line
+-
+  !python {model: account.invoice}: |
+    acc_id=self.browse(cr, uid, ref("account_invoice_supplier5"))
+    assert(acc_id.move_id)
+-
+  6.8.- I will create and post an account voucher of amount 3450.00 for the partner Maria Garcia.
+-
+  !record {model: account.voucher, id: account_voucher2, view: account_voucher.view_vendor_payment_form}:
+    type: payment
+    account_id: account.cash
+    amount: 3450.00
+    company_id: base.main_company
+    journal_id: bank_journal_debit
+    name: Voucher x
+    narration: PC Assemble SC234
+    line_dr_ids:
+      - account_id: account.cash
+        amount: 3450.00
+        name: Voucher x
+    partner_id: res_partner12
+    reference: none
+-
+  6.9.- I check that initially vendor voucher is in the "Draft" state
+-
+  !assert {model: account.voucher, id: account_voucher2}:
+    - state == 'draft'
+-
+  6.10.- I create voucher by clicking on create button
+-
+  !workflow {model: account.voucher, action: proforma_voucher, ref: account_voucher2}
+-
+  6.11.- I check that the voucher state is "Posted"
+-
+  !assert {model: account.voucher, id: account_voucher2}:
+    - state == 'posted'
+-
+  6.12.- I check if the state of the invoice is "Paid"
+-
+  !assert {model: account.invoice, id: account_invoice_supplier5}:
+    - state == 'paid'
+-
+  6.13.- I create a supplier refund using the method "Create refund, reconcile and create a new draft invoice" and 
+  check the state of the account invoice refund resulting
+-
+  !python {model: account.invoice.refund}: |
+    import time
+    values = {
+        'date': time.strftime('%Y-%m-%d'),
+        'period': self.pool.get('account.period').find(
+            cr, uid, time.strftime('%Y/%m/%d'), {'company_id': ref('base.main_company')})[0],
+        'journal_id': ref('purchase_refund_journal_debit'),
+        'description': 'Desde el test',
+        'filter_refund': 'modify',
+    }
+    wzr_id = self.create(cr, uid, values)
+    assert wzr_id, 'The wizard instance was not created.'
+    context['wzr_id'] = wzr_id
+    context['active_ids'] = [ref('account_invoice_supplier5')]
+    rfnd_id = self.invoice_refund(cr, uid, [wzr_id], context=context)
+    assert rfnd_id, 'The refund was not created.'
+    context['rfnd_id'] = rfnd_id
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, rfnd_id['domain'][1][2])
+    assert rfnd_supplier_inv_brw[0].state == 'paid', 'The supplier refund was not properly created, the state must be Paid.'
+-
+  6.14.- I check the correct asigning of parent_id in the supplier refund created, for that
+  the supplier refund must have the supplier invoice id in the fiel parent_id
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    rfnd_supplier_inv_brw = obj_inv.browse(cr, uid, context['rfnd_id']['domain'][1][2])
+    supplier_inv_brw = obj_inv.browse(cr, uid, ref('account_invoice_supplier5'))    
+    assert (rfnd_supplier_inv_brw[0].parent_id.id == supplier_inv_brw.id), 'The supplier refund dont have the correct parent_id.'
+-
+  6.15.- I check that the new supplier invoice was created in Draft state
+-
+  !python {model: account.invoice.refund}: |
+    obj_inv = self.pool.get('account.invoice')
+    inv_src = obj_inv.search(cr, uid, [], order='id')
+    supplier_inv_brw = obj_inv.browse(cr, uid, inv_src[-1])
+    assert supplier_inv_brw.state == 'draft', 'The new supplier invoice was not properly created, the state must be Draft.'
 
=== modified file 'debit_credit_note/wizard/account_invoice_refund.py'
--- debit_credit_note/wizard/account_invoice_refund.py	2014-01-17 21:53:54 +0000
+++ debit_credit_note/wizard/account_invoice_refund.py	2014-02-03 16:27:19 +0000
@@ -92,24 +92,21 @@
             period_id = ids[0]
         return period_id
 
-    def _get_orig(self, cr, uid, inv, context={}):
-        """
-        Return  default origin value
-        """
-        nro_ref = ''
+    def _get_orig(self, cr, uid, inv, ref, context={}):
+        """ Return  default origin value
+        """
+        nro_ref = ref
         if inv.type == 'out_invoice':
             nro_ref = inv.number
-        orig = _('INV REFUND:') + (nro_ref or '') + _('- DATE:') + (
-            inv.date_invoice or '') + (' TOTAL:' + str(inv.amount_total) or '')
+        orig = 'Devolucion FACT:' +(nro_ref or '') + '- DE FECHA:' + (inv.date_invoice or '') + (' TOTAL:' + str(inv.amount_total) or '')
         return orig
-
+    
     def compute_refund(self, cr, uid, ids, mode='refund', context=None):
-        """
-        @param cr: the current row, from the database cursor,
-        @param uid: the current user’s ID for security checks,
+        """ 
         @param ids: the account invoice refund’s ID or list of IDs
-
         """
+        wzd_brw = self.browse(cr,uid,ids[0],context=context)
+        brw = self.browse(cr,uid,ids[0],context=context)
         inv_obj = self.pool.get('account.invoice')
         reconcile_obj = self.pool.get('account.move.reconcile')
         account_m_line_obj = self.pool.get('account.move.line')
@@ -121,70 +118,66 @@
         res_users_obj = self.pool.get('res.users')
         if context is None:
             context = {}
-
         for form in self.browse(cr, uid, ids, context=context):
             created_inv = []
             date = False
             period = False
             description = False
-            company = res_users_obj.browse(
-                cr, uid, uid, context=context).company_id
-            journal_id = form.journal_id.id
-            for inv in inv_obj.browse(cr, uid, context.get('active_ids'),
-                                      context=context):
+            company = res_users_obj.browse(cr, uid, uid, context=context).company_id
+            journal_brw = form.journal_id or False
+            for inv in inv_obj.browse(cr, uid, context.get('active_ids'), context=context):
                 if inv.state in ['draft', 'proforma2', 'cancel']:
-                    raise osv.except_osv(_('Error!'), _(
-                        'Cannot %s draft/proforma/cancel invoice.') % (mode))
+                    raise osv.except_osv(_('Error !'), _('Can not %s draft/proforma/cancel invoice.') % (mode))
                 if inv.reconciled and mode in ('cancel', 'modify'):
-                    raise osv.except_osv(_('Error!'), _(
-                        'Cannot %s invoice which is already reconciled, '
-                        'invoice should be unreconciled first. You can only '
-                        'refund this invoice.') % (mode))
-                if form.period.id:
-                    period = form.period.id
-                else:
-                    period = inv.period_id and inv.period_id.id or False
+                    raise osv.except_osv(_('Error !'), _('Can not %s invoice which is already reconciled, invoice should be unreconciled first. You can only Refund this invoice') % (mode))
+                period = form.period.id or False
+                if not period:
+	                #Take period from the current date
+                    period = self.pool.get('account.period').find(cr, uid, context=context)
+                    period = period and period[0] or False
+                    if not period:
+                        raise osv.except_osv(_('No Period Defined'), \
+                                _('You have been left empty the period field that automatically fill with the current period. However there is not period defined for the current company. Please check in Accounting/Configuration/Periods'))
+                    self.write(cr, uid, ids, {'period': period }, context=context)
 
-                if not journal_id:
+                if not journal_brw:
                     journal_id = inv.journal_id.id
+                else:
+                    journal_id=journal_brw.id
 
                 if form.date:
                     date = form.date
-                    if not form.period.id:
-                        cr.execute("select name from ir_model_fields \
-                                        where model = 'account.period' \
-                                        and name = 'company_id'")
-                        result_query = cr.fetchone()
-                        if result_query:
-                            cr.execute("""select p.id from account_fiscalyear y
-                                            , account_period p
-                                            where y.id=p.fiscalyear_id \
-                                and date(%s) between p.date_start AND
-                                p.date_stop and y.company_id = %s limit 1""",
-                                      (date, company.id,))
-                        else:
-                            cr.execute("""SELECT id
-                                    from account_period where date(%s)
-                                    between date_start AND  date_stop  \
-                                    limit 1 """, (date,))
-                        res = cr.fetchone()
-                        if res:
-                            period = res[0]
+                    if not form.period:
+                            cr.execute("select name from ir_model_fields \
+                                            where model = 'account.period' \
+                                            and name = 'company_id'")
+                            result_query = cr.fetchone()
+                            if result_query:
+                                cr.execute("""select p.id from account_fiscalyear y, account_period p where y.id=p.fiscalyear_id \
+                                    and date(%s) between p.date_start AND p.date_stop and y.company_id = %s limit 1""", (date, company.id,))
+                            else:
+                                cr.execute("""SELECT id
+                                        from account_period where date(%s)
+                                        between date_start AND  date_stop  \
+                                        limit 1 """, (date,))
+                            res = cr.fetchone()
+                            if res:
+                                period = res[0]
                 else:
-                    date = inv.date_invoice
+                    #Take current date
+                    #date = inv.date_invoice
+                    date = time.strftime('%Y-%m-%d')
                 if form.description:
                     description = form.description
                 else:
                     description = inv.name
 
                 if not period:
-                    raise osv.except_osv(_('Insufficient Data!'),
-                                         _('No period found on the invoice.'))
-
-                refund_id = inv_obj.refund(cr, uid, [
-                                           inv.id], date, period,
-                                           description, journal_id,
-                                           context=context)
+                    raise osv.except_osv(_('Data Insufficient !'), \
+                                            _('No Period found on Invoice!'))
+
+                refund_id = inv_obj.refund(cr, uid, [inv.id], date, period, description, journal_id)
+
                 refund = inv_obj.browse(cr, uid, refund_id[0], context=context)
                 # Add parent invoice
                 inv_obj.write(cr, uid, [refund.id],
@@ -202,43 +195,37 @@
                             to_reconcile_ids[line.account_id.id] = [line.id]
                         if type(line.reconcile_id) != osv.orm.browse_null:
                             reconcile_obj.unlink(cr, uid, line.reconcile_id.id)
-                    wf_service.trg_validate(uid, 'account.invoice',
-                                            refund.id, 'invoice_open', cr)
-                    refund = inv_obj.browse(
-                        cr, uid, refund_id[0], context=context)
-                    for tmpline in refund.move_id.line_id:
+                    wf_service.trg_validate(uid, 'account.invoice', \
+                                        refund.id, 'invoice_open', cr)
+                    
+                    refund = inv_obj.browse(cr, uid, refund_id[0], context=context)
+
+                    for tmpline in  refund.move_id.line_id:
                         if tmpline.account_id.id == inv.account_id.id:
-                            to_reconcile_ids[
-                                tmpline.account_id.id].append(tmpline.id)
+                            to_reconcile_ids[tmpline.account_id.id].append(tmpline.id)
                     for account in to_reconcile_ids:
-                        account_m_line_obj.reconcile(
-                            cr, uid, to_reconcile_ids[account],
-                            writeoff_period_id=period,
-                            writeoff_journal_id=inv.journal_id.id,
-                            writeoff_acc_id=inv.account_id.id
-                        )
+                        account_m_line_obj.reconcile(cr, uid, to_reconcile_ids[account],
+                                        writeoff_period_id=period,
+                                        writeoff_journal_id = inv.journal_id.id,
+                                        writeoff_acc_id=inv.account_id.id
+                                        )
                     if mode == 'modify':
                         invoice = inv_obj.read(cr, uid, [inv.id],
-                                               ['name', 'type', 'number',
-                                                'reference', 'comment',
-                                                'date_due', 'partner_id',
-                                                'partner_insite',
-                                                'partner_contact',
-                                                'partner_ref', 'payment_term',
-                                                'account_id', 'currency_id',
-                                                'invoice_line', 'tax_line',
-                                                'journal_id', 'period_id'],
-                                               context=context)
+                                    ['name', 'type', 'number', 'supplier_invoice_number',
+                                    'comment', 'date_due', 'partner_id',
+                                    'partner_insite', 'partner_contact',
+                                    'partner_ref', 'payment_term', 'account_id',
+                                    'currency_id', 'invoice_line', 'tax_line',
+                                    'journal_id', 'period_id'], context=context)
                         invoice = invoice[0]
                         del invoice['id']
-                        invoice_lines = inv_line_obj.browse(
-                            cr, uid, invoice['invoice_line'], context=context)
-                        invoice_lines = inv_obj._refund_cleanup_lines(
-                            cr, uid, invoice_lines, context=context)
-                        tax_lines = inv_tax_obj.browse(
-                            cr, uid, invoice['tax_line'], context=context)
-                        tax_lines = inv_obj._refund_cleanup_lines(
-                            cr, uid, tax_lines, context=context)
+                        invoice_lines = inv_line_obj.browse(cr, uid, invoice['invoice_line'], context=context)
+
+                        invoice_lines = inv_obj._refund_cleanup_lines(cr, uid, invoice_lines)
+                        tax_lines = inv_tax_obj.browse(cr, uid, invoice['tax_line'], context=context)
+                        tax_lines = inv_obj._refund_cleanup_lines(cr, uid, tax_lines)
+                        #Add origin value
+                        orig = self._get_orig(cr, uid, inv, invoice['supplier_invoice_number'], context)
                         invoice.update({
                             'type': inv.type,
                             'date_invoice': date,
@@ -248,36 +235,99 @@
                             'tax_line': tax_lines,
                             'period_id': period,
                             'name': description,
-                            'origin': self._get_orig(cr, uid, inv, context={}),
+                            'origin': orig,
                         })
-                        for field in (
-                            'partner_id', 'account_id', 'currency_id',
-                                'payment_term', 'journal_id'):
-                                invoice[field] = invoice[
-                                    field] and invoice[field][0]
+                        for field in ( 'partner_id',
+                                'account_id', 'currency_id', 'payment_term', 'journal_id'):
+                                invoice[field] = invoice[field] and invoice[field][0]
                         inv_id = inv_obj.create(cr, uid, invoice, {})
                         if inv.payment_term.id:
-                            data = inv_obj.onchange_payment_term_date_invoice(
-                                cr, uid, [inv_id], inv.payment_term.id, date)
+                            data = inv_obj.onchange_payment_term_date_invoice(cr, uid, [inv_id], inv.payment_term.id, date)
                             if 'value' in data and data['value']:
                                 inv_obj.write(cr, uid, [inv_id], data['value'])
                         created_inv.append(inv_id)
-            xml_id = (inv.type == 'out_refund') and 'action_invoice_tree1' or \
-                     (inv.type == 'in_refund') and 'action_invoice_tree2' or \
-                     (inv.type == 'out_invoice') and 'action_invoice_tree3' or \
-                     (inv.type == 'in_invoice') and 'action_invoice_tree4'
+
+                        new_inv_brw = inv_obj.browse(cr,uid,created_inv[1],context=context)
+                        inv_obj.write(cr,uid,created_inv[0],{'name':wzd_brw.description,'origin':new_inv_brw.origin},context=context)
+                        inv_obj.write(cr,uid,created_inv[1],{'origin':inv.origin,'name':wzd_brw.description},context=context)
+            if inv.type in ('out_invoice', 'out_refund'):
+                xml_id = 'action_invoice_tree3'
+            else:
+                xml_id = 'action_invoice_tree4'
             result = mod_obj.get_object_reference(cr, uid, 'account', xml_id)
             id = result and result[1] or False
             result = act_obj.read(cr, uid, id, context=context)
             invoice_domain = eval(result['domain'])
             invoice_domain.append(('id', 'in', created_inv))
             result['domain'] = invoice_domain
+            
+            if wzd_brw.filter_refund == 'cancel':
+                orig = self._get_orig(cr, uid, inv, inv.supplier_invoice_number, context)
+                inv_obj.write(cr,uid,created_inv[0],{'origin':orig,'name':wzd_brw.description},context=context)
+            
+            if wzd_brw.filter_refund == 'refund':
+                orig = self._get_orig(cr, uid, inv, inv.supplier_invoice_number, context)
+                inv_obj.write(cr,uid,created_inv[0],{'origin':inv.origin,'name':wzd_brw.description},context=context)
             return result
+            
+    def validate_total_payment_inv(self, cr, uid, ids, context=None):
+        """ Method that validate if invoice is totally paid.
+        @param ids: list of invoices.
+        return: True: if invoice is paid.
+                False: if invoice is not paid.
+        """
+        res = False
+        inv_obj = self.pool.get('account.invoice')
+        for inv in inv_obj.browse(cr, uid, ids, context=context):
+            res = inv.reconciled
+        return res
+
+    def unreconcile_paid_invoices(self, cr, uid, invoiceids, context=None):
+        """ Method that unreconcile the payments of invoice.
+        @param invoiceids: list of invoices.
+        return: True: unreconcile successfully.
+                False: unreconcile unsuccessfully.
+        """
+        inv_obj = self.pool.get('account.invoice')
+        moveline_obj = self.pool.get('account.move.line')
+        voucher_pool = self.pool.get('account.voucher')
+        res = True
+        rec = []
+        mid = []
+        if self.validate_total_payment_inv(cr, uid, invoiceids, context=context):
+            for inv in inv_obj.browse(cr, uid, invoiceids, context=context):
+                movelineids = inv_obj.move_line_id_payment_get(cr, uid,[inv.id])
+                for moveline in moveline_obj.browse(cr, uid, movelineids,context=context):
+                    if moveline.reconcile_id:
+                        rec += [moveline.reconcile_id.id]
+                    if moveline.reconcile_partial_id:
+                        rec += [moveline.reconcile_partial_id.id]
+                movelines = moveline_obj.search(cr, uid, [('|'),('reconcile_id','in',rec),('reconcile_partial_id','in',rec)],context=context)
+                for mids in moveline_obj.browse(cr, uid, movelines, context=context):
+                    mid +=[mids.move_id.id]
+                voucherids = voucher_pool.search(cr, uid,[('move_id','in',mid)])
+            if voucherids:
+                voucher_pool.cancel_voucher(cr, uid, voucherids, context=context)
+            else:
+                res = False
+        return res
 
     def invoice_refund(self, cr, uid, ids, context=None):
-        data_refund = self.read(cr, uid, ids, [
-                                'filter_refund'],
-                                context=context)[0]['filter_refund']
+        """ Create a invoice refund
+        """
+        if context is None:
+            context = {}
+        inv_obj = self.pool.get('account.invoice')
+        period_obj = self.pool.get('account.period')
+        wzr_brw = self.browse(cr,uid,ids,context=context)[0]
+        date = wzr_brw.date
+        period = wzr_brw and wzr_brw.period and wzr_brw.period.id 
+        period_ids = date and period_obj.search(cr,uid,[('date_start', '<=', date),('date_stop', '>=', date),('special', '=', False)],context=context)
+        if period not in period_ids:
+            raise osv.except_osv(_('Error !'), \
+                                     _('The date should be chosen to belong to the period'))    
+        self.unreconcile_paid_invoices(cr, uid, context.get('active_ids'), context=context)
+        data_refund = self.browse(cr, uid, ids, context=context)[0].filter_refund
         return self.compute_refund(cr, uid, ids, data_refund, context=context)