Closed (fixed)
Project:
Nodequeue
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 May 2009 at 12:47 UTC
Updated:
15 Jun 2009 at 16:10 UTC
My work on #468136: Nodequeue: Add test cases led me to this issue.
// load node queue with qid 1
$queue1 = nodequeue_load(1);
// $queue now contains a nodequeue object
// let's delete the nodequeue:
nodequeue_delete(1);
// and try to load it, bypassing the cache to make sure it's gone
$queue2 = nodequeue_load_queues(array(1), TRUE);
// at this point i would expect $queue2 to be NULL or empty; however, at this point $queue2 contains the nodequeue object with qid 1
The attached patch attempts to fix this by making sure the cache is actually bypassed when $bypass_cache is true.
| Comment | File | Size | Author |
|---|---|---|---|
| nodequeue_load_queues_fixed.patch | 2.28 KB | wulff |
Comments
Comment #1
ezra-g commentedThanks for posting a patch -- Marking as needs review.
Comment #2
ezra-g commentedThis resolves the malfunction while preserving cache functionality. Committed. Also, thanks for the clear testing instructions.
Thanks!