storage_thumbnail: 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

@@ -29,9 +29,10 @@ def tearDownClass(cls):
         cls.loader.restore_registry()
         super().tearDownClass()
 
-    def _create_thumbnail(self):
+    def _create_thumbnail(self, **values):
         # create thumbnail
         vals = {"name": "TEST THUMB"}
+        vals.update(values)
         return self.env["storage.thumbnail"].create(vals)
 
     def _create_image(self, resize=False, **kw):
@@ -51,6 +52,17 @@ def test_thumbnail(self):
         thumb.unlink()
         self.assertTrue(file_id.to_delete)
 
+    def test_image_get_or_create_thumbnail():
+        image = self._create_image()
+        self.assertTrue(image.url)
+        self.assertEqual(2, len(image.thumbnail_ids))
+        thumb = image.thumb_small_id
+        thumb.url_key = "test"
+        existing_thumb = image.get_existing_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)