On Fri, Jun 6, 2014 at 4:18 PM, Justin Deoliveira <[email protected]
> wrote:
>
>
>
> On Thu, Jun 5, 2014 at 8:04 AM, Andrea Aime <[email protected]>
> wrote:
>
>> On Thu, Jun 5, 2014 at 3:48 PM, Justin Deoliveira <
>> [email protected]> wrote:
>>
>>> I've just changed it to:
>>>>
>>>> //push a new context for children, declaring the default prefix
>>>> to be the one of this
>>>> // element
>>>> this.namespaces.pushContext();
>>>>
>>>> if (uri != null) {
>>>> String existingUri = namespaces.getURI(uri);
>>>> if (!uri.equals(existingUri)) {
>>>> this.namespaces.declarePrefix("", uri);
>>>> }
>>>> }
>>>>
>>>> Not sure if it's a typo but "String existingUri =
>>> namespaces.getURI(uri)" doesn't make much sense to me as getURI() is meant
>>> to take a prefix?
>>>
>>
>> Ops you're right, it's a typo... funny enough, the full gt and gs builds
>> (hmm.. did not check app-schema tests though) pass anyways with this change?
>>
>
> Right... makes me think that in general this full blown namespace handling
> is probably not needed. I do remember some tricky case for cite tests. Be
> good to validate any changes with that.
>
All right, will do
Cheers
Andrea
--
==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.
==
Ing. Andrea Aime
@geowolf
Technical Lead
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
-------------------------------------------------------
------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their
applications. Written by three acclaimed leaders in the field,
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/NeoTech
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
