Hi Ede, Michael,

yes, would be a good addition to EnableCheck

cheers,
stefan

On 12/11/2014 10:50 AM, [email protected] wrote:
> 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


------------------------------------------------------------------------------
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

Reply via email to