Wonder if that is a functionality change since I wrote
GML.decodeFeatureCollection (hard to know).

Still the point of a facade class is to hook up these methods to the best
available implementation, and the changes you are making sound like an
improvement.

--
Jody Garnett

On 5 February 2016 at 10:28, Andrea Aime <andrea.a...@geo-solutions.it>
wrote:

> On Fri, Feb 5, 2016 at 7:25 PM, Jody Garnett <jody.garn...@gmail.com>
> wrote:
>
>> Looks like from your code example that you are doing a much smarter job
>> then the GML
>> <https://github.com/geotools/geotools/blob/master/modules/library/xml/src/main/java/org/geotools/GML.java>
>> utility class  decodeFeatureCollection(InputStream in) method.
>>
>
>> When a schema is not available the parser returns a stream of
>> Map<String,Object> entries. The simpleFeatureCollection( collection method)
>> uses uses simpleType(Object)
>> <https://github.com/geotools/geotools/blob/master/modules/library/xml/src/main/java/org/geotools/GML.java#L634>
>> to generate a schema for the first parsed Map<String,Object>, and then
>> creates the features based on that.
>>
>
> I am actually getting back a SimpleFeatureCollection regardless, it's just
> that it's limited to the attributes
> of the first feature (see my first mail for a description of the original
> problem).
>
>
>>
>> Once your new code is ready I would like to update
>> GML.decodeFeatureCollection( InputStream ) to use it.
>>
>
> I could go for that, sure
>
> Cheers
> Andrea
>
> --
> ==
> GeoServer Professional Services from the experts! Visit
> http://goo.gl/it488V 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
>
> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>
> Le informazioni contenute in questo messaggio di posta elettronica e/o
> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
> darcene notizia via e-mail e di procedere alla distruzione del messaggio
> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
> principi dettati dal D.Lgs. 196/2003.
>
>
>
> The information in this message and/or attachments, is intended solely for
> the attention and use of the named addressee(s) and may be confidential or
> proprietary in nature or covered by the provisions of privacy act
> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
> Code).Any use not in accord with its purpose, any disclosure, reproduction,
> copying, distribution, or either dissemination, either whole or partial, is
> strictly forbidden except previous formal approval of the named
> addressee(s). If you are not the intended recipient, please contact
> immediately the sender by telephone, fax or e-mail and delete the
> information in this message that has been received in error. The sender
> does not give any warranty or accept liability as the content, accuracy or
> completeness of sent messages and accepts no responsibility  for changes
> made after they were sent or for other risks which arise as a result of
> e-mail transmission, viruses, etc.
>
> -------------------------------------------------------
>
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to