Hello,

thanks for the work! Loving this module.

Steps to reproduce:

1. Create 2 content types
2. Make one content type reference the other using entity reference (cardinality does not matter) ALSO MAKE THE FIELD REQUIRED!
3. Go to the reference field's "Manage Form Display" Page and select "Inline entity form - Multiple value" (Inline entity form - Single value does not work, see next bug report)
4. Save.
5. Go ahead and create a node referencing the other content type. Fill in the required fields and save.
6. Error message "field is required" appears, even though everything is filled in.

The problem seems to be here: function inline_entity_form_required_field in inline_entity_form.module (line 518)

CommentFileSizeAuthor
#3 inline_entity_form-form-field-validation-fail-2475969-3.patch795 byteswoprrr
#1 01.png226.4 KBwoprrr
#1 02.png231.01 KBwoprrr
#1 03.png243.01 KBwoprrr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

woprrr’s picture

woprrr CreditAttribution: woprrr as a volunteer commented
FileSize
03.png243.01 KB
02.png231.01 KB
01.png226.4 KB

Hi brutzelspeck,

If I create my valid entity and that I have no information in the required fields, validation warned me that I have not completed the required fields. And when I filled correctly create my content is well done and I have made my entity as a table.

For the single value problem see https://www.drupal.org/node/2475981 issue :) you are welcome :)

webiator GmbH’s picture

webiator GmbH CreditAttribution: webiator GmbH commented
Status: Active » Closed (fixed)

tested.Seems to be fixed in the latest dev version.
Thanks!

woprrr’s picture

woprrr CreditAttribution: woprrr as a volunteer commented
Priority: Normal » Major
Status: Closed (fixed) » Needs review
FileSize
inline_entity_form-form-field-validation-fail-2475969-3.patch795 bytes

You're right Brutzelspeck !! I managed to reproduce this bug malgres The latest version dailleurs (cache empty). Here is the patch, in my case it works very well. See you soon on IRC :)

I board the issue by Major because it is a blocking point that the field does not work when the reference field is required.

Thank you for your return good job guys :)

  • webflo committed 25016cc on 8.x-1.x 
    Issue #2475969 by brutzelspeck, woprrr, webflo: Required Field...
webflo’s picture

webflo CreditAttribution: webflo at UEBERBIT GmbH commented
Status: Needs review » Fixed

Thanks! Changed the type to \Drupal\Core\Field\FieldDefinitionInterface on commit, because thats the type we are using in WidgetBase.

Status: Fixed » Closed (fixed)

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