Closed (fixed)
Project:
Forum Access
Version:
5.x-1.9
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Feb 2008 at 07:04 UTC
Updated:
27 Feb 2008 at 14:12 UTC
While writing some code that was using forum_access.module for a few things, I encountered a bug in forum_access_access(). The static caching was only caching the result corresponding to the $tid passed in, and not caching results for different $type or $account values.
This could (and did in my case) result in incorrect information being returned by forum_access_access() on subsequent calls.
To fix this, I've patched against forum_access.module 5.x-1.9 to cache the other pertinent argument values along with the result they yield. Patch is attached.
| Comment | File | Size | Author |
|---|---|---|---|
| patch1.diff | 1.12 KB | Josh Benner |
Comments
Comment #1
salvisCommitted to 1.x-dev — thanks, that was a serious bug!
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.