The function nodequeue_load_random_node($sqid) does not return content [#1167846]

The function nodequeue_load_random_node($sqid) is intended to return a random node from the subqueue specified by the id. Instead, because the query is malformed, the function returns nothing. I've attached a patch which should correct the issue on the alpha2 release.

Comment	File	Size	Author
#2	1167846.patch	2.22 KB	Gekiboy
#2
#1	1167846.patch	2.23 KB	amateescu
#1
	nodequeue.random_node.patch	892 bytes	Gekiboy

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

amateescu CreditAttribution: amateescu commented 25 May 2011 at 08:38

Version:	7.x-2.0-alpha2	» 7.x-2.x-dev
Status:	Active	» Fixed

File	Size
1167846.patch	2.23 KB

Thanks for the patch. I found the same problem in nodequeue_view_random_node() and also fixed a related issue in nodequeue_view_nodes().

Commited attached patch to 7.x. Thanks again :)

http://drupalcode.org/project/nodequeue.git/commit/964859f

Comment #2

Gekiboy CreditAttribution: Gekiboy commented 27 May 2011 at 23:56

File	Size
1167846.patch	2.22 KB

Apparently Drush Make needs the patch to be generated with the --no-prefix option to work properly with the make files. I applied your patch and regenerated the patch file with the additional option.