theme_field_ui_table() expects $variables['elements']['#regions'][...]['rows_order'] to be an array, but if an entity type has no fields, this array key is not set by \Drupal\field_ui\FieldOverview::buildForm(), causing an undefined index error.

CommentFileSizeAuthor
#1 drupal_2203271_1.patch775 bytesxano

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Active » Needs review
StatusFileSize
new drupal_2203271_1.patch775 bytes

I confirmed that the patch works for Payment, of which \Drupal\payment\Tests\Controller\PaymentTypeWebTest failed because of this.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Fix looks valid and suddenly HEAD is failing with this too?

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed a457090 and pushed to 8.x. Thanks!

yesct’s picture

yesct commented
Related issues: +#2193609: No page title on manage display & form for non-default view mode

was the head fail traced back to #2193609: No page title on manage display & form for non-default view mode ? and it's change in using account settings instead of node for a test?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Yes, I did trace it back. There's nothing wrong with that issue, other than it somehow conflicted with another issue that apparently changed the code in a way that results in those noticed. Just bad timing :)

Status: Fixed » Closed (fixed)

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