Problem/Motivation

Reproduce

  • Enable ief and paragraphs
  • Create new paragraph foo
  • Add required, single value, entity reference field to content type page
  • Use IEF Simple Widget for that field in form display
  • Connect paragraph with article and use preview as "edit mode" in paragraph widget
  • Add node with one paragraph
  • Save node
  • Edit node and try to add a further paragraph

Recoverable fatal error: Argument 4 passed to Drupal\Core\Field\WidgetBase::setWidgetState() must be of the type array, null given, called in modules/inline_entity_form/src/Plugin/Field/FieldWidget/InlineEntityFormSimple.php on line 166 and defined in Drupal\Core\Field\WidgetBase::setWidgetState() (line 459 of /Users/christianfritsch/repositories/drupal/core/lib/Drupal/Core/Field/WidgetBase.php).

CommentFileSizeAuthor
#36 ief_simple_widget_not-2722097-36.patch4.17 KBjohnchque
#36 ief_simple_widget_not-2722097-36-test-only.patch3.15 KBjohnchque
#36 interdiff-2722097-17-36.txt6.07 KBjohnchque
#17 ief_simple_widget_not-2722097-17_TEST_ONLY.patch4.56 KBchr.fritsch
#17 ief_simple_widget_not-2722097-17.patch5.59 KBchr.fritsch
#13 ief_simple_widget_not-2722097-12_TEST_ONLY.patch4.66 KBchr.fritsch
#13 ief_simple_widget_not-2722097-12.patch5.69 KBchr.fritsch
#10 ief_simple_widget_not-2722097-10_TEST_ONLY.patch4.33 KBchr.fritsch
#10 ief_simple_widget_not-2722097-10.patch5.68 KBchr.fritsch
#3 simple_widget_not-2722097-3_TEST_ONLY.patch4.33 KBchr.fritsch
#3 simple_widget_not-2722097-3.patch5.36 KBchr.fritsch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chr.fritsch created an issue. See original summary.

miro_dietiker’s picture

This issue was referenced in #2708033-13: Edit modes "closed" and "preview" choke on invalid entity references
As it's committed, we should be able to fix it easily. We still need additional test coverage.

chr.fritsch’s picture

The last submitted patch, 3: simple_widget_not-2722097-3.patch, failed testing.

chr.fritsch’s picture

Project: Inline Entity Form » Paragraphs

Moved issue into paragraphs module, because the issue have to be fixed there

The last submitted patch, 3: simple_widget_not-2722097-3.patch, failed testing.

chr.fritsch’s picture

Title: Simple Widget not working in paragraphs with preview mode » IEF Simple Widget not working in paragraphs with preview mode

The last submitted patch, 3: simple_widget_not-2722097-3.patch, failed testing.

The last submitted patch, 3: simple_widget_not-2722097-3.patch, failed testing.

The last submitted patch, 10: ief_simple_widget_not-2722097-10.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 10: ief_simple_widget_not-2722097-10_TEST_ONLY.patch, failed testing.

The last submitted patch, 13: ief_simple_widget_not-2722097-12.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 13: ief_simple_widget_not-2722097-12_TEST_ONLY.patch, failed testing.

Berdir’s picture

The test dependency patch needs to be a standalone patch that does nothing but that, so that it can be committed separately.

Are you sure you are using the latest version? your patch doesn't apply and I think we recently made some changes about exactly something like this.

chr.fritsch’s picture

Status: Needs work » Needs review
FileSize
5.59 KB
4.56 KB

The issue still occurs with latest dev. So here is a new patch

The last submitted patch, 17: ief_simple_widget_not-2722097-17.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 17: ief_simple_widget_not-2722097-17_TEST_ONLY.patch, failed testing.

chr.fritsch’s picture

chr.fritsch’s picture

The last submitted patch, 10: ief_simple_widget_not-2722097-10.patch, failed testing.

The last submitted patch, 10: ief_simple_widget_not-2722097-10_TEST_ONLY.patch, failed testing.

The last submitted patch, 13: ief_simple_widget_not-2722097-12.patch, failed testing.

The last submitted patch, 13: ief_simple_widget_not-2722097-12_TEST_ONLY.patch, failed testing.

The last submitted patch, 17: ief_simple_widget_not-2722097-17.patch, failed testing.

The last submitted patch, 17: ief_simple_widget_not-2722097-17_TEST_ONLY.patch, failed testing.

The last submitted patch, 10: ief_simple_widget_not-2722097-10.patch, failed testing.

The last submitted patch, 13: ief_simple_widget_not-2722097-12.patch, failed testing.

The last submitted patch, 13: ief_simple_widget_not-2722097-12_TEST_ONLY.patch, failed testing.

The last submitted patch, 17: ief_simple_widget_not-2722097-17.patch, failed testing.

The last submitted patch, 17: ief_simple_widget_not-2722097-17_TEST_ONLY.patch, failed testing.

The last submitted patch, 3: simple_widget_not-2722097-3_TEST_ONLY.patch, failed testing.

The last submitted patch, 10: ief_simple_widget_not-2722097-10_TEST_ONLY.patch, failed testing.

johnchque’s picture

Assigned: Unassigned » johnchque

I will try to fix the tests.

The last submitted patch, 36: ief_simple_widget_not-2722097-36-test-only.patch, failed testing.

The last submitted patch, 36: ief_simple_widget_not-2722097-36-test-only.patch, failed testing.

The last submitted patch, 36: ief_simple_widget_not-2722097-36-test-only.patch, failed testing.

miro_dietiker’s picture

Status: Needs review » Fixed

Great, yeah this limitation to edit is important.

Status: Fixed » Closed (fixed)

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

brooke_heaton’s picture

I'm getting this error again after the latest Paragraphs update :(