Fork me on GitHub
Subscribe 1

Ticket #190 (fixed task)

Cache censored words

  • Created: 2010-11-15 16:54:23
  • Reported by: Franz
  • Assigned to: Franz
  • Milestone: 1.4.4
  • Component: caching
  • Priority: normal

If censoring is enabled, the censor_words() function is currently run on quite a lot of requests (viewtopic.php, viewforum.php, post.php and a few more).

That means there is a query run on every request to those pages, just for data that is not often changed. And while we make sure that the query is only run once, if at all, already, we could improve this (and thus, the overall performance) easily by caching the censored words.

This was brought up by Visman here: http://fluxbb.org/forums/viewtopic.php?id=4821

History

Franz 2010-12-08 21:06:51

  • Owner set to Franz.

Franz 2010-12-08 22:40:26

Commit 1f514d1 to fluxbb fluxbb-1.4

Ticket #190: Add caching support for censored words. Patch provided by Visman, thanks.

Franz 2010-12-08 22:42:43

  • Status changed from open to fixed.

And implemented.Thanks goes to Visman for the patch.

I think generating the cache when the censoring option is off, too, is no problem - especially as we'd have to mess with regenerating the cache when turning it on again, too.