Javascripts are unset, unconditionally?

Problem/Motivation

It seems that the `eu_cookie_compliance_js_alter` alter hook removes JS from the `disabled_javascripts` config unconditionally so it seems? We currently had issues for the admin pages where all the Maps were unable to render, even when we used the "Exclude admin pages" option. Debugging this further in the module, I've found the eu_cookie_compliance_js_alter hook. Correct me if I'm wrong, but I think this function should unset javascript only when permissions have been given OR when exclude conditions apply (excluded user roles, path matches, exclude admin pages etc...).

Disclaimer; I'm hesitant to call this issue, since I expect that this piece of code has been reviewed on disabling JS correctly, so I'm probably wrong in claiming this issue.

/**
 * Implements hook_js_alter().
 */
function eu_cookie_compliance_js_alter(&$javascript, AttachedAssetsInterface $assets) {
  $config = \Drupal::config('eu_cookie_compliance.settings');
  $disabled_javascripts = $config->get('disabled_javascripts');
  $disabled_javascripts = _eu_cookie_compliance_explode_multiple_lines($disabled_javascripts);

  foreach ($disabled_javascripts as $script) {
    // Remove 'category:' if present.
    $parts = explode('%3A', $script);
    if (count($parts) > 2) {
      array_shift($parts);
      $script = implode(':', $parts);
    }
    else {
      $script = end($parts);
    }
    // Parse the string and drop the parameter that is a behavior name.
    if (strpos($script, '%7C') !== FALSE) {
      @list($script, $attach_name) = explode('%7C', $script);
    }
    _eu_cookie_compliance_convert_relative_uri($script);

    unset($javascript[$script]);
  }
}

Steps to reproduce

We've provided the following geolocation scripts to the Disabled javascripts config. Note that when you add a Geolocation field with Google Maps form view that the field will not render its map.

modules/contrib/geolocation/js/geolocation-google-maps-api.js
modules/contrib/geolocation/js/geolocation-common-map.js|geolocationCommonMap

When removing the lines, the map will render again?

Proposed resolution

Make the unsetting of the disabled_javascripts conditional?

Remaining tasks

Tba

User interface changes

N/a

API changes

N/a

Data model changes

N/a