Page Cache

mikeytown2 - June 8, 2009 - 02:01
Project:Beautify
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

if using the CacheExclude module with this one, the cache is still set. Here's a patch that corrects for this.
http://api.drupal.org/api/function/drupal_page_footer/6
http://api.drupal.org/api/function/page_set_cache/6

AttachmentSize
beautify.module.patch343 bytes

#1

mikeytown2 - June 12, 2009 - 11:31
Status:active» needs review

#2

psynaptic - June 21, 2009 - 15:48

Looks good thanks. Would you like CVS access so you can commit this yourself?

#3

mikeytown2 - June 21, 2009 - 17:53

sure, but I might end up fixing other things as well. Just a FYI

#4

psynaptic - June 21, 2009 - 19:26

That is the kind of warning I love! I have granted you access, feel free to do whatever you think is right.

I think we should probably remove the search and replace processing provided by this module (the bits where it removes newlines) as it messes up content submitted via textareas inputs.

#5

mikeytown2 - June 22, 2009 - 03:44
Status:needs review» reviewed & tested by the community

#6

mikeytown2 - November 10, 2009 - 04:38
Status:reviewed & tested by the community» needs work

#7

mikeytown2 - November 10, 2009 - 04:56
Status:needs work» needs review

works with pressflow now

AttachmentSize
beautify-484756.patch 3.78 KB
 
 

Drupal is a registered trademark of Dries Buytaert.