Merge pull request #628 from laslabs/bugfix/9.0/base_multi_image-unlink_bypass

[FIX][9.0] base_multi_image: Adhere to image delete bypass

base_multi_image/README.rst

@@ -121,6 +121,7 @@ Contributors
 * Rafael Blasco <rafabn@antiun.com>
 * Jairo Llopis <yajo.sk8@gmail.com>
 * Sodexis <dev@sodexis.com>
+* Dave Lasley <dave@laslabs.com>
 
 Maintainer
 ----------

base_multi_image/__openerp__.py

@@ -9,8 +9,9 @@
     "name": "Multiple images base",
     "summary": "Allow multiple images for database objects",
     "version": "9.0.1.1.0",
-    "author": "Serv. Tecnol. Avanzados - Pedro M. Baeza, "
+    "author": "Tecnativa, "
               "Antiun Ingeniería, S.L., Sodexis, "
+              "LasLabs, "
               "Odoo Community Association (OCA)",
     "license": "AGPL-3",
     "website": "http://www.antiun.com",

base_multi_image/models/owner.py

@@ -90,9 +90,12 @@ class Owner(models.AbstractModel):
 
     @api.multi
     def unlink(self):
-        """Mimic `ondelete="cascade"` for multi images."""
+        """Mimic `ondelete="cascade"` for multi images.
+
+        Will be skipped if ``env.context['bypass_image_removal']`` == True
+        """
         images = self.mapped("image_ids")
         result = super(Owner, self).unlink()
-        if result:
+        if result and not self.env.context.get('bypass_image_removal'):
             images.unlink()
         return result