Changeset 2994 for trunk/jmessenger/js/trophyim.js

Timestamp:

07/05/10 15:52:19 (14 years ago)

Author:

alexandrecorreia

Message:

Ticket #1116 - As vezes a operacao de remover um contato da lista nao funcionava direito.

File:

• trunk/jmessenger/js/trophyim.js (modified) (7 diffs)

trunk/jmessenger/js/trophyim.js

@@ -594 +594 @@
                                                 group_array[group_array.length] = groups[g].firstChild.nodeValue;
                                 }
-                                
-                                if( getAttribute('ask') )
-                                        TrophyIM.rosterObj.addContact(getAttribute('jid'), getAttribute('ask'), getAttribute('name'), group_array);
+
+                                if( getAttribute('ask') && getAttribute('ask').toString() === "subscribe" )
+                                {
+                                        if( getAttribute('subscription').toString() === "none" )
+                                        {
+                                                TrophyIM.rosterObj.addContact( getAttribute('jid'), getAttribute('ask'), getAttribute('name'), group_array );
+                                        }
+                                        
+                                        if( getAttribute('subscription').toString() === "remove" )
+                                        {
+                                                TrophyIM.rosterObj.removeContact(getAttribute('jid'));
+                                        }
+                                }
                                 else
-                                        TrophyIM.rosterObj.addContact(getAttribute('jid'), getAttribute('subscription'), getAttribute('name'), group_array);
+                                {
+                                        if( getAttribute('subscription').toString() === "remove" )
+                                        {
+                                                TrophyIM.rosterObj.removeContact(getAttribute('jid'));
+                                        }
+                                        else
+                                        {
+                                                TrophyIM.rosterObj.addContact( getAttribute('jid'), getAttribute('subscription'), getAttribute('name'), group_array );
+                                        }
+                                }
                         }
         }
@@ -1246 +1265 @@
                                                         case "from" :
                                                                 
-                                                                status          = " (( " + i18n.AUTH_ASK + " )) ";
+                                                                status          = " (( " + i18n.AUTHORIZED + " )) ";
                                                                 statusColor = "green";
                                                                 break;
@@ -1260 +1279 @@
                                                                 status          = " (( " + i18n.ASK_FOR_AUTH_QUESTION  + " )) ";
                                                                 statusColor     = "orange";     
+                                                                break;
+                                                                
+                                                        default :
+                                                                
                                                                 break;
                                                 }
@@ -1397 +1420 @@
                                 var _subscription = TrophyIM.rosterObj.roster[users[i]].contact.subscription;
                                 
-                                if( _subscription == "to" || _subscription == "not-in-roster")
+                                if( _subscription === "to" )
+                                {
+                                        flag = true;
+                                }
+                                
+                                if(  _subscription === "not-in-roster")
                                 {
                                         flag = true;
@@ -1458 +1486 @@
         var _id = TrophyIM.connection.getUniqueId();
         
-        TrophyIM.connection.send($pres( ).attrs( {to: jidTo, from: jidFrom, type: _typeSubscription, id: _id}).tree());
+        TrophyIM.connection.send($pres( ).attrs({ from: jidFrom, to: jidTo, type: _typeSubscription, id: _id }).tree());
         },
     
@@ -1696 +1724 @@
         this.addContact = function(jid, subscription, name, groups )
         {
-                switch( subscription )
-                {
-                        case 'remove'   :
-
-                                this.removeContact(jid);
-                                break;
-                                
-                        default :
-                                
-                                var contact             = { jid:jid, subscription:subscription, name:name, groups:groups }
-                        var jid_lower   = jid.toLowerCase();
-                
-                                if ( this.roster[jid_lower] )
-                                {
-                            this.roster[jid_lower]['contact'] = contact;
-                        }
-                                else
-                                {
-                            this.roster[jid_lower] = {contact:contact};
-                        }
-        
-                                groups = groups ? groups : [''];
-                        
-                                for ( var g = 0; g < groups.length; g++ )
-                                {
-                                        if ( !this.groups[groups[g]] )
-                                        {
-                                this.groups[groups[g]] = {};
-                            }
-                            
-                                        this.groups[groups[g]][jid_lower] = jid_lower;
-                        }
-                }
-                
-                /*if( subscription.indexOf("remove") >= 0 )
+                if( subscription === "remove" )
         {
                         this.removeContact(jid);
@@ -1759 +1753 @@
                                 this.groups[groups[g]][jid_lower] = jid_lower;
                 }
-        }*/
+        }
     }