X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/67a5809c8a486b5e8cc92dabd08624afc1a12e6c..5e7ab68721b2d7fe916690b8521bd94c60fbe583:/app/controllers/user_blocks_controller.rb?ds=sidebyside diff --git a/app/controllers/user_blocks_controller.rb b/app/controllers/user_blocks_controller.rb index e77f83999..7027359cc 100644 --- a/app/controllers/user_blocks_controller.rb +++ b/app/controllers/user_blocks_controller.rb @@ -70,8 +70,7 @@ class UserBlocksController < ApplicationController def update if @valid_params - if current_user != @user_block.creator && - current_user != @user_block.revoker + if cannot?(:update, @user_block) flash[:error] = t(@user_block.revoker ? ".only_creator_or_revoker_can_edit" : ".only_creator_can_edit") redirect_to :action => "edit" else @@ -81,7 +80,10 @@ class UserBlocksController < ApplicationController @user_block.ends_at = Time.now.utc + @block_period.hours @user_block.deactivates_at = (@user_block.ends_at unless @user_block.needs_view) @user_block.revoker = current_user if user_block_was_active && !@user_block.active? - if !user_block_was_active && @user_block.active? + if user_block_was_active && @user_block.active? && current_user != @user_block.creator + flash.now[:error] = t(".only_creator_can_edit_without_revoking") + render :action => "edit" + elsif !user_block_was_active && @user_block.active? flash.now[:error] = t(".inactive_block_cannot_be_reactivated") render :action => "edit" else