X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/3d55db46ed24d563d82cf42f91fd064e40447c4d..506c0b5f0db8aca79ef61fa45cad508b7167817a:/app/models/user_block.rb?ds=inline diff --git a/app/models/user_block.rb b/app/models/user_block.rb index f8c05c92c..eb0daba65 100644 --- a/app/models/user_block.rb +++ b/app/models/user_block.rb @@ -4,32 +4,54 @@ class UserBlock < ActiveRecord::Base belongs_to :user, :class_name => "User", :foreign_key => :user_id belongs_to :creator, :class_name => "User", :foreign_key => :creator_id belongs_to :revoker, :class_name => "User", :foreign_key => :revoker_id - + PERIODS = USER_BLOCK_PERIODS + ## + # scope to match active blocks + def self.active + where("needs_view or ends_at > ?", Time.now.getutc) + end + + ## + # return a renderable version of the reason text. + def reason + RichText.new(self[:reason_format], self[:reason]) + end + ## # returns true if the block is currently active (i.e: the user can't # use the API). def active? - needs_view or ends_at > Time.now.getutc + needs_view || ends_at > Time.now.getutc + end + + ## + # returns true if the block is a "zero hour" block + def zero_hour? + # if the times differ more than 1 minute we probably have more important issues + needs_view && (ends_at.to_i - updated_at.to_i) < 60 end ## # revokes the block, allowing the user to use the API again. the argument # is the user object who is revoking the ban. def revoke!(revoker) - update_attributes({ :ends_at => Time.now.getutc(), - :revoker_id => revoker.id, - :needs_view => false }) + update_attributes( + :ends_at => Time.now.getutc, + :revoker_id => revoker.id, + :needs_view => false + ) end private + ## # validate that only moderators are allowed to change the # block. this should be caught and dealt with in the controller, # but i've also included it here just in case. def moderator_permissions - errors.add(:base, I18n.t('user_block.model.non_moderator_update')) if creator_id_changed? and !creator.moderator? - errors.add(:base, I18n.t('user_block.model.non_moderator_revoke')) unless revoker_id.nil? or revoker.moderator? + errors.add(:base, I18n.t("user_block.model.non_moderator_update")) if creator_id_changed? && !creator.moderator? + errors.add(:base, I18n.t("user_block.model.non_moderator_revoke")) unless revoker_id.nil? || revoker.moderator? end end