Standard install, editing admin user listing view this notice is thrown

Location http://drupal/admin/structure/views/view/user_admin_people
Message Notice: Undefined index: provider in Drupal\views\Plugin\views\field\Broken->adminLabel() (line 24 of core\modules\views\lib\Drupal\views\Plugin\views\field\Broken.php).
Severity Notice

CommentFileSizeAuthor
drupal-views.view_.user_admin_people.yml-missing-provider-value.patch529 bytesAlan D.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +VDC

Good catch!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Nice catch! Hm, wonder how that happened? I guess that view per-dates the introduction of the key?

I thought about tests for this, but can't really think of a reasonable one, since all future Views exports will already have this key in them, and we don't write tests against old versions of APIs.

Ergo, committed and pushed to 8.x. Thanks!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

I guess that view per-dates the introduction of the key?

That or the fact that it is optional could have led to the result.

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented

Weird, I was certain this was fixed in another patch... I guess not :)

Status: Fixed » Closed (fixed)

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