Add the regex path exclusion from Javascript Aggregator module to core

Created on 13 May 2009, over 15 years ago
Updated 18 July 2024, about 2 months ago

Today I found that I needed to exclude the Ubercart JS from aggregation, because we pass the aggregated JS through Lighttpd, and that wasn't working for Ubercart JS.

Unfortunately, the aggregator at admin/settings/performance doesn't let you exclude by path, like it did in the 5.x Javascript Aggregator module.

It would be great if that functionality could be added back in. As it was, I had to hack includes/common.inc as follows:

function drupal_get_js($scope = 'header', $javascript = NULL) {
...
foreach ($javascript as $type => $data) {
...
switch ($type) {
...
default:

foreach ($data as $path => $info) {
		
if(strpos($path, "uc_")) { $info['preprocess'] = FALSE; } // added to exclude Ubercart JS

...
✨ Feature request
Status

Closed: won't fix

Version

11.0 πŸ”₯

Component
SystemΒ  β†’

Last updated 1 day ago

No maintainer
Created by

πŸ‡ΊπŸ‡ΈUnited States EvanDonovan

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024