app/models/unidom/authorization/concerns/as_permission.rb in unidom-authorization-1.3 vs app/models/unidom/authorization/concerns/as_permission.rb in unidom-authorization-1.4
- old
+ new
@@ -6,13 +6,12 @@
has_many :authorizings, class_name: 'Unidom::Authorization::Authorizing'
def authorize!(authorized, by: nil, at: Time.now)
- raise ArgumentError.new('The authorized argument is required.') if authorized.blank?
- raise ArgumentError.new('The by argument is required.' ) if by.blank?
- raise ArgumentError.new('The at argument is required.' ) if at.blank?
+ assert_present! :authorized, authorized
+ assert_present! :at, at
attributes = { authorized: authorized, opened_at: at }
if by.present?
attributes[:authorizer] = by
else
@@ -23,11 +22,11 @@
end
def authorize?(authorized, at: Time.now)
- raise ArgumentError.new('The authorized argument is required.') if authorized.blank?
- raise ArgumentError.new('The at argument is required.' ) if at.blank?
+ assert_present! :authorized, authorized
+ assert_present! :at, at
authorizings.authorized_is(authorized).valid_at(now: at).alive.exists?
end