I'd never noticed this but seems harmless +1

Ian

On 24 August 2017 at 23:29, Ben Caradoc-Davies <[email protected]> wrote:

> +0. No objection here. While this is a slight behavioural change, it must
> be backwards compatible and I do not see how the new behaviour can cause
> any harm.
>
> Kind regards,
> Ben.
>
> On 25/08/17 02:31, Nuno Oliveira wrote:
>
>> Hi,
>>
>> I just made this PR [1] that fix this issue [2].
>>
>> Currently when doing a layer preview of a coverage (or multiple
>> coverages) filtering capabilities
>> (CQL, OGC and FeatureID) are deactivated, they are only active if a
>> vector layer is present. My
>> PR changes that behavior to activate filtering capabilities if at least
>> one of the used coverage
>> layers reader supports filtering (e.g. image mosaic).
>>
>> I would like to backport this to GS 2.11.x right away, this issue is
>> between a bug and an improvement.
>> The nature of the changes are additive and self contained to the
>> Openlayers output format.
>>
>> In the users point of view the only difference will be the possibility to
>> use the filtering capabilities when
>> doing the preview of a coverage whose reader supports filtering.
>>
>> Any objections to backporting right away ?
>>
>> Cheers,
>>
>> [1] https://github.com/geoserver/geoserver/pull/2494
>> [2] https://osgeo-org.atlassian.net/browse/GEOS-8263
>>
>>
> --
> Ben Caradoc-Davies <[email protected]>
> Director
> Transient Software Limited <http://transient.nz/>
> New Zealand
>
>
> ------------------------------------------------------------
> ------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Geoserver-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>



-- 
Ian Turton
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to