openerp-dev-web team mailing list archive

["Ujjvala Collins \(OpenERP\)" <uco@xxxxxxxxxxx>]

Ujjvala Collins (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-bug-758699-uco into lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #758699 in OpenERP Addons: "delete project don´t delete analytic account"
  https://bugs.launchpad.net/openobject-addons/+bug/758699

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-758699-uco/+merge/57622

[FIX] project:
--------------------
* If a project is deleted its related analytic account should also be deleted. (lp:758699)
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-758699-uco/+merge/57622
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-758699-uco.

=== modified file 'project/project.py'
--- project/project.py	2011-04-07 11:41:45 +0000
+++ project/project.py	2011-04-14 05:37:31 +0000
@@ -118,9 +118,11 @@
         return result.keys()
 
     def unlink(self, cr, uid, ids, *args, **kwargs):
+        analytic_obj = self.pool.get('account.analytic.account')
         for proj in self.browse(cr, uid, ids):
             if proj.tasks:
                 raise osv.except_osv(_('Operation Not Permitted !'), _('You can not delete a project with tasks. I suggest you to deactivate it.'))
+            analytic_obj.unlink(cr, uid, [proj.analytic_account_id.id])
         return super(project, self).unlink(cr, uid, ids, *args, **kwargs)
 
     _columns = {