Closed (won't fix)
Project:
Drupal Commons
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Aug 2011 at 00:28 UTC
Updated:
4 Apr 2013 at 01:04 UTC
We display the status stream with information relevant to the user at /user -- Let's make 'stream' the global status stream for the site.
Comments
Comment #1
ezra-g commentedhttps://github.com/acquia/commons/pull/12
Comment #2
ezra-g commentedWe're not passing the right arguments to the callback here.
Comment #3
icecreamyou commentedBefore I left Acquia, Jay and I agreed on the following:
Additionally, administrators should have an option to set the default value of this setting. For anonymous users, /stream always shows the global, site-wide stream.
This is how it currently works:
For anonymous users, /stream is empty.
This implies the following nontrivial changes:
The hardest part of this is making user profiles just show the user's own activity. The easiest approach is to add the acting user's UID as a column to the {activity_log_messages} table and then filter on that column appropriately. But doing this cleanly is going to require some re-thinking about the way we approach the visibility of activity messages. (That's a good thing; that part needs some re-thinking.)
Comment #4
markconroy commented...
This is a big issue for me. I would presume when an anonymous user looks at the site and has access to "view all activity streams", that /stream would show all streams.
Is this going to be fixed/changed or is there something I can do to make it work. Here's my test environment:
http://bit.ly/qIej1h - if successful, it'll be the framework for the biggest, most ambitious web project in Ireland ever.
Comment #5
icecreamyou commentedYes, this issue is about fixing it.
Tomorrow I will try to come up with a plan for how this should work. It's pretty closely related to the question of re-architecting how activity visibility works.
Comment #6
icecreamyou commentedRan out of time today.
Comment #7
icecreamyou commentedPlan described at #1259306: Re-architect activity message visibility
Comment #8
ezra-g commentedAlso, for anyone looking at this on their Commons site, Commons_status_Streams currently implements
return theme('facebook_status_form_display', NULL, NULL, 'activity_log_stream');for the /stream callback, overriding the configured $view that's set by the context.Comment #9
japerryIn all likelihood, this issue was fixed in a later version, but since Since we only support 6.x-2.12 and 7.x-3.x, marking this as won't fix. Feel free to re-open if you find this to be in error