@@ -76,8 +76,8 class Member < ActiveRecord::Base | |||||
76 | end |
|
76 | end | |
77 | end |
|
77 | end | |
78 |
|
78 | |||
79 |
|
|
79 | # Set member role ids ignoring any change to roles that | |
80 |
|
|
80 | # user is not allowed to manage | |
81 | def set_editable_role_ids(ids, user=User.current) |
|
81 | def set_editable_role_ids(ids, user=User.current) | |
82 | ids = (ids || []).collect(&:to_i) - [0] |
|
82 | ids = (ids || []).collect(&:to_i) - [0] | |
83 | editable_role_ids = user.managed_roles(project).map(&:id) |
|
83 | editable_role_ids = user.managed_roles(project).map(&:id) | |
@@ -86,17 +86,17 class Member < ActiveRecord::Base | |||||
86 | self.role_ids = untouched_role_ids + touched_role_ids |
|
86 | self.role_ids = untouched_role_ids + touched_role_ids | |
87 | end |
|
87 | end | |
88 |
|
88 | |||
89 |
|
|
89 | # Returns true if one of the member roles is inherited | |
90 | def any_inherited_role? |
|
90 | def any_inherited_role? | |
91 | member_roles.any? {|mr| mr.inherited_from} |
|
91 | member_roles.any? {|mr| mr.inherited_from} | |
92 | end |
|
92 | end | |
93 |
|
93 | |||
94 |
|
|
94 | # Returns true if the member has the role and if it's inherited | |
95 | def has_inherited_role?(role) |
|
95 | def has_inherited_role?(role) | |
96 | member_roles.any? {|mr| mr.role_id == role.id && mr.inherited_from.present?} |
|
96 | member_roles.any? {|mr| mr.role_id == role.id && mr.inherited_from.present?} | |
97 | end |
|
97 | end | |
98 |
|
98 | |||
99 |
|
|
99 | # Returns true if the member's role is editable by user | |
100 | def role_editable?(role, user=User.current) |
|
100 | def role_editable?(role, user=User.current) | |
101 | if has_inherited_role?(role) |
|
101 | if has_inherited_role?(role) | |
102 | false |
|
102 | false | |
@@ -105,7 +105,7 class Member < ActiveRecord::Base | |||||
105 | end |
|
105 | end | |
106 | end |
|
106 | end | |
107 |
|
107 | |||
108 |
|
|
108 | # Returns true if the member is deletable by user | |
109 | def deletable?(user=User.current) |
|
109 | def deletable?(user=User.current) | |
110 | if any_inherited_role? |
|
110 | if any_inherited_role? | |
111 | false |
|
111 | false | |
@@ -114,14 +114,14 class Member < ActiveRecord::Base | |||||
114 | end |
|
114 | end | |
115 | end |
|
115 | end | |
116 |
|
116 | |||
117 |
|
|
117 | # Destroys the member | |
118 | def destroy |
|
118 | def destroy | |
119 | member_roles.reload.each(&:destroy_without_member_removal) |
|
119 | member_roles.reload.each(&:destroy_without_member_removal) | |
120 | super |
|
120 | super | |
121 | end |
|
121 | end | |
122 |
|
122 | |||
123 |
|
|
123 | # Returns true if the member is user or is a group | |
124 |
|
|
124 | # that includes user | |
125 | def include?(user) |
|
125 | def include?(user) | |
126 | if principal.is_a?(Group) |
|
126 | if principal.is_a?(Group) | |
127 | !user.nil? && user.groups.include?(principal) |
|
127 | !user.nil? && user.groups.include?(principal) |
General Comments 0
You need to be logged in to leave comments.
Login now