Thanks for the information about the official way for contributing.
I'll try to submit my patch regarding the minimum requirements.
Regards
Paul
Von: andrea.a...@gmail.com [mailto:andrea.a...@gmail.com] Im Auftrag von
Andrea Aime
Gesendet: Mittwoch, 20. Februar 2013 12:59
An: Biskup, Paul
Cc: Justin Deoliveira; geoserver-devel@lists.sourceforge.net
Betreff: Re: [Geoserver-devel] JSON output - total number of features
for paging
On Wed, Feb 20, 2013 at 9:50 AM, Biskup, Paul
<paul.bis...@fit.fichtner.de> wrote:
I'm glad to hear, that you want to integrate this extra
attribute in JSON-output!
Since I have no experience in the contribution of source code,
maybe I'll just send you my patched "GeoJSONOutputFormat.java"-file.
I've added the function "getNumberOfFeaturesFromRequest", which
gets called before the JSON-object gets written.
This function also respects filters. So if a filter gets passed,
only the count of the filtered features gets returned.
The positions I have edited are marked with PBI-comment.
I hope that this is sufficient.
Generally speaking we don't accept changes unless they are accompained
by a test.
The bare minimum is a patch (not the full files) against the current
GeoServer master that contains both the change to the code, and the test
showing it works.
If that's too hard you can open an improvement request on
jira.codehaus.org, attach the file above, and wait
for a good soul that completes the minimum requirements for you.
Cheers
Andrea
--
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it 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
-------------------------------------------------------
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
