hey Mike,

just quickly hacked down
 EnableCheckFactory.createSelectedLayerablesMustBeEither(final Class[] classes) 
and committed it.

your EnableCheck should be created via
 EnableCheckFactory.createSelectedLayerablesMustBeEither(new Class[]{ 
Layer.class })

i didn't at all test it so far, so it might have some issues. maybe you can go 
ahead, use and fine-tune it. if not i will do so later ;)

..ede


On 11.12.2014 14:55, sstein wrote:
> 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
> 

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