Merge pull request #18500 from edx/arch/remove-course-structures-table

Remove course_structures app (part 2)

openedx/core/djangoapps/content/course_structures/migrations/0002_delete_coursestructure.py

@@ -0,0 +1,18 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.13 on 2018-07-02 19:43
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('course_structures', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.DeleteModel(
+            name='CourseStructure',
+        ),
+    ]

openedx/core/djangoapps/content/course_structures/models.py

@@ -12,72 +12,3 @@ from util.models import CompressedTextField
 
 
 logger = logging.getLogger(__name__)  # pylint: disable=invalid-name
-
-
-class CourseStructure(TimeStampedModel):
-    """
-    The CourseStructure model is an aggregated representation of the course content tree
-    """
-
-    class Meta(object):
-        app_label = 'course_structures'
-
-    course_id = CourseKeyField(max_length=255, db_index=True, unique=True, verbose_name='Course ID')
-
-    # Right now the only thing we do with the structure doc is store it and
-    # send it on request. If we need to store a more complex data model later,
-    # we can do so and build a migration. The only problem with a normalized
-    # data model for this is that it will likely involve hundreds of rows, and
-    # we'd have to be careful about caching.
-    structure_json = CompressedTextField(verbose_name='Structure JSON', blank=True, null=True)
-
-    # JSON mapping of discussion ids to usage keys for the corresponding discussion modules
-    discussion_id_map_json = CompressedTextField(verbose_name='Discussion ID Map JSON', blank=True, null=True)
-
-    @property
-    def structure(self):
-        """
-        Deserializes a course structure JSON object
-        """
-        if self.structure_json:
-            return json.loads(self.structure_json)
-        return None
-
-    @property
-    def ordered_blocks(self):
-        """
-        Return the blocks in the order with which they're seen in the courseware. Parents are ordered before children.
-        """
-        if self.structure:
-            ordered_blocks = OrderedDict()
-            self._traverse_tree(self.structure['root'], self.structure['blocks'], ordered_blocks)
-            return ordered_blocks
-
-    @property
-    def discussion_id_map(self):
-        """
-        Return a mapping of discussion ids to usage keys of the corresponding discussion modules.
-        """
-        if self.discussion_id_map_json:
-            result = json.loads(self.discussion_id_map_json)
-            for discussion_id in result:
-                # Usage key strings might not include the course run, so we add it back in with map_into_course
-                result[discussion_id] = UsageKey.from_string(result[discussion_id]).map_into_course(self.course_id)
-            return result
-        return None
-
-    def _traverse_tree(self, block, unordered_structure, ordered_blocks, parent=None):
-        """
-        Traverses the tree and fills in the ordered_blocks OrderedDict with the blocks in
-        the order that they appear in the course.
-        """
-        # find the dictionary entry for the current node
-        cur_block = unordered_structure[block]
-
-        if parent:
-            cur_block['parent'] = parent
-
-        ordered_blocks[block] = cur_block
-
-        for child_node in cur_block['children']:
-            self._traverse_tree(child_node, unordered_structure, ordered_blocks, parent=block)