WMS dimension filter for tileindex #5209

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@edigiacomo
Contributor

edigiacomo commented Dec 11, 2015

WMS dimensions in raster layers, similar to TIME support for raster layers.

@edigiacomo edigiacomo changed the title from Dimensions filter is applied to tileindex too to WMS dimension filter for tileindex Dec 11, 2015

@sdlime

This comment has been minimized.

Show comment
Hide comment
@sdlime

sdlime Dec 23, 2015

Member

Any WMS maintainers available to review this patch? Seems reasonable but it's not an area of the code I am familiar with. --Steve

@dmorissette

Member

sdlime commented Dec 23, 2015

Any WMS maintainers available to review this patch? Seems reasonable but it's not an area of the code I am familiar with. --Steve

@dmorissette

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Jan 4, 2016

Member

@edigiacomo I found no place in the code where a TIME filter was being explicitely set on an underlying tileindex layer as it seems that you are doing in the generic dimension case here. Can you point me where in the code you are "copying" this from ?

Member

tbonfort commented Jan 4, 2016

@edigiacomo I found no place in the code where a TIME filter was being explicitely set on an underlying tileindex layer as it seems that you are doing in the generic dimension case here. Can you point me where in the code you are "copying" this from ?

@edigiacomo

This comment has been minimized.

Show comment
Hide comment
@edigiacomo

edigiacomo Jan 7, 2016

Contributor

See msRASTERLayerSetTimeFilter in maprasterquery.c (it's the LayerSetTimeFilter of a raster layer).

Contributor

edigiacomo commented Jan 7, 2016

See msRASTERLayerSetTimeFilter in maprasterquery.c (it's the LayerSetTimeFilter of a raster layer).

tbonfort added a commit that referenced this pull request Feb 24, 2016

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Feb 24, 2016

Member

Applied to branch-7-0 in 7daec58 , thanks

Member

tbonfort commented Feb 24, 2016

Applied to branch-7-0 in 7daec58 , thanks

@tbonfort tbonfort closed this Feb 24, 2016

@tbonfort tbonfort added this to the 7.0.1 milestone Feb 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment