fixes #519; don't use IDT text as key for modifications if two types …

…of modifications have the same IDT text
UC-Davis-molecular-computing · Nov 20, 2020 · 59a2638 · 59a2638
1 parent c1180d1
commit 59a2638
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/src/state/modification.dart b/lib/src/state/modification.dart
@@ -65,7 +65,7 @@ abstract class Modification5Prime
     implements Built<Modification5Prime, Modification5PrimeBuilder>, Modification {
   factory Modification5Prime(
       {String display_text, String id, String idt_text, BuiltMap<String, Object> unused_fields}) {
-    String id_to_assign = id ?? idt_text;
+    String id_to_assign = id ?? ("5'-" + idt_text);
     var unused_fields_to_assign = unused_fields ?? BuiltMap<String, Object>();
     return Modification5Prime.from((b) => b
       ..display_text = display_text
@@ -122,7 +122,7 @@ abstract class Modification3Prime
     implements Built<Modification3Prime, Modification3PrimeBuilder>, Modification {
   factory Modification3Prime(
       {String display_text, String id, String idt_text, BuiltMap<String, Object> unused_fields}) {
-    String id_to_assign = id ?? idt_text;
+    String id_to_assign = id ?? ("3'-" + idt_text);
     var unused_fields_to_assign = unused_fields ?? BuiltMap<String, Object>();
     return Modification3Prime.from((b) => b
       ..display_text = display_text
@@ -183,7 +183,7 @@ abstract class ModificationInternal
       String idt_text,
       BuiltSet<String> allowed_bases,
       BuiltMap<String, Object> unused_fields}) {
-    String id_to_assign = id ?? idt_text;
+    String id_to_assign = id ?? ("internal-" + idt_text);
     var unused_fields_to_assign = unused_fields ?? BuiltMap<String, Object>();
     var allowed_bases_to_assign = allowed_bases ?? BuiltSet<String>(['A', 'C', 'G', 'T']);
     return ModificationInternal.from((b) => b