I ran into a problem implementing nodequeue on MyClaySun.com using the latest generated downloadable file.

Nodequeue tab was showing on content, but when clicked, generated an error message that no nodequeues existed for this content type. I had created a couple of queues, assigned them to the blog type, enabled role access, and checked carefully to ensure that nothing was amiss in the database.

I put in some watchdog() code into nodequeue.module that revealed the following:

$sql= SELECT nq.qid FROM {nodequeue_queue} nq INNER JOIN {nodequeue_types} nt ON nt.qid = nq.qid INNER JOIN {nodequeue_roles} nr ON nr.qid = nq.qid WHERE nt.type = '%s' AND nr.rid IN (%d,%d)

$type = blog

There did not seem to be any values supplied for the %d placeholder.

I pulled a working nodequeue.module from BlufftonToday.com that has a much simpler, earlier version of nodequeue_node_access, and everything is working just peachy now. However, I can't find a version number for it.

Comments

merlinofchaos’s picture

merlinofchaos commented

I've fixed these bugs in the Drupal 5 branch, but need to back port them to the 4.7 branch. I guess I was being a little lazy and hoping I could ignore the 4.7 branch, since the 5.0 branch has some changes to use jquery. But obviously it's going to be difficult for some sites to update to Drupal 5.

dreed47’s picture

dreed47 commented

Any idea on an ETA? Or can you tell me which recent 5 commit fixed the problem so I can patch my version of 4.7?

dreed47’s picture

dreed47 commented

Or tell me which version number on the 4.7 branch I can roll back to that might not have the bug.

dreed47’s picture

dreed47 commented

yelvington, would you be willing to share your working 4.7 vesion in the mean time, until this gets fixed?

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Fixed

This should be fixed now. I'm not completely confident because I basically cut & pasted the code over but I think nothing is different that I didn't already catch.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)