10:13 <yxhuvud> amk: looks mostly fine. I can bet the xml parser has a way to pick out specific children, both lists of items and individual fields, though, so no need to loop like that. I'd also look at the fields of entry and see if any of those fields needs to always exist, as working with nilables is not as nice as working with non-nil values.

11:54 <amk> yxhuvud: thanks for looking! I've changed it now so it uses xpath which lets met pick nodes which is a lot nicer