mirrors/prosody
1
0
Fork 0
mirror of https://github.com/bjc/prosody.git synced 2025-04-04 13:47:41 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

util.datamapper: Fix arrays nesting one level too deep

Browse source
This commit is contained in:
Kim Alvefur 2021-03-19 01:17:59 +01:00
parent 3c3cdcd0c7
commit 7f04df223a
3 changed files with 31 additions and 25 deletions
Download patch file Download diff file Expand all files Collapse all files

22
spec/util_datamapper_spec.lua
View file

@ -46,10 +46,13 @@ describe("util.datampper", function()
type = "string"; type = "string";
xml = {name = "origin-id"; namespace = "urn:xmpp:sid:0"; x_single_attribute = "id"}; xml = {name = "origin-id"; namespace = "urn:xmpp:sid:0"; x_single_attribute = "id"};
}; };
reactions = { react = {
type = "array"; type = "object";
xml = {namespace = "urn:xmpp:reactions:0"; wrapped = true}; xml = {namespace = "urn:xmpp:reactions:0"; name = "reactions"};
items = {type = "string"; xml = {name = "reaction"}}; properties = {
to = {type = "string"; xml = {attribute = true; name = "id"}};
reactions = {type = "array"; items = {type = "string"; xml = {name = "reaction"}}};
};
}; };
}; };
}; };
@ -80,9 +83,12 @@ describe("util.datampper", function()
state = "active"; state = "active";
fallback = true; fallback = true;
origin_id = "qgkmMdPB"; origin_id = "qgkmMdPB";
reactions = { react = {
"👋", to = "744f6e18-a57a-11e9-a656-4889e7820c76";
"🐢", reactions = {
"👋",
"🐢",
};
}; };
}; };
end); end);
@ -102,6 +108,8 @@ describe("util.datampper", function()
assert.equal(x:get_child_text("delay", "urn:xmpp:delay"), u:get_child_text("delay", "urn:xmpp:delay")); assert.equal(x:get_child_text("delay", "urn:xmpp:delay"), u:get_child_text("delay", "urn:xmpp:delay"));
assert.same(x:get_child("delay", "urn:xmpp:delay").attr, u:get_child("delay", "urn:xmpp:delay").attr); assert.same(x:get_child("delay", "urn:xmpp:delay").attr, u:get_child("delay", "urn:xmpp:delay").attr);
assert.same(x:get_child("origin-id", "urn:xmpp:sid:0").attr, u:get_child("origin-id", "urn:xmpp:sid:0").attr); assert.same(x:get_child("origin-id", "urn:xmpp:sid:0").attr, u:get_child("origin-id", "urn:xmpp:sid:0").attr);
assert.same(x:get_child("reactions", "urn:xmpp:reactions:0").attr, u:get_child("reactions", "urn:xmpp:reactions:0").attr);
assert.same(2, #u:get_child("reactions", "urn:xmpp:reactions:0").tags);
for _, tag in ipairs(x.tags) do for _, tag in ipairs(x.tags) do
if tag.name ~= "UNRELATED" then if tag.name ~= "UNRELATED" then
assert.truthy(u:get_child(tag.name, tag.attr.xmlns) or u:get_child(tag.name), tag:top_tag()) assert.truthy(u:get_child(tag.name, tag.attr.xmlns) or u:get_child(tag.name), tag:top_tag())

16
teal-src/util/datamapper.tl
View file

@ -216,7 +216,7 @@ local function parse (schema : json_schema_object, s : st.stanza_t) : table
end end
end end
local function unparse ( schema : json_schema_object, t : table, current_name : string, current_ns : string ) : st.stanza_t local function unparse ( schema : json_schema_object, t : table, current_name : string, current_ns : string, ctx : st.stanza_t ) : st.stanza_t
if schema.xml then if schema.xml then
if schema.xml.name then if schema.xml.name then
@ -228,7 +228,7 @@ local function unparse ( schema : json_schema_object, t : table, current_name :
-- TODO prefix? -- TODO prefix?
end end
local out = st.stanza(current_name, { xmlns = current_ns }) local out = ctx or st.stanza(current_name, { xmlns = current_ns })
if schema.type == "object" then if schema.type == "object" then
@ -303,15 +303,13 @@ local function unparse ( schema : json_schema_object, t : table, current_name :
out:add_direct_child(c); out:add_direct_child(c);
end end
elseif proptype == "array" and propschema is json_schema_object and v is table then elseif proptype == "array" and propschema is json_schema_object and v is table then
local c = unparse(propschema, v, name, namespace); if value_where == "in_wrapper" then
if c then local c = unparse(propschema, v, name, namespace);
if value_where == "in_wrapper" then if c then
local w = st.stanza(propschema.xml.name or name, { xmlns = propschema.xml.namespace or namespace })
w:add_direct_child(c);
out:add_direct_child(w);
else
out:add_direct_child(c); out:add_direct_child(c);
end end
else
unparse(propschema, v, name, namespace, out);
end end
else else
error "NYI" error "NYI"

18
util/datamapper.lua
View file

@ -1,5 +1,7 @@
local st = require("util.stanza"); local st = require("util.stanza");
local schema_t = {}
local function toboolean(s) local function toboolean(s)
if s == "true" or s == "1" then if s == "true" or s == "1" then
return true return true
@ -181,7 +183,7 @@ local function parse(schema, s)
end end
end end
local function unparse(schema, t, current_name, current_ns) local function unparse(schema, t, current_name, current_ns, ctx)
if schema.xml then if schema.xml then
if schema.xml.name then if schema.xml.name then
@ -193,7 +195,7 @@ local function unparse(schema, t, current_name, current_ns)
end end
local out = st.stanza(current_name, {xmlns = current_ns}) local out = ctx or st.stanza(current_name, {xmlns = current_ns})
if schema.type == "object" then if schema.type == "object" then
@ -268,15 +270,13 @@ local function unparse(schema, t, current_name, current_ns)
out:add_direct_child(c); out:add_direct_child(c);
end end
elseif proptype == "array" and type(propschema) == "table" and type(v) == "table" then elseif proptype == "array" and type(propschema) == "table" and type(v) == "table" then
local c = unparse(propschema, v, name, namespace); if value_where == "in_wrapper" then
if c then local c = unparse(propschema, v, name, namespace);
if value_where == "in_wrapper" then if c then
local w = st.stanza(propschema.xml.name or name, {xmlns = propschema.xml.namespace or namespace})
w:add_direct_child(c);
out:add_direct_child(w);
else
out:add_direct_child(c); out:add_direct_child(c);
end end
else
unparse(propschema, v, name, namespace, out);
end end
else else
error("NYI") error("NYI")