storage_thumbnail: fix, always use slugified url_key

Both for creating and searching thumbnails.

This avoid an infinite creation of thumbnails on images as soon as an
`url_key` is provided.

This is a regression introduced by commit 1976597

storage_thumbnail/models/thumbnail_mixin.py

@@ -101,6 +101,7 @@ def get_existing_thumbnail(self, size_x, size_y, url_key=None):
         return thumbnail
 
     def get_or_create_thumbnail(self, size_x, size_y, url_key=None):
+        url_key = self._get_url_key(url_key)
         thumbnail = self.get_existing_thumbnail(size_x, size_y, url_key=url_key)
         if not thumbnail and self.data:
             vals = self.env["storage.thumbnail"]._prepare_thumbnail(

storage_thumbnail/tests/test_thumbnail.py

@@ -51,6 +51,18 @@ def test_thumbnail(self):
         thumb.unlink()
         self.assertTrue(file_id.to_delete)
 
+    def test_image_get_or_create_thumbnail_no_duplicate(self):
+        # Ensure no duplicate is generated thanks to unique url_key
+        image = self._create_image()
+        self.assertTrue(image.url)
+        self.assertEqual(len(image.thumbnail_ids), 2)
+        thumb = image.thumb_small_id
+        thumb.url_key = "test"
+        existing_thumb = image.get_or_create_thumbnail(
+            thumb.size_x, thumb.size_y, url_key="TEST"
+        )
+        self.assertEqual(thumb, existing_thumb)
+
     def test_model(self):
         image = self._create_image()
         self.assertTrue(image.url)