If you set a content of a selected content type to unpublished it's still viewable in glider.

CommentFileSizeAuthor
#3 unpublished_content-1236414-2.patch772 bytesalesr
#1 unpublished_content-1236414.patch508 bytesalesr

Comments

alesr’s picture

alesr commented
Status: Active » Needs review
StatusFileSize
new unpublished_content-1236414.patch508 bytes

patch which adds sql condition to check the status of a content

xjm’s picture

xjm
she/her
Primary language English
commented
  1. It would be better to use the constant NODE_PUBLISHED instead of 1
  2. However, node access modules can grant access to unpublished nodes in D7, so it would be better still to obey node access.
  3. The fact that node access is apparently ignored makes me think there is another fundamental bug in the module.

Leaving at "needs review" since I do not maintain nor use this module. :)

alesr’s picture

alesr commented
StatusFileSize
new unpublished_content-1236414-2.patch772 bytes

Patch #2 obeying node access

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

The patch in #3 looks good; it should respect all node access. @alesr tested the patch and it resolves the original issue for unpublished nodes.

filoquin’s picture

filoquin commented
Status: Reviewed & tested by the community » Closed (fixed)

I push the with the patch.
I am not using this module in any website. I appreciate the help.