diff options
author | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2019-11-08 06:26:33 +0000 |
---|---|---|
committer | Toshi MARUYAMA <marutosijp2@yahoo.co.jp> | 2019-11-08 06:26:33 +0000 |
commit | ea859aedca167730d2c64bb31e5f256875ff39e0 (patch) | |
tree | a3c9424b72c3c70a50f136d0f995f8318be5958a /app/models/role.rb | |
parent | 7f59b33e390c979fd3bdc4ff6f42123c4482a596 (diff) | |
download | redmine-ea859aedca167730d2c64bb31e5f256875ff39e0.tar.gz redmine-ea859aedca167730d2c64bb31e5f256875ff39e0.zip |
cleanup: rubocop: fix Layout/AlignArguments in app/models/role.rb
git-svn-id: http://svn.redmine.org/redmine/trunk@18947 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app/models/role.rb')
-rw-r--r-- | app/models/role.rb | 20 |
1 files changed, 12 insertions, 8 deletions
diff --git a/app/models/role.rb b/app/models/role.rb index dd1091ce7..5c4795c42 100644 --- a/app/models/role.rb +++ b/app/models/role.rb @@ -78,17 +78,21 @@ class Role < ActiveRecord::Base validates_presence_of :name validates_uniqueness_of :name validates_length_of :name, :maximum => 255 - validates_inclusion_of :issues_visibility, + validates_inclusion_of( + :issues_visibility, :in => ISSUES_VISIBILITY_OPTIONS.collect(&:first), - :if => lambda {|role| role.respond_to?(:issues_visibility) && role.issues_visibility_changed?} - validates_inclusion_of :users_visibility, + :if => lambda {|role| role.respond_to?(:issues_visibility) && role.issues_visibility_changed?}) + validates_inclusion_of( + :users_visibility, :in => USERS_VISIBILITY_OPTIONS.collect(&:first), - :if => lambda {|role| role.respond_to?(:users_visibility) && role.users_visibility_changed?} - validates_inclusion_of :time_entries_visibility, + :if => lambda {|role| role.respond_to?(:users_visibility) && role.users_visibility_changed?}) + validates_inclusion_of( + :time_entries_visibility, :in => TIME_ENTRIES_VISIBILITY_OPTIONS.collect(&:first), - :if => lambda {|role| role.respond_to?(:time_entries_visibility) && role.time_entries_visibility_changed?} + :if => lambda {|role| role.respond_to?(:time_entries_visibility) && role.time_entries_visibility_changed?}) - safe_attributes 'name', + safe_attributes( + 'name', 'assignable', 'position', 'issues_visibility', @@ -98,7 +102,7 @@ class Role < ActiveRecord::Base 'managed_role_ids', 'permissions', 'permissions_all_trackers', - 'permissions_tracker_ids' + 'permissions_tracker_ids') # Copies attributes from another role, arg can be an id or a Role def copy_from(arg, options={}) |