Various plugin classes need to have a leading backslash and we use "Contains" now as standard:

/**
 * @file
 * Definition of Drupal\Component\Plugin\PluginManagerInterface
 */
CommentFileSizeAuthor
#4 2078991-3.patch2.38 KBpwolanin
#2 2078991-2.patch504 bytespwolanin
#1 2078991-1.patch509 bytespwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Active » Reviewed & tested by the community
FileSize
2078991-1.patch509 bytes

trivial doc patch

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Reviewed & tested by the community » Needs review
FileSize
2078991-2.patch504 bytes

oops, should be "Contains" it seems..

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Title: Add leading backslash to PluginManagerInterface » fix @file doxygen for PluginManagerInterface
pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Title: fix @file doxygen for PluginManagerInterface » Fix @file doxygen for classes under Drupal\Component\Plugin
FileSize
2078991-3.patch2.38 KB

Let's just fix all in that directory and the Mapper

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Issue summary: View changes

Updated issue summary.

dawehner’s picture

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

Perfect small changes

webchick’s picture

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

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.