Changeset 6279 for trunk/qt4-gui/src/core/usermenu.cpp

Timestamp:

06/13/08 03:40:47 (6 months ago)

Author:

flynd

Message:

Use new user group handling function so we need less code here in the plugin.

Files:

• trunk/qt4-gui/src/core/usermenu.cpp (modified) (3 diffs)

trunk/qt4-gui/src/core/usermenu.cpp

@@ -537 +537 @@
 {
   unsigned int gid = action->data().toUInt();
-
-  ICQUser* u = gUserManager.FetchUser(myId.toLatin1(), myPpid, LOCK_W);
-
-  if (u == NULL)
-    return;
-
-  // Don't let the user to be deleted from local group
-  // if he is a member of the same server group
-  if (!action->isChecked() &&                             // delete
-      u->GetSID() &&                                      // server-side
-      gUserManager.GetGroupFromID(u->GetGSID()) == gid)   // same group
-  {
-    gUserManager.DropUser(u);
-    return;
-  }
-
-  u->SetInGroup(GROUPS_USER, gid, action->isChecked());
-
-  gUserManager.DropUser(u);
+  gUserManager.SetUserInGroup(myId.toLatin1(), myPpid, GROUPS_USER, gid,
+      action->isChecked(), false);
 
   // The daemon does not send an update when group membership changes
@@ -566 +549 @@
   unsigned int gid = action->data().toUInt();
 
-  ICQUser* u = NULL;
-  if (gid == GROUP_NEW_USERS || gid == GROUP_ONLINE_NOTIFY || gid == GROUP_IGNORE_LIST)
-  {
-    u = gUserManager.FetchUser(myId.toLatin1(), myPpid, LOCK_W);
+  if (gid == GROUP_IGNORE_LIST && !action->isChecked())
+  {
+    ICQUser* u = gUserManager.FetchUser(myId.toLatin1(), myPpid, LOCK_R);
     if (u == NULL)
       return;
-  }
-
-  switch (gid)
-  {
-    case GROUP_NEW_USERS:
-      u->SetNewUser(action->isChecked());
-      break;
-
-    case GROUP_ONLINE_NOTIFY:
-      u->SetOnlineNotify(action->isChecked());
-      break;
-
-    case GROUP_VISIBLE_LIST:
-      gLicqDaemon->ProtoToggleVisibleList(myId.toLatin1(), myPpid);
-      break;
-
-    case GROUP_INVISIBLE_LIST:
-      gLicqDaemon->ProtoToggleInvisibleList(myId.toLatin1(), myPpid);
-      break;
-
-    case GROUP_IGNORE_LIST:
-      if (!action->isChecked() ||
-          QueryYesNo(this, tr("Do you really want to add\n%1 (%2)\nto your ignore list?")
-            .arg(QString::fromUtf8(u->GetAlias())).arg(myId)))
-      {
-        u->SetIgnoreList(action->isChecked());
-        gUserManager.DropUser(u);
-        u = NULL;
-        gLicqDaemon->icqToggleIgnoreList(myId.toLatin1(), myPpid); // network only
-      }
-      break;
-  }
-
-  if (u != NULL)
+
+    QString alias = QString::fromUtf8(u->GetAlias());
     gUserManager.DropUser(u);
+
+    if(!QueryYesNo(this, tr("Do you really want to add\n%1 (%2)\nto your ignore list?")
+        .arg(alias).arg(myId)))
+      return;
+  }
+
+  gUserManager.SetUserInGroup(myId.toLatin1(), myPpid, GROUPS_SYSTEM, gid,
+      action->isChecked(), true);
 
   // The daemon does not send an update when group membership changes
@@ -615 +573 @@
 void UserMenu::setServerGroup(QAction* action)
 {
-  gUserManager.AddUserToGroup(myId.toLatin1(), myPpid, action->data().toUInt());
+  unsigned int gid = action->data().toUInt();
+  gUserManager.SetUserInGroup(myId.toLatin1(), myPpid, GROUPS_USER, gid, true, true);
 
   // The daemon does not send an update when group membership changes