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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xano’s picture

Status: Active » Needs review
FileSize
775 bytes

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

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

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

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a457090 and pushed to 8.x. Thanks!

YesCT’s picture

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

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.