Skip to content

Commit 7571152

Browse files
committed
fix issue adding users to groups
1 parent b260572 commit 7571152

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

app/models/group.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,8 @@ def self.user_trust_level_change!(user_id, trust_level)
7676
GroupUser.where(group_id: trust_group_ids, user_id: user_id).delete_all
7777

7878
if group = Group[name]
79-
group_users.build(user_id: user_id)
80-
group_users.save!
79+
group.group_users.build(user_id: user_id)
80+
group.save!
8181
else
8282
refresh_automatic_group!(name)
8383
end

spec/models/group_spec.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,11 @@
4747

4848
Group[:trust_level_1].user_ids.should == []
4949
Group[:trust_level_2].user_ids.should == [user.id]
50+
51+
user2 = Fabricate(:coding_horror)
52+
user2.change_trust_level!(:regular)
53+
54+
Group[:trust_level_2].user_ids.sort.should == [user.id, user2.id].sort
5055
end
5156

5257
end

0 commit comments

Comments
 (0)