Download & Extend
Nodequeue » Issues

Repeated nodequeue_load_queues() in nodequeue_nodeapi()

Posted by wulff on June 5, 2009 at 5:41pm
Project:Nodequeue
Version:7.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

The hook_nodeapi implementation currently contains this:

$queues = nodequeue_load_queues(array($obj->qid));
$queue = nodequeue_load_queues(array($obj->qid));

This doesn't look right, and results in a PHP notice as well.

Patch attached.

AttachmentSizeStatusTest resultOperations
nodequeue_hook_nodeapi.patch1.02 KBIgnored: Check issue status.NoneNone
Login or register to post comments

Comments

#1

Posted by ezra-g on October 24, 2009 at 8:35pm
Status:active» fixed

Good catch. Committed. Thanks!

#2

Posted by System Message on November 7, 2009 at 8:40pm
Status:fixed» closed (fixed)

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