Hi Ede, > 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 Right Ede. Difficult to say if future layarables will be raster or vector and even if future Layer extensions will be vector or raster, but I missed ReferencedImagesLayer which will be more easy to use than ReferencedImageStyle.
Indeed, I missed ReferencedImagesLayer because I did not found it in http://jump-pilot.sourceforge.net/javadoc/openjump_javadoc/ Wonder why as the class is probably quite old. Do you know how to update/complete this inline API ? Michaël > > 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
