Include Drupal core's improvement to drupal_build_css_cache() to prevent unnecessary aggregates [#1888824]

As reported in #1404380: Unnecessary aggregation of CSS/JS, unnecessary CSS aggregates are sometimes created. Since the CDN module replaces Core's drupal_build_css_cache() function with its own nearly identical function, this improvement should be included in CDN's version of the function. This will help resolve the same issue when the CDN module is enabled.

Comment	File	Size	Author
#1	cdn-core-css-aggregation-improvement-1888824-1.patch	677 bytes	Chaulky

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Chaulky CreditAttribution: Chaulky commented 15 January 2013 at 02:10

File	Size
cdn-core-css-aggregation-improvement-1888824-1.patch	677 bytes

Patch applies cleanly against 7.x-2.x-dev commit http://drupalcode.org/project/cdn.git/commit/7f05ccf

Comment #2

Wim Leers

Ghent 🇧🇪🇪🇺

CreditAttribution: Wim Leers commented 28 February 2013 at 19:14

Title:	Include Core's improvement to drupal_build_css_cache to prevent unnecessary aggregates	» Include Drupal core's improvement to drupal_build_css_cache() to prevent unnecessary aggregates
Status:	Active	» Fixed

AWESOME!

This also explains #1905468: Variables table is not designed to be used as a cache.

Committed: http://drupalcode.org/project/cdn.git/commit/f22a12e

I really should be working on getting CDN's modified version of CSS aggregation in, so that the CDN module can get rid of all that bloat (i.e. no longer duplicate 99.5% of the functionality to add in .5%). The only reason it needs to duplicate things is because it needs to be able to alter file URLs inside aggregated CSS files.
Maybe you want to help with that? :)