Basically it doesn't filter values when they're empty
It causes the situation where is not possible to remove the plugin with configuration in the field with the cardinality = unlimited

CommentFileSizeAuthor
#8 plugin-selector-widget-empty-value-2981112-8.patch1.06 KBjnorton
#7 plugin-selector-widget-empty-value-2981112-7.patch881 bytesdaniel kulbe
#2 2981112-2.patch933 bytesalphawebgroup

Comments

alphawebgroup created an issue. See original summary.

alphawebgroup’s picture

alphawebgroup
he/him
Primary language English
Location Kyiv / Toronto
commented
StatusFileSize
new 2981112-2.patch933 bytes
alphawebgroup’s picture

alphawebgroup
he/him
Primary language English
Location Kyiv / Toronto
commented
Assigned: alphawebgroup » Unassigned
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: 2981112-2.patch, failed testing. View results

dubs’s picture

dubs commented

+1 for to have this submitted - it works for me :-)

joachim’s picture

joachim commented

> It causes the situation where is not possible to remove the plugin with configuration in the field with the cardinality = unlimited

I've applied the patch and I'm still unable to remove items from an unlimited-value field.

daniel kulbe’s picture

daniel kulbe
he/him
Primary language German
Location Berlin
commented
Status: Needs work » Needs review
StatusFileSize
new plugin-selector-widget-empty-value-2981112-7.patch881 bytes

New patch ...

jnorton’s picture

jnorton commented
StatusFileSize
new plugin-selector-widget-empty-value-2981112-8.patch1.06 KB

Drupal 9 patch attached for Plugin version 8.x-2.8.

ckaotik’s picture

ckaotik
Primary language German
Location Berlin
commented
Title: PluginSelector::massageFormValues() fix » Cannot remove value due to PluginSelector::massageFormValues()

This issue resolved itself for me when using #2647312: Use SubFormState in plugin selectors.