Fix student data is per mentoring, not shared

mentoring/light_children.py

@@ -302,11 +302,12 @@ class LightChild(Plugin, LightChildrenMixin):
 
         student_id = self.xmodule_runtime.anonymous_student_id
         course_id = self.xmodule_runtime.course_id
+        url_name = "%s-%s" % (self.xblock_container.url_name, name)
 
         lightchild_data, created = LightChildModel.objects.get_or_create(
             student_id=student_id,
             course_id=course_id,
-            name=name,
+            name=url_name,
         )
         return lightchild_data
 

mentoring/mentoring.py

@@ -179,12 +179,19 @@ class MentoringBlock(XBlockWithLightChildren):
 
         xml_content = submissions['xml_content']
         try:
-            etree.parse(StringIO(xml_content))
+            xml = etree.parse(StringIO(xml_content))
+            root = xml.getroot()
+            self.url_name = root.attrib['url_name']
         except etree.XMLSyntaxError as e:
             response = {
                 'result': 'error',
                 'message': e.message
             }
+        except KeyError as e:
+            response = {
+                'result': 'error',
+                'message': 'mentoring "url_name" attribute is missing'
+            }
         else:
             response = {
                 'result': 'success',

mentoring/migrations/0005_auto__chg_field_lightchild_name.py

+# -*- coding: utf-8 -*-
+import datetime
+from south.db import db
+from south.v2 import SchemaMigration
+from django.db import models
+
+
+class Migration(SchemaMigration):
+
+    def forwards(self, orm):
+
+        # Changing field 'LightChild.name'
+        db.alter_column('mentoring_lightchild', 'name', self.gf('django.db.models.fields.CharField')(max_length=100))
+
+    def backwards(self, orm):
+
+        # Changing field 'LightChild.name'
+        db.alter_column('mentoring_lightchild', 'name', self.gf('django.db.models.fields.CharField')(max_length=50))
+
+    models = {
+        'mentoring.answer': {
+            'Meta': {'unique_together': "(('student_id', 'course_id', 'name'),)", 'object_name': 'Answer'},
+            'course_id': ('django.db.models.fields.CharField', [], {'max_length': '50', 'db_index': 'True'}),
+            'created_on': ('django.db.models.fields.DateTimeField', [], {'auto_now_add': 'True', 'blank': 'True'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'modified_on': ('django.db.models.fields.DateTimeField', [], {'auto_now': 'True', 'blank': 'True'}),
+            'name': ('django.db.models.fields.CharField', [], {'max_length': '50', 'db_index': 'True'}),
+            'student_id': ('django.db.models.fields.CharField', [], {'max_length': '32', 'db_index': 'True'}),
+            'student_input': ('django.db.models.fields.TextField', [], {'default': "''", 'blank': 'True'})
+        },
+        'mentoring.lightchild': {
+            'Meta': {'unique_together': "(('student_id', 'course_id', 'name'),)", 'object_name': 'LightChild'},
+            'course_id': ('django.db.models.fields.CharField', [], {'max_length': '50', 'db_index': 'True'}),
+            'created_on': ('django.db.models.fields.DateTimeField', [], {'auto_now_add': 'True', 'blank': 'True'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'modified_on': ('django.db.models.fields.DateTimeField', [], {'auto_now': 'True', 'blank': 'True'}),
+            'name': ('django.db.models.fields.CharField', [], {'max_length': '100', 'db_index': 'True'}),
+            'student_data': ('django.db.models.fields.TextField', [], {'default': "''", 'blank': 'True'}),
+            'student_id': ('django.db.models.fields.CharField', [], {'max_length': '32', 'db_index': 'True'})
+        }
+    }
+
+    complete_apps = ['mentoring']
\ No newline at end of file

mentoring/models.py

@@ -61,7 +61,7 @@ class LightChild(models.Model):
         app_label = 'mentoring'
         unique_together = (('student_id', 'course_id', 'name'),)
 
-    name = models.CharField(max_length=50, db_index=True)
+    name = models.CharField(max_length=100, db_index=True)
     student_id = models.CharField(max_length=32, db_index=True)
     course_id = models.CharField(max_length=50, db_index=True)
     student_data = models.TextField(blank=True, default='')