feat: remove the broken Zooming Image Tool

The Zooming Image Tool does not work, and the security implications of getting such external Javascript to function properly are unclear at best. We should simply remove it from the list of HTML block templates.
openedx · Dec 12, 2022 · f0f50d7 · f0f50d7
1 parent c1ad89b
commit f0f50d7
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 86 deletions.
diff --git a/cms/djangoapps/contentstore/views/tests/test_item.py b/cms/djangoapps/contentstore/views/tests/test_item.py
@@ -247,11 +247,11 @@ def test_split_test(self):
                                   boilerplate='announcement.yaml')
         self.assertEqual(resp.status_code, 200)
         resp = self.create_xblock(parent_usage_key=split_test_usage_key, category='html',
-                                  boilerplate='zooming_image.yaml')
+                                  boilerplate='latex_html.yaml')
         self.assertEqual(resp.status_code, 200)
         html, __ = self._get_container_preview(split_test_usage_key)
         self.assertIn('Announcement', html)
-        self.assertIn('Zooming', html)
+        self.assertIn('LaTeX', html)
 
     def test_split_test_edited(self):
         """

diff --git a/xmodule/templates/html/zooming_image.yaml b/xmodule/templates/html/zooming_image.yaml
diff --git a/xmodule/templates/test/zooming_image.yaml b/xmodule/templates/test/zooming_image.yaml
diff --git a/xmodule/tests/test_resource_templates.py b/xmodule/tests/test_resource_templates.py
@@ -18,7 +18,6 @@ class ResourceTemplatesTests(unittest.TestCase):
     def test_templates(self):
         expected = {
             'latex_html.yaml',
-            'zooming_image.yaml',
             'announcement.yaml',
             'anon_user_id.yaml'}
         got = {t['template_id'] for t in TestClass.templates()}
@@ -38,7 +37,6 @@ def test_get_custom_template(self):
     def test_custom_templates(self):
         expected = {
             'latex_html.yaml',
-            'zooming_image.yaml',
             'announcement.yaml',
             'anon_user_id.yaml'}
         got = {t['template_id'] for t in TestClassResourceTemplate.templates()}