The error occurs when line 179 runs a function that returns nodequeue_nodequeue_access($node->type) && nodequeue_queue_access($queue, $subqueue).
You would not expect the superadmin to get a response of FALSE, but it does
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 1470570-enabled-node-types-6.patch | 1.19 KB | fizk |
Comments
Comment #1
carlhinton commentedThe error occurs when the following steps are followed:
Suggested solution:
When the user removes the node type at step 4, the code needs to identify the existance of nodes of this type and ask the user if these should be deleted or not
Comment #2
carlhinton commentedComment #3
amateescu commentedI agree with the suggested solution, but there is no patch here so setting back to active until someone steps up and creates a patch :)
Comment #4
pat redmond commentedconfirming this is an issue.
The module need to either warn the user that removing a content type will delete those nodes from the list (as mentioned in #1), or prevent the user from removing content types which exist in the nodequeue.
Comment #5
finedesign commentedAlso confirming this is an issue. I didn't setup the current site and didn't know only certain content types were available to a certain NodeQueue, so when I couldn't add a node, I just added the [nid: 123] manually. It worked, but when I realized it was not the correct content type, I could not remove it from the queue until I enabled that content type in the nodequeue settings.
Comment #6
fizk commentedHere's a patch that displays an error message if these types of nodes exist. The user is asked to either enable the node type(s) or remove those nodes from the queue first.
Comment #7
fizk commentedComment #8
fizk commentedComment #9
othermachines commented@fzk, I could not duplicate this issue using latest 7.x-2.x-dev following the steps in #1. After unchecking the content type in the list of permitted content types, I was able to remove from the queue nodes belonging to that content type both as User #1 and a regular user with permissive role. I'll let you decide if this should be closed.
Comment #10
fizk commented@othermachines Strange, I can't reproduce this anymore either...
Comment #11
fizk commented@CarlHinton, @finedesign, @Pat Redmond, If anyone can reproduce this using the latest 7.x-2.x-dev, please describe the steps to reproduce.
I'll close this after 7 days if we can't reproduce.
Comment #12
fizk commentedClosing. If you can reproduce using the latest 7.x-2.x-dev, please describe the steps.