<div dir="ltr">Hi Craig,<div><br></div><div>it looks reasonable to me (not sure if the "continue" is necessary or it would break if there are more than one remote service). Could you go on with a PR?</div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-08-30 3:39 GMT+02:00 Craig Stephenson <span dir="ltr"><<a href="mailto:crstephenson@alaska.edu" target="_blank">crstephenson@alaska.edu</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi All,</div><div><br></div><div>My colleagues and I discovered that an exception is produced if you add a remote layer (via the Remote Services feature) to a map, then try to download the map by going to:</div><div><br></div><div>Download Map -> Download Data Layers -> Start downloading this map</div><div><br></div><div>We modified GeoNode to skip over Remote Services layers like this:</div><div><br></div><div><a href="https://github.com/ua-snap/geonode/commit/c23d8d19b65328901145a5d74e12f98f4aa0e537" target="_blank">https://github.com/ua-snap/<wbr>geonode/commit/<wbr>c23d8d19b65328901145a5d74e12f9<wbr>8f4aa0e537</a></div><div><br></div><div>Is this a reasonable fix? And if so, should I submit this as a PR to your project?</div><div><br></div><div>Some context: I had tried making a map with a Shapefile layer, a GeoTIFF layer, and a Remote Services layer. The j_layer["service"] value for each shows up as:</div><div><br></div><div>Shapefile: WFS</div><div>GeoTIFF: WCS</div><div>Remote Services: None</div><div><br></div><div>Which is why I'm performing this check:</div><div><br></div><div>if j_layer["service"] is None</div><div><br></div><div>Or would it be better to do something like this?</div><div><br></div><div>if j_layer["service"] is not "WFS" and j_layer["service"] is not "WCS"</div><div><br></div><div>Thanks,</div><div>Craig</div></div>
<br>______________________________<wbr>_________________<br>
geonode-devel mailing list<br>
<a href="mailto:geonode-devel@lists.osgeo.org">geonode-devel@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/geonode-devel" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/geonode-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Simone </div>
</div>