Jody Garnett ha scritto: > Andrea Aime wrote: >> Jody Garnett ha scritto: >>> Running udig trunk today; looks like I am getting some exceptions on >>> Expression.NIL; the resulting visual display results in massive point >>> symbolizers :-) I will grab a stack trace tomorrow and see what I can >>> see... >> >> Jody, this may be related to my changes to SLDStyleFactory, remember >> you suggested I use Expression.NIL to spot if the symbolizer size was >> missing? > It is; and that is why I expected to have some fun testing :-)
Meh :( > Here is the stack trace; and you are right Expression.NIL is found to be > at fault - I suspect the Filter encoder needs to know about it when an > SLD file is generated... as for the Size I think BIG is the new default > is it not? The SLD standard asks for 16pixel wide icons when there is no way to determine a natural size. Much bigger than the older 6 pixel wide, but not sure if as big as what you're seeing. Can you open a jira? Cheers Andrea >> Caused by: java.lang.IllegalArgumentException: Cannot encode >> org.opengis.filter.expression.NilExpression >> at >> org.geotools.filter.FilterTransformer$FilterTranslator.encode(FilterTransformer.java:283) >> >> >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.element(SLDTransformer.java:158) >> >> >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.visit(SLDTransformer.java:551) >> >> >> at org.geotools.styling.GraphicImpl.accept(GraphicImpl.java:334) >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.visit(SLDTransformer.java:522) >> >> >> at >> org.geotools.styling.PointSymbolizerImpl.accept(PointSymbolizerImpl.java:84) >> >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.visit(SLDTransformer.java:494) >> >> >> at org.geotools.styling.RuleImpl.accept(RuleImpl.java:209) >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.visit(SLDTransformer.java:805) >> >> >> at >> org.geotools.styling.SLDTransformer$SLDTranslator.visit(SLDTransformer.java:770) >> >> >> at org.geotools.styling.StyleImpl.accept(StyleImpl.java:116) >> ... 25 more > So two things here; the SLDTransformer needs some love. And BIG is > probably what the SLD specification asked for :-) > > Jody > > > !DSPAM:4022,4852c1ed176301804284693! > > ------------------------------------------------------------------------ > ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
