Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Hello, I have used this module on my website and I found a problem, in the user profiles I can`t see "Skip blog approvals" checkbox. I found solution: in line 231
if (user_access('create '. $node->type .' content', $account)) {
should be replaced with
if ((user_access('create '. $node->type .' content', $account)) || (user_access('create '. $node->type .' entries', $account))) {
because we have 'create article content', but 'create blog entries'.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | skip-approval.patch | 1.45 KB | rschwab |
| #2 | skip-approval.patch | 2.16 KB | rschwab |
Comments
Comment #1
GDI commentedComment #2
rschwab commentedThis happens not only for blog entries but for contributed module content types that don't follow the Node module's permission style. For example, the chatroom module has a permission to "create chat rooms", and hence gets left out of the account screen.
In this patch I removed the access check for creating content altogether. It seems more important that all content types are represented than to have that safety net for site builders that don't properly set their permissions (ie giving permission to approve comments on content types the role can't even create).
I've also included the changes from the patch at #980026: "Skip $node->type approvals" should be "Skip $node->name approvals" here, for simplicities sake.
Comment #3
rschwab commented#2 is a regression, so to fix I'll use node_access instead of user_access. Will patch and commit once I figure out which branch we're working from.
Comment #4
rschwab commentedThis patch changes the access check to use node_access() in order to correctly query the permission for each node type, and also implements the patch at #980026: "Skip $node->type approvals" should be "Skip $node->name approvals"
Comment #5
rschwab commentedCommitted #4 to 2.x-dev.