Justin Deoliveira ha scritto:
> David,
>
> I notice you made the following change for GEOS-3192:
>
> ---
> trunk/src/wms/src/main/java/org/vfny/geoserver/wms/responses/helpers/WMSCapsTransformer.java
>
> 2009-06-24 20:56:00 UTC (rev 12700)
> +++
> trunk/src/wms/src/main/java/org/vfny/geoserver/wms/responses/helpers/WMSCapsTransformer.java
>
> 2009-06-24 21:50:46 UTC (rev 12701)
> @@ -682,7 +682,7 @@
> AttributesImpl qatts = new AttributesImpl();
> qatts.addAttribute("", "queryable", "queryable", "", "1");
> start("Layer", qatts);
> - element("Name", layer.getName());
> + element("Name",
> layer.getResource().getNamespace().getPrefix() + ":" + layer.getName());
> //REVISIT: this is bad, layer should have title and
> anbstract by itself
> element("Title", layer.getResource().getTitle());
> element("Abstract", layer.getResource().getAbstract());
>
> While I realize this is a single line change, and that the change is
> probably worth while as it fixes an issue, changing anything (no matter
> how trivial) in the capabilities document can seriously affect existing
> clients, so such changes should not be made lightly.
>
> I am not against the change, but it went forward with absolutely no
> discussion on the developers list. Was there discussion on IRC that I
> missed? Did you verify this change with existing clients? I am guessing
> openlayers but what about udig, qgis and other clients?
I saw no discussion. Looking at the diff it would seem this change
fixes a regression introduced during the NY sprint? (using
layer and their names instead of resources and their qualified names?)
Cheers
Andrea
------------------------------------------------------------------------------
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
