The function responding to "notifications/autocomplete/node/title" (used for NID autocomplete in Thread subscription) do not take into account the per-content-type thread subscription setting, returning instead ALL the nodes.

This issue should also take into account Group privacy setting (as stated in #906168)

Regards

CommentFileSizeAuthor
#2 notifications_autocomplete-1008996-2.patch1.4 KBduellj

Comments

thepanz’s picture

thepanz commented
Status: Active » Needs review

I've fixed this issue (with many others) on my Git repository (https://github.com/thePanz/notifications), in particular on commit https://github.com/thePanz/notifications/commit/edb1eb8764a60224e815c0a5...

The base branch of notifications have been taken fom the git.drupalsource.org repository

duellj’s picture

duellj commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new notifications_autocomplete-1008996-2.patch1.4 KB

Fix works as expected. After applying, notifications autocomplete for adding nodes takes into account per-content type settings.

Also uploading patch here based on thePanz's git fixes

danepowell’s picture

danepowell commented
Title: Thread subscription: does not take into account per-content-type setting » Thread subscription not taking into account per-content-type setting
Status: Reviewed & tested by the community » Fixed

http://drupalcode.org/project/notifications.git/commit/54d8463

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.