Opened 19 years ago

Closed 17 years ago

#1261 closed defect (fixed)

[WMS} time support : overriding Filter paramter

Reported by: assefa Owned by: assefa
Priority: high Milestone: 5.0 release
Component: WMS Server Version: 5.0
Severity: normal Keywords:
Cc: uli@…

Description

e-mail from uli mueller : uli.mueller@GMX.CH


I finally got some nice WMS with time support running. Now I fell into
the problem that any time filter seems to override everything in the
PostGIS layer's FILTER setting.
I had read most of http://mapserver.gis.umn.edu/bugs/show_bug.cgi?id=872
so I understand that WMS time support is done (mis)using the FILTER
parameter. However it would be preferrable if any existing FILTER could
persist. Why not combine an existing filter and a WMS time filter
through SQL-AND?

Change History (5)

comment:1 by uli@…, 19 years ago

Cc: uli@… added

comment:2 by assefa, 18 years ago

Owner: changed from mapserverbugs to assefa
I have updated the setting so the an And is used if the layer has already a 
filter of an expression type. This was added for regular layers 9all beside 
postgis).
I will add the postgis next.

comment:3 by assefa, 17 years ago

Milestone: 5.0 release
Version: 4.55.0

this Patch was sent and tested by Denis Nadeau dnadeau@…

mappostgis.c:2061 replace :

if(lp->filteritem) free(lp->filteritem);

lp->filteritem = strdup(timefield); if (&lp->filter)

freeExpression(&lp->filter);

for:

if (&lp->filter)

{

/* if the filter is set and it's a string type, concatenate it with

the time. If not just free it */

if (lp->filter.type == MS_EXPRESSION)

{ strcat(buffer, "("); strcat(buffer, lp->filter.string); strcat(buffer, ") and "); }

else

{ freeExpression(&lp->filter); }

}

Patch applied in r6412 Denis please give it a try from svn and if all goes well I will close the bug

comment:4 by dnadeau, 17 years ago

I have tested the patch and everything works fine.

I have tried FILTER and TIME without problem.

Good job.

comment:5 by assefa, 17 years ago

Resolution: fixed
Status: newclosed

Thnaks for the tests. closing bug.

Note: See TracTickets for help on using tickets.