[Geotools-devel] [jira] (GEOT-4302) WMS 1.3.0 GetFeatureInfo

2012-11-02 Thread Jody Garnett (JIRA) 














































Jody Garnett
 created  GEOT-4302


WMS 1.3.0 GetFeatureInfo















Issue Type:


Sub-task



Affects Versions:


9.0-M0



Assignee:


Jody Garnett



Components:


wms extension



Created:


03/Nov/12 1:12 AM



Description:


In a similar fashion to WMS 1.3.0 GetMap we need to be very careful submitting a bounding box to a WMS 1.3.0 Server (to ensure we have provided the request with the axis order supplied by the server).

For more information (and example request) see UDIG-1958




Project:


GeoTools



Priority:


Major



Reporter:


Jody Garnett




























This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira





--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] Can I ask for caution on master

2012-11-02 Thread Jody Garnett 
Justin has advised me to ask for a code freeze ( rather than delay
patches ) while the feature collection cleanup patch is applied.

Instead can I ask that we exercise caution on any patches using
feature collection. There are a few difficult sections ( especially
GML parsing ) that I would ask everyone to steer clear of.

I am going to try and release 8.0-M0 this weekend, and be in position
to apply feature collection the moment GeoServer is ready.
--
Jody Garnett

--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Jody Garnett 
Thanks Justin!

If it helps there is a BaseFeatureCollection that just requires a
features() method, which is great for wrappers that formally supplied an
iteration() method.

I am on IRC on and off this weekend if needed - trying to release a
milestone release (not even sure where to start with the new scripts -
perhaps the GeoServer directions).

The uDig migration ready on a branch. The change to FeatureEvent means I
have to throw a FeatureSource around for anything editable rather than
FeatureCollection :(. Expected but still a pain.

--
Jody Garnett

On 02/11/2012, at 12:41 PM, Justin Deoliveira  wrote:

I took an initial run at updating GeoServer for the new api... for the most
part it is straight forward but there seems to be one tricky bit when it
comes to some feature collection wrappers that are used for security. MIght
not be a problem but need to understand what they are being used for, and
update tests. Will take another run over the weekeend.

On Fri, Nov 2, 2012 at 6:53 AM, Jody Garnett  wrote:

>  On Thursday, 1 November 2012 at 8:38 PM, Jody Garnett wrote:
>
>  I have deployed 9.0-M0-SNAPSHOT as preview for anyone wanting to try
> this out locally.
>
> It will also help work on the GeoServer and uDig migration branches,
> indeed it may allow me to do a pull request for uDig early.
>
> - pull request is ready https://github.com/uDig/udig-platform/pull/161
>
>
>
> --
> LogMeIn Central: Instant, anywhere, Remote PC access and management.
> Stay in control, update software, and manage PCs from one command center
> Diagnose problems and improve visibility into emerging IT issues
> Automate, monitor and manage. Do more in less time with Central
> http://p.sf.net/sfu/logmein12331_d2d
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Justin Deoliveira 
I took an initial run at updating GeoServer for the new api... for the most
part it is straight forward but there seems to be one tricky bit when it
comes to some feature collection wrappers that are used for security. MIght
not be a problem but need to understand what they are being used for, and
update tests. Will take another run over the weekeend.

On Fri, Nov 2, 2012 at 6:53 AM, Jody Garnett  wrote:

> On Thursday, 1 November 2012 at 8:38 PM, Jody Garnett wrote:
>
>  I have deployed 9.0-M0-SNAPSHOT as preview for anyone wanting to try
> this out locally.
>
> It will also help work on the GeoServer and uDig migration branches,
> indeed it may allow me to do a pull request for uDig early.
>
> - pull request is ready https://github.com/uDig/udig-platform/pull/161
>
>
>
> --
> LogMeIn Central: Instant, anywhere, Remote PC access and management.
> Stay in control, update software, and manage PCs from one command center
> Diagnose problems and improve visibility into emerging IT issues
> Automate, monitor and manage. Do more in less time with Central
> http://p.sf.net/sfu/logmein12331_d2d
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] removing xerces dependency

2012-11-02 Thread Justin Deoliveira 
Nice. Thanks Ben!

On Fri, Nov 2, 2012 at 1:43 AM, Ben Caradoc-Davies <
ben.caradoc-dav...@csiro.au> wrote:

> Justin,
>
> I ported AppSchemaValidator to use EntityResolver2, required for relative
> schema locations:
> https://github.com/jdeolive/**geotools/pull/1
>
> Note that xmlcodegen still fails to build (I commented it out for testing).
>
> With noxmldeps GeoTools, there is a unit test failure in GeoServer
> (app-schema-test SecondaryNamespaceExceptionTes**t). Looks like a side
> effect of your dexercesification is that undefined namespace prefixes
> (null:) are now passed through the GML32OutputFormat XSLT rather that
> causing an ows:ExceptionReport. Not sure why. I also needed to remove the
> xerces dependencies in the release module.
>
> Nice work! You are pretty close.
>
> Now just that pesky Xalan (app-schema GeoServer plugin, so off-topic). Any
> XSLT gurus willing to help backport to Xalan 2.6 (in the JDK)?
> https://jira.codehaus.org/**browse/GEOS-5352
>
> Kind regards,
> Ben.
>
>
> On 23/10/12 08:29, Justin Deoliveira wrote:
>
>> Hi all,
>>
>> So I recently I took a crack at removing the xerces dependency as we
>> discussed in our last skype meeting. For the most part this was
>> relatively straight forward. Unfortunately one thing i wasn't quite able
>> to figure out was some stuff in app-schema-resolver module. The
>> AppSchemaValidator class uses an implementation of what
>> was XMLEntityResolver directly in xerces. I tried porting this to the
>> jaxp EntityResolver interface which is included in the JDK, but couldn't
>> make it work. So I think i will need some help there. Or we can simply
>> keep xerces as a dependency of just app-schema.
>>
>> Here is the branch I am currently working on.
>>
>> https://github.com/jdeolive/**geotools/tree/noxmldeps
>> And the corresponding geoserver branch:
>>
>> https://github.com/jdeolive/**geoserver/tree/noxmldeps
>>
>> WIth these changes all unit tests pass in both code bases, and so do the
>> GeoServer cite tests.
>>
>> -Justin
>>
>> --
>> Justin Deoliveira
>> OpenGeo - http://opengeo.org
>> Enterprise support for open source geospatial.
>>
>>
> --
> Ben Caradoc-Davies 
> Software Engineer
> CSIRO Earth Science and Resource Engineering
> Australian Resources Research Centre
>



-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] feature collection cleanup -> 9.0-M0-SNAPSHOT

2012-11-02 Thread Jody Garnett 
On Thursday, 1 November 2012 at 8:38 PM, Jody Garnett wrote:
> I have deployed 9.0-M0-SNAPSHOT as preview for anyone wanting to try this out 
> locally. 
> 
> It will also help work on the GeoServer and uDig migration branches, indeed 
> it may allow me to do a pull request for uDig early.
- pull request is ready https://github.com/uDig/udig-platform/pull/161 

--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] removing xerces dependency

2012-11-02 Thread Ben Caradoc-Davies 
Justin,

I ported AppSchemaValidator to use EntityResolver2, required for 
relative schema locations:
https://github.com/jdeolive/geotools/pull/1

Note that xmlcodegen still fails to build (I commented it out for testing).

With noxmldeps GeoTools, there is a unit test failure in GeoServer 
(app-schema-test SecondaryNamespaceExceptionTest). Looks like a side 
effect of your dexercesification is that undefined namespace prefixes 
(null:) are now passed through the GML32OutputFormat XSLT rather that 
causing an ows:ExceptionReport. Not sure why. I also needed to remove 
the xerces dependencies in the release module.

Nice work! You are pretty close.

Now just that pesky Xalan (app-schema GeoServer plugin, so off-topic). 
Any XSLT gurus willing to help backport to Xalan 2.6 (in the JDK)?
https://jira.codehaus.org/browse/GEOS-5352

Kind regards,
Ben.

On 23/10/12 08:29, Justin Deoliveira wrote:
> Hi all,
>
> So I recently I took a crack at removing the xerces dependency as we
> discussed in our last skype meeting. For the most part this was
> relatively straight forward. Unfortunately one thing i wasn't quite able
> to figure out was some stuff in app-schema-resolver module. The
> AppSchemaValidator class uses an implementation of what
> was XMLEntityResolver directly in xerces. I tried porting this to the
> jaxp EntityResolver interface which is included in the JDK, but couldn't
> make it work. So I think i will need some help there. Or we can simply
> keep xerces as a dependency of just app-schema.
>
> Here is the branch I am currently working on.
>
> https://github.com/jdeolive/geotools/tree/noxmldeps
> And the corresponding geoserver branch:
>
> https://github.com/jdeolive/geoserver/tree/noxmldeps
>
> WIth these changes all unit tests pass in both code bases, and so do the
> GeoServer cite tests.
>
> -Justin
>
> --
> Justin Deoliveira
> OpenGeo - http://opengeo.org
> Enterprise support for open source geospatial.
>

-- 
Ben Caradoc-Davies 
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

--
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel