From b194d99054fa82aa70394792a4eeef107c12e157 Mon Sep 17 00:00:00 2001
From: Gaston Dombiak <gaston@jivesoftware.com>
Date: Tue, 2 Dec 2003 01:53:39 +0000
Subject: [PATCH] Fixes a problem renaming a RosterGroup

git-svn-id: http://svn.igniterealtime.org/svn/repos/smack/trunk@2176 b35dd754-fafc-0310-a699-88a17e54d16e
---
 source/org/jivesoftware/smack/RosterGroup.java | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/source/org/jivesoftware/smack/RosterGroup.java b/source/org/jivesoftware/smack/RosterGroup.java
index 1f8a140ba..8e608aa38 100644
--- a/source/org/jivesoftware/smack/RosterGroup.java
+++ b/source/org/jivesoftware/smack/RosterGroup.java
@@ -92,18 +92,23 @@ public class RosterGroup {
     }
 
     /**
-     * Sets the name of the group.
+     * Sets the name of the group. Changing the group's name is like moving all the group entries
+     * of the group to a new group specified by the new name. Since this group won't have entries 
+     * it will be removed from the roster. This means that all the references to this object will 
+     * be invalid and will need to be updated to the new group specified by the new name.
      *
      * @param name the name of the group.
      */
     public void setName(String name) {
-        this.name = name;
         synchronized (entries) {
             for (int i=0; i<entries.size(); i++) {
                 RosterPacket packet = new RosterPacket();
                 packet.setType(IQ.Type.SET);
                 RosterEntry entry = (RosterEntry)entries.get(i);
-                packet.addRosterItem(RosterEntry.toRosterItem(entry));
+                RosterPacket.Item item = RosterEntry.toRosterItem(entry);
+                item.removeGroupName(this.name);
+                item.addGroupName(name);
+                packet.addRosterItem(item);
                 connection.sendPacket(packet);
             }
         }