Browse Source

Extension editable, dropdown display the value instead of label (#7386)

* Include value in <A> tag only if formatter is used #7081

* Fix linting errors
Karel Kozlik 1 year ago
parent
commit
5632a2372f
1 changed files with 3 additions and 1 deletions
  1. 3 1
      src/extensions/editable/bootstrap-table-editable.js

+ 3 - 1
src/extensions/editable/bootstrap-table-editable.js

@@ -58,6 +58,8 @@ $.BootstrapTable = class extends $.BootstrapTable {
         }
       }
 
+      const formatterIsSet = column.formatter ? true : false
+
       $.each(this.options, processDataOptions)
 
       column.formatter = column.formatter || (value => value)
@@ -95,7 +97,7 @@ $.BootstrapTable = class extends $.BootstrapTable {
           data-name="${column.field}"
           data-pk="${row[this.options.idField]}"
           data-value="${value || ''}"
-          ${editableDataMarkup}>${result}</a>` // expand all data-editable-XXX
+          ${editableDataMarkup}>${formatterIsSet ? result : ''}</a>` // expand all data-editable-XXX
       }
     })
   }