Hi. I've faced an issue when I tried to install this module from scratch. After some investigation, I found that we have highcharts drupal module in dependencies as that module required 'highcharts.js' library to be installed. So I definitely think that we should add this info to the module documentation page.
For example something like this:
After enabling performance_toolkit module you need to manually add a JS library hightchars.js. For that you can complete the following steps:
1. Create folder sites/all/libraries/highcharts/js
2. Download file from https://code.highcharts.com/highcharts.js to the above folder.

CommentFileSizeAuthor
#2 add_hook_requirements-3036993-2.patch1.17 KBdriverok

Comments

driverok created an issue. See original summary.

driverok’s picture

StatusFileSize
new1.17 KB

Hi. I created a patch which implements hook_requirements() in order to check that required JS library 'highcharts' exists.
This hook checks on installation and runtime phases.
Also please consider updating the module page with information proposed by my previous comment (#1). Thanks for the great job!.

e.bogatyrev’s picture

Status: Active » Fixed

  • e.bogatyrev committed fc8fd4d on 7.x-1.x authored by driverok
    Issue #3036993 by driverok: Add installation and runtime check for...

Status: Fixed » Closed (fixed)

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