Home » Download » Modules » Memcache API and Integration » Issues

Mollom should call cache_set() with a timestamp, not a number of seconds

Damien Tournoud - August 13, 2009 - 09:59
Project:Memcache API and Integration
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

cache_set() takes a UNIX timestamp as the $expire parameter, not a number of seconds (oups!). Let's fix that.

» Login or register to post comments

#1

Damien Tournoud - August 13, 2009 - 10:03
Status:active» needs review
AttachmentSize
548078-cache-set-oups.patch 793 bytes
Login or register to post comments

#2

mikl - August 13, 2009 - 10:48
Status:needs review» reviewed & tested by the community

Looks fine. Go for it :)

Login or register to post comments

#3

Dries - August 13, 2009 - 11:06
Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks!

Login or register to post comments

#4

System Message - August 27, 2009 - 11:10
Status:fixed» closed

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

Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.