Posted by mikeytown2 on June 8, 2009 at 2:01am
| Project: | Beautify |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
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
| Attachment | Size |
|---|---|
| beautify.module.patch | 343 bytes |
Comments
#1
#2
Looks good thanks. Would you like CVS access so you can commit this yourself?
#3
sure, but I might end up fixing other things as well. Just a FYI
#4
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
#6
#7
works with pressflow now