Remove group_type based on group class because we handle STI separa…

…tely
dwbutler · May 10, 2017 · aa8411c · aa8411c
1 parent 87fc7ec
commit aa8411c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/groupify/adapter/active_record/group.rb b/lib/groupify/adapter/active_record/group.rb
@@ -111,7 +111,7 @@ def merge!(source_group, destination_group)
           end
 
           source_group.transaction do
-            source_group.group_memberships_as_group.update_all(group_id: destination_group.id, group_type: destination_group.class.model_name.to_s)
+            source_group.group_memberships_as_group.update_all(group_id: destination_group.id)
             source_group.destroy
           end
         end
@@ -149,7 +149,7 @@ def <<(*args)
               to_add_directly << member unless include?(member)
               # add a second entry for the given membership type
               if membership_type
-                membership = member.group_memberships_as_member.where(group_id: group.id, group_type: group.class.model_name.to_s, membership_type: membership_type).first_or_initialize
+                membership = member.group_memberships_as_member.where(group_id: group.id, membership_type: membership_type).first_or_initialize
                 to_add_with_membership_type << membership unless membership.persisted?
               end
               member.__send__(:clear_association_cache)