The $libraries variable is initialized to an empty array at the start of LibraryDiscoveryParser::buildByExtension() and then a few lines below a second time with the result from the call to $this->parseLibraryInfo().

CommentFileSizeAuthor
#2 3035552-2.patch606 bytespfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen at Randstad Digital for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Active » Needs review
FileSize
3035552-2.patch606 bytes
angel.h’s picture

angel.h
Primary language English
Location Plovdiv
CreditAttribution: angel.h commented
Status: Needs review » Reviewed & tested by the community

Indeed the $libraries variable is redeclared further down the code and is not used in between.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Version: 8.6.x-dev » 8.7.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 825e514 and pushed to 8.7.x. Thanks!

  • catch committed 825e514 on 8.7.x 
    Issue #3035552 by pfrenssen, angel.h: $libraries variable declared twice...

Status: Fixed » Closed (fixed)

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