From 4ecf288a935ba26546967f27168448bc4cc2d4d4 Mon Sep 17 00:00:00 2001
From: Kim Alvefur <zash@zash.se>
Date: Fri, 7 Sep 2018 01:04:53 +0200
Subject: mod_vcard_legacy: Handle avatar without vcard4

Since vcards are just avatar containers in many modern clients, aborting
in case of no vcard4 data is not optimal.

The upgrade mechanism needs further tweaks.
---
 plugins/mod_vcard_legacy.lua | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/plugins/mod_vcard_legacy.lua b/plugins/mod_vcard_legacy.lua
index 302dc7f8..d6383114 100644
--- a/plugins/mod_vcard_legacy.lua
+++ b/plugins/mod_vcard_legacy.lua
@@ -107,10 +107,6 @@ module:hook("iq-get/bare/vcard-temp:vCard", function (event)
 				vcard_temp:up();
 			end
 		end
-	else
-		local legacy_vcard = st.deserialize(vcards:get(jid_split(stanza.attr.to) or origin.username));
-		origin.send(st.reply(stanza):add_child(legacy_vcard or vcard_temp));
-		return true;
 	end
 
 	local meta_ok, avatar_meta = pep_service:get_items("urn:xmpp:avatar:metadata", stanza.attr.from);
-- 
cgit v1.2.3