Fix select all logic in v-field-list-item component #20315
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Scope
What's changed:
When we are adding fields one by one, we are emitting
field.key
as seen here:directus/app/src/components/v-field-list/v-field-list-item.vue
Line 132 in 4a36355
which is accurate and doesn't include groups in it (but still includes related collection if it's a related collection field).
Hence this PR replaces the original select all logic to re-use
field.key
of the children rather than re-constructing it manually.Before
Currently when we add a field one by one, the fields are added as intended. However when we use the
Select All
button, fields within groups are being added as if it's a related collection field:chrome_2PWIs0SXx3.mp4
After
chrome_NgRLR2dg5t.mp4
Potential Risks / Drawbacks
Fixes #19567