On 11.12.2014 13:43, [email protected] wrote: > + private boolean isVectorLayer(Layerable layerable) { > + if (layerable instanceof RasterImageLayer) return false; > + if (layerable instanceof WMSLayer) return false; > + if (layerable instanceof Layer && > ((Layer)layerable).getStyle(ReferencedImageStyle.class) != null) return false; > + return true; > + }
that should test if (layerable instanceof Layer && ! (layerable instanceof ReferencedImagesLayer) ) return true and return false in all instances. this way it will not fail on future layerable implementations which are unknown today. this would make sense as a method in EnableCheckFactory don't you think? ..ede ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk _______________________________________________ Jump-pilot-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
