Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
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
None.
API changes
None.
Related Issues
Find the doc standards meta and link it here.
Related #2002624: Definition of to Contains \Drupal in field
#1310084: [meta] API documentation cleanup sprint
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | definition-of-to-contains-2002618.patch | 2.87 KB | leanderl |
Comments
Comment #0.0
YesCT CreditAttribution: YesCT commentedadding related (maybe contains a duplicate)
Comment #1
leanderl CreditAttribution: leanderl commentedHere is the standards doc which explains the pattern http://drupal.org/node/1354#file
Comment #2
leanderl CreditAttribution: leanderl commentedChanged "Definition of Drupal\..."
to Contains "\Drupal\..."
Comment #3
cilefen CreditAttribution: cilefen commentedI checked over the changes carefully and everything looks correct.
Comment #4
alexpottCommitted 4d2a10e and pushed to 8.x. Thanks!
Comment #5.0
(not verified) CreditAttribution: commentedrelated