Changeset 6372 for trunk/console/src/console_menu.cpp

Timestamp:

07/01/08 05:59:12 (5 months ago)

Author:

flynd

Message:

More uin usage removed from console plugin.

Files:

• trunk/console/src/console_menu.cpp (modified) (18 diffs)

trunk/console/src/console_menu.cpp

@@ -3 +3 @@
 
 #include <ctype.h>
+#include <string>
+
+using std::string;
 
 const unsigned short NUM_COMMANDS = 24;
@@ -539 +542 @@
     if (nStatus == ICQ_STATUS_OFFLINE)
     {
-      gUserManager.DropOwner(nPPID);
+      gUserManager.DropOwner(o);
       licqDaemon->ProtoLogoff(nPPID);
       continue;
@@ -548 +551 @@
     // call the right function
     bool b = o->StatusOffline();
-    gUserManager.DropOwner(nPPID);
+    gUserManager.DropOwner(o);
     if (b)
     {
@@ -591 +594 @@
  * CLicqConsole::GetUinFromArg
  *-------------------------------------------------------------------------*/
-unsigned long CLicqConsole::GetUinFromArg(char **p_szArg)
+string CLicqConsole::GetUserFromArg(char** p_szArg)
 {
   char *szAlias, *szCmd;
-  unsigned long nUin = 0;
+  string id;
   bool bCheckUin = true;
   char *szArg = *p_szArg;
@@ -611 +614 @@
     {
       winMain->wprintf("%CUnbalanced quotes.\n", COLOR_RED);
-      return (unsigned long)-1;
+      return "-";
     }
     *szCmd++ = '\0';
@@ -619 +622 @@
   {
     *p_szArg = NULL;
-    return gUserManager.OwnerUin();
+    return gUserManager.OwnerId(LICQ_PPID);
   }
   else if (szArg[0] == '$')
   {
     *p_szArg = NULL;
-    return winMain->nLastUin;
+    return winMain->myLastId;
   }
   else
@@ -639 +642 @@
   *p_szArg = szCmd;
 
-  // Find the user
-  // See if all the chars are digits
-  if (bCheckUin)
-  {
-    char *sz = szAlias;
-    while (isdigit(*sz)) sz++;
-    if (*sz == '\0') nUin = atol(szAlias);
-  }
-
-  if (nUin == 0)
-  {
-    FOR_EACH_PROTO_USER_START(LICQ_PPID, LOCK_R)
-    {
-      if (strcasecmp(szAlias, pUser->GetAlias()) == 0)
-      {
-        nUin = pUser->Uin();
-        FOR_EACH_USER_BREAK;
-      }
-    }
-    FOR_EACH_PROTO_USER_END
-    if (nUin == 0)
-    {
-      winMain->wprintf("%CInvalid user: %A%s\n", COLOR_RED, A_BOLD, szAlias);
-      return (unsigned long)-1;
-    }
-  }
-  else
-  {
-    if (!gUserManager.IsOnList(nUin))
-    {
-      winMain->wprintf("%CInvalid uin: %A%lu\n", COLOR_RED, A_BOLD, nUin);
-      return (unsigned long)-1;
-    }
+  FOR_EACH_PROTO_USER_START(LICQ_PPID, LOCK_R)
+  {
+    if (strcasecmp(szAlias, pUser->GetAlias()) == 0 ||
+        (bCheckUin && strcasecmp(szAlias, pUser->IdString()) == 0))
+    {
+      id = pUser->IdString();
+      FOR_EACH_USER_BREAK;
+    }
+  }
+  FOR_EACH_PROTO_USER_END
+
+  if (id.empty())
+  {
+    winMain->wprintf("%CInvalid user: %A%s\n", COLOR_RED, A_BOLD, szAlias);
+    return "-";
   }
 
   // Save this as the last user
-  if (winMain->nLastUin != nUin)
-  {
-    winMain->nLastUin = nUin;
+  if (winMain->myLastId != id)
+  {
+    winMain->myLastId = id;
     PrintStatus();
   }
 
-  return nUin;
+  return id;
 }
 
@@ -738 +723 @@
   {
     *p_szArg = NULL;
-    ICQOwner *o = gUserManager.FetchOwner(LOCK_R);
+    ICQOwner* o = gUserManager.FetchOwner(LICQ_PPID, LOCK_R);
     scon.szId = o->IdString();
-    scon.nPPID = o->PPID(); 
-    gUserManager.DropOwner();
+    scon.nPPID = o->PPID();
+    gUserManager.DropOwner(o);
     return scon;
   }
@@ -815 +800 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == gUserManager.OwnerUin())
+  string id = GetUserFromArg(&sz);
+
+  if (gUserManager.FindOwner(id.c_str(), LICQ_PPID) != NULL)
     winMain->wprintf("%CSetting personal info not implemented yet.\n", COLOR_RED);
-  else if (nUin == 0) {
-    char szUin[24];
-    sprintf(szArg, "%lu", gUserManager.OwnerUin());
-    UserCommand_Info(szUin, LICQ_PPID, sz);
-  } else if (nUin != (unsigned long)-1)
-    sprintf(szArg, "%lu", nUin);
-    UserCommand_Info(szArg, LICQ_PPID, sz);
+  else if (id.empty())
+    UserCommand_Info(gUserManager.OwnerId(LICQ_PPID).c_str(), LICQ_PPID, sz);
+  else if (id != "-")
+    UserCommand_Info(id.c_str(), LICQ_PPID, sz);
 }
 
@@ -836 +818 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == gUserManager.OwnerUin())
+  string id = GetUserFromArg(&sz);
+
+  if (gUserManager.FindOwner(id.c_str(), LICQ_PPID) != NULL)
     winMain->wprintf("%CYou can't send URLs to yourself!\n", COLOR_RED);
-  else if (nUin == 0)
+  else if (id.empty())
     winMain->wprintf("%CYou must specify a user to send a URL to.\n", COLOR_RED);
-  else if (nUin != (unsigned long)-1)
-    sprintf(szArg, "%lu", nUin);
-    UserCommand_Url(szArg, LICQ_PPID, sz);
-
+  else if (id != "-")
+    UserCommand_Url(id.c_str(), LICQ_PPID, sz);
 }
 
@@ -856 +836 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == 0)
+  string id = GetUserFromArg(&sz);
+
+  if (id.empty())
     winMain->wprintf("%CInvalid user\n", COLOR_RED);
-  else if (nUin != (unsigned long)-1)
-  {
-    sprintf(szArg, "%lu", nUin);
-    UserCommand_Sms(szArg, LICQ_PPID, sz);
-  }
+  else if (id != "-")
+    UserCommand_Sms(id.c_str(), LICQ_PPID, sz);
 }
 
@@ -887 +864 @@
     ICQOwner *o = gUserManager.FetchOwner(LICQ_PPID, LOCK_R);
     unsigned short nNumMsg = o->NewMessages();
-    gUserManager.DropOwner();
+    gUserManager.DropOwner(o);
     if (nNumMsg > 0)
     {
       //TODO which owner?
-      char szUin[24];
-      sprintf(szUin, "%lu", gUserManager.OwnerUin());
-      UserCommand_View(szUin, LICQ_PPID, NULL);
+      UserCommand_View(gUserManager.OwnerId(LICQ_PPID).c_str(), LICQ_PPID, NULL);
       return;
     }
@@ -927 +902 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == gUserManager.OwnerUin())
+  string id = GetUserFromArg(&sz);
+
+  if (gUserManager.FindOwner(id.c_str(), LICQ_PPID) != NULL)
     winMain->wprintf("%CYou can't establish a secure connection to yourself!\n", COLOR_RED);
-  else if (nUin == 0)
+  else if (id.empty())
     winMain->wprintf("%CYou must specify a user to talk to.\n", COLOR_RED);
-  else if (nUin != (unsigned long)-1)
-  {
-    sprintf(szArg, "%lu", nUin);
-    UserCommand_Secure(szArg, LICQ_PPID, sz);
-  }
+  else if (id != "-")
+    UserCommand_Secure(id.c_str(), LICQ_PPID, sz);
 }
 
@@ -947 +919 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == gUserManager.OwnerUin())
+  string id = GetUserFromArg(&sz);
+
+  if (gUserManager.FindOwner(id.c_str(), LICQ_PPID) != NULL)
     winMain->wprintf("%CYou can't send files to yourself!\n", COLOR_RED);
-  else if (nUin == 0)
+  else if (id.empty())
   {
     bool bNum = false;
@@ -970 +942 @@
     }
   }
-  else if (nUin != (unsigned long)-1)
-  {
-    sprintf(szArg, "%lu", nUin);
-    UserCommand_SendFile(szArg, LICQ_PPID, sz);
-  }
+  else if (id != "-")
+    UserCommand_SendFile(id.c_str(), LICQ_PPID, sz);
 }
 
@@ -984 +953 @@
 {
   char *sz = szArg;
-  unsigned long nUin = GetUinFromArg(&sz);
-
-  if (nUin == gUserManager.OwnerUin())
+  string id = GetUserFromArg(&sz);
+
+  if (gUserManager.FindOwner(id.c_str(), LICQ_PPID) != NULL)
   {
     wattron(winMain->Win(), A_BOLD);
@@ -995 +964 @@
     winMain->wprintf("%B%CAuto response:\n%b%s\n",
                      COLOR_WHITE, o->AutoResponse());
-    gUserManager.DropOwner();
+    gUserManager.DropOwner(o);
     wattron(winMain->Win(), A_BOLD);
     for (unsigned short i = 0; i < winMain->Cols() - 10; i++)
@@ -1003 +972 @@
     wattroff(winMain->Win(), A_BOLD);
   }
-  else if (nUin == 0)
+  else if (id.empty())
     UserCommand_SetAutoResponse(NULL, LICQ_PPID, sz);
-  else if (nUin != (unsigned long)-1)
+  else if (id != "-")
     UserCommand_FetchAutoResponse(NULL, LICQ_PPID, sz);
 }