<div><div dir="auto">Hello Andreas,</div></div><div dir="auto"><br></div><div dir="auto">I also have no idea if that is the right approach :) but I think the next step with your patch is to both run the test suite and add a few new cases with single examples and the combined one if at all possible in the form of a pull request.</div><div dir="auto"><br></div><div dir="auto">I recently got started with the test suite, so if you need help setting that up just shoot me an email.</div><div dir="auto"><br></div><div dir="auto">Best,</div><div dir="auto">Ariel</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 14 Apr 2020 at 2:28 AM Andreas Zapke <andreas.zapke@geomap.immo> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
I have 2 wms sources for getfeatureinfo requests. Both uses xlst-transformation to output html. This works well for each of them if the other one is disabled, but fails for both with:<br>
<br>
File "/usr/lib/python3.7/site-packages/mapproxy/featureinfo.py", line 125, in combine<br>
body = tree.body.getchildren()<br>
AttributeError: 'lxml.etree._XSLTResultTree' object has no attribute 'body'<br>
<br>
I use: python 3.7, lxml 4.3.4 und MapProxy 1.12.0<br>
<br>
It seems that the lxml ElementTree has no body Element here and a additional getroot() is required.<br>
With the following changes in mapproxy/featureinfo.py the concatanation works, but maybe its not the best way?<br>
<br>
125c125<br>
<                 body = tree.body.getchildren()<br>
---<br>
>                 body = tree.getroot().body.getchildren()<br>
127,128c127,128<br>
<                 body = tree.getchildren()<br>
<             result_tree.body.extend(body)<br>
---<br>
>                 body = tree.getroot().getchildren()<br>
>             result_tree.getroot().body.extend(body)<br>
<br>
<br>
Best regards<br>
Andreas<br>
_______________________________________________<br>
MapProxy mailing list<br>
<a href="mailto:MapProxy@lists.osgeo.org" target="_blank">MapProxy@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/mapproxy" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/mapproxy</a></blockquote></div></div>