<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 12 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p
{mso-style-priority:99;
mso-margin-top-alt:auto;
margin-right:0in;
mso-margin-bottom-alt:auto;
margin-left:0in;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Would be nice to have separate pull requests then (my rationale for 1 ticket)…<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Yves Jacolin [mailto:yves.jacolin@camptocamp.com]
<br>
<b>Sent:</b> Thursday, September 20, 2012 2:15 PM<br>
<b>To:</b> thomas bonfort<br>
<b>Cc:</b> Lime, Steve D (DNR); mapserver-dev List<br>
<b>Subject:</b> Re: [mapserver-dev] [WFS] define the geometry column in WFS as datasource<o:p></o:p></span></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal" style="margin-bottom:12.0pt">I agree with Thomas, those two issues are not related each other.<br>
<br>
I will do more test tomorrow for the namespace part with MapServer and next week with ArcGIS if possible.<br>
<br>
Y.<o:p></o:p></p>
<div>
<p class="MsoNormal">2012/9/20 thomas bonfort <<a href="mailto:thomas.bonfort@gmail.com" target="_blank">thomas.bonfort@gmail.com</a>><o:p></o:p></p>
<p class="MsoNormal">I'm going to have to disagree a bit with Steve here: please open two<br>
distinct pull requests as those seem to be very unrelated issues, so<br>
we can decide to incorporate one or the other independently.<br>
<br>
thanks,<br>
thomas<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal"><br>
On Thu, Sep 20, 2012 at 6:13 PM, Lime, Steve D (DNR)<br>
<<a href="mailto:Steve.Lime@state.mn.us">Steve.Lime@state.mn.us</a>> wrote:<br>
> I think I'd recommend creating a single ticket that refers to your pull request. The ticket will should provide the necessary context to understand what you've done. I think once we get a look at the changes we can make a decision on if and when to include
the changes. The metadata addition doesn't seem disruptive since by default things should work as they do now. The other seems, potentially, more disruptive.<br>
><br>
> We welcome contributions from all users, hardcore devs or not so *thank you*!<br>
><br>
> Steve<br>
><br>
> -----Original Message-----<br>
> From: <a href="mailto:mapserver-dev-bounces@lists.osgeo.org">mapserver-dev-bounces@lists.osgeo.org</a> [mailto:<a href="mailto:mapserver-dev-bounces@lists.osgeo.org">mapserver-dev-bounces@lists.osgeo.org</a>] On Behalf Of Yves Jacolin<br>
> Sent: Thursday, September 20, 2012 10:43 AM<br>
> To: mapserver-dev List<br>
> Subject: [mapserver-dev] [WFS] define the geometry column in WFS as datasource<br>
><br>
> Hello,<br>
><br>
> This week I find a limitation in MapServer configuration when using a WFS servcie as a datasource (aka MS as a WFS client).<br>
><br>
> As you can see here [1] the geometry column is hardcoded in code but some WFS services are not using Geometry as a column name (ArcGIS WFS sercices are using Shape instead, GeoServer is depending configuration of the layer).<br>
><br>
> I prepared a PR this week and tested it with a GeoServer instance. this PR add a new metadata for WFS client nammed geometryName (it could be a better name of course) to setup the name of the geometry column in the filter send by MS.<br>
><br>
> I have also some modifications related to namespace issue which should be added in tags, two examples:<br>
><br>
> <ogc:Filter xmlns:ogc='<a href="http://www.opengis.net/ogc" target="_blank">http://www.opengis.net/ogc</a>'><br>
> <gml:Box xmlns:gml="<a href="http://www.opengis.net/gml" target="_blank">http://www.opengis.net/gml</a>"><br>
><br>
> I would like to know what is the best process: do I need to create two tickets and propose my PR in some way?<br>
><br>
> Does MapServer developers are interesting about this two modifications?<br>
><br>
> Does this contribution could be added in the next release of MS (aka 6.2), is it too late? Are you interesting to get a PR in 6.0.x branche?<br>
><br>
> The last information is that I am **not** a developper at all but it was not so hard to do it finally :)<br>
><br>
> Thanks,<br>
><br>
> Y.<br>
> [1] <a href="https://github.com/mapserver/mapserver/blob/master/mapwfslayer.c#L242" target="_blank">
https://github.com/mapserver/mapserver/blob/master/mapwfslayer.c#L242</a><br>
> --<br>
> Responsable Formation et Support<br>
> Camptocamp France SAS<br>
> Savoie Technolac, BP 352<br>
> 73377 Le Bourget du Lac, Cedex<br>
> Tel (France) : <a href="tel:%2B33%204%2079%2026%2057%2098">+33 4 79 26 57 98</a><br>
> Mob. : <a href="tel:%2B33%206%2018%2075%2042%2021">+33 6 18 75 42 21</a><br>
> Fax : 04 79 70 15 81<br>
> Mail : <a href="mailto:yves.jacolin@camptocamp.com">yves.jacolin@camptocamp.com</a><br>
> <a href="http://www.camptocamp.com" target="_blank">http://www.camptocamp.com</a><br>
> _______________________________________________<br>
> mapserver-dev mailing list<br>
> <a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">
http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
><br>
><br>
> _______________________________________________<br>
> mapserver-dev mailing list<br>
> <a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">
http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><o:p></o:p></p>
</div>
</div>
</div>
<p class="MsoNormal"><br>
<br clear="all">
<br>
-- <o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Responsable Formation et Support<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Camptocamp France SAS<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Savoie Technolac, BP 352<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">73377 Le Bourget du Lac, Cedex<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Tel (France) : +33 4 79 26 57 98<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Mob. : +33 6 18 75 42 21<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Fax : 04 79 70 15 81<o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt">Mail : <a href="mailto:yves.jacolin@camptocamp.com" target="_blank">
yves.jacolin@camptocamp.com</a><o:p></o:p></p>
<p style="margin:0in;margin-bottom:.0001pt"><a href="http://www.camptocamp.com" target="_blank">http://www.camptocamp.com</a><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>