-<%= _('Are you sure you want to leave %s?') % @to_leave.name %> -
- -<% form_tag do %> - <%= hidden_field_tag(:confirmation, 1) %> - <%= submit_button(:ok, _("Yes, I want to leave.") % @to_leave.name) %> - <%= button(:cancel, _("No, I don't want."), :action => 'index') %> -<% end %> diff --git a/app/views/profile/leave.rhtml b/app/views/profile/leave.rhtml new file mode 100644 index 0000000..bd3937c --- /dev/null +++ b/app/views/profile/leave.rhtml @@ -0,0 +1,15 @@ ++<%= _('Are you sure you want to leave %s?') % profile.name %> +
+ +<% form_tag do %> + <%= hidden_field_tag(:confirmation, 1) %> + <%= submit_button(:ok, _("Yes, I want to leave.") % profile.name) %> + <% if logged_in? && request.xhr? %> + <%= lightbox_close_button(_("No, I don't want")) %> + <% else %> + <%= button(:cancel, _("No, I don't want."), profile.url) %> + <% end %> +<% end %> diff --git a/app/views/shared/join_community_popup.rhtml b/app/views/shared/join_community_popup.rhtml index 56540d8..0e131c2 100644 --- a/app/views/shared/join_community_popup.rhtml +++ b/app/views/shared/join_community_popup.rhtml @@ -3,9 +3,9 @@ <% button_bar do %> <% if logged_in? %> - <%= button(:ok, _('Yes'), { :controller => 'profile', :action => 'join', :profile => profile.identifier, :confirmation => '1'}, :method => :post) %> + <%= button(:ok, _('Yes'), profile.join_url.merge(:confirmation => '1'), :method => :post) %> <% else %> - <%= button(:ok, _('Yes'), :controller => 'profile', :action => 'join', :profile => profile.identifier) %> + <%= button(:ok, _('Yes'), profile.join_url) %> <% end %> <%= button_to_remote(:cancel, _('Not now'), :url => profile.url.merge({:controller => 'profile', :action => 'refuse_for_now', :profile => profile.identifier}), :loaded => '$("join-community-popup").hide()') %> <%= button(:cancel, _('No and don\'t ask again'), :controller => 'profile', :action => 'refuse_join', :profile => profile.identifier) if logged_in? %> diff --git a/lib/authenticated_system.rb b/lib/authenticated_system.rb index 7e73acb..70ded91 100644 --- a/lib/authenticated_system.rb +++ b/lib/authenticated_system.rb @@ -79,8 +79,8 @@ module AuthenticatedSystem # Store the URI of the current request in the session. # # We can return to this location by calling #redirect_back_or_default. - def store_location - session[:return_to] = request.request_uri + def store_location(location = request.request_uri) + session[:return_to] = location end # Redirect to the URI stored by the most recent store_location call or diff --git a/test/functional/cms_controller_test.rb b/test/functional/cms_controller_test.rb index be9ebb6..ffe1256 100644 --- a/test/functional/cms_controller_test.rb +++ b/test/functional/cms_controller_test.rb @@ -1121,4 +1121,5 @@ class CmsControllerTest < Test::Unit::TestCase get :view, :profile => profile.identifier, :id => profile.blog.id assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/upload_files?parent_id=#{profile.blog.id}"} end + end diff --git a/test/functional/memberships_controller_test.rb b/test/functional/memberships_controller_test.rb index 84d78a2..0556331 100644 --- a/test/functional/memberships_controller_test.rb +++ b/test/functional/memberships_controller_test.rb @@ -109,41 +109,7 @@ class MembershipsControllerTest < Test::Unit::TestCase community = Community.create!(:name => 'my test community', :description => 'description test') community.add_member(profile) get :index, :profile => profile.identifier - assert_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/memberships/leave/#{community.id}" }, :content => 'Leave' - end - - should 'present confirmation before leaving a profile' do - community = Community.create!(:name => 'my test community') - community.add_member(profile) - get :leave, :profile => profile.identifier, :id => community.id - - assert_response :success - assert_template 'leave' - end - - should 'actually leave profile' do - community = Community.create!(:name => 'my test community') - community.add_member(profile) - assert_includes profile.memberships, community - post :leave, :profile => profile.identifier, :id => community.id, :confirmation => '1' - - assert_response :redirect - assert_redirected_to :action => 'index' - - profile = Profile.find(@profile.id) - assert_not_includes profile.memberships, community - end - - should 'leave profile when on wizard' do - community = Community.create!(:name => 'my test community') - community.add_member(profile) - post :leave, :profile => profile.identifier, :id => community.id, :confirmation => '1', :wizard => true - - assert_response :redirect - assert_redirected_to :controller => 'search', :action => 'assets', :asset => 'communities', :wizard => true - - profile = Profile.find(@profile.id) - assert_not_includes profile.memberships, community + assert_tag :tag => 'a', :attributes => { :href => "/profile/#{community.identifier}/leave" }, :content => 'Leave' end should 'current user is added as admin after create new community' do diff --git a/test/functional/profile_controller_test.rb b/test/functional/profile_controller_test.rb index a6aa7b2..b593905 100644 --- a/test/functional/profile_controller_test.rb +++ b/test/functional/profile_controller_test.rb @@ -12,6 +12,7 @@ class ProfileControllerTest < Test::Unit::TestCase @profile = create_user('testuser').person end + attr_reader :profile def test_local_files_reference assert_local_files_reference @@ -224,7 +225,8 @@ class ProfileControllerTest < Test::Unit::TestCase community = Community.create!(:name => 'my test community') community.add_member(@profile) get :index, :profile => community.identifier - assert_tag :tag => 'a', :attributes => { :href => "/myprofile/#{@profile.identifier}/memberships/leave/#{community.id}" } + assert_tag :tag => 'a', + :attributes => { :href => "/profile/#{community.identifier}/leave" } end should 'not show Leave This Community button for non-registered users' do @@ -510,4 +512,126 @@ class ProfileControllerTest < Test::Unit::TestCase assert_equal ((2..10).to_a + [community.id]), @request.session[:no_asking] end + should 'present confirmation before leaving a profile' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + login_as(profile.identifier) + get :leave, :profile => community.identifier + + assert_template 'leave' + assert_tag :tag => 'input', :attributes => {:value => 'Yes, I want to leave.', :type => 'submit'} + end + + should 'actually leave profile' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + assert_includes profile.memberships, community + + login_as(profile.identifier) + post :leave, :profile => community.identifier, :confirmation => '1' + + profile = Profile.find(@profile.id) + assert_not_includes profile.memberships, community + end + + should 'leave profile when on wizard' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + login_as(profile.identifier) + post :leave, :profile => community.identifier, :confirmation => '1', :wizard => true + + assert_response :redirect + assert_redirected_to :controller => 'search', :action => 'assets', :asset => 'communities', :wizard => true + + profile = Profile.find(@profile.id) + assert_not_includes profile.memberships, community + end + + should "offer button to close 'leave community' lightbox" do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + login_as(profile.identifier) + get :index, :profile => community.identifier + + assert_tag :tag => 'a', :content => 'Leave', :attributes => { :href => "/profile/#{community.identifier}/leave", :class => /^lbOn/ } + end + + should 'offer button to cancel leaving community' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + login_as(profile.identifier) + get :leave, :profile => community.identifier + + assert_tag :tag => 'a', :content => "No, I don't want." + end + + should 'render without layout when use lightbox to leave community' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + @request.stubs(:xhr?).returns(true) + login_as(profile.identifier) + get :leave, :profile => community.identifier + + assert_no_tag :tag => 'body' # e.g. no layout + end + + should 'require login to leave community' do + community = Community.create!(:name => 'my test community') + get :leave, :profile => community.identifier + + assert_redirected_to :controller => 'account', :action => 'login' + end + + should 'redirect to stored location after leave community' do + community = Community.create!(:name => 'my test community') + community.add_member(profile) + + @request.session[:return_to] = "/profile/#{community.identifier}/to_go" + login_as(profile.identifier) + + post :leave, :profile => community.identifier, :confirmation => '1' + + assert_redirected_to "/profile/#{community.identifier}/to_go" + end + + should 'store referer location when request leave via get' do + community = Community.create!(:name => 'my test community') + login_as(profile.identifier) + + assert_nil @request.session[:return_to] + @request.expects(:referer).returns("/profile/redirect_to") + + get :leave, :profile => community.identifier + + assert_equal '/profile/redirect_to', @request.session[:return_to] + end + + should 'store referer location when request join via get' do + community = Community.create!(:name => 'my test community') + login_as(profile.identifier) + + assert_nil @request.session[:return_to] + @request.expects(:referer).returns("/profile/redirect_to") + + get :join, :profile => community.identifier + + assert_equal '/profile/redirect_to', @request.session[:return_to] + end + + should 'redirect to stored location after join community' do + community = Community.create!(:name => 'my test community') + + @request.session[:return_to] = "/profile/#{community.identifier}/to_go" + login_as(profile.identifier) + + post :join, :profile => community.identifier, :confirmation => '1' + + assert_redirected_to "/profile/#{community.identifier}/to_go" + end + end diff --git a/test/unit/profile_test.rb b/test/unit/profile_test.rb index d6799d1..a0e1775 100644 --- a/test/unit/profile_test.rb +++ b/test/unit/profile_test.rb @@ -1370,6 +1370,16 @@ class ProfileTest < Test::Unit::TestCase assert_equal "header customized", person.custom_header end + should 'provide URL to leave' do + profile = Profile.create!(:name => "Test Profile", :identifier => 'testprofile', :environment_id => create_environment('mycolivre.net').id) + assert_equal({ :profile => 'testprofile', :controller => 'profile', :action => 'leave'}, profile.leave_url) + end + + should 'provide URL to join' do + profile = Profile.create!(:name => "Test Profile", :identifier => 'testprofile', :environment_id => create_environment('mycolivre.net').id) + assert_equal({ :profile => 'testprofile', :controller => 'profile', :action => 'join'}, profile.join_url) + end + private def assert_invalid_identifier(id) -- libgit2 0.21.2