This patch contains two changes:

  1. I implemented simple regexp support (wildcard '*'). (based on the SecurePages module, but simpler).
  2. I moved the settings (previously on its own page "?q=admin/settings/cacheexclude") to the cache fieldset of the performance settings form "?q=admin/settings/performance". This removes clutter from the administration pages and adds the CacheExclude settings to a place where it belongs: the cache settings => easier administration, less clicking. (less code also).

because almost all code changed, I did not attach a patch but the whole module file (as txt).

cheers

CommentFileSizeAuthor
cacheexclude_0.txt1.51 KBsoxofaan

Comments

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

I'd suggest to use the code used by path.module instead.

Note that this functionality is directly available in the Drupal 6 API, in the drupal_match_path() function.

Crell’s picture

Crell commented

Yeah, no that it's nicely factored out in Drupal 6 my plan is to just backport that function for the Drupal 5 version, or even just leave this feature until D6 where I can just use the existing match functionality. This module is why I was so happy to see the patch for that function go in. :-) Unfortunately it's low on my priority list at the moment. :-(

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Heh, no worries, I was just posting it for reference here ;) :)

Crell’s picture

Crell commented
Status: Needs review » Fixed

I've just tagged a new 2.x release that uses a backport of drupal_match_path().

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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