Merge pull request OCA#83 from dreispt/fix-reapply-sla

Fix bug on the Reapply of Contract SLAs. Closes OCA#93.

Author: dreispt

Diff for: project_sla/analytic_account.py

@@ -44,15 +44,16 @@ def _reapply_sla(self, cr, uid, ids, recalc_closed=False, context=None):
             ctrl_models = set([sla.control_model for sla in contract.sla_ids])
             for model_name in ctrl_models:
                 model = self.pool[model_name]
-                domain = [] if recalc_closed else [('stage_id.fold', '=', 1)]
+                base = [] if recalc_closed else [('stage_id.fold', '=', 0)]
+                doc_ids = []
                 if 'analytic_account_id' in model._columns:
-                    domain.append(
-                        ('analytic_account_id', '=', contract.id))
+                    domain = base + [
+                        ('analytic_account_id', '=', contract.id)]
+                    doc_ids += model.search(cr, uid, domain, context=context)
                 if 'project_id' in model._columns:
-                    domain.append(
-                        ('project_id.analytic_account_id', '=', contract.id))
-
-                doc_ids = model.search(cr, uid, domain, context=context)
+                    domain = base + [
+                        ('project_id.analytic_account_id', '=', contract.id)]
+                    doc_ids += model.search(cr, uid, domain, context=context)
                 if doc_ids:
                     model = self.pool[model_name]
                     docs = model.browse(cr, uid, doc_ids, context=context)

Diff for: project_sla/test/project_sla.yml

@@ -45,10 +45,10 @@
     self.write(cr, uid, [ref('project.project_project_1_account_analytic_account')],
         {'sla_ids': [(4, ref('sla_response'))]})
 -
-  Button to Reapply the SLA Definition
+  Button to Reapply the SLA Definition on the Contract
 -
-  !python {model: project.sla}: |
-    self._reapply_slas(cr, uid, [ref('sla_resolution')], recalc_closed=True)
+  !python {model: account.analytic.account}: |
+    self._reapply_sla(cr, uid, [ref('project.project_project_1_account_analytic_account')], recalc_closed=True)
 -
   There should be two Service Level lines generated on the Issue
 -