From 725eb3c53891e5b9e9086deda3793e182d1009b6 Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Mon, 10 Feb 2020 23:04:58 +1100
Subject: [PATCH] Do not duplicate images when copying a part

- Simply reference the existing image
---
 InvenTree/part/models.py | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/InvenTree/part/models.py b/InvenTree/part/models.py
index fb270f06f0..7d33a782de 100644
--- a/InvenTree/part/models.py
+++ b/InvenTree/part/models.py
@@ -136,18 +136,9 @@ def rename_part_image(instance, filename):
     """
 
     base = 'part_images'
+    fname = os.path.basename(filename)
 
-    if filename.count('.') > 0:
-        ext = filename.split('.')[-1]
-    else:
-        ext = ''
-
-    fn = 'part_{pk}_img'.format(pk=instance.pk)
-
-    if ext:
-        fn += '.' + ext
-
-    return os.path.join(base, fn)
+    return os.path.join(base, fname)
 
 
 def match_part_names(match, threshold=80, reverse=True, compare_length=False):
@@ -832,10 +823,8 @@ class Part(models.Model):
         # Copy the part image
         if kwargs.get('image', True):
             if other.image:
-                image_file = ContentFile(other.image.read())
-                image_file.name = rename_part_image(self, other.image.url)
-
-                self.image = image_file
+                # Reference the other image from this Part
+                self.image = other.image
 
         # Copy the BOM data
         if kwargs.get('bom', False):