add debug for erase from AvatarHashMap, repair check for display name

assignment-client/src/Agent.cpp

@@ -132,7 +132,7 @@ void Agent::readPendingDatagrams() {
                 // let this continue through to the NodeList so it updates last heard timestamp
                 // for the sending audio mixer
                 NodeList::getInstance()->processNodeData(senderSockAddr, receivedPacket);
-            } else if (datagramPacketType == PacketTypeAvatarData
+            } else if (datagramPacketType == PacketTypeBulkAvatarData
                        || datagramPacketType == PacketTypeAvatarIdentity
                        || datagramPacketType == PacketTypeAvatarBillboard
                        || datagramPacketType == PacketTypeKillAvatar) {

interface/src/avatar/AvatarManager.cpp

@@ -135,7 +135,6 @@ AvatarSharedPointer AvatarManager::newSharedAvatar() {
 
 AvatarHash::iterator AvatarManager::erase(const AvatarHash::iterator& iterator) {
     if (iterator.key() != MY_AVATAR_KEY) {
-        qDebug() << "Removing Avatar with UUID" << iterator.key() << "from AvatarManager hash.";
         if (reinterpret_cast<Avatar*>(iterator.value().data())->isInitialized()) {
             _avatarFades.push_back(iterator.value());
         }

libraries/avatars/src/AvatarHashMap.cpp

@@ -25,6 +25,7 @@ void AvatarHashMap::insert(const QUuid& id, AvatarSharedPointer avatar) {
 }
 
 AvatarHash::iterator AvatarHashMap::erase(const AvatarHash::iterator& iterator) {
+    qDebug() << "Removing Avatar with UUID" << iterator.key() << "from AvatarHashMap.";
     return _avatarHash.erase(iterator);
 }
 
@@ -71,6 +72,8 @@ bool AvatarHashMap::containsAvatarWithDisplayName(const QString& displayName) {
             }
             
             break;
+        } else {
+            ++avatarIterator;
         }
     }