Summary of Changes

1. Cache::setWorkarounds() incorrectly returns ALL current assets even if headerbefore option exists and a module/view didn't actually add any own assets.

This results in incremental caching of all assets in the cache of each module. Tons of WebAssets events are next executed on module cache load via Cache::getWorkarounds() and the load time is seriously affected.

2. Iterating over $wa->getManagerState()['activeAssets'] does not requires checking the ASSET_STATE_ACTIVE state because this array holds only active assets

3. The resulting array of getHeadData()['assetManager']['assets'] didn't have the asset name as key which prevents the fast cache workarounds comparison

Testing Instructions

Enable Joomla conservative caching, browse frontend pages as guest

Actual result BEFORE applying this Pull Request

Explore the contents of module cache files, see that they contain all WebAssetItem objects existing at the time of caching.
Note the cache file size.

Expected result AFTER applying this Pull Request

See that cache file size in much smaller now (delete old files first), see that the module cache now only have the assets really added by this module.

Documentation Changes Required

No.