social_stream-base 0.18.1 → 0.19.0

data/db/migrate/20120111141717_create_social_stream.rb

@@ -0,0 +1,265 @@
+class CreateSocialStream < ActiveRecord::Migration
+  def change
+    create_table "activities", :force => true do |t|
+      t.integer  "activity_verb_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "ancestry"
+      t.integer  "channel_id"
+    end
+
+    add_index "activities", ["activity_verb_id"], :name => "index_activities_on_activity_verb_id"
+    add_index "activities", ["channel_id"], :name => "index_activities_on_channel_id"
+
+    create_table "activity_object_activities", :force => true do |t|
+      t.integer  "activity_id"
+      t.integer  "activity_object_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "object_type"
+    end
+
+    add_index "activity_object_activities", ["activity_id"], :name => "index_activity_object_activities_on_activity_id"
+    add_index "activity_object_activities", ["activity_object_id"], :name => "index_activity_object_activities_on_activity_object_id"
+
+    create_table "activity_objects", :force => true do |t|
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "object_type", :limit => 45
+      t.integer  "like_count",                :default => 0
+      t.integer  "channel_id"
+    end
+
+    add_index "activity_objects", ["channel_id"], :name => "index_activity_objects_on_channel_id"
+
+    create_table "activity_verbs", :force => true do |t|
+      t.string   "name",       :limit => 45
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    create_table "actors", :force => true do |t|
+      t.string   "name"
+      t.string   "email",              :default => "",   :null => false
+      t.string   "slug"
+      t.string   "subject_type"
+      t.boolean  "notify_by_email",    :default => true
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.integer  "activity_object_id"
+      t.integer  "follower_count",     :default => 0
+    end
+
+    add_index "actors", ["activity_object_id"], :name => "index_actors_on_activity_object_id"
+    add_index "actors", ["email"], :name => "index_actors_on_email"
+    add_index "actors", ["slug"], :name => "index_actors_on_slug", :unique => true
+
+    create_table "audiences", :force => true do |t|
+      t.integer "relation_id"
+      t.integer "activity_id"
+    end
+
+    add_index "audiences", ["activity_id"], :name => "index_audiences_on_activity_id"
+    add_index "audiences", ["relation_id"], :name => "index_audiences_on_relation_id"
+
+    create_table "authentications", :force => true do |t|
+      t.integer  "user_id"
+      t.string   "provider"
+      t.string   "uid"
+      t.datetime "created_at", :null => false
+      t.datetime "updated_at", :null => false
+    end
+
+    add_index "authentications", ["user_id"], :name => "index_authentications_on_user_id"
+
+    create_table "avatars", :force => true do |t|
+      t.integer  "actor_id"
+      t.string   "logo_file_name"
+      t.string   "logo_content_type"
+      t.integer  "logo_file_size"
+      t.datetime "logo_updated_at"
+      t.boolean  "active",            :default => true
+    end
+
+    add_index "avatars", ["actor_id"], :name => "index_avatars_on_actor_id"
+
+    create_table "channels", :force => true do |t|
+      t.integer  "author_id"
+      t.integer  "owner_id"
+      t.integer  "user_author_id"
+      t.datetime "created_at",     :null => false
+      t.datetime "updated_at",     :null => false
+    end
+
+    add_index "channels", ["author_id"], :name => "index_channels_on_author_id"
+    add_index "channels", ["owner_id"], :name => "index_channels_on_owner_id"
+    add_index "channels", ["user_author_id"], :name => "index_channels_on_user_author_id"
+
+    create_table "comments", :force => true do |t|
+      t.integer  "activity_object_id"
+      t.text     "text"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    add_index "comments", ["activity_object_id"], :name => "index_comments_on_activity_object_id"
+
+    create_table "contacts", :force => true do |t|
+      t.integer  "sender_id"
+      t.integer  "receiver_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.integer  "inverse_id"
+      t.integer  "ties_count",  :default => 0
+    end
+
+    add_index "contacts", ["inverse_id"], :name => "index_contacts_on_inverse_id"
+    add_index "contacts", ["receiver_id"], :name => "index_contacts_on_receiver_id"
+    add_index "contacts", ["sender_id"], :name => "index_contacts_on_sender_id"
+
+    create_table "groups", :force => true do |t|
+      t.integer  "actor_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    add_index "groups", ["actor_id"], :name => "index_groups_on_actor_id"
+
+    create_table "permissions", :force => true do |t|
+      t.string   "action"
+      t.string   "object"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    create_table "posts", :force => true do |t|
+      t.integer  "activity_object_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.text     "text"
+    end
+
+    add_index "posts", ["activity_object_id"], :name => "index_posts_on_activity_object_id"
+
+    create_table "profiles", :force => true do |t|
+      t.integer  "actor_id"
+      t.date     "birthday"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "organization", :limit => 45
+      t.string   "phone",        :limit => 45
+      t.string   "mobile",       :limit => 45
+      t.string   "fax",          :limit => 45
+      t.string   "address"
+      t.string   "city"
+      t.string   "zipcode",      :limit => 45
+      t.string   "province",     :limit => 45
+      t.string   "country",      :limit => 45
+      t.integer  "prefix_key"
+      t.string   "description"
+      t.string   "experience"
+      t.string   "website"
+      t.string   "skype",        :limit => 45
+      t.string   "im",           :limit => 45
+    end
+
+    add_index "profiles", ["actor_id"], :name => "index_profiles_on_actor_id"
+
+    create_table "relation_permissions", :force => true do |t|
+      t.integer  "relation_id"
+      t.integer  "permission_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    add_index "relation_permissions", ["permission_id"], :name => "index_relation_permissions_on_permission_id"
+    add_index "relation_permissions", ["relation_id"], :name => "index_relation_permissions_on_relation_id"
+
+    create_table "relations", :force => true do |t|
+      t.integer  "actor_id"
+      t.string   "type"
+      t.string   "name"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+      t.string   "sender_type"
+      t.string   "receiver_type"
+      t.string   "ancestry"
+    end
+
+    add_index "relations", ["actor_id"], :name => "index_relations_on_actor_id"
+    add_index "relations", ["ancestry"], :name => "index_relations_on_ancestry"
+
+    create_table "ties", :force => true do |t|
+      t.integer  "contact_id"
+      t.integer  "relation_id"
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    add_index "ties", ["contact_id"], :name => "index_ties_on_contact_id"
+    add_index "ties", ["relation_id"], :name => "index_ties_on_relation_id"
+
+    create_table "users", :force => true do |t|
+      t.string   "encrypted_password",     :limit => 128, :default => "",     :null => false
+      t.string   "password_salt"
+      t.string   "reset_password_token"
+      t.datetime "reset_password_sent_at"
+      t.datetime "remember_created_at"
+      t.integer  "sign_in_count",                         :default => 0
+      t.datetime "current_sign_in_at"
+      t.datetime "last_sign_in_at"
+      t.string   "current_sign_in_ip"
+      t.string   "last_sign_in_ip"
+      t.string   "authentication_token"
+      t.datetime "created_at",                                                :null => false
+      t.datetime "updated_at",                                                :null => false
+      t.integer  "actor_id"
+      t.string   "language"
+    end
+
+    add_index "users", ["actor_id"], :name => "index_users_on_actor_id"
+    add_index "users", ["reset_password_token"], :name => "index_users_on_reset_password_token", :unique => true
+
+    add_foreign_key "activities", "activity_verbs", :name => "index_activities_on_activity_verb_id"
+    add_foreign_key "activities", "channels", :name => "index_activities_on_channel_id"
+
+    add_foreign_key "activity_object_activities", "activities", :name => "index_activity_object_activities_on_activity_id"
+    add_foreign_key "activity_object_activities", "activity_objects", :name => "activity_object_activities_on_activity_object_id"
+
+    add_foreign_key "activity_objects", "channels", :name => "index_activity_objects_on_channel_id"
+
+    add_foreign_key "actors", "activity_objects", :name => "actors_on_activity_object_id"
+
+    add_foreign_key "audiences", "activities", :name => "audiences_on_activity_id"
+    add_foreign_key "audiences", "relations", :name => "audiences_on_relation_id"
+
+    add_foreign_key "authentications", "users", :name => "authentications_on_user_id"
+
+    add_foreign_key "avatars", "actors", :name => "avatars_on_actor_id"
+
+    add_foreign_key "channels", "actors", :name => "index_channels_on_author_id", :column => "author_id"
+    add_foreign_key "channels", "actors", :name => "index_channels_on_owner_id", :column => "owner_id"
+    add_foreign_key "channels", "actors", :name => "index_channels_on_user_author_id", :column => "user_author_id"
+
+    add_foreign_key "comments", "activity_objects", :name => "comments_on_activity_object_id"
+
+    add_foreign_key "contacts", "actors", :name => "contacts_on_receiver_id", :column => "receiver_id"
+    add_foreign_key "contacts", "actors", :name => "contacts_on_sender_id", :column => "sender_id"
+
+    add_foreign_key "groups", "actors", :name => "groups_on_actor_id"
+
+    add_foreign_key "posts", "activity_objects", :name => "posts_on_activity_object_id"
+
+    add_foreign_key "profiles", "actors", :name => "profiles_on_actor_id"
+
+    add_foreign_key "relation_permissions", "permissions", :name => "relation_permissions_on_permission_id"
+    add_foreign_key "relation_permissions", "relations", :name => "relation_permissions_on_relation_id"
+
+    add_foreign_key "relations", "actors", :name => "relations_on_actor_id"
+
+    add_foreign_key "ties", "contacts", :name => "ties_on_contact_id"
+    add_foreign_key "ties", "relations", :name => "ties_on_relation_id"
+
+    add_foreign_key "users", "actors", :name => "users_on_actor_id"
+  end
+end

data/db/migrate/20120326083509_object_channels_to_actions.rb

@@ -0,0 +1,45 @@
+class ObjectChannelsToActions < ActiveRecord::Migration
+  def up
+    add_column :activity_actions, :author, :boolean, :default => false
+    add_column :activity_actions, :user_author, :boolean, :default => false
+    add_column :activity_actions, :owner, :boolean, :default => false
+
+    ActivityAction.reset_column_information
+    ActivityAction.record_timestamps = false
+    ActivityObject.record_timestamps = false
+
+    ActivityObject.all.each do |ao|
+      channel = Channel.find ao.channel_id
+
+      %w{ author user_author owner }.each do |role|
+        next unless channel.__send__ "#{ role }_id"
+
+        ao.__send__"#{ role }_id=", channel.__send__("#{ role }_id")
+      end
+
+      ao.received_actions.each do |a|
+        a.created_at = a.updated_at = ao.created_at
+      end
+
+      ao.save!
+    end
+
+    remove_foreign_key :activity_objects, :name => "index_activity_objects_on_channel_id"
+    remove_column :activity_objects, :channel_id
+
+    ActivityObject.reset_column_information
+
+    ActivityAction.record_timestamps = true
+    ActivityObject.record_timestamps = true
+  end
+
+  def down
+    remove_column :activity_actions, :author
+    remove_column :activity_actions, :user_author
+    remove_column :activity_actions, :owner
+
+    add_column :activity_objects, :channel_id, :integer
+    add_index  :activity_objects, :channel_id
+    add_foreign_key :activity_actions, :channel_id, :name => "index_activity_objects_on_channel_id"
+  end
+end

data/lib/social_stream/ability/base.rb

@@ -22,11 +22,11 @@ module SocialStream
           end
 
           can :update, klass do |k| # can :update, Post do |post|
-            [k.channel.author_id, k.channel.owner_id].include?(Actor.normalize_id(subject))
+            [k.author_id, k.owner_id].include?(Actor.normalize_id(subject))
           end
 
           can :destroy, klass do |k| # can :destroy, Post do |post|
-            [k.channel.author_id, k.channel.owner_id].include?(Actor.normalize_id(subject))
+            [k.author_id, k.owner_id].include?(Actor.normalize_id(subject))
           end
         end
 

data/lib/social_stream/base/version.rb

@@ -1,5 +1,5 @@
 module SocialStream
   module Base
-    VERSION = "0.18.1".freeze
+    VERSION = "0.19.0".freeze
   end
 end

data/lib/social_stream/controllers/cancan_devise_integration.rb

@@ -3,7 +3,7 @@ module SocialStream
     # Common methods added to ApplicationController
     module CancanDeviseIntegration
       extend ActiveSupport::Concern
-      
+
       private
 
       # Catch some authorization errors:
@@ -17,14 +17,18 @@ module SocialStream
         if user_signed_in?
           if params[:s].present? && controller_name != 'home'
             redirect_to :home
-          else
-            raise exception
+            return
           end
         else
-          redirect_to new_user_session_path
+          if request.get?
+            session["user_return_to"] = request.fullpath
+            redirect_to new_user_session_path
+            return
+          end
         end
+
+        raise exception
       end
     end
   end
 end
-

data/lib/social_stream/models/object.rb

@@ -5,15 +5,9 @@ module SocialStream
       extend ActiveSupport::Concern
 
       included do
-        attr_writer   :_relation_ids
-        attr_accessor :_activity_parent_id
-
         subtype_of :activity_object,
                    :build => { :object_type => to_s }
 
-        has_one  :channel, :through => :activity_object
-        has_many :activity_object_activities, :through => :activity_object
-
         unless self == Actor
           validates_presence_of :author_id, :owner_id, :user_author_id
 
@@ -28,72 +22,6 @@ module SocialStream
             merge(ActivityObject.authored_by(subject))
         }
       end
-
-      # All the activities with this object
-      def activities
-        Activity.
-          includes(:activity_objects => self.class.to_s.underscore).
-          where("#{ self.class.quoted_table_name }.id" => self.id)
-      end
-
-      # Build the post activity when this object is not saved
-      def build_post_activity
-        Activity.new :channel      => channel!,
-                     :relation_ids => Array(_relation_ids)
-      end
-
-      def _contact
-        @_contact ||= author && owner && author.contact_to!(owner)
-      end
-
-      def _contact_id
-        _contact.try(:id)
-      end
-
-      def _relation_ids
-        @_relation_ids ||=
-          if _contact_id.nil?
-            nil
-          else
-            # FIXME: repeated in Activity#fill_relations
-            if _contact.reflexive?
-              _contact.sender.relation_customs.map(&:id)
-            else
-               _contact.
-                 receiver.
-                 relation_customs.
-                 allow(_contact.sender, 'create', 'activity').
-                 map(&:id)
-            end
-          end
-      end
-
-      def _activity_parent
-        @_activity_parent ||= Activity.find(_activity_parent_id)
-      end
-
-      private
-
-      def create_post_activity
-        create_activity "post"
-      end
-
-      def create_update_activity
-        return if _contact_id.blank?
-        
-        create_activity "update"
-      end
-
-      def create_activity(verb)
-        a = Activity.new :verb         => verb,
-                         :channel      => channel,
-                         :relation_ids => _relation_ids,
-                         :parent_id    => _activity_parent_id
-
-        a.activity_objects << activity_object
-
-        a.save!
-      end
     end
   end
 end

data/lib/social_stream/models/subject.rb

@@ -22,6 +22,7 @@ module SocialStream
         subtype_of :actor,
                    :build => { :subject_type => to_s }
         
+        has_one :activity_object, :through => :actor
         has_one :profile, :through => :actor
         
         validates_presence_of :name
@@ -48,7 +49,7 @@ module SocialStream
 
         scope :followed, lambda { 
           joins(:actor).
-            order("actors.follower_count DESC")
+            merge(Actor.followed)
         }
 
         scope :liked, lambda { 

data/lib/social_stream/models/subtype.rb

@@ -53,7 +53,12 @@ module SocialStream #:nodoc:
         # the subtype. Example: user.foo should raise "foo is not defined in user"
         # and not "in actor"
         rescue NameError => supertype_error
-          raise subtype_error
+          if supertype_error.name == subtype_error.name &&
+               supertype_error.message =~ /#{ self.class.supertype_name.to_s.classify }/
+            raise subtype_error
+          else
+            raise supertype_error
+          end
         end
       end
 

data/spec/controllers/groups_controller_spec.rb

@@ -13,6 +13,12 @@ describe GroupsController do
       assert_response :success
     end
 
+    it "should render index with most followed" do
+      get :index, :most => 'followed'
+
+      response.should be_success
+    end
+
     it "should render show" do
       get :show, :id => Factory(:group).to_param
 

data/spec/factories/post.rb

@@ -9,3 +9,9 @@ Factory.define :public_post, :parent => :post do |p|
   p.owner_id  { |q| q.author_id }
   p._relation_ids { |q| Array(Relation::Public.instance.id) }
 end
+
+Factory.define :self_post, :parent => :post do |p|
+  p.author_id { Factory(:user).actor_id }
+  p.owner_id  { |q| q.author_id }
+  p.user_author_id { |q| q.author_id }
+end

data/spec/models/activity_action_spec.rb

@@ -22,5 +22,40 @@ describe ActivityAction do
 
       action.reload.should_not be_follow
     end
+
+    describe "where posting to other owner" do
+      before do
+        @post = Factory(:post)
+      end
+
+      it "should not be duplicated" do
+        @post.received_actions.count.should == 2
+      end
+    end
+
+    describe "where posting to self" do
+      before do
+        @post = Factory(:self_post)
+      end
+
+      it "should not be duplicated" do
+        @post.received_actions.count.should == 1
+      end
+    end
+
+    describe "where building the post" do
+      before do
+        user = Factory(:user)
+        @post = Post.new :text => "Testing",
+                         :author => user,
+                         :owner  => user,
+                         :user_author => user
+        @post.save!
+      end
+
+      it "should not be duplicated" do
+        @post.received_actions.count.should == 1
+      end
+    end
   end
 end