Changeset 6284 for trunk/qt-gui/src/mainwin.cpp

Timestamp:

06/14/08 03:48:20 (7 months ago)

Author:

flynd

Message:

Updated to work with new group handling. Contact list is now updated when groups are changed from other plugins.

Files:

• trunk/qt-gui/src/mainwin.cpp (modified) (17 diffs)

trunk/qt-gui/src/mainwin.cpp

@@ -1484 +1484 @@
       if (m_bThreadView && m_nGroupType == GROUPS_USER && m_nCurrentGroup == 0)
       {
-        CUserViewItem* i = userView->firstChild();
-
-        while (i)
+        for (CUserViewItem* i = userView->firstChild(); i != NULL; i = i->nextSibling())
         {
-          if (u->GetInGroup(GROUPS_USER, i->GroupId()))
+          unsigned short groupId = i->GroupId();
+
+          // Should user be shown in this group
+          bool showInGroup = show_user(u) &
+              ((groupId != 0 && u->GetInGroup(GROUPS_USER, groupId)) ||
+              (groupId == 0 && u->GetGroups().empty() && !u->IgnoreList()));
+
+
+          // Remove old user item if present
+          for (CUserViewItem* it = i->firstChild(); it != NULL; it = it->nextSibling())
           {
-            CUserViewItem* it = i->firstChild();
-
-            while (it)
+            char* szRealId = 0;
+            ICQUser::MakeRealId(it->ItemId(), it->ItemPPID(), szRealId);
+            bool equal = strcasecmp(szRealId, szId) == 0 && it->ItemPPID() == nPPID;
+            delete [] szRealId;
+            if (equal)
             {
-              char *szRealId = 0;
-              ICQUser::MakeRealId(it->ItemId(), it->ItemPPID(), szRealId);
-              if(strcasecmp(szRealId, szId) == 0 && it->ItemPPID() == nPPID)
-              {
-                delete it;
-                delete [] szRealId;
-                if (show_user(u))
-                  (void) new CUserViewItem(u, i);
-                break;
-              }
-              delete [] szRealId;
-              it = it->nextSibling();
-            }
-            if (it == NULL)
-            {
-              if ( show_user(u) &
-                   ((i->GroupId() != 0 && u->GetInGroup(GROUPS_USER, i->GroupId())) ||
-                    (i->GroupId() == 0 && u->GetGroups(GROUPS_USER) == 0 && !u->IgnoreList())))
-                (void) new CUserViewItem(u, i);
+              delete it;
+              break;
             }
           }
-          i = i->nextSibling();
+
+          // If user should be shown in group, create a new item
+          if (showInGroup)
+            (void) new CUserViewItem(u, i);
         }
       }
@@ -1748 +1743 @@
     case LIST_GROUP_REMOVED:
     case LIST_GROUP_CHANGED:
+    case LIST_GROUP_REORDERED:
       updateGroups();
       updateUserWin();
@@ -1809 +1805 @@
   if (doGroupView)
   {
-    CUserViewItem* gi = new CUserViewItem(0, tr("Other Users").local8Bit(), userView);
+    CUserViewItem* gi = new CUserViewItem(0, tr("Other Users").local8Bit(), 65535, userView);
     gi->setOpen(m_nGroupStates & 1);
-    GroupList *g = gUserManager.LockGroupList(LOCK_R);
-    for (unsigned short i = 0; i < g->size(); i++)
-    {
-      gi = new CUserViewItem(i+1, (*g)[i], userView);
-      gi->setOpen(m_nGroupStates & (1<<QMIN(i+1, 31)));
-    }
-    gUserManager.UnlockGroupList();
+    int i = 1;
+    FOR_EACH_GROUP_START_SORTED(LOCK_R)
+    {
+      gi = new CUserViewItem(pGroup->id(), pGroup->name().c_str(), pGroup->sortIndex(), userView);
+      gi->setOpen(m_nGroupStates & (1<<QMIN(pGroup->id(), 31)));
+      ++i;
+    }
+    FOR_EACH_GROUP_END
   }
   FOR_EACH_USER_START(LOCK_R)
@@ -1835 +1832 @@
       {
         if((gi->GroupId() != 0 && pUser->GetInGroup(GROUPS_USER, gi->GroupId())) ||
-           (gi->GroupId() == 0 && pUser->GetGroups(GROUPS_USER) == 0 && !pUser->IgnoreList()))
+           (gi->GroupId() == 0 && pUser->GetGroups().empty() && !pUser->IgnoreList()))
           (void) new CUserViewItem(pUser, gi);
       }
@@ -1924 +1921 @@
 void CMainWindow::setCurrentGroup(int index)
 {
-  m_nCurrentGroup = index;
-  m_nGroupType = GROUPS_USER;
   unsigned short nNumGroups = gUserManager.NumGroups();
-  if (m_nCurrentGroup > nNumGroups)
-  {
-    m_nCurrentGroup -= nNumGroups;
+  if (index > nNumGroups)
+  {
+    m_nCurrentGroup = index - nNumGroups;
     m_nGroupType = GROUPS_SYSTEM;
+  }
+  else
+  {
+    m_nCurrentGroup = myGroupIds[index];
+    m_nGroupType = GROUPS_USER;
   }
   // Update the combo box
@@ -1959 +1959 @@
   mnuGroup->clear();
   mnuServerGroup->clear();
+  myGroupIds.clear();
 
   QString group = Strings::getSystemGroupName(GROUP_ALL_USERS);
@@ -1964 +1965 @@
   mnuUserGroups->insertItem(group);
   mnuUserGroups->insertSeparator();
+  myGroupIds.push_back(0);
 
   // take care of this first
@@ -1969 +1971 @@
   mnuGroup->insertSeparator();
 
-  GroupList *g = gUserManager.LockGroupList(LOCK_R);
-  for (unsigned short i = 0; i < g->size(); i++)
-  {
-    cmbUserGroups->insertItem(QString::fromLocal8Bit((*g)[i]));
-    mnuUserGroups->insertItem(QString::fromLocal8Bit((*g)[i]));
-    mnuGroup->insertItem(QString::fromLocal8Bit((*g)[i]), i+1);
-    mnuServerGroup->insertItem(QString::fromLocal8Bit((*g)[i]), i+1);
-  }
-  gUserManager.UnlockGroupList();
+  unsigned short i = 0;
+  FOR_EACH_GROUP_START_SORTED(LOCK_R)
+  {
+    myGroupIds.push_back(pGroup->id());
+    QString name = QString::fromLocal8Bit(pGroup->name().c_str());
+    cmbUserGroups->insertItem(name);
+    mnuUserGroups->insertItem(name);
+    mnuGroup->insertItem(name, i+1);
+    mnuServerGroup->insertItem(name, i+1);
+    ++i;
+  }
+  FOR_EACH_GROUP_END
   mnuUserGroups->insertSeparator();
   mnuGroup->insertSeparator();
@@ -1997 +2002 @@
   }
 
-  int index = m_nCurrentGroup;
+  unsigned short index = 0;
   if (m_nGroupType == GROUPS_SYSTEM)
-    index += gUserManager.NumGroups();
+    index = m_nCurrentGroup + gUserManager.NumGroups();
+  else
+    for (unsigned short i = 0; i < myGroupIds.size(); ++i)
+      if (myGroupIds[i] == m_nCurrentGroup)
+        index = i;
   setCurrentGroup(index);
 }
@@ -3270 +3279 @@
   mnuGroup->setItemChecked(1000+GROUP_NEW_USERS, u->NewUser());
 
-  GroupList *g = gUserManager.LockGroupList(LOCK_R);
-  for (unsigned short i = 0; i < g->size(); i++)
-    mnuGroup->setItemChecked(i+1, u->GetInGroup(GROUPS_USER, i+1));
-  gUserManager.UnlockGroupList();
+  for (unsigned short i = 0; i < myGroupIds.size()-1; i++)
+    mnuGroup->setItemChecked(i+1, u->GetInGroup(GROUPS_USER, myGroupIds[i+1]));
   gUserManager.DropUser(u);
 }
@@ -3282 +3289 @@
   bool add = mnuGroup->isItemChecked(id);
   GroupType gtype = (id < 1000 ? GROUPS_USER : GROUPS_SYSTEM);
-  unsigned short groupId = (id < 1000 ? id : id - 1000);
+  unsigned short groupId = (id < 1000 ? myGroupIds[id] : id - 1000);
 
   if (gtype == GROUPS_SYSTEM && groupId == GROUP_IGNORE_LIST && add)
@@ -3302 +3309 @@
     RemoveUserFromGroup(gtype, groupId, m_szUserMenuId, m_nUserMenuPPID, this);
   else
-  {
     gUserManager.SetUserInGroup(m_szUserMenuId, m_nUserMenuPPID,
         gtype, groupId, true, false);
-    updateUserWin();
-  }
 }
 
@@ -3326 +3330 @@
     gUserManager.DropUser(u);
 
-    GroupList* g = gUserManager.LockGroupList(LOCK_R);
+    LicqGroup* g = gUserManager.FetchGroup(group, LOCK_R);
     if (g == NULL)
       return true;
-    QString groupName = QString::fromLocal8Bit((*g)[group - 1]);
-    gUserManager.UnlockGroupList();
+    QString groupName = QString::fromLocal8Bit(g->name().c_str());
+    gUserManager.DropGroup(g);
 
     QString warning(tr("Are you sure you want to remove\n%1 (%2)\nfrom the '%3' group?")
@@ -3340 +3344 @@
 
   gUserManager.SetUserInGroup(id, ppid, gtype, group, false);
-  updateUserWin();
   return true;
 }
@@ -3352 +3355 @@
   for (uint index = 0; index < mnuServerGroup->count(); index++)
   {
-    const int id = mnuServerGroup->idAt(index);
-    const QCString text = mnuServerGroup->text(id).local8Bit();
+    int id = mnuServerGroup->idAt(index);
 
     bool checked = false;
-    if (u->GetSID() && (u->GetGSID() == gUserManager.GetIDFromGroup(text.data())))
+    if (u->GetSID() && (u->GetGSID() == gUserManager.GetIDFromGroup(myGroupIds[id])))
       checked = true;
 
@@ -3372 +3374 @@
   if (u == NULL) return;
 
-  GroupList *g = gUserManager.LockGroupList(LOCK_R);
-  for (unsigned int i = 0; i < g->size(); i++)
+  for (unsigned int i = 0; i < myGroupIds.size()-1; i++)
     mnuServerGroup->setItemChecked(i+1, (int)(i+1) == n);
 
-  gUserManager.UnlockGroupList();
   gUserManager.DropUser(u);
 
   gUserManager.SetUserInGroup(m_szUserMenuId, m_nUserMenuPPID, GROUPS_USER,
       n, true, true);
-  updateUserWin();
 }
 
@@ -4697 +4696 @@
 void CMainWindow::showEditGrpDlg()
 {
-  EditGrpDlg *d = new EditGrpDlg;
+  EditGrpDlg *d = new EditGrpDlg(licqSigMan);
   connect (d, SIGNAL(signal_updateGroups()), this, SLOT(updateGroups()));
   d->show();