Hi Gabriel, Patch looks good, bad assumption on my part.
-Justin On Thu, 15 Oct 2009 15:57:18 -0300, Gabriel Roldan <[email protected]> wrote: > Hey, > > trying to persist a coverage info got stuck with a can't persist > coverage.xml message. > Digging into it found that XStreamPersister assumes a NumberRange being > of Doubles. > The following patch seems to fix it, does it look ok? > > Index: src/main/java/org/geoserver/config/util/XStreamPersister.java > =================================================================== > --- src/main/java/org/geoserver/config/util/XStreamPersister.java > (revision 13509) > +++ src/main/java/org/geoserver/config/util/XStreamPersister.java > (working copy) > @@ -954,7 +954,7 @@ > NumberRange range = (NumberRange) original; > > writer.startNode("min"); > - if ( Double.isInfinite( > ((Double)range.getMinValue()).doubleValue() ) ) { > + if ( Double.isInfinite( > ((Number)range.getMinValue()).doubleValue() ) ) { > context.convertAnother( "-inf" ); > } > else { > @@ -963,7 +963,7 @@ > writer.endNode(); > > writer.startNode("max"); > - if ( Double.isInfinite( > ((Double)range.getMaxValue()).doubleValue() )) { > + if ( Double.isInfinite( > ((Number)range.getMaxValue()).doubleValue() )) { > context.convertAnother( "inf"); > } > else { ------------------------------------------------------------------------------ Come build with us! The BlackBerry(R) Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9 - 12, 2009. Register now! http://p.sf.net/sfu/devconference _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
