SamplePointLocations upgrade - should be on senaite.samplepointlocations

bikalims · Sep 21, 2024 · b0e24cd · b0e24cd
1 parent 4e425f5
commit b0e24cd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/senaite/core/upgrade/utils.py b/src/senaite/core/upgrade/utils.py
@@ -299,7 +299,7 @@ def permanently_allow_type_for(portal_type, allowed_type):
     # get the current allowed types for the object
     allowed_types = fti.allowed_content_types
     # append the allowed type
-    fti.allowed_content_types = allowed_types + (allowed_type, )
+    fti.allowed_content_types = allowed_types + [allowed_type, ]
 
 
 def set_uid(obj, uid):

diff --git a/src/senaite/core/upgrade/v02_06_000.py b/src/senaite/core/upgrade/v02_06_000.py
@@ -1328,6 +1328,11 @@ def migrate_samplepoint_to_dx(src, destination=None):
     # NOTE: always convert string values to unicode for dexterity fields!
     target.title = api.safe_unicode(src.Title() or "")
     target.description = api.safe_unicode(src.Description() or "")
+    target.sample_point_id = api.safe_unicode(src.SamplePointId or "")
+    target.sample_point_location = [src.aq_parent.UID()]
+    target.equipment_id = api.safe_unicode(src.EquipmentID or "")
+    target.equipment_type = api.safe_unicode(src.EquipmentType or "")
+    target.equipment_description = api.safe_unicode(src.EquipmentDescription or "")
 
     # we set the fields with our custom setters
     target.setLatitude(src.getLatitude())