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

ezra-g’s picture

Status: Active » Needs review
ezra-g’s picture

Status: Needs review » Needs work

We're not passing the right arguments to the callback here.

icecreamyou’s picture

Before I left Acquia, Jay and I agreed on the following:

  • /user/UID always shows UID's activity, including for the current user
  • For authenticated users, by default, /stream shows the global, site-wide stream of all activity. However, there should be a setting in users' profiles that allows users to change their stream to show just the following:
    • the current user's activity
    • the current user's relationships' activity
    • the current user's groups' activity

    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:

  • /user/UID always shows UID's activity, except for the current user
  • For authenticated users, /stream shows:
    • the current user's activity
    • the current user's relationships' activity
    • the current user's groups' activity

    For anonymous users, /stream is empty.

  • /user/UID shows the same thing as /stream for the current user
  • There is no global, site-wide stream.

This implies the following nontrivial changes:

  • Make user profiles just show the user's own activity.
  • Add a user profile setting to see all activity or just your stream at /stream
  • Add an administrative setting for the default value of the user profile setting (default is all activity)
  • Decide what kinds of activity should appear in the global stream

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.)

markconroy’s picture

This is how it currently works:

...

For anonymous users, /stream is empty.

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.

icecreamyou’s picture

Is this going to be fixed/changed or is there something I can do to make it work.

Yes, 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.

icecreamyou’s picture

Ran out of time today.

icecreamyou’s picture

ezra-g’s picture

Also, 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.

japerry’s picture

Version: » 6.x-2.x-dev
Status: Needs work » Closed (won't fix)

In 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