Add share_with fancybox on search page, fix a couple of queries

app/controllers/aspects_controller.rb

@@ -11,7 +11,7 @@ class AspectsController < ApplicationController
   def index
     @posts  = current_user.visible_posts(:_type => "StatusMessage").paginate :page => params[:page], :per_page => 15, :order => 'created_at DESC'
     @post_hashes = hashes_for_posts @posts
-    @contacts = current_user.contacts(:pending => false)
+    @contacts = Contact.all(:user_id => current_user.id, :pending => false)
     @aspect_hashes = hashes_for_aspects @aspects.all, @contacts, :limit => 8
     @aspect = :all
 
@@ -62,7 +62,7 @@ class AspectsController < ApplicationController
     unless @aspect
       render :file => "#{Rails.root}/public/404.html", :layout => false, :status => 404
     else
-      @aspect_contacts = hashes_for_contacts @aspect.contacts(:pending => false)
+      @aspect_contacts = hashes_for_contacts Contact.all(:user_id => current_user.id, :aspect_ids.in => [@aspect.id], :pending => false)
       @aspect_contacts_count = @aspect_contacts.count
 
       @posts = @aspect.posts.find_all_by__type("StatusMessage", :order => 'created_at desc').paginate :page => params[:page], :per_page => 15

app/controllers/people_controller.rb

@@ -113,6 +113,22 @@ class PeopleController < ApplicationController
     end
   end
 
+  def share_with
+    @person = Person.find(params[:id].to_id)
+    @contact = current_user.contact_for(@person)
+    @aspects_with_person = []
+
+    if @contact
+      @aspects_with_person = @contact.aspects
+    end
+
+    @aspects_without_person = @aspects.reject do |aspect|
+      @aspects_with_person.include?(aspect)
+    end
+
+    render :layout => nil
+  end
+
   private
   def hashes_for_posts posts
     post_ids = posts.map{|p| p.id}

app/models/profile.rb

@@ -34,12 +34,12 @@ class Profile
   validates_length_of :first_name, :maximum => 32
   validates_length_of :last_name,  :maximum => 32
 
+
   before_save :strip_names
 
   attr_accessible :first_name, :last_name, :image_url, :image_url_medium,
     :image_url_small, :birthday, :gender, :bio, :searchable, :date
 
-
   def person
     self._parent_document
   end

app/views/people/_aspect_list.haml

@@ -37,23 +37,14 @@
       = link_to "edit aspect membership", "#", :id=> "edit_contact_aspects"
 
   .edit{:class => ("hidden" if contact)}
-    .contact_list#aspects_list
-      %ul
-        - for aspect in aspects_with_person
-          %li{:data=>{:guid=>aspect.id}}
-            %span.name
-              = link_to aspect.name, aspect
-            .right
-              = aspect_membership_button(aspect.id, contact, person)
 
-        - for aspect in aspects_without_person
-          %li{:data=>{:guid=>aspect.id}}
-            %span.name
-              = link_to aspect.name, aspect
-            .right
-              = aspect_membership_button(aspect.id, contact, person)
+    = render :partial => 'people/share_with_pane',
+        :locals => {:person => person,
+          :contact => contact,
+          :aspects_with_person => aspects_with_person,
+          :aspects_without_person => aspects_without_person}
 
+    - if contact
       .right
         = link_to "done editing", "#", :id => "done_contact_aspects"
-    - if contact
       = link_to t('people.profile_sidebar.remove_contact'), person, :confirm => t('are_you_sure'), :method => :delete

app/views/people/_person.html.haml

@@ -4,22 +4,27 @@
 
 %li.message{:id => person.id}
 
-  = person_image_link(person)
-
-  .content
-    %span.from
-      =person_link(person)
-
   .right{:style=>"display:inline;"}
     - if person.owner_id == current_user.id
       = t('.thats_you')
     - elsif contact && !contact.pending
       = t('.already_connected')
     - elsif (contact && contact.pending) || request
-          = link_to t('.pending_request'), aspects_manage_path
+      = t('.pending_request')
     - else
       - single_aspect_form ||= nil
-          = render(:partial => request_partial(single_aspect_form), :locals => {:aspects => aspects, :destination_handle => person.diaspora_handle})
+      = link_to "start sharing",
+        {:controller => "people",
+        :action => "share_with",
+        :id => person.id},
+        :class => 'button share_with_button'
+
+  = person_image_link(person)
+
+  .content
+    %span.from
+      =person_link(person)
+
 
     .info
       = person.diaspora_handle

app/views/people/index.html.haml

@@ -5,6 +5,9 @@
 - content_for :page_title do
   = t('search')
 
+- content_for :head do
+  = include_javascripts :people
+
 .span-24.last
   - if params[:q].blank?
     %h2

app/views/shared/_stream_element.html.haml

@@ -3,17 +3,17 @@
 -#   the COPYRIGHT file.
 
 %li.message{:data=>{:guid=>post.id}}
-  = person_image_link(person, :size => :thumb_small)
-
-  .content
-    .from
   - if person.owner_id == current_user.id
-        .right
+    .right.controls
       - reshare_aspects = aspects_without_post(aspects, post)
       - unless reshare_aspects.empty?
         = render 'shared/reshare', :aspects => reshare_aspects, :post => post
       = link_to t('delete'), status_message_path(post), :confirm => t('are_you_sure'), :method => :delete, :remote => true, :class => "delete"
 
+  = person_image_link(person, :size => :thumb_small)
+
+  .content
+    .from
       %h4
         =person_link(person)
 

config/assets.yml

@@ -28,6 +28,8 @@ javascripts:
     - public/javascripts/vendor/jquery-ui-1.8.6.custom.min.js
     - public/javascripts/aspect-edit.js
     - public/javascripts/contact-list.js
+  people:
+    - public/javascripts/contact-list.js
   photos:
     - public/javascripts/photo-show.js
 

config/routes.rb

@@ -8,6 +8,10 @@ Diaspora::Application.routes.draw do
   resources :requests,        :only => [:destroy, :create]
   resources :services
   resources :posts,           :only => [:show], :path => '/p/'
+
+
+
+  match '/people/share_with' => 'people#share_with', :as => 'share_with'
   resources :people do
     resources :status_messages
     resources :photos

public/javascripts/view.js

@@ -82,7 +82,8 @@ var View = {
       ".invite_user_button",
       ".add_photo_button",
       ".remove_person_button",
-      ".question_mark"
+      ".question_mark",
+      ".share_with_button"
     ]
   },
 

public/stylesheets/sass/application.sass

@@ -617,10 +617,12 @@ ul.comments
   a
     :padding 3px
 
-li.message .from .right
+li.message .right
   :display none
   :position absolute
   :right 12px
+
+  &.controls
     :font
       :size 12px
     :color #999
@@ -2041,3 +2043,22 @@ h3,h4
   .aspect_badge
     :top -0.2em
 
+#share_with
+  :min-width 400px
+  :max-width 400px
+
+  .share_with_header
+    :min-height 100px
+
+  .avatar
+    :width 100px
+    :height 100px
+    :float left
+  h3,p
+    :padding
+      :left 120px
+  h3
+    :margin 0
+
+#aspects_list
+  :height auto

spec/controllers/people_controller_spec.rb

@@ -14,6 +14,15 @@ describe PeopleController do
     sign_in :user, user
   end
 
+  describe '#share_with' do
+    before do
+      @person = Factory.create(:person)
+    end
+    it 'succeeds' do
+      get :share_with, :id => @person.id
+      response.should be_success
+    end
+  end
   describe '#hashes_from_people' do
     before do
       @everyone = []