fix(ISSUE-279): 🐛 set fieldset description as attribute, not node, wh…

…en migrating from PFG
collective · Jun 11, 2021 · 1cae079 · 1cae079
1 parent ce31ba0
commit 1cae079
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/news/279.bugfix b/news/279.bugfix
@@ -0,0 +1,2 @@
+Set fieldset description as attribute, not node, when migrating from PFG.
+[thomasmassmann]
diff --git a/src/collective/easyform/migration/fields.py b/src/collective/easyform/migration/fields.py
@@ -128,6 +128,13 @@ def append_or_set_title(field, name, value):
         append_node(field, name, value)
 
 
+def append_or_set_description(field, name, value):
+    if field.tag == 'fieldset':
+        set_attribute(field, 'description', value)
+    else:
+        append_node(field, name, value)
+
+
 def convert_tales_expressions(value):
     if value == u'here/memberEmail':
         return u"python:member and member.getProperty('email', '') or ''"
@@ -171,7 +178,7 @@ def to_text(value):
 }
 
 PROPERTIES_MAPPING = {
-    'description': Property('description', append_node),
+    'description': Property('description', append_or_set_description),
     'fgDefault': Property('default', append_default_node),
     'fgmaxlength': Property('max_length', append_maxlength_node),
     'fgsize': None,  # Not available in collective.easyform