Don't check the cache in getVCard since we don't call it so often anymore

converse.js

@@ -1043,7 +1043,6 @@
                 partner_jid = from;
                 resource = Strophe.getResourceFromJid(message_from);
             }
-
             chatbox = this.get(partner_jid);
             if (!chatbox) {
                 xmppchat.getVCard(
@@ -1222,19 +1221,6 @@
     });
 
     xmppchat.getVCard = function (jid, callback, errback) {
-        /* First we check if we don't already have a RosterItem, since it will
-         * contain all the vCard details.
-         */
-        var model = xmppchat.roster.getItem(jid);
-        if (model) {
-            callback(
-                model.get('jid'), 
-                model.get('fullname'), 
-                model.get('image'),
-                model.get('image_type'), 
-                model.get('url')
-            );
-        } else {
         xmppchat.connection.vcard.get($.proxy(function (iq) {
             $vcard = $(iq).find('vCard');
             var fullname = $vcard.find('FN').text(),
@@ -1244,7 +1230,6 @@
             callback(jid, fullname, img, img_type, url);
         }, this), jid, errback);
     }
-    }
 
     xmppchat.RosterItems = Backbone.Collection.extend({
         model: xmppchat.RosterItem,