Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#3224 closed defect (fixed)

WFS hits count is incorrect if the request conatin 2 layers or more

Reported by: assefa Owned by: assefa
Priority: normal Milestone: 5.6 release
Component: WFS Server Version: unspecified
Severity: normal Keywords: wfs hits
Cc: dmorissette

Description

WFS hits count is incorrect if the request conatin 2 layers or more: return only the hits for the first layer. This is a safe change.

--- mapwfs.c (revision 9571) +++ mapwfs.c (working copy) @@ -1680,7 +1680,6 @@

if (GET_LAYER(map, j)->resultcache && GET_LAYER(map, j)->resultcache->numresults > 0) {

iNumberOfFeatures += GET_LAYER(map, j)->resultcache->numresults;

  • break;

}

}

Change History (4)

comment:1 by assefa, 14 years ago

should I wait for the release or just commit the change which is strait forward? I really would not want anything to hold up tomorrow's final release.

comment:2 by dmorissette, 14 years ago

Cc: dmorissette added
Owner: changed from mapserverbugs to assefa

Is this new to 5.6? If this has been around for several releases then I'd say wait for 5.6.1, otherwise if that's new to 5.6 and you are confident that this is the right fix then please go ahead now.

Is it clearly stated somewhere in the spec whether the count is per layer/typename or global?

comment:3 by assefa, 14 years ago

Resolution: fixed
Status: newclosed

This was part of #2907 and was new to 5.6. The count if for the whole GetFeature request, so for all results. committed in r9572.

Thanks

comment:4 by assefa, 14 years ago

note that the previous commit was incomplete. Added missing test in r9573.

Note: See TracTickets for help on using tickets.