Repurposed sfmoma oembed workaround for fluent bug into icekit for AC…

…MI ticket 46,

django-fluent/django-fluent-contents#65

icekit/apps.py

@@ -9,8 +9,9 @@
 from django.conf import settings
 from django.core.management.color import no_style
 from django.db import connection
-from django.db.models.signals import post_migrate
+from django.db.models.signals import post_migrate, pre_save
 from django.utils.module_loading import autodiscover_modules
+from fluent_contents.plugins.oembeditem.models import OEmbedItem
 
 
 def update_site(sender, **kwargs):
@@ -58,3 +59,25 @@ def ready(self):
 
         # Import plugins from installed apps.
         autodiscover_modules('plugins')
+
+
+def handle_soundcloud_malformed_widths_for_oembeds(sender, instance, **kwargs):
+    """
+    A signal receiver that prevents soundcloud from triggering ValueErrors
+    when we save an OEmbedItem.
+    Soundcloud returns "100%", instead of an integer for an OEmbed's width property.
+    This is against the OEmbed spec, but actually makes sense in the context of
+    responsive design. Unfortunately, the OEmbedItems can only store integers in
+    the `width` field, so this becomes a 500 throwing issue.
+    The issue is tracked https://github.com/edoburu/django-fluent-contents/issues/65
+    """
+    if instance.width == '100%':
+        instance.width = -100
+
+
+class OEmbedAppConfig(AppConfig):
+    name = 'fluent_contents.plugins.oembeditem'
+
+    def ready(self):
+        from fluent_contents.plugins.oembeditem.models import OEmbedItem
+pre_save.connect(handle_soundcloud_malformed_widths_for_oembeds, sender=OEmbedItem)

icekit/project/settings/_base.py

@@ -705,7 +705,7 @@
     'icekit.plugins.links',
     'icekit.plugins.map',
     'icekit.plugins.map_with_text',
-    'icekit.plugins.oembed_with_caption',
+    'icekit.apps.OEmbedAppConfig',
     'icekit.plugins.page_anchor',
     'icekit.plugins.page_anchor_list',
     'icekit.plugins.quote',