Currently, if the comments for a node are spread across more than one page, those comment pages are not flushed (urls with "?page=#" appended). Wondering if this is on the roadmap. Thanks!

Comments

mr.j’s picture

mr.j CreditAttribution: mr.j commented

I have noticed that too. Its not an easy one to fix the way the module just sends an array of URLs when it calls its hook. Without changing that I assume you would have to specify each page as a separate URL with the ?page=1, ?page=2 etc appended.

If you purge whatever cache you use with wildcards for example you could inadvertently clear a whole bunch of other pages. eg When a user updates a forum node on our site, the taxonomy part of this module purges the /forum URL as well as the actual forum node page like /forum/discussion/topic/12345. But if you purged /forum/* that would clear every page in the whole forum. So you need to either specify the pages individually, or somehow recognise that the URL is for a specific node and not a taxonomy term or homepage, and then apply a wildcard to it.

mr.j’s picture

mr.j CreditAttribution: mr.j commented

I have some code here that does this when using core's db caching:
http://drupal.org/node/1308252#comment-5133192

Its not perfect and relies on the ordering of the URLs passed through by the module, but so far it works great for my needs.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Issue summary: View changes
Status: Active » Closed (outdated)

Drupal 6 is no longer supported. If this issue exists in Drupal 7, please reopen and provide details.