nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge branch 'master' of github.com:diaspora/diaspora

Browse source
This commit is contained in:
Sarah Mei 2010-10-26 19:05:32 -07:00
commit 601bd91346
27 changed files with 329 additions and 279 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
README.md
View file

@ -20,7 +20,7 @@ You can find an introduction to the source code [here](http://github.com/diaspor
Bugs and pending features are on our [issue tracker](http://bugs.joindiaspora.com). Here are a few good places to start:
- Run "rake spec" to run our [Rspec](http://blog.davidchelimsky.net/2007/05/14/an-introduction-to-rspec-part-i/)
unit test suite. Take a look at the pending specs, make one pass!
unit test suite. [Here](http://github.com/diaspora/diaspora/wiki/Introduction-to-Our-Rspec-Convention) is an introduction to our Rspec convention. Take a look at the pending specs, make one pass!
- Run "rake cucumber" to run our [Cucumber](http://rubylearning.com/blog/2010/10/05/outside-in-development/)
integration test suite. As you can see, we need more integration tests. Pick a feature and write one!

2
app/controllers/aspects_controller.rb
View file

@ -48,7 +48,7 @@ class AspectsController < ApplicationController
render :file => "#{Rails.root}/public/404.html", :layout => false, :status => 404
else
@friends = @aspect.people
@posts = current_user.visible_posts( :by_members_of => @aspect ).paginate :per_page => 15, :order => 'created_at DESC'
@posts = current_user.visible_posts( :by_members_of => @aspect ).paginate :page => params[:page], :per_page => 15, :order => 'created_at DESC'
respond_with @aspect
end
end

36
app/controllers/people_controller.rb
View file

@ -23,8 +23,6 @@ class PeopleController < ApplicationController
@profile = @person.profile
@aspects_with_person = current_user.aspects_with_person(@person)
@posts = current_user.visible_posts(:person_id => @person.id).paginate :page => params[:page], :order => 'created_at DESC'
@latest_status_message = current_user.raw_visible_posts.find_all_by__type_and_person_id("StatusMessage", params[:id]).last
@post_count = @posts.count
respond_with @person
end
end
@ -34,4 +32,38 @@ class PeopleController < ApplicationController
respond_with :location => root_url
end
def edit
@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
prep_image_url(params[:person])
if current_user.update_profile params[:person][:profile]
flash[:notice] = "Profile updated"
else
flash[:error] = "Failed to update profile"
end
redirect_to edit_person_path
end
private
def prep_image_url(params)
url = APP_CONFIG[:pod_url].dup
url.chop! if APP_CONFIG[:pod_url][-1,1] == '/'
if params[:profile][:image_url].empty?
params[:profile].delete(:image_url)
else
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

2
app/controllers/status_messages_controller.rb
View file

@ -10,7 +10,7 @@ class StatusMessagesController < ApplicationController
def create
public_flag = params[:status_message][:public]
public_flag.match(/(true)/) ? public_flag = true : public_flag = false
public_flag.to_s.match(/(true)/) ? public_flag = true : public_flag = false
params[:status_message][:public] = public_flag
data = clean_hash params[:status_message]

30
app/controllers/users_controller.rb
View file

@ -14,11 +14,8 @@ class UsersController < ApplicationController
respond_to :html
def edit
@aspect = :user_edit
@user = current_user
@person = @user.person
@profile = @user.person.profile
@photos = current_user.visible_posts(:person_id => current_user.person.id, :_type => 'Photo').paginate :page => params[:page], :order => 'created_at DESC'
@aspect = :user_edit
@user = current_user
end
def update
@ -32,16 +29,9 @@ class UsersController < ApplicationController
else
flash[:error] = "Password Change Failed"
end
else
prep_image_url(params[:user])
if @user.update_profile params[:user][:profile]
flash[:notice] = "Profile updated"
else
flash[:error] = "Failed to update profile"
end
end
redirect_to edit_user_path(@user)
redirect_to edit_user_path(@user)
end
def destroy
@ -100,19 +90,5 @@ class UsersController < ApplicationController
end
private
def prep_image_url(params)
url = APP_CONFIG[:pod_url].dup
url.chop! if APP_CONFIG[:pod_url][-1,1] == '/'
if params[:profile][:image_url].empty?
params[:profile].delete(:image_url)
else
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

2
app/models/request.rb
View file

@ -26,8 +26,6 @@ class Request
validates_presence_of :destination_url, :callback_url
before_validation :clean_link
scope :for_user, lambda{ |user| where(:destination_url => user.person.receive_url) }
def self.instantiate(options = {})
person = options[:from]
self.new(:destination_url => options[:to],

10
app/models/user.rb
View file

@ -32,11 +32,11 @@ class User
key :invites, Integer, :default => 5
key :invitation_token, String
key :invitation_sent_at, DateTime
key :inviter_ids, Array
key :friend_ids, Array
key :pending_request_ids, Array
key :visible_post_ids, Array
key :visible_person_ids, Array
key :inviter_ids, Array, :typecast => 'ObjectId'
key :friend_ids, Array, :typecast => 'ObjectId'
key :pending_request_ids, Array, :typecast => 'ObjectId'
key :visible_post_ids, Array, :typecast => 'ObjectId'
key :visible_person_ids, Array, :typecast => 'ObjectId'
key :invite_messages, Hash

5
app/views/layouts/application.html.haml
View file

@ -55,8 +55,9 @@
.avatar
= owner_image_tag
= link_to current_user.real_name, '#'
%li= link_to "profile", current_user.person
%li= link_to "settings", edit_user_path(current_user)
%li= link_to "view profile", current_user.person
%li= link_to "edit profile", edit_person_path(current_user.person)
%li= link_to "account settings", edit_user_path(current_user)
%li= link_to t('.logout.'), destroy_user_session_path
= render "shared/aspect_nav"

67
app/views/people/edit.html.haml Normal file
View file

@ -0,0 +1,67 @@
-# Copyright (c) 2010, Diaspora Inc. This file is
-# licensed under the Affero General Public License version 3 or later. See
-# the COPYRIGHT file.
#section_header
%h2
Settings
%ul#settings_nav
%li=link_to 'Profile', edit_person_path(current_user.person)
%li=link_to 'Account', edit_user_path(current_user)
%li=link_to 'Services', services_path
.span-19.prepend-5.last
%h2 Profile
= form_for @person do |person|
= person.error_messages
= person.fields_for :profile do |p|
%h3="#{t('.picture')}"
%div#image_picker
= p.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
%br
%h3="#{t('.info')}"
%p
%b
="#{t('.diaspora_username')}:"
= @person.diaspora_handle
%p
= p.label :first_name
= p.text_field :first_name, :value => @profile.first_name
%p
= p.label :last_name
= p.text_field :last_name, :value => @profile.last_name
.submit_block
= link_to t('.cancel'), edit_user_path(current_user)
= t('.or')
= person.submit t('.update_profile')
#content_bottom
.back
= link_to "⇧ #{t('.home')}", root_path

8
app/views/services/index.html.haml
View file

@ -6,8 +6,8 @@
%h2
Settings
%ul#settings_nav
%li=link_to 'Profile', '#', :class => 'profile'
%li=link_to 'Account', '#', :class => 'account'
%li=link_to 'Profile', edit_person_path(current_user.person)
%li=link_to 'Account', edit_user_path(current_user)
%li=link_to 'Services', services_path
.span-19.prepend-5.last
@ -23,6 +23,6 @@
= service.nickname
= link_to "disconnect", service, :confirm => "disconnect #{service.provider}?", :method => :delete
%h4= link_to "Connect to twitter", "/auth/twitter"
%h4= link_to "Connect to facebook", "/auth/facebook"
%h4= link_to "Connect to twitter", "/auth/twitter" if SERVICES['twitter']['consumer_key']!= ""
%h4= link_to "Connect to facebook", "/auth/facebook" if SERVICES['facebook']['app_id'] !=""

41
app/views/users/_account.haml
View file

@ -1,41 +0,0 @@
-# Copyright (c) 2010, Diaspora Inc. This file is
-# licensed under the Affero General Public License version 3 or later. See
-# the COPYRIGHT file.
%h2 Account
= link_to "invite friends", new_user_invitation_path(current_user)
%br
%br
%br
%h3 Change Password
= form_for @user do |f|
= f.error_messages
%p
= f.label :password, "New Password"
= f.password_field :password
%p
= f.label :password_confirmation
= f.password_field :password_confirmation
.submit_block
= link_to "Cancel", edit_user_path(current_user)
or
= f.submit 'Change password'
%h3 Export Data
= link_to "download my xml", users_export_path, :class => "button"
= link_to "download my photos", users_export_photos_path, :class => "button"
%br
%br
%br
%h3 Close Account
= link_to "Close Account", current_user,
:confirm => "Are you sure?", :method => :delete,
:class => "button"

58
app/views/users/_profile.haml
View file

@ -1,58 +0,0 @@
-# Copyright (c) 2010, Diaspora Inc. This file is
-# licensed under the Affero General Public License version 3 or later. See
-# the COPYRIGHT file.
%h2 Profile
= form_for @user do |f|
= f.error_messages
= f.fields_for :profile do |p|
%h3="#{t('.picture')}"
%div#image_picker
= p.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
%br
%h3="#{t('.info')}"
%p
%b
="#{t('.diaspora_username')}:"
= @user.diaspora_handle
%p
= p.label :first_name
= p.text_field :first_name, :value => @profile.first_name
%p
= p.label :last_name
= p.text_field :last_name, :value => @profile.last_name
.submit_block
= link_to t('.cancel'), edit_user_path(current_user)
= t('.or')
= f.submit t('.update_profile')
#content_bottom
.back
= link_to "⇧ #{t('.home')}", root_path

42
app/views/users/edit.html.haml
View file

@ -16,14 +16,44 @@
%h2
Settings
%ul#settings_nav
%li=link_to 'Profile', '#', :class => 'profile'
%li=link_to 'Account', '#', :class => 'account'
%li=link_to 'Profile', edit_person_path(current_user.person)
%li=link_to 'Account', edit_user_path(current_user)
%li=link_to 'Services', services_path
.span-19.prepend-5.last
#profile.settings_pane{:style=>"display:block;"}
= render 'users/profile'
%h2 Account
#account.settings_pane
= render 'users/account'
= link_to "invite friends", new_user_invitation_path(current_user)
%br
%br
%br
%h3 Change Password
= form_for @user do |f|
= f.error_messages
%p
= f.label :password, "New Password"
= f.password_field :password
%p
= f.label :password_confirmation
= f.password_field :password_confirmation
.submit_block
= link_to "Cancel", edit_user_path(current_user)
or
= f.submit 'Change password'
%h3 Export Data
= link_to "download my xml", users_export_path, :class => "button"
= link_to "download my photos", users_export_photos_path, :class => "button"
%br
%br
%br
%h3 Close Account
= link_to "Close Account", current_user,
:confirm => "Are you sure?", :method => :delete,
:class => "button"

8
config/deploy.rb
View file

@ -56,9 +56,9 @@ namespace :deploy do
run "ln -s -f #{shared_path}/app_config.yml #{current_path}/config/app_config.yml"
end
task :symlink_fb_config do
run "touch #{shared_path}/fb_config.yml"
run "ln -s -f #{shared_path}/fb_config.yml #{current_path}/config/fb_config.yml"
task :symlink_oauth_keys_config do
run "touch #{shared_path}/oauth_keys.yml"
run "ln -s -f #{shared_path}/oauth_keys.yml #{current_path}/config/oauth_keys.yml"
end
task :start do
@ -157,4 +157,4 @@ namespace :db do
end
after "deploy:symlink", "deploy:symlink_images", "deploy:symlink_bundle", 'deploy:symlink_config', 'deploy:symlink_fb_config'
after "deploy:symlink", "deploy:symlink_images", "deploy:symlink_bundle", 'deploy:symlink_config', 'deploy:symlink_oauth_keys_config'

24
config/locales/diaspora/en.yml
View file

@ -116,18 +116,6 @@ en:
users:
edit:
editing_profile: "Editing profile"
profile:
cancel: "Cancel"
update_profile: "Update Profile"
home: "Home"
diaspora_username: "DIASPORA USERNAME"
info: "Info"
picture: "Picture"
editing_profile: "Editing profile"
albums: "Albums"
you_dont_have_any_photos: "You don't have any photos! Go to the"
page_to_upload_some: "page to upload some."
or: "or"
destroy: "Account successfully closed."
comments:
comment:
@ -218,6 +206,18 @@ en:
are_you_sure: "Are you sure?"
remove_friend: "remove friend"
no_posts: "no posts to display!"
edit:
cancel: "Cancel"
update_profile: "Update Profile"
home: "Home"
diaspora_username: "DIASPORA USERNAME"
info: "Info"
picture: "Picture"
editing_profile: "Editing profile"
albums: "Albums"
you_dont_have_any_photos: "You don't have any photos! Go to the"
page_to_upload_some: "page to upload some."
or: "or"
requests:
new_request:
add_a_new_friend_to: "Add a new friend to"

39
config/locales/diaspora/fr-informal.yml
View file

@ -6,6 +6,15 @@
# See http://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
fr-informal:
activemodel:
errors:
models:
user:
attributes:
username:
taken: "est déjà pris"
email:
taken: "est déjà pris"
hello: "Bonjour tout le monde"
application:
helper:
@ -38,6 +47,7 @@ fr-informal:
share: "Partager"
aspect_friends:
add_friends: "ajouter des amis"
photos: "photos"
albums:
album:
you: "toi"
@ -69,10 +79,14 @@ fr-informal:
friends_albums: "Albums d’amis"
your_albums: "Tes albums"
aspects:
index:
photos: "photos"
show:
photos: "photos"
no_friends_message:
nobody: "Tu as forcément des amis, n’hésite pas à les ramener sur Diaspora !"
nobody_in_aspect: "Ton aspect '%{aspect_name}' est vide."
add_friend: "Ajouter un ami"
add_friend_to: "Ajouter quelqu’un à %{aspect_name}"
invite: "Inviter quelqu’un à rejoindre Diaspora !"
no_posts_message:
start_talking: "Personne n’a encore rien dit. Commence dès à présent la conversation !"
manage:
add_a_new_aspect: "Ajouter un nouvel aspect"
add_a_new_friend: "Ajouter un nouvel ami"
@ -90,12 +104,12 @@ fr-informal:
success: "%{name} a été supprimé avec succès."
update:
success: "Ton aspect, %{name}, a été édité avec succès."
move_friends:
failure: "L’édition de l’aspect concernant ton ami %{real_name} a échoué."
success: "Les aspects ont été édités avec succès."
move_friend:
failure: "ne fonctionne pas %{inspect}"
success: "Tu montres à présent à ton ami un aspect différent au tiens."
success: "Personne déplacée vers le nouvel aspect"
add_to_aspect:
failure: "L’ajout de l’ami vers l’aspect a échoué."
success: "Tu as ajouté l’ami à l’aspect avec succès."
helper:
remove: "supprimer"
aspect_not_empty: "L’aspect n’est pas vide"
@ -136,6 +150,8 @@ fr-informal:
photo:
show_comments: "afficher les commentaires"
posted_a_new_photo_to: "a publié une nouvelle photo dans"
delete: "Supprimer"
are_you_sure: "Es-tu sûr(e) ?"
new:
new_photo: "Nouvelle photo"
back_to_list: "Retourner à la liste"
@ -159,6 +175,7 @@ fr-informal:
sent: 'Ton invitation a été envoyée.'
no_more: 'Tu n’as plus d’invitation.'
already_sent: 'Tu as déjà invité cette personne.'
already_friends: 'Tu es déjà ami avec cette personne'
invitation_token_invalid: 'Le jeton d’invitation fourni n’est pas valide !'
updated: 'Ton mot de passe a été réglé avec succès. Tu es à présent connecté.'
@ -200,6 +217,7 @@ fr-informal:
save: "sauver"
are_you_sure: "Es-tu sûr(e) ?"
remove_friend: "supprimer de mes amis"
no_posts: "aucun message à afficher !"
requests:
new_request:
add_a_new_friend_to: "Ajouter un nouvel ami à"
@ -212,6 +230,9 @@ fr-informal:
ignore: "Requête d’ami ignorée."
create:
error: "Aucun résultat Diaspora n’a été trouvé avec cette adresse e-mail !"
already_friends: "Vous êtes déjà amis avec %{destination_url}!"
invalid_identity: "Cette identité n’est pas proprement formatée"
error_server: "Un problème est survenu avec l’autre serveur. Il n’existe peut-être pas ?"
yourself: "Tu ne peux pas être ami avec toi-même !"
already_friends: "Tu es déjà ami avec %{destination_url}!"
success: "Une requête d’ami a été envoyée à %{destination_url}."
horribly_wrong: "Quelque chose d’horrible s’est produit."

39
config/locales/diaspora/fr.yml
View file

@ -6,6 +6,15 @@
# See http://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
fr:
activemodel:
errors:
models:
user:
attributes:
username:
taken: "est déjà pris"
email:
taken: "est déjà pris"
hello: "Bonjour tout le monde"
application:
helper:
@ -38,6 +47,7 @@ fr:
share: "Partager"
aspect_friends:
add_friends: "ajouter des amis"
photos: "photos"
albums:
album:
you: "vous"
@ -69,10 +79,14 @@ fr:
friends_albums: "Albums d’amis"
your_albums: "Vos albums"
aspects:
index:
photos: "photos"
show:
photos: "photos"
no_friends_message:
nobody: "Vous avez forcément des amis, n’hésitez pas à les ramener sur Diaspora !"
nobody_in_aspect: "Votre aspect '%{aspect_name}' est vide."
add_friend: "Ajouter un ami"
add_friend_to: "Ajouter quelqu’un à %{aspect_name}"
invite: "Inviter quelqu’un à rejoindre Diaspora !"
no_posts_message:
start_talking: "Personne n’a encore rien dit. Commencez dès à présent la conversation !"
manage:
add_a_new_aspect: "Ajouter un nouvel aspect"
add_a_new_friend: "Ajouter un nouvel ami"
@ -90,12 +104,12 @@ fr:
success: "%{name} a été supprimé avec succès."
update:
success: "Votre aspect, %{name}, a été édité avec succès."
move_friends:
failure: "L’édition de l’aspect concernant votre ami %{real_name} a échoué."
success: "Les aspects ont été édités avec succès."
move_friend:
failure: "ne fonctionne pas %{inspect}"
success: "Vous montrez à présent à votre ami un aspect différent au vôtre."
success: "Personne déplacée vers le nouvel aspect"
add_to_aspect:
failure: "L’ajout de l’ami vers l’aspect a échoué."
success: "Vous avez ajouté l’ami vers l’aspect avec succès."
helper:
remove: "supprimer"
aspect_not_empty: "L’aspect n’est pas vide"
@ -136,6 +150,8 @@ fr:
photo:
show_comments: "afficher les commentaires"
posted_a_new_photo_to: "a publié une nouvelle photo dans"
delete: "Supprimer"
are_you_sure: "Êtes-vous sûr(e) ?"
new:
new_photo: "Nouvelle photo"
back_to_list: "Retourner à la liste"
@ -159,6 +175,7 @@ fr:
sent: 'Votre invitation a été envoyée.'
no_more: 'Vous n’avez plus d’invitation.'
already_sent: 'Vous avez déjà invité cette personne.'
already_friends: 'Vous êtes déjà ami avec cette personne'
invitation_token_invalid: 'Le jeton d’invitation fourni n’est pas valide !'
updated: 'Votre mot de passe a été réglé avec succès. Vous êtes à présent connecté.'
@ -200,6 +217,7 @@ fr:
save: "sauver"
are_you_sure: "Êtes-vous sûr(e) ?"
remove_friend: "supprimer de mes amis"
no_posts: "aucun message à afficher !"
requests:
new_request:
add_a_new_friend_to: "Ajouter un nouvel ami à"
@ -212,6 +230,9 @@ fr:
ignore: "Requête d’ami ignorée."
create:
error: "Aucun résultat Diaspora n’a été trouvé avec cette adresse e-mail !"
already_friends: "Vous êtes déjà amis avec %{destination_url}!"
invalid_identity: "Cette identité n’est pas proprement formatée"
error_server: "Un problème est survenu avec l’autre serveur. Il n’existe peut-être pas ?"
yourself: "Vous ne pouvez pas être ami avec vous-même !"
already_friends: "Vous êtes déjà ami avec %{destination_url}!"
success: "Une requête d’ami a été envoyée à %{destination_url}."
horribly_wrong: "Quelque chose d’horrible s’est produit."

2
config/routes.rb
View file

@ -3,7 +3,7 @@
# the COPYRIGHT file.
Diaspora::Application.routes.draw do
resources :people, :only => [:index, :show, :destroy]
resources :people
resources :status_messages, :only => [:create, :destroy, :show]
resources :comments, :except => [:index]
resources :requests, :except => [:edit, :update]

11
lib/diaspora/user/friending.rb
View file

@ -29,9 +29,8 @@ module Diaspora
end
def accept_friend_request(friend_request_id, aspect_id)
request = Request.find_by_id(friend_request_id)
pending_requests.delete(request)
request = pending_requests.find!(friend_request_id)
pending_request_ids.delete(request.id.to_id)
activate_friend(request.person, aspect_by_id(aspect_id))
request.reverse_for(self)
@ -45,16 +44,16 @@ module Diaspora
end
def accept_and_respond(friend_request_id, aspect_id)
requester = Request.find_by_id(friend_request_id).person
requester = pending_requests.find!(friend_request_id).person
reversed_request = accept_friend_request(friend_request_id, aspect_id)
dispatch_friend_acceptance reversed_request, requester
end
def ignore_friend_request(friend_request_id)
request = Request.find_by_id(friend_request_id)
request = pending_requests.find!(friend_request_id)
person = request.person
self.pending_requests.delete(request)
self.pending_request_ids.delete(request.id)
self.save
person.save

6
public/javascripts/aspect-edit.js
View file

@ -25,8 +25,12 @@ $(function() {
revert: true,
start: function(event,ui){
$(this).children("img").animate({'height':80, 'width':80, 'opacity':0.8},200);
$(this).children("img").tipsy("hide");
$(".draggable_info").fadeIn(100);
},
drag: function(event,ui){
$(this).children("img").tipsy("hide"); //ensure this is hidden
},
stop: function(event,ui){
$(this).children("img").animate({'height':70, 'width':70, 'opacity':1},200);
$(".draggable_info").fadeOut(100);
@ -88,7 +92,7 @@ $(function() {
'aspect_id' : person.attr('data-aspect_id') }
});
}
person.fadeOut('slow', $(this).remove());
person.fadeOut(400, function(){person.remove();});
}
}
});

1
public/javascripts/view.js
View file

@ -76,6 +76,7 @@ $(document).ready(function(){
$("img", "#left_pane").tipsy({live:true});
$(".add_aspect_button", "#aspect_nav").tipsy({gravity:'w'});
$(".person img", ".dropzone").tipsy({live:true});
});//end document ready

3
public/stylesheets/sass/application.sass
View file

@ -855,6 +855,9 @@ h1.big_text
:padding 4px 10px
:color #CCC
:background
:color #222
&:hover
:background
:color #000

33
spec/controllers/people_controller_spec.rb
View file

@ -6,22 +6,22 @@ require 'spec_helper'
describe PeopleController do
render_views
before do
@user = Factory.create(:user)
sign_in :user, @user
@user.aspect(:name => "lame-os")
let(:user) { Factory(:user) }
let!(:aspect) { user.aspect(:name => "lame-os") }
before do
sign_in :user, user
end
it "index should yield search results for substring of person name" do
eugene = Factory.create(:person, :profile => {:first_name => "Eugene", :last_name => "w"})
get :index, :q => "Eu"
assigns[:people].should include eugene
end
it 'should go to the current_user show page' do
get :show, :id => @user.person.id
get :show, :id => user.person.id
end
it "doesn't error out on an invalid id" do
@ -29,6 +29,25 @@ describe PeopleController do
end
it "doesn't error out on a nonexistent person" do
get :show, :id => @user.id
get :show, :id => user.id
end
describe '#update' do
context 'with a profile photo set' do
it "doesn't overwrite the profile photo when an empty string is passed in" do
user.person.profile.image_url = "http://tom.joindiaspora.com/images/user/tom.jpg"
user.person.profile.save
params = {"profile"=>
{"image_url" => "",
"last_name" => user.person.profile.last_name,
"first_name" => user.person.profile.first_name}}
image_url = user.person.profile.image_url
put("update", :id => user.person.id, "person" => params)
user.person.profile.image_url.should == image_url
end
end
end
end

72
spec/controllers/users_controller_spec.rb
View file

@ -5,10 +5,14 @@
require 'spec_helper'
describe UsersController do
let(:user) { Factory(:user) }
let!(:aspect) { user.aspect(:name => "lame-os") }
let!(:old_password) { user.encrypted_password }
before do
@user = Factory.create(:user)
sign_in :user, @user
@user.aspect(:name => "lame-os")
sign_in :user, user
end
describe '#export' do
@ -18,67 +22,29 @@ describe UsersController do
end
end
describe '#update' do
context 'with a profile photo set' do
before do
@user.person.profile.image_url = "http://tom.joindiaspora.com/images/user/tom.jpg"
@user.person.profile.save
@params = {"profile"=>
{"image_url" => "",
"last_name" => @user.person.profile.last_name,
"first_name" => @user.person.profile.first_name}}
end
it "doesn't overwrite the profile photo when an empty string is passed in" do
image_url = @user.person.profile.image_url
put("update", :id => @user.id, "user" => @params)
@user.person.profile.image_url.should == image_url
end
it "doesn't overwrite random attributes" do
new_user = Factory.create(:user)
@params[:owner_id] = new_user.id
person = @user.person
put('update', :id => @user.id, "user" => @params)
Person.find(person.id).owner_id.should == @user.id
end
it "doesn't overwrite random attributes" do
params = {:diaspora_handle => "notreal@stuff.com"}
proc{ put 'update', :id => user.id, "user" => params }.should_not change(user, :diaspora_handle)
end
context 'should allow the user to update their password' do
it 'should change a users password ' do
old_password = @user.encrypted_password
put("update", :id => @user.id, "user"=> {"password" => "foobaz", 'password_confirmation' => "foobaz","profile"=>
{"image_url" => "",
"last_name" => @user.person.profile.last_name,
"first_name" => @user.person.profile.first_name}})
@user.reload
@user.encrypted_password.should_not == old_password
put("update", :id => user.id, "user"=> {"password" => "foobaz", 'password_confirmation' => "foobaz"})
user.reload
user.encrypted_password.should_not == old_password
end
it 'should not change a password if they do not match' do
old_password = @user.encrypted_password
put("update", :id => @user.id, "user"=> {"password" => "foobarz", 'password_confirmation' => "not_the_same","profile"=>
{"image_url" => "",
"last_name" => @user.person.profile.last_name,
"first_name" => @user.person.profile.first_name}})
@user.reload
@user.encrypted_password.should == old_password
put("update", :id => user.id, "user"=> {"password" => "foobarz", 'password_confirmation' => "not_the_same"})
user.reload
user.encrypted_password.should == old_password
end
it 'should not update if the password fields are left blank' do
old_password = @user.encrypted_password
put("update", :id => @user.id, "user"=> {"password" => "", 'password_confirmation' => "","profile"=>
{"image_url" => "",
"last_name" => @user.person.profile.last_name,
"first_name" => @user.person.profile.first_name}})
@user.reload
@user.encrypted_password.should == old_password
put("update", :id => user.id, "user"=> {"password" => "", 'password_confirmation' => ""})
user.reload
user.encrypted_password.should == old_password
end
end
end

11
spec/models/request_spec.rb
View file

@ -31,17 +31,6 @@ describe Request do
xml.should include user.exported_key
end
it 'should allow me to see only friend requests sent to me' do
remote_person = Factory.build(:person, :diaspora_handle => "robert@grimm.com", :url => "http://king.com/")
Request.instantiate(:into => aspect.id, :from => user.person, :to => remote_person.receive_url).save
Request.instantiate(:into => aspect.id, :from => user.person, :to => remote_person.receive_url).save
Request.instantiate(:into => aspect.id, :from => user.person, :to => remote_person.receive_url).save
Request.instantiate(:into => aspect.id, :from => remote_person, :to => user.receive_url).save
Request.for_user(user).all.count.should == 1
end
it 'should strip the destination url' do
person_request = Request.new
person_request.destination_url = " http://google.com/ "

53
spec/models/user/user_friending_spec.rb
View file

@ -25,7 +25,7 @@ describe Diaspora::UserModules::Friending do
end
context 'friend requesting' do
it "should assign a request to a aspect" do
it "should assign a request to a aspect for the user that sent it out" do
aspect.requests.size.should == 0
user.send_friend_request_to(friend, aspect)
@ -34,21 +34,42 @@ describe Diaspora::UserModules::Friending do
aspect.requests.size.should == 1
end
it "should be able to accept a pending friend request" do
r = Request.instantiate(:to => user.receive_url, :from => friend)
r.save
describe '#receive_friend_request' do
it 'adds a request to pending if it was not sent by user' do
r = Request.instantiate(:to => user.receive_url, :from => friend)
r.save
user.receive_friend_request(r)
user.reload.pending_requests.should include r
end
it 'should autoaccept a request the user sent' do
request = user.send_friend_request_to(user2.person, aspect)
request.reverse_for(user2)
proc{user.receive_friend_request(request)}.should change(user.reload.friends, :count).by(1)
end
proc { user.accept_friend_request(r.id, aspect.id) }.should change {
Request.for_user(user).all.count }.by(-1)
end
it 'should be able to ignore a pending friend request' do
friend = Factory.create(:person)
r = Request.instantiate(:to => user.receive_url, :from => friend)
r.save
context 'received a friend request' do
proc { user.ignore_friend_request(r.id) }.should change {
Request.for_user(user).count }.by(-1)
let(:request_for_user) {Request.instantiate(:to => user.receive_url, :from => friend)}
let(:request2_for_user) {Request.instantiate(:to => user.receive_url, :from => person_one)}
before do
request_for_user.save
user.receive_friend_request(request_for_user)
user.receive_friend_request(request2_for_user)
user.reload
end
it "should delete an accepted friend request" do
proc { user.accept_friend_request(request2_for_user.id, aspect.id) }.should change(
user.reload.pending_requests, :count ).by(-1)
end
it 'should be able to ignore a pending friend request' do
proc { user.ignore_friend_request(request_for_user.id) }.should change (
user.reload.pending_requests, :count ).by(-1)
end
end
it 'should not be able to friend request an existing friend' do
@ -168,20 +189,20 @@ describe Diaspora::UserModules::Friending do
user.receive_friend_request @request
person_two.destroy
user.pending_requests.size.should be 1
user.reload.pending_requests.size.should be 1
user.friends.size.should be 0
user.receive_friend_request @request_two
user.pending_requests.size.should be 2
user.reload.pending_requests.size.should be 2
user.friends.size.should be 0
user.accept_friend_request @request.id, aspect.id
user.pending_requests.size.should be 1
user.reload.pending_requests.size.should be 1
user.friends.size.should be 1
user.friends.include?(person_one).should be true
user.ignore_friend_request @request_two.id
user.pending_requests.size.should be 0
user.reload.pending_requests.size.should be 0
user.friends.size.should be 1
user.friends.include?(person_two).should be false
end

1
spec/spec_helper.rb
View file

@ -67,6 +67,7 @@ ImageUploader.enable_processing = false
def friend_users(user1, aspect1, user2, aspect2)
request = user1.send_friend_request_to(user2.person, aspect1)
user2.receive_friend_request(request)
reversed_request = user2.accept_friend_request( request.id, aspect2.id)
user1.reload
user1.receive reversed_request.to_diaspora_xml, user2.person