Ticket #94 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

multi-layer query broken when one of the layer defines a feature filter

Reported by: tomfukushima Owned by: madair
Priority: P2 Milestone: 1.1
Component: MapGuide Version: 1.0.6
Severity: Major Keywords:
Cc: External ID:
state: Pending Approval Browser: All
Operating System: All

Description

See the MapGuide trac ticket for more details:  http://trac.osgeo.org/mapguide/ticket/651. A patch is attached for your review. In case the patch doesn't work (because I hand edited the paths) the corresponding MG change is  http://trac.osgeo.org/mapguide/changeset/3260.

Attachments

ticket651.patch Download (1.2 KB) - added by tomfukushima 4 years ago.
Defects-1058939.mgp Download (213.8 KB) - added by edzimmerman 4 years ago.

Change History

Changed 4 years ago by tomfukushima

Changed 4 years ago by edzimmerman

Changed 4 years ago by edzimmerman

I attached a package containing an example map. To reproduce the defect 1) Apply the attached package. 2) In MapGuide Studio, create a fusion web layout referencnig the sample map contained in the package. 3) Launch the web browser on fusion web layout url. 4) Set layer visibility as follows: airprt020 (OFF); airprt020 filtered (ON); Sample_World_Countries (ON). 5) Do a window select around some points.

Before the fix, the selection will fail. After the fix, it will succeed. The problem was that use of the $filter variable in Query.php was being overloaded to both store the possible incoming sql filter and to store the possible combined filter of the sql filter and the layer's feature filter. I fixed this be adding a new $combinedFilter variable to eliminate the overloaded use of $filter.

Changed 3 years ago by pagameba

  • state changed from New to Pending Approval

Changed 3 years ago by madair

(In [1615]) re #94: patch applied

Changed 3 years ago by madair

  • status changed from new to closed
  • resolution set to fixed

(In [1616]) closes #94: patch applied to trunk

Changed 3 years ago by tomfukushima

Thanks Mike

Note: See TracTickets for help on using tickets.