Fix crash on empty association due to kill (#1177)

superdesk · Aug 10, 2023 · 4847ea3 · 4847ea3
1 parent 3f26da3
commit 4847ea3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/newsroom/wire/search.py b/newsroom/wire/search.py
@@ -551,7 +551,7 @@ def permission_embeds_in_item(self, item, permitted_products):
         """
         disable_download = []
         for key, embed_item in item.get("associations", {}).items():
-            if key.startswith("editor_") and embed_item.get('type') in ['audio', 'video']:
+            if key.startswith("editor_") and embed_item and (embed_item.get('type', '')) in ['audio', 'video']:
                 # get the list of products that the embedded item matched in Superdesk
                 embed_products = [p.get('code') for p in
                                   ((item.get('associations') or {}).get(key) or {}).get('products', [])]

diff --git a/tests/test_download.py b/tests/test_download.py
@@ -221,7 +221,8 @@ def setup_embeds(client, app):
                 }
             },
             "products": [{"code": "888"}]
-        }
+        },
+        "editor_3": None
     }
     app.data.update('items', item['_id'], {'associations': associations, 'body_html':
                                            '<p>Par 1</p><!-- EMBED START Audio {id: \"editor_0\"} --><figure>'