Closed (duplicate)
Project:
Drupal Commons
Version:
7.x-3.3
Component:
Email Notifications
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Oct 2013 at 16:04 UTC
Updated:
13 Nov 2013 at 20:22 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
behoppe333 commentedI did a bit more research, aided by fact that I already had maillog module installed on my site. Maillog's report of sent emails is consistent with the 0 shown in the tab title: when new posts are added to my moderated group, no email notifications go to John I Smith.
I have changed title of this issue to indicate that I think error is probably in the body, not the title, of each tab on the user notifications page.
Comment #2
WebSinPat commentedI see similar behavior in my installation. in my case, I think the discrepancy comes that although the "title" says "John Smith Follows", the body is actually showing the information for the logged in user. The body stays the same no matter what uid is in the URL user/[uid]/notification-settings. Whereas the title does change to reflect the URL, showing "John Smith Follows" for that user vs "you follow" for the logged in user.
So I'm guessing it's a configuration in the view that displays the info.
Comment #3
WebSinPat commented@behoppe333, let me know if you think I'm taking over your OP if you think your issue is not the same.
I found reference to similar behavior in #2001702: Message Subscribe sends emails regardless of flag checkmarks in notification-settings of always showing the groups followed by the logged in user even when claiming to be for another user. The issue was supposedly fixed for Commons 3.3, but I am still seeing the problem.
The fix in #3 of that issue at https://drupal.org/comment/7483678#comment-7483678 does pass the user object $account as an argument to the relevant functions, but then the view itself (e.g commons_follow_node) is not configured to expect a uid as an argument.
This is the original view configuration which does not have any user relationships and indeed displays flags "by current user"
Here is the modification I made to incorporate the user argument (according to instructions I found in an unrelated issue #1191228: Display Nodes Flagged by a Given User (rather than current user) in a View )

This change is giving me the behavior I expect when I visit user/[uid]/notification-settings.
I'm not sure if this is the right fix for the issue. I see that in message_subscribe_ui.module
so the uid is attempted to be passed programmatically rather than by the uid in the URL.
but all the same it does seem that the view is not receiving the argument with the original configuration.
Comment #4
WebSinPat commentedI found this related issue #1920824: Admin viewing another user's notification settings tab shows admin's notification settings that discussed this same issue. The conclusion there seemed to be that another solution would be to disable anyone (ie. user1) from viewing another user's notification-settings page at all, rather than dealing with fixing the relationship and contextual filters of the flag/view code. There is a patch at that issue, which I have not yet tested.
Comment #5
BarisW commentedPlease focus on this fix: #1920824: Admin viewing another user's notification settings tab shows admin's notification settings