The update went okay, the only changes for GeoServer involved the KMLWriter (there is a couple of cases where it tries to figure out the opacity with out using a Feature). BEFORE: opacity.getValue( null ) AFTER: opacity.evaluate( null, Float.class )
Similar methods exist in the Filters and SLD utility classes ... for some reason we all want to to turn the opacity into a float. The first person to make opacity dependent on an expression is going to show up some problems :-D Gabriel you commented on just this kind of thing previously (the fact that we need to use evaluate); I hope you will find the majority of cases covered by now. The hardest part was as usual updating some of the FilterVisistor implementations, and figuring out what they all do. You can find a new wiki page on this very topic. Cheers, Jody Aside: Jesse I was unable to find the time to update uDig trunk; hopefully you get a chance (and it can serve as a second code review). I would also ask you to rename your new clean copying filter visitor to "DeepCopyFilterVisitor" - since I cannot keep things straight as they are now. ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
