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'];
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3031385-3.patch | 1.24 KB | mferanda |
Comments
Comment #2
mferanda commentedComment #3
mferanda commentedComment #5
mferanda commentedComment #6
mferanda commented