On large installations it is customary to serve static files like JS and images from a dedicated server.

The attached patch adds support for javascript aggregator for this, through the settings page.

CommentFileSizeAuthor
yh.patch1.99 KByhager
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

derjochenmeyer’s picture

derjochenmeyer CreditAttribution: derjochenmeyer commented
Status: Needs review » Fixed

Commited: http://drupal.org/cvs?commit=353784

Thanks!

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented

For Drupal 7, I believe it's already supported via http://api.drupal.org/api/function/file_create_url/7 . CDN module for Drupal 6 needs a core patch, unfortunately, I believe.... Not completely sure really. We should port parts of this sucker to Drupal 7 and find out!

Kiphaas7’s picture

Kiphaas7 CreditAttribution: Kiphaas7 commented

@1: Isn't it better to use parallel module for this? It has more options and pretty much does what you want (and some more).
http://drupal.org/project/parallel

(So I'd prefer to rollback this patch)

EDIT: even more since this issue is fixed: #770374: Incompatible with Parallel module

nevermind, subtle difference apparently.

Status: Fixed » Closed (fixed)

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