* assigning a name that shouldn't conflict

* avoiding changes to asset_version.content on save

Certain incorrectly deployed surveys (lacking names) might
have records that fail to export correctly, but we can
make adjustments in the future

kpi/models/asset_version.py

@@ -33,12 +33,9 @@ class AssetVersion(models.Model):
     class Meta:
         ordering = ['-date_modified']
 
-    def save(self, *args, **kwargs):
-        if self.deployed and self.deployed_content is None:
-            self.deployed_content = self._deployed_content()
-        super(AssetVersion, self).save(*args, **kwargs)
-
     def _deployed_content(self):
+        if self.deployed_content is not None:
+            return self.deployed_content
         legacy_names = self._reversion_version is not None
         if legacy_names:
             return to_xlsform_structure(self.version_content,
@@ -49,7 +46,7 @@ def _deployed_content(self):
 
     def to_formpack_schema(self):
         return {
-            'content': self.deployed_content or self._deployed_content(),
+            'content': self._deployed_content(),
             'version': self.uid,
             'version_id_key': '__version__',
         }

kpi/utils/autoname.py

@@ -10,7 +10,7 @@
 
 from kpi.utils.sluggify import (sluggify, sluggify_label, is_valid_nodeName)
 
-
+from formpack.utils.json_hash import json_hash
 
 
 
@@ -51,8 +51,7 @@ def autoname_fields__depr(surv_content):
             if 'label' in surv_row:
                 next_name = sluggify_valid_xml__depr(surv_row['label'])
             else:
-                raise ValueError('Label cannot be translated: %s' %
-                                 json.dumps(surv_row))
+                next_name = 'unnamable_row_{}'.format(json_hash(surv_row))
             while next_name in kuid_names.values():
                 next_name = _increment(next_name)
             if 'kuid' not in surv_row: