Merge lp:~vauxoo/addons-vauxoo/7.0-HER-zero_aml-hbto into lp:addons-vauxoo/7.0

[hbto [Vauxoo] http://www.vauxoo.com]

Getting Rid of Journal Entry Lines in Zero Both debit and credit.

[Julio Serna-http://www.vauxoo.com (hearthebreaker)]

review: Approve

[Gabriela Quilarque (gabrielaquilarque97)]

Hi Humberto,

I did functional test, and still make the journal entries in Zero.

View the screenshot before the pay.
http://awesomescreenshot.com/07c376vo01

View the screenshot after the pay.
http://awesomescreenshot.com/0be376x6aa

This success for partial pay, and complete pay.

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

It seem only one of the move has been deleted there are still some missing.

Could please tell at which steps are these Entries being generated,

so that it could be easily pointed,

thanks a lot.

2014-07-23 11:16 GMT-04:30 Gabriela Quilarque (Vauxoo) <email address hidden>
:

> Hi Humberto,
>
> I did functional test, and still make the journal entries in Zero.
>
> View the screenshot before the pay.
> http://awesomescreenshot.com/07c376vo01
>
> View the screenshot after the pay.
> http://awesomescreenshot.com/0be376x6aa
>
>
> This success for partial pay, and complete pay.
> --
>
> https://code.launchpad.net/~vauxoo/addons-vauxoo/7.0-HER-zero_aml-hbto/+merge/227925
> You proposed lp:~vauxoo/addons-vauxoo/7.0-HER-zero_aml-hbto for merging.
>

[Gabriela Quilarque (gabrielaquilarque97)]

I just made a video, where I show the journal entries in Zero, when is partial pay and full pay.

http://youtu.be/ByfQi9XQVHQ

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

I have tested at my end and it seems to me it is working fine,

In runbot this MP has gone green,

https://docs.google.com/a/vauxoo.com/file/d/0B1w9gokcJL9hV2QyTkVJazJYRVE/edit?usp=drivesdk

I will wait for Gaby's comment to finally merge this branch onto addons-vauxoo/7.0

review: Approve

[Gabriela Quilarque (gabrielaquilarque97)]

I test again, and everything work perfect.

review: Approve

=== modified file 'hr_expense_replenishment/model/hr_expense.py'
--- hr_expense_replenishment/model/hr_expense.py	2014-05-14 00:27:08 +0000
+++ hr_expense_replenishment/model/hr_expense.py	2014-07-23 14:32:58 +0000
@@ -1016,6 +1016,23 @@
             'datas': datas
         }
 
+    def action_receipt_create(self, cr, uid, ids, context=None):
+        '''
+        main function that is called when trying to create the accounting entries related to an expense
+        then this super tries to get rid of the Journal Entry Lines in zero
+        '''
+        super(hr_expense_expense, self).action_receipt_create(cr, uid, ids, context=context)
+        aml_obj = self.pool.get('account.move.line')
+        res = []
+        for exp in self.browse(cr, uid, ids, context=context):
+            if not exp.account_move_id.journal_id.entry_posted:
+                for aml_brw in exp.account_move_id.line_id:
+                    if not aml_brw.debit and not aml_brw.credit:
+                        res.append(aml_brw.id)
+        if res:
+            aml_obj.unlink(cr, uid, res, context=context)
+        return True
+
 class account_voucher(osv.Model):
     _inherit = 'account.voucher'
     def create(self, cr, uid, vals, context=None):