fixed labels export: do not default to node def name (#3137)

Co-authored-by: Stefano Ricci <[email protected]>
openforis · Nov 13, 2023 · 743bb6f · 743bb6f
1 parent 2cd805d
commit 743bb6f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/core/survey/nodeDef.js b/core/survey/nodeDef.js
@@ -255,8 +255,8 @@ export const getMeta = R.propOr({}, keys.meta)
 export const getMetaHierarchy = R.pathOr([], [keys.meta, metaKeys.h])
 
 // Utils
-export const getLabel = (nodeDef, lang, type = NodeDefLabelTypes.label) => {
-  let firstPart = null
+export const getLabel = (nodeDef, lang, type = NodeDefLabelTypes.label, defaultToName = true) => {
+  let firstPart = ''
   const name = getName(nodeDef)
 
   if (type === NodeDefLabelTypes.name) {
@@ -267,13 +267,13 @@ export const getLabel = (nodeDef, lang, type = NodeDefLabelTypes.label) => {
     if (!StringUtils.isBlank(label)) {
       if (type === NodeDefLabelTypes.label) {
         firstPart = label
-      } else if (type === NodeDefLabelTypes.labelAndName && !StringUtils.isBlank(label)) {
+      } else if (type === NodeDefLabelTypes.labelAndName) {
         firstPart = `${label} [${name}]`
       }
     }
   }
   // default to name
-  if (StringUtils.isBlank(firstPart)) {
+  if (StringUtils.isBlank(firstPart) && defaultToName) {
     firstPart = name
   }
 

diff --git a/server/modules/survey/service/surveyLabelsExport.js b/server/modules/survey/service/surveyLabelsExport.js
@@ -21,7 +21,12 @@ const exportLabels = async ({ surveyId, outputStream }) => {
         ...languages.reduce(
           (labelsAcc, lang) => ({
             ...labelsAcc,
-            [SurveyLabelsExportModel.getLabelColumn(lang)]: NodeDef.getLabel(nodeDef, lang),
+            [SurveyLabelsExportModel.getLabelColumn(lang)]: NodeDef.getLabel(
+              nodeDef,
+              lang,
+              NodeDef.NodeDefLabelTypes.label,
+              false
+            ),
           }),
           {}
         ),