Internal paragraph type selection does not work with exclude filter

Beaware/!\ The paragraph table is designed for a single paragraph type. You can select multiple paragraphs, but only the first one will be selected for display.

That is correct, but the paragraph field settings allow to select / filter multiple paragraph types. In that case, using the “Paragraphs Table” formatter on the form view should prevent any other paragraph types from being added. (The view formatter actually works well, as it creates a separate table for every type added to the paragraphs field, so why not just change the type filtering behavior and call it a day?)

As it stands right now, a user is able to evoke undefined behavior in the module without changing code, and that is a bug, regardless of the module's purpose or design.

That said, we would nonetheless be eager to propose a solution to this as soon as we have the time. :)

Yeah this is clearly a bug with the (newer)

Exclude the selected below

setting on the paragraphs field. Should work in both directions and use the same logic as paragraphs itself uses, from my perspective.

Still @lazzyvn is totally correct with his point. It's focused on one type, otherwise creates separate tables due to the different fields = columns. We understand the reason and that shouldn''t be changed!

Alright, in that case we should probably prevent the user from adding more than one different paragraph type to the form field. That would be my first thought on a possible solution at least. We somehow have to prevent that buggy result, but I can totally understand if we so not want to undermine the module's design here.

Alright, understandable that we don't want to undermine the module's core concepts here. But the proposed solution should still work with that, wouldn't it?

Yes, it's just a clean bugfix, that still makes sense IMHO. Not changing the module logics at all.

Oh I think fixed it is not complicate
in widget/formatter looking for target_bundles just check handler_settings[negate] and use array_reverse.

lazzyvn → closed merge request !16

Thanks for the fix @lazzyvn! That was fast!!

Small regression can be found here: 🐛 Warning: Undefined array key 0 in ParagraphsTableFormatter.php on line 474 Active

Automatically closed - issue fixed for 2 weeks with no activity.