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