Closed (fixed)
Project:
FlashVideo
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
25 Jun 2009 at 04:42 UTC
Updated:
25 Jul 2009 at 21:00 UTC
Why does the module use a direct TRUNCATE sql statement to clear the cache when performing postop, rather than using Drupal's provided API for clearing cache: cache_clear_all() ?
Aside from assuming too much about the structure of Drupal's cache table, a direct call to TRUNCATE will actually fail -- as I found out -- if you've properly set up your db to disallow TRUNCATE permission to the db user. (If someone hacks your Drupal site, you don't want to let them clean out entire tables, at least not with a single statement.)
Update in a future version?
Comments
Comment #1
attheshow commentedGood point. Fixed in the development version.