photo uploader on edit profile page.

app/controllers/people_controller.rb

@@ -37,7 +37,6 @@ class PeopleController < ApplicationController
     @aspect  = :person_edit
     @person  = current_user.person
     @profile = @person.profile
-    @photos  = current_user.visible_posts(:person_id => @person.id, :_type => 'Photo').paginate :page => params[:page], :order => 'created_at DESC'
   end
 
   def update
@@ -64,15 +63,17 @@ class PeopleController < ApplicationController
 
   private
   def prep_image_url(params)
-    url = APP_CONFIG[:pod_url].dup
+    if params[:profile] && params[:profile][:image_url]
-    url.chop! if APP_CONFIG[:pod_url][-1,1] == '/'
+      url = APP_CONFIG[:pod_url].dup
-    if params[:profile][:image_url].empty?
+      url.chop! if APP_CONFIG[:pod_url][-1,1] == '/'
-      params[:profile].delete(:image_url)
+      if params[:profile][:image_url].empty?
-    else
+        params[:profile].delete(:image_url)
-      if /^http:\/\// =~ params[:profile][:image_url]
-        params[:profile][:image_url] = params[:profile][:image_url]
       else
-        params[:profile][:image_url] = url + params[:profile][:image_url]
+        if /^http:\/\// =~ params[:profile][:image_url]
+          params[:profile][:image_url] = params[:profile][:image_url]
+        else
+          params[:profile][:image_url] = url + params[:profile][:image_url]
+        end
       end
     end
   end

app/controllers/photos_controller.rb

@@ -51,6 +51,11 @@ class PhotosController < ApplicationController
 
       @photo = current_user.post(:photo, params[:photo])
 
+      # set image as profile photo
+      if params[:set_profile_image] && !params[:set_profile_image].empty?
+        current_user.update_profile(:image_url=>@photo.url(:thumb_medium))
+      end
+
       respond_to do |format|
         format.json{render(:layout => false , :json => {"success" => true, "data" => @photo}.to_json )}
       end

app/views/people/_profile_photo_upload.html.haml

@@ -0,0 +1,13 @@
+-#   Copyright (c) 2010, Diaspora Inc.  This file is
+-#   licensed under the Affero General Public License version 3 or later.  See
+-#   the COPYRIGHT file.
+
+
+#profile_photo_upload
+  = owner_image_tag
+  = render 'photos/new_photo', :album_id => nil, :aspect_id => 'all', :set_profile_image => true
+
+  %p
+    \...or select one from your already existing
+    = link_to 'photos', person_photos_path(@person)
+    \.

app/views/people/edit.html.haml

@@ -42,26 +42,7 @@
 
       %h4
         Your photo
-        %div#image_picker
+        = render 'people/profile_photo_upload'
-          = profile.hidden_field :image_url, :value => (@profile.image_url if @profile.image_url), :id => 'image_url_field'
-
-          - unless @photos.nil? || @photos.empty?
-            - for photo in @photos
-              - if @profile.image_url && @profile.image_url.include?(photo.url(:thumb_medium))
-                %div.small_photo{:id => photo.url(:thumb_medium), :class=>'selected'}
-                  = check_box_tag 'checked_photo', true, true
-                  = link_to image_tag(photo.url(:thumb_medium)), "#"
-              - else
-                %div.small_photo{:id => photo.url(:thumb_medium)}
-                  = check_box_tag 'checked_photo'
-                  = link_to image_tag(photo.url(:thumb_medium)), "#"
-
-          - else
-            =t('.you_dont_have_any_photos')
-            = link_to t('.albums'), albums_path(:aspect => 'all')
-            =t('.page_to_upload_some')
-
-        =will_paginate @photos
 
     .submit_block
       = link_to t('.cancel'), edit_user_path(current_user)

app/views/photos/_new_photo.haml

@@ -6,7 +6,7 @@
   function createUploader(){
     var uploader = new qq.FileUploader({
         element: document.getElementById('file-upload'),
-        params: {'photo' : { 'album_id' : "#{album_id}", 'to' : "#{aspect_id}"}},
+        params: {'photo' : { 'album_id' : "#{album_id}", 'to' : "#{aspect_id}"}, 'set_profile_image' : "#{set_profile_image if defined?(set_profile_image)}"},
         allowedExtensions: ['jpg', 'jpeg', 'png', 'gif'],
         action: "#{photos_path}"
     });

app/views/users/getting_started/_step_1.html.haml

@@ -33,26 +33,7 @@
 
     %h4
       Your photo
-    %div#image_picker
+      = render 'people/profile_photo_upload'
-      = profile.hidden_field :image_url, :value => (@profile.image_url if @profile.image_url), :id => 'image_url_field'
-
-      - unless @photos.nil? || @photos.empty?
-        - for photo in @photos
-          - if @profile.image_url && @profile.image_url.include?(photo.url(:thumb_medium))
-            %div.small_photo{:id => photo.url(:thumb_medium), :class=>'selected'}
-              = check_box_tag 'checked_photo', true, true
-              = link_to image_tag(photo.url(:thumb_medium)), "#"
-          - else
-            %div.small_photo{:id => photo.url(:thumb_medium)}
-              = check_box_tag 'checked_photo'
-              = link_to image_tag(photo.url(:thumb_medium)), "#"
-
-      - else
-        =t('.you_dont_have_any_photos')
-        = link_to t('.albums'), albums_path(:aspect => 'all')
-        =t('.page_to_upload_some')
-
-      =will_paginate @photos
 
     = hidden_field_tag :getting_started, @step
 

public/stylesheets/sass/application.sass

@@ -1377,3 +1377,17 @@ ul.aspects
   :display inline
   :left 600px
   :top 0
+
+#profile_photo_upload
+  :margin
+    :top 5px
+  img
+    :position absolute
+    :left 0
+    :height 100px
+    :width 100px
+    :border-radius 5px
+    :-webkit-box-shadow 0 1px 2px #666
+
+  :padding
+    :left 120px