Problem/Motivation

With the addition of per plugin config for the behavior plugins some of them will not need to show any field in the behavior form that is displayed while editing paragraphs.
With the current implementation a behavior plugin MUST provide at least one field for the behavior form, which won't be always the case, otherwise there will be a white display of error.

Proposed resolution

Allow behavior plugins not adding any behavior field.

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yongt9412 created an issue. See original summary.

johnchque’s picture

This actually fixes the bug, but we need tests.

johnchque’s picture

Status: Active » Needs review
miro_dietiker’s picture

Status: Needs review » Fixed

Committed to allow us move forward with other issues...

Let's make sure we have test coverage for every bug we discover in future...

johnchque’s picture

Adding tests, the test-only patch reverts the change to show that it was broken before the fix. :)

The last submitted patch, 6: allow_behavior_plugins-2843047-6-test-only.patch, failed testing.

The last submitted patch, 6: allow_behavior_plugins-2843047-6-test-only.patch, failed testing.

miro_dietiker’s picture

Component: Code » Experimental Widget
miro_dietiker’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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