Some class files use the old standard of Definition of Drupal...

Proposed resolution

Standard now is Contains \Drupal...

Do just field_ui.
Here is a grep that shows the filenames that need updated:

lib/Drupal/field_ui/DisplayOverview.php: * Definition of Drupal\field_ui\DisplayOverview.
lib/Drupal/field_ui/FieldOverview.php: * Definition of Drupal\field_ui\FieldOverview.
lib/Drupal/field_ui/OverviewBase.php: * Definition of Drupal\field_ui\OverviewBase.
lib/Drupal/field_ui/Tests/FieldUiTestBase.php: * Definition of Drupal\field_ui\Tests\FieldUiTestBase.
lib/Drupal/field_ui/Tests/ManageDisplayTest.php: * Definition of Drupal\field_ui\Tests\ManageDisplayTest.
lib/Drupal/field_ui/Tests/ManageFieldsTest.php: * Definition of Drupal\field_ui\Tests\ManageFieldsTest.

Remaining tasks

Make the patch that updates just these files and just these lines to keep the patch reviewable.

User interface changes


API changes


Find the doc standards meta and link it here.
Related #2002624: Definition of to Contains \Drupal in field
#1310084: [meta] API documentation cleanup sprint

#2 definition-of-to-contains-2002618.patch2.87 KBleanderl
PASSED: [[SimpleTest]]: [MySQL] 55,909 pass(es).
[ View ]


YesCT’s picture

Issue summary:View changes

adding related (maybe contains a duplicate)

leanderl’s picture

Here is the standards doc which explains the pattern

leanderl’s picture

Status:Active» Needs review
new2.87 KB
PASSED: [[SimpleTest]]: [MySQL] 55,909 pass(es).
[ View ]

Changed "Definition of Drupal\..."
to Contains "\Drupal\..."

cilefen’s picture

Status:Needs review» Reviewed & tested by the community

I checked over the changes carefully and everything looks correct.

alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 4d2a10e and pushed to 8.x. Thanks!

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

Anonymous’s picture

Issue summary:View changes