social_stream 0.26.2 → 0.27.0

data/base/app/assets/javascripts/social_stream.timeline.js

@@ -31,6 +31,8 @@ SocialStream.Timeline = (function(SS, $, undefined){
 		$(fullId).hide();
 		$(summaryId).tipsy({
 			html: true,
+			hoverable: true,
+			opacity: 0.95,
 			title: function(){
 				return $(this).siblings('.activity_audience').html();
 			}

data/base/app/assets/javascripts/social_stream.wall.js.erb

@@ -38,6 +38,12 @@ SocialStream.Wall = (function(SS, $, undefined){
     });
   }
 
+  var changeSecurityImage = function(type) {
+    $('#security_chzn .chzn-choices').
+      css('background-image', 'url(<%= asset_path('btn/') %>' + type + '.png)');
+  }
+
+
   var initSecuritySelect = function(){
     var default_security_width = '100px';
 
@@ -63,6 +69,9 @@ SocialStream.Wall = (function(SS, $, undefined){
               chosen.results_update_field();
             }
           }
+          changeSecurityImage('public');
+        } else {
+          changeSecurityImage('restricted');
         }
       });
 
@@ -145,6 +154,6 @@ SocialStream.Wall = (function(SS, $, undefined){
     addInitCallback: addInitCallback,
       init: init,
       activateAntiRebounds: activateAntiRebounds,
-      unblockForms: unblockForms  
+      unblockForms: unblockForms
   };
 }) (SocialStream, jQuery)

data/base/app/assets/stylesheets/activities.css.scss.erb

@@ -27,7 +27,7 @@
 
 #security_chzn .chzn-choices {
 	padding-left: 20px;
-	background: url('btn/btn_security.png') no-repeat 10px center;
+	background: url('btn/restricted.png') no-repeat 10px center;
 }
 /******************* WALL - ACTIVITY ***************/
 .super_activity{width: 95%;}
@@ -55,7 +55,6 @@
 
 .security, .post_time_ago, .verb_comment, .verb_like, .verb_comment, .verb_delete{ display: inline-block;}
 
-
 /************* ACTIVITY - FOOTER *********/
 .space_comments {	padding-top: 1px; 	padding-bottom: 1px;}
 .space_activities {	text-align:center;	padding: 0px 0px 0px 1px;	width:100%;}
@@ -88,3 +87,15 @@
 .input_new_comments_container{min-height: 30px; position: relative;  }
 
 .activity_likes{ margin-bottom: 2px; }
+
+.timeline-130 {
+  width: 130px;
+  margin: 10px;
+  float: left;
+}
+
+.audience_logo {
+  padding: 2px;
+  display: inline-block;
+}
+

data/base/app/assets/stylesheets/base.css.scss

@@ -137,7 +137,7 @@ div.chzn-container ul.chzn-choices li.search-field input{ cursor:pointer;}
 
 /*************MENU ICON SECTION**********/
 .menu_header{  border-bottom: thin solid $separation-color;  width:14.5em;  padding-left:3px;  padding-right:3px;}
-.menu_icon{	vertical-align: middle; padding-bottom: 3px;	padding-right: 3px;  padding-left: 3px;	
+.menu_icon{	vertical-align: middle; padding-bottom: 3px;	padding-right: 2px;  padding-left: 2px;	
             display: inline-block;}
 
 /*************SPACES SECTION *************/

data/base/app/controllers/audience_controller.rb

@@ -0,0 +1,16 @@
+class AudienceController < ApplicationController
+  before_filter :read_activity
+
+  respond_to :js
+
+  def index
+  end
+
+  private
+
+  def read_activity
+    @activity = Activity.find params[:activity_id]
+
+    authorize! :read, @activity
+  end
+end

data/base/app/models/activity.rb

@@ -1,21 +1,32 @@
 # Activities follow the {Activity Streams}[http://activitystrea.ms/] standard.
 #
-# == {Activity Activities}, {Channel Channels} and Audiences
-# Every activity is attached to a {Channel}, which defines the sender and the receiver of the {Activity}
+# Every {Activity} has an {#author}, {#user_author} and {#owner}
 #
-# Besides, the activity is attached to one or more relations, which define the audience of the activity,
-# in other words, the {actors Actor} that can reach it and their {permissions Permission}
+# author:: Is the {SocialStream::Models::Subject subject} that originated
+#          the activity. The entity that posted something, liked, etc..
 #
-# == Wall
-# The Activity.wall(args) scope provides all the activities appearing in a wall
+# user_author:: The {User} logged in when the {Activity} was created.
+#               If the {User} has not changed the session to represent
+#               other entity (a {Group} for example), the user_author
+#               will be the same as the author.
 #
-# There are two types of wall, :home and :profile. Check {Actor#wall} for more information
+# owner:: The {SocialStream::Models::Subject subject} whose wall was posted
+#         or comment was liked, etc..
+#
+# == {Audience Audiences} and visibility
+# Each activity is attached to one or more {Relation relations}, which define
+# the {SocialStream::Models::Subject subject} that can reach the activity
+#
+# In the case of a {Relation::Public public relation} everyone will be
+# able to see the activity.
+#
+# In the case of {Relation::Custom custom relations}, only the subjects
+# that have a {Tie} with that relation (in other words, the contacts that
+# have been added as friends with that relation} will be able to reach the {Activity}
 #
-include NotificationsHelper
-
 class Activity < ActiveRecord::Base
-  # See {SocialStream::Models::Channeled}
-  channeled
+  # FIXME: this does not follow the Rails way
+  include NotificationsHelper
 
   # This has to be declared before 'has_ancestry' to work around rails issue #670
   # See: https://github.com/rails/rails/issues/670
@@ -27,6 +38,13 @@ class Activity < ActiveRecord::Base
 
   belongs_to :activity_verb
 
+  belongs_to :author,
+             :class_name => "Actor"
+  belongs_to :owner,
+             :class_name => "Actor"
+  belongs_to :user_author,
+             :class_name => "Actor"
+
   has_many :audiences, :dependent => :destroy
   has_many :relations, :through => :audiences
 
@@ -35,10 +53,26 @@ class Activity < ActiveRecord::Base
   has_many :activity_objects,
            :through => :activity_object_activities
 
+  scope :authored_by, lambda { |subject|
+    where(:author_id => Actor.normalize_id(subject))
+  }
+  scope :owned_by, lambda { |subject|
+    where(:owner_id => Actor.normalize_id(subject))
+  }
+  scope :authored_or_owned_by, lambda { |subjects|
+    ids = Actor.normalize_id(subjects)
+
+    where(arel_table[:author_id].in(ids).or(arel_table[:owner_id].in(ids)))
+  }
+
+  scope :shared_with, lambda { |subject|
+    joins(:audiences).
+      merge(Audience.where(:relation_id => Relation.ids_shared_with(subject)))
+  }
+
   scope :wall, lambda { |args|
     q =
       select("DISTINCT activities.*").
-      joins(:channel).
       joins(:audiences).
       joins(:relations).
       roots
@@ -48,14 +82,13 @@ class Activity < ActiveRecord::Base
             where('activity_objects.object_type' => args[:object_type])
     end
 
-    channels   = Channel.arel_table
     audiences  = Audience.arel_table
     relations  = Relation.arel_table
 
     owner_conditions =
-      channels[:author_id].eq(Actor.normalize_id(args[:owner])).
-        or(channels[:user_author_id].eq(Actor.normalize_id(args[:owner]))).
-        or(channels[:owner_id].eq(Actor.normalize_id(args[:owner])))
+      arel_table[:author_id].eq(Actor.normalize_id(args[:owner])).
+        or(arel_table[:user_author_id].eq(Actor.normalize_id(args[:owner]))).
+        or(arel_table[:owner_id].eq(Actor.normalize_id(args[:owner])))
 
     audience_conditions =
       audiences[:relation_id].in(args[:relation_ids]).
@@ -65,16 +98,16 @@ class Activity < ActiveRecord::Base
       case args[:type]
       when :home
         followed_conditions =
-          channels[:author_id].in(args[:followed]).
-            or(channels[:owner_id].in(args[:followed]))
+          arel_table[:author_id].in(args[:followed]).
+            or(arel_table[:owner_id].in(args[:followed]))
 
         owner_conditions.
           or(followed_conditions.and(audience_conditions))
       when :profile
         if args[:for].present?
           visitor_conditions =
-            channels[:author_id].eq(Actor.normalize_id(args[:for])).
-              or(channels[:owner_id].eq(Actor.normalize_id(args[:for])))
+            arel_table[:author_id].eq(Actor.normalize_id(args[:for])).
+              or(arel_table[:owner_id].eq(Actor.normalize_id(args[:for])))
 
           owner_conditions.
             and(visitor_conditions.or(audience_conditions))
@@ -93,7 +126,7 @@ class Activity < ActiveRecord::Base
   after_create  :increment_like_count
   after_destroy :decrement_like_count, :delete_notifications
 
-  validates_presence_of :relations
+  validates_presence_of :author_id, :user_author_id, :owner_id, :relations
  
   #For now, it should be the last one
   #FIXME
@@ -109,12 +142,37 @@ class Activity < ActiveRecord::Base
     self.activity_verb = ActivityVerb[name]
   end
 
+  # The {SocialStream::Models::Subject subject} author
+  def author_subject
+    author.subject
+  end
+
+  # The {SocialStream::Models::Subject subject} owner
+  def owner_subject
+    owner.subject
+  end
+
+  # The {SocialStream::Models::Subject subject} user actor
+  def user_author_subject
+    user_author.subject
+  end
+
+  # Does this {Activity} have the same sender and receiver?
+  def reflexive?
+    author_id == owner_id
+  end
+
+  # Is the author represented in this {Activity}?
+  def represented_author?
+    author_id != user_author_id
+  end
+
   # The {Actor} author of this activity
   #
   # This method provides the {Actor}. Use {#sender_subject} for the {SocialStream::Models::Subject Subject}
   # ({User}, {Group}, etc..)
   def sender
-    channel.author
+    author
   end
 
   # The {SocialStream::Models::Subject Subject} author of this activity
@@ -122,7 +180,7 @@ class Activity < ActiveRecord::Base
   # This method provides the {SocialStream::Models::Subject Subject} ({User}, {Group}, etc...).
   # Use {#sender} for the {Actor}.
   def sender_subject
-    channel.author_subject
+    author_subject
   end
 
   # The wall where the activity is shown belongs to receiver
@@ -130,7 +188,7 @@ class Activity < ActiveRecord::Base
   # This method provides the {Actor}. Use {#receiver_subject} for the {SocialStream::Models::Subject Subject}
   # ({User}, {Group}, etc..)
   def receiver
-    channel.owner
+    owner
   end
 
   # The wall where the activity is shown belongs to the receiver
@@ -138,7 +196,7 @@ class Activity < ActiveRecord::Base
   # This method provides the {SocialStream::Models::Subject Subject} ({User}, {Group}, etc...).
   # Use {#receiver} for the {Actor}.
   def receiver_subject
-    channel.owner_subject
+    owner_subject
   end
 
   # The comments about this activity
@@ -152,7 +210,7 @@ class Activity < ActiveRecord::Base
   end
 
   def liked_by(user) #:nodoc:
-    likes.joins(:channel).merge(Channel.subject_authored_by(user))
+    likes.authored_by(user)
   end
 
   # Does user like this activity?
@@ -162,12 +220,11 @@ class Activity < ActiveRecord::Base
 
   # Build a new children activity where subject like this
   def new_like(subject, user)
-    channel = Channel.new :author      => Actor.normalize(subject),
-                          :user_author => Actor.normalize(user),
-                          :owner       => owner
-    a = children.new :verb => "like",
-                     :channel => channel,
-                     :relation_ids => self.relation_ids
+    a = children.new :verb           => "like",
+                     :author_id      => Actor.normalize_id(subject),
+                     :user_author_id => Actor.normalize_id(user),
+                     :owner_id       => owner_id,
+                     :relation_ids   => self.relation_ids
 
     if direct_activity_object.present?
       a.activity_objects << direct_activity_object
@@ -218,7 +275,7 @@ class Activity < ActiveRecord::Base
       participants.each do |p|
         p.notify(notification_subject, "Youre not supposed to see this", self) unless p == sender
       end
-    elsif ['like','follow','make-friend','post','update'].include? verb and !channel.reflexive?
+    elsif ['like','follow','make-friend','post','update'].include? verb and !reflexive?
       receiver.notify(notification_subject, "Youre not supposed to see this", self)
     end
     true
@@ -241,15 +298,15 @@ class Activity < ActiveRecord::Base
 
   # Is subject allowed to perform action on this {Activity}?
   def allow?(subject, action)
-    return false if channel.blank?
+    return false if author.blank?
 
     case action
     when 'create'
-      return false if subject.blank? || channel.author_id != Actor.normalize_id(subject)
+      return false if subject.blank? || author_id != Actor.normalize_id(subject)
 
       rels = Relation.normalize(relation_ids)
 
-      own_rels = rels.select{ |r| r.actor_id == channel.author_id }
+      own_rels = rels.select{ |r| r.actor_id == author_id }
       # Consider Relation::Single as own_relations
       own_rels += rels.select{ |r| r.is_a?(Relation::Single) }
 
@@ -268,12 +325,12 @@ class Activity < ActiveRecord::Base
 
       return false if subject.blank?
 
-      return true if [channel.author_id, channel.owner_id].include?(Actor.normalize_id(subject))
+      return true if [author_id, owner_id].include?(Actor.normalize_id(subject))
     when 'update'
-      return true if [channel.author_id, channel.owner_id].include?(Actor.normalize_id(subject))
+      return true if [author_id, owner_id].include?(Actor.normalize_id(subject))
     when 'destroy'
       # We only allow destroying to sender and receiver by now
-      return [channel.author_id, channel.owner_id].include?(Actor.normalize_id(subject))
+      return [author_id, owner_id].include?(Actor.normalize_id(subject))
     end
 
     Relation.
@@ -298,6 +355,21 @@ class Activity < ActiveRecord::Base
       allow?(subject, 'update')
   end
 
+  # Is this activity public?
+  def public?
+    relation_ids.include? Relation::Public.instance.id
+  end
+
+  # The {Actor Actors} this activity is shared with
+  def audience
+    raise "Cannot get the audience of a public activity!" if public?
+
+    [ author, user_author, owner ].uniq |
+      Actor.
+        joins(:received_ties).
+        merge(Tie.where(:relation_id => relation_ids))
+  end
+
   # The {Relation} with which activity is shared
   def audience_in_words(subject, options = {})
     options[:details] ||= :full

data/base/app/models/activity_object.rb

@@ -217,15 +217,7 @@ class ActivityObject < ActiveRecord::Base
 
     @valid_relations = true
 
-    self.relation_ids =
-      if SocialStream.relation_model == :custom
-        owner.
-          relations.
-          allowing('read', 'activity').
-          map(&:id)
-      else
-        Array.wrap Relation::Public.instance.id
-      end
+    self.relation_ids = owner.activity_relation_ids
   end
 
   # validate method

data/base/app/models/actor.rb

@@ -73,16 +73,6 @@ class Actor < ActiveRecord::Base
   has_many :relations,
            :dependent => :destroy
 
-  has_many :authored_channels,
-           :class_name => "Channel",
-           :foreign_key => :author_id,
-           :dependent => :destroy
-
-  has_many :owned_channels,
-           :class_name => "Channel",
-           :foreign_key => :owner_id,
-           :dependent => :destroy
-
   has_many :sent_actions,
            :class_name => "ActivityAction",
            :dependent  => :destroy
@@ -328,11 +318,6 @@ class Actor < ActiveRecord::Base
     ties_to(subject).with_permissions(action, object).any?
   end
 
-  # The {Channel} of this {Actor} to self (totally close!)
-  def self_channel
-    Channel.find_or_create_by_author_id_and_user_author_id_and_owner_id id, id, id
-  end
-
   # Return the {ActivityAction} model to an {ActivityObject}
   def action_to(activity_object)
     sent_actions.received_by(activity_object).first
@@ -391,12 +376,26 @@ class Actor < ActiveRecord::Base
         any?
   end
 
-  # An {Activity} can be shared with multiple {audicences Audience}, which corresponds to a {Relation}.
-  #
-  # This method returns all the {relations Relation} that this actor can use to broadcast an Activity
+  # The default {Relation Relations} for sharing an {Activity} owned
+  # by this {Actor}
+  def activity_relations
+    SocialStream.relation_model == :custom ?
+      relations.
+        allowing('read', 'activity') :
+      [ Relation::Public.instance ]
+  end
+
+  # The ids of the default {Relation Relations} for sharing an {Activity}
+  # owned by this {Actor}
+  def activity_relation_ids
+    activity_relations.map(&:id)
+  end
+
+  # This method returns all the {relations Relation} that subject can choose to broadcast an Activity in this {Actor}'s wall
   #
+  # See {Activity} on how they can be shared with multiple {audicences Audience}, which corresponds to a {Relation}.
   #
-  def activity_relations(subject, options = {})
+  def activity_relations_for(subject, options = {})
     if Actor.normalize(subject) == self
       return relation_customs + Array.wrap(Relation::Public.instance)
     else
@@ -404,9 +403,9 @@ class Actor < ActiveRecord::Base
     end
   end
 
-  # Are there any activity_relations present?
-  def activity_relations?(*args)
-    activity_relations(*args).any?
+  # Are {#activity_relations} available for subject?
+  def activity_relations_for?(subject, options = {})
+    activity_relations(subject, options).any?
   end
 
   # Is this {Actor} allowed to create a comment on activity?
@@ -459,37 +458,30 @@ class Actor < ActiveRecord::Base
   #             the wall for members of the group.
   #             
   def wall(type, options = {})
-    args = {}
-
-    args[:type]  = type
-    args[:owner] = self
-    # Preserve this options
-    [ :for, :object_type ].each do |opt|
-      args[opt]   = options[opt]
-    end
+    options[:for] = self if type == :home
 
-    if type == :home
-      args[:followed] = Actor.followed_by(self).map(&:id)
-    end
+    wall =
+      Activity.
+        select("DISTINCT activities.*").
+        roots.
+        includes(:author, :user_author, :owner, :activity_objects, :activity_verb, :relations)
 
-    # TODO: this is not scalling for sure. We must use a fact table in the future
-    args[:relation_ids] =
+    actor_ids =
       case type
       when :home
-        # The relations from followings that can be read
-        Relation.allow(self, 'read', 'activity').map(&:id)
+        following_actor_and_self_ids
       when :profile
-        # FIXME: options[:relation]
-        #
-        # The relations that can be read by options[:for]
-        options[:for].present? ?
-          Relation.allow(options[:for], 'read', 'activity').map(&:id) :
-          []
+        id
       else
         raise "Unknown type of wall: #{ type }"
       end
 
-    Activity.wall args
+    wall = wall.authored_or_owned_by(actor_ids)
+
+    # Authentication
+    wall = wall.shared_with(options[:for])
+
+    wall = wall.order("created_at desc")
   end
  
   def logo
@@ -507,7 +499,7 @@ class Actor < ActiveRecord::Base
   end
   
   def liked_by(subject) #:nodoc:
-    likes.joins(:channel).merge(Channel.subject_authored_by(subject))
+    likes.authored_by(subject)
   end
   
   # Does subject like this {Actor}?
@@ -517,14 +509,11 @@ class Actor < ActiveRecord::Base
   
   # Build a new activity where subject like this
   def new_like(subject, user)
-    channel =
-      Channel.
-        find_or_create_by_author_id_and_user_author_id_and_owner_id Actor.normalize_id(subject),
-                                                                    Actor.normalize_id(user),
-                                                                    id
-    a = Activity.new :verb => "like",
-                     :channel => channel,
-                     :relation_ids => Array(Relation::Public.instance.id)
+    a = Activity.new :verb           => "like",
+                     :author_id      => Actor.normalize_id(subject),
+                     :user_author_id => Actor.normalize_id(user),
+                     :owner_id       => id,
+                     :relation_ids   => Array(Relation::Public.instance.id)
     
     a.activity_objects << activity_object