$config_directories is a core global but not in the list which leads to false positives. Add it to the list. Found whilst working on #2656176: Fix issues identified by phpcs using Drupal standards and patch tested on that project - it works great.

CommentFileSizeAuthor
#2 2656186-2.patch723 bytesalexpott
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

alexpott created an issue. See original summary.

alexpott’s picture

Status: Active » Needs review
FileSize
723 bytes
alexpott’s picture

Everything else in the list on https://api.drupal.org/api/drupal/globals is present.

alexpott’s picture

pfrenssen’s picture

Status: Needs review » Reviewed & tested by the community

Nice find, thanks!

  • klausi committed 717cb8b on authored by alexpott
    Issue #2656186 by alexpott: Add missing $config_directories to core...
klausi’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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