Part of #2909370: Fix 'Drupal.Commenting.VariableComment.IncorrectVarType' coding standard

If multiple types are possible, separate them by a vertical bar ("|").

See https://www.drupal.org/docs/develop/coding-standards/api-documentation-a...

CommentFileSizeAuthor
#2 fix-2926121-2.patch669 byteszaporylie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zaporylie created an issue. See original summary.

zaporylie’s picture

zaporylie
he/him
Primary language Polish
CreditAttribution: zaporylie at Ny Media AS commented
Status: Active » Needs review
FileSize
fix-2926121-2.patch669 bytes
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and at Dazzle commented
Status: Needs review » Reviewed & tested by the community

Tested this with ag (ag "@var(.*)\sor\s(.*)"), the remaining results are valid.

  • Gábor Hojtsy committed 8e461b3 on 8.5.x 
    Issue #2926121 by zaporylie, borisson_: Multiple data types must be...

  • Gábor Hojtsy committed a5fc709 on 8.4.x 
    Issue #2926121 by zaporylie, borisson_: Multiple data types must be...
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed

Looks good, thanks!

Status: Fixed » Closed (fixed)

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