Another small patch to add only published node to the queue. (Useful when nodes are created by users who cannot check/uncheck the add to queue option).

Comments

davidfells81@gmail.com’s picture

I wonder if this should be an option. It seems like an overall workflow type situation. I can't think of a reason to add an unpublished node but who knows how those queues are being used? We can't be sure. I'll have to think about this one.

davidfells81@gmail.com’s picture

Status: Needs review » Closed (works as designed)

Not acting on this one right now.

scotwith1t’s picture

Issue summary: View changes

yeah, i'd say that's the job of the view that's based on the queue to handle not including unpublished nodes, but that's just my 2 cents. i'm sure there are circumstances where the patch is useful. it would be more useful to have a setting to set a variable which then determines whether the node should be added to the queue or not (checkbox on a config page for "Only add published nodes to the nodequeue", maybe a drupal_set_message that reminds the user it wasn't included if they save a node as unpublished and the auto nodequeue checkbox was checked?) There are lots of potential configuration options for this module, but don't want to get too far off the OP here :)