← Back to team overview

openerp-expert-accounting team mailing list archive

[Bug 519133] Re: [5.0] The Centralisation check is required at write() as well.

 

Thank you for the patch
Actually, there are no extra bugs but the behaviour is still incorrect:

When we modify the date of an existing move line, all the move lines are
still updated with this date.

I think the correction should something like :

on line 789 :

                  if todo_date:
+                       if not journal.centralisation:
                        self.pool.get('account.move').write(cr, uid, [line.move_id.id], {'date': todo_date}, context=context)
+                        else
+                       self.pool.get('account.move').write(cr, uid, [line.move_id.id], {'date':???}, context=context)

-- 
[5.0] The Centralisation check is required at write() as well.
https://bugs.launchpad.net/bugs/519133
You received this bug notification because you are a member of OpenERP's
Accounting Experts, which is a direct subscriber.

Status in OpenObject Addons Modules: In Progress

Bug description:
Hello

When modifying an entry line in a centralized journal (journal.centralisation=true), all the entry line dates get modified and take the modified entry line value.

The entry line dates should not be changed

I guess the solution is in the write function in account_move_line.py where we should find some 'if journal.centralisation:...' as it is in the create function.

Regards