Opened 10 years ago

Closed 9 years ago

#501 closed defect (fixed)

Buffer command does not buffer all selected features

Reported by: ronnielouie Owned by:
Priority: high Milestone:
Component: General Version: 2.0.0
Severity: major Keywords:
Cc: External ID: 1057442

Description

On any web layout, using any viewer, select a moderately large number of features (>50 or so).

Choose to buffer the features.

Expected result: All selected features to be buffered.

Actual result: Only a subset of the features are buffered.

Change History (6)

comment:1 Changed 10 years ago by ronnielouie

Submitted fix for 2.0.x stream for basic weblayout.

http://trac.osgeo.org/mapguide/changeset/3093

Fixes for trunk stream and for Flexible weblayout will need to be addressed at a later date.

comment:2 Changed 10 years ago by ronnielouie

The fix submitted for http://trac.osgeo.org/mapguide/changeset/3093 was a temporary fix particular to the 2.0.x stream.

The correct solution is to fix the limitation issue in the GenerateFilter API. When this is done, the GenerateFilters method will no longer be available.

comment:3 Changed 10 years ago by nclayton

Since the GenerateFilters? method is temporary, the code that calls it should do a check to see if it exists before calling it, otherwise use GenerateFilter?.

For example, in PHP:

if(method_exists($selectionObject,'GenerateFilters?')) {

$filters = $selectionObject->GenerateFilters?($layer,$layerClassName,$scnt); $selString = $filters->GetItem?(0);

} else {

$selString = $selectionObject->GenerateFilter?($layer,$layerClassName);

}

This will save you from having to alter your code again once the GenerateFilters? method is removed.

comment:4 Changed 9 years ago by jbirch

Can this be closed? I don't think that these changes got ported back to trunk, and GenerateFilter? may no longer exist?

comment:5 Changed 9 years ago by jbirch

Duh, I mean GenerateFilters?.

comment:6 Changed 9 years ago by jbirch

Resolution: fixed
Status: newclosed

Closing. If still a problem with 2.1, please re-open and reset the version.

Note: See TracTickets for help on using tickets.