[Context] context 1.0 write code uses WMS instead of OGC:WMS

[bartvde@…]

When outputting a WMC, Mapserver uses WMS for the value of the service
attribute of Server, whereas the OGC example (on schemas.opengis.net) as
well as Ionic use OGC:WMS

<Server service="OGC:WMS"

See also the example in the OGC WMC 1.0 spec:
http://www.opengeospatial.org/docs/03-036r2.pdf

[jlacroix]

Note this is fixed in WMC 1.1 support.
Before that, the WMC only specified "string" as the type for the Server's
service parameter. If I remember well, we write only "WMS" because at the time
the context support was developped "OGC:WMS" was not used anywhere (version
0.1.4 at the time). So I'm not sure from which version we should change it to
"OGC:WMS".

Also since in 1.0.0 specify only the string type it's legal to use "WMS"...

What do you think?

[bartvde@…]

Hi Julien,

what Mapserver does is valid according to the schema, that's true. But I think
it would be more wise to follow the examples from the specification document
though, only ofcourse if it won't break backward compatibility which I don't
think it does (since the Ionic OGC:WMS in the WMC 1.0 document was parsed
correctly to CONNECTIONTYPE WMS).

Bart

[dmorissette]

Reassigned to Julien.

[tomkralidis]

It looks like this has been done (see mapcontext.c, line 1940). Julien: can you verify?

[bartvde]

Replying to tomkralidis:

It looks like this has been done (see mapcontext.c, line 1940). Julien: can you verify?

Hi Tom, it has not been done yet (see ​http://cvs.gis.umn.edu/cgi-bin/viewcvs/mapserver/mapcontext.c?rev=1.81&content-type=text/vnd.viewcvs-markup). For WMC 1.1 it outputs OGC:WMS, but for WMC 1.0 it outputs WMS instead of OGC:WMS. I think we should use OGC:WMS also for WMC 1.0, but Julien and I did not agree on that part if I remember correctly.

[tomkralidis]

Ah, ok. I misunderstood this one. :)

OK, if we set Server/@service"OGC:WMS" for 1.0.0 and 1.1.0, I think that's fine. I don't know that alot of clients actually use the value when binding as opposed to an identifier. Infact, I think this attribute may have originally been a placeholder for future versions of WMC which may support other OGC types.

Julien: is this okay with you?

[jlacroix]

Thanks for the precision. I updated the code to write "OGC:WMS" for the 1.0.0 version. I left previous versions use "WMS" because the examples in those documents used "WMS".

Marking as FIXED

[jlacroix]

BTW, I updated only the SVN head version. Should it be back ported to 4.10?

[bartvde]

I think svn head is fine.