In certain heavy-load circumstances the css_js_query_string feature can add a major performance hit. It would be good to have an option to disable this.

CommentFileSizeAuthor
#1 zen-n772234-6.x-1.x.patch1.15 KBDamienMcKenna
#1 zen-n772234-6.x-2.x.patch1.17 KBDamienMcKenna
#1 zen-n772234-7.x-3.x.patch1.17 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
zen-n772234-7.x-3.x.patch1.17 KB
zen-n772234-6.x-2.x.patch1.17 KB
zen-n772234-6.x-1.x.patch1.15 KB

Here are three patches that work off a new variable called "skip_css_js_query_string", which defaults to FALSE and allows for the query string addition to be skipped.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

FYI a related patch for Drupal core has been submitted: #772328: Optionally disable using css_js_query_string - D6 for now, a D7 version will be provided soon.

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Status: Needs review » Closed (won't fix)

Based on the core issue, closing this.