Bug 463 – [Filter expression] Filter espression returns nothing if WFS filter clear is hit

Bug 463 - [Filter expression] Filter espression returns nothing if WFS filter clear is hit

Summary:

[Filter expression] Filter espression returns nothing if WFS filter clear is hit

Status:	RESOLVED FIXED

Product:	Chameleon
Component:	Widget
Version:	1.99
Platform:	PC Linux

Importance:	P2 normal
Target Milestone:	1.99 beta 4
Assigned To:	Julien-Samuel Lacroix

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2004-06-22 15:34 by Julien-Samuel Lacroix
Modified:	2004-12-02 21:31 (History)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note

You need to log in before you can comment on or make changes to this bug.

Description From Julien-Samuel Lacroix 2004-06-22 15:34:00

The Filter expression widget returns nothing if a query is cleared using the WFS
filter clear button before the filter expression button is pressed.

------- Comment #1 From Julien-Samuel Lacroix 2004-07-05 13:41:14 -------

Working on this

------- Comment #2 From Julien-Samuel Lacroix 2004-07-05 13:42:59 -------

Oups! sorry!  forgot assign to me before marking as ASSIGNED

working on this

------- Comment #3 From Darren Redfern 2004-07-08 13:00:28 -------

Changed target to 1.99 RC 1.

------- Comment #4 From Julien-Samuel Lacroix 2004-07-08 17:00:15 -------

Few! Finally got it! I had a hard time to track it down.
The problem was in LegendTemplate.widget.php. It happens because the
LegendTemplate widget keep a list of all layers comma separated in the hidden
form variable LEGENDTEMPLATE_LAYERS.

The problem was "a list of all layers"...even those deleted by other widgets. So
I added a validation when building this string.

417,418c417,422
<             $szLayerNames .= $szSep . $oLayer->name;
<             $szSep = ",";
---
>             // Don't include layers that will be deleted.
>             if( $oLayer->status != MS_DELETE )
>             {
>                 $szLayerNames .= $szSep . $oLayer->name;
>                 $szSep = ",";
>             }

Marking as FIXED.

------- Comment #5 From Darren Redfern 2004-08-10 15:38:10 -------

Changing Version to 1.99 (2.0 not being worked on yet).