Problem/Motivation

A warning banner is being displayed on profile install with aggrid. The source is the fact that variables are being assigned before an if statement.

Proposed resolution

Move the variables below inside of the aggrid.install if area.

// Verify the library is installed.
$aggrid_library_community = \Drupal::service('library.discovery')->getLibraryByName('aggrid', 'ag-grid-community');
$install_path_community = $aggrid_library_community['js'][0]['data'];
$aggrid_library_enterprise = \Drupal::service('library.discovery')->getLibraryByName('aggrid', 'ag-grid-enterprise');
$install_path_enterprise = $aggrid_library_enterprise['js'][0]['data'];

CommentFileSizeAuthor
#3 3031385-3.patch1.24 KBmferanda

Comments

mferanda created an issue. See original summary.

mferanda’s picture

Issue tags: +install, +requirements
mferanda’s picture

StatusFileSize
new1.24 KB

  • mferanda committed 81dbf29 on 8.x-1.x
    Issue #3031385 by mferanda: Warning on profile install due to aggrid...
mferanda’s picture

Status: Active » Fixed
mferanda’s picture

Status: Fixed » Closed (fixed)