Hi,
> just quickly hacked down
> EnableCheckFactory.createSelectedLayerablesMustBeEither(final Class[]
> classes)
> and committed it.
>
> your EnableCheck should be created via
> EnableCheckFactory.createSelectedLayerablesMustBeEither(new Class[]{
> Layer.class })
But it will not save me from creating a customized enableCheck to exclude
ReferencedImagesLayer or did I miss something ?
Anyway, it will simplify the creation of the check and make it less
error prone.
Thanks,
Michaël
>
> 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
>
------------------------------------------------------------------------------
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
