sp-dev-fx-controls-react: [Dynamic Form] Update Form crashes on loading, if an item has TaxonomyFieldTypeMulti field. [3.16.1] !!!

Version

Please specify what version of the library you are using: [3.16.1]

Observed Behavior

If an item has a TaxonomyFieldTypeMulti field , the Dynamic Form crashes on loading. And get next error:

`Error get field informations TypeError: defaultValue.split is not a function
    at eval (DynamicForm.js:591:1)
    at step (DynamicForm.js:58:1)
    at Object.eval [as next] (DynamicForm.js:39:45)
    at step (DynamicForm.js:43:1)
    at Object.eval [as next] (DynamicForm.js:39:45)
    at fulfilled (DynamicForm.js:30:42)`

It assumes that value should be a string , but it is an Array of objects:

Screenshot 2024-01-09 135139

About this issue

Original URL
State: closed
Created 6 months ago
Comments: 21 (10 by maintainers)

Commits related to this issue

Fixing #1733 for non-standard cases A possible fix for #1733 when the data gets returned in a non-standard way. — committed to IRRDC/sp-dev-fx-controls-react by IRRDC 5 months ago

Most upvoted comments

I created a new fix (to fix the fix): https://github.com/pnp/sp-dev-fx-controls-react/pull/1736

IRRDC on Jan 11, 2024

@joelfmrodrigues Why did you close the bug without comment when there is an open pull request to fix existing issues?

Hi @IRRDC this issue has been closed because its original PR (which was a rollback from has been released in 3.16.1) has been merged and closed, then released in minor version.

Your PR is linked to a global behavior for which we didn’t figure out the root cause, thus not linked to this issue in particular.

Before addressing this one, we need to understand what causes this results sub-array to appear unconsistently, not only providing a solution that can handle both scenarios. We’ll keep your PR open meanwhile of course.

I’ll open a new issue.

michaelmaillot on Feb 7, 2024

Hi @IRRDC & @bva999,

A new version has been released which includes this fix.

Can you check and tell me if that’s ok for you?

Hi @michaelmaillot. It’s works for me.

bva999 on Jan 22, 2024