Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Default value of hidden nested components keys - not cleared from the submission when inside another edit grid. #1686

Open
llaurentiu opened this issue Jan 15, 2024 · 2 comments

Comments

@llaurentiu
Copy link

llaurentiu commented Jan 15, 2024

Having:

  • A parent edit grid edited in a modal (grid 'template' has modal = true)
  • Another edit grid/container inside the parent edit grid.
  • A component (eg: radio/checkbox) that controls the visibility of the inner grid/container

then:

  • When we show the inner grid/container by triggering the conditional
  • Then we hide the inner grid/container again

=> the inner grid/container keys (the default value - [] / {}) are not removed from the submission.
image

Correct behavior:
components that are hidden by logic, that have clearOnHide = true, should be removed from the submission.

JsFiddle here: https://jsfiddle.net/tcLbx7pe/

happens in the last stable: 4.18.0 too.

@llaurentiu llaurentiu changed the title Container & edit grid keys not cleared from the submission when inside another edit grid. [BUG] Container & edit grid keys not cleared from the submission when inside another edit grid. Jan 16, 2024
@llaurentiu llaurentiu changed the title [BUG] Container & edit grid keys not cleared from the submission when inside another edit grid. [BUG] Nested components keys not cleared from the submission when inside another edit grid. Jan 16, 2024
@llaurentiu llaurentiu changed the title [BUG] Nested components keys not cleared from the submission when inside another edit grid. [BUG] Default value of hidden nested components keys - not cleared from the submission when inside another edit grid. Jan 16, 2024
@heather-formio
Copy link
Contributor

@llaurentiu Thank you for the report, we have a ticket on our backlog to address this, and appreciate the contribution here.

@olgabann
Copy link

For internal reference: FIO-8344

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants