Aggregate External CSS & JS

I might want to move it out into the relocation sub module. It's already inlineing google fonts, doing the same for JS makes sense. FB has a really short TTL (20 min) so it's a little bit harder to do in comparison to GA but should be possible.

Example for me

https://www.googletagmanager.com/gtm.js?id=GTM-56CK7R (15 minutes)
https://connect.facebook.net/en_US/fbevents.js (20 minutes)
https://connect.facebook.net/signals/config/1540626602827177?v=2.7.16 (20 minutes)
https://www.google-analytics.com/plugins/ua/linkid.js (60 minutes)
https://www.google-analytics.com/analytics.js?org68b (2 hours)

Going too see what I can do...

Inline GA code without script loader
(function(i,s,o,r){i["GoogleAnalyticsObject"]=r;i[r]=i[r]||function(){(i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date()})(window,document,"script","ga");ga("create", "UA-XXXXXXX-X", {"cookieDomain":"auto"});ga("set", "anonymizeIp", true);ga("send", "pageview");

No UI, no control, proof on concept code.

Still need to handle @import statements for external CSS files, better code doc, put in a UI, and merge duplicate functionality. Looking good though.

@import needs to use

  $contents = preg_replace_callback('%@import\s*+(?:url\(\s*+)?+[\'"]?+(?![a-z]++:|/)([^\'"()\s]++)[\'"]?+\s*+\)?+\s*+;%i', '_advagg_load_stylesheet', $contents);

with a different _advagg_load_stylesheet() function.

Got this working with import now. Still need code doc, move code to .inc, UI for control, and merge duplicate functionality. Also need to keep #2889776: Remove no longer needed dns-prefetch and preconnect link tags in mind. Should also check if fonts loaded via deep import statements are handled. Need to try drupal_http_request again on failure.

Appears that there is a bug with nested import statements... might be core related.

Quick testing code

  drupal_static_reset('drupal_add_css');
  drupal_static_reset('drupal_add_js');
  drupal_add_css($GLOBALS['base_url'] . '/sites/all/modules/advagg/tests/css_test_files/css_input_with_import.css', array('type' => 'external'));
  $variables = array();
  _advagg_process_html($variables);
  $a = explode('link type="text/css" rel="stylesheet" href="', $variables['styles']);
  $c = array();
  foreach ($a as $b) {
    $z = explode('" media="all" />', $b);
    if (count($z) > 1) {
      $c[] = str_replace($GLOBALS['base_path'], $GLOBALS['base_url'] . '/', $z[0]);
    }
  }
  echo httprl_pr($c);

Functionally it's there. UI is there, but not really plugged in the code; need to add in the states array as well. Look into merging duplicate code functionality.

UI needs states array; need to pass variables into the $aggregate_settings array.

Should be good to go.

You're welcome! Let me know how it works for you. The Facebook pixel code was quite difficult to pull off, but I have it working on our site; getting a 100/100 on google page speed when using google analytics & tag manager as well as FB pixel https://developers.google.com/speed/pagespeed/insights/?utm_source=analy...