Leave most logic to base module.

OCA · May 30, 2016 · fa8459e · fa8459e
1 parent 2034582
commit fa8459e
Showing 1 changed file with 7 additions and 23 deletions.
diff --git a/website_snippet_mass_mailing_partner/controllers/main.py b/website_snippet_mass_mailing_partner/controllers/main.py
@@ -10,27 +10,11 @@ class MassMailingPartner(MassMailController):
     @route()
     def subscribe(self, list_id, email, **post):
         """Handle name if provided."""
-        result = super(MassMailingPartner, self).subscribe(
-            list_id, email, **post)
+        Partner = request.env["res.partner"].sudo()
         name = post.get("name")
-        if name:
-            Contact = request.env["mail.mass_mailing.contact"].sudo()
-            Partner = request.env["res.partner"].sudo()
-            contacts = Contact.search([
-                ("email", "=ilike", email),
-                ("list_id", "=", int(list_id)),
-                ("opt_out", "=", False),
-            ])
-            partner = Partner.search(
-                [("email", "=ilike", email), ("name", "=ilike", name)],
-                limit=1)
-            if not partner:
-                partner = Partner.create({
-                    "name": name,
-                    "email": email,
-                })
-            partner.opt_out = True
-            contacts.write({
-                "partner_id": partner.id,
-            })
-        return result
+
+        # Update partner's name, to make it get updated in contact list later
+        Partner.search([("email", "=", email)], limit=1).write({"name": name})
+
+        return super(MassMailingPartner, self).subscribe(
+            list_id, email, **post)