GetFeatureInfo broken

[assefa]

initially reported on the list by Bart,

"QUERY_STRING=map=/data/OGC_UMN_services/test_vector.map&LAYERS=ORA_POLY&FORMAT=image%2Fpng&TRANSPARENT=TRUE&SERVICE=WMS&VERSION=1.1.1&REQUEST=GetFeatureInfo&STYLES=&EXCEPTIONS=application%2Fvnd.ogc.se_xml&SRS=EPSG%3A28992&BBOX=122757.714116%2C434907.834101%2C204041.825055%2C469124.423963&X=452&Y=129&INFO_FORMAT=application%2Fvnd.ogc.gml&QUERY_LAYERS=ORA_POLY&FEATURE_COUNT=1&WIDTH=1031&HEIGHT=434" [Tue Jul 27 08:34:28 2010].343306 msOracleSpatialLayerOpen called with: SHAPE FROM (SELECT OBJECTID, HECTARES, ZONENR, NULZONES, AREA, PERIMETER, SHAPE FROM GRENZEN.OV_ZONERING) USING UNIQUE OBJECTID (Layer pointer 0x8630840) [Tue Jul 27 08:34:28 2010].464171 msOracleSpatialLayerOpen. Shared connection not available. Creating one. [Tue Jul 27 08:34:28 2010].464892 msConnPoolRegister(ORA_POLY,RWS_LEZEN/RWS_LEZEN@…:1521/PGPR.rws.nl,0x87722b0) [Tue Jul 27 08:34:28 2010].478015 msOracleSpatialLayerFreeItemInfo was called. [Tue Jul 27 08:34:28 2010].478734 msOracleSpatialLayerFreeItemInfo was called. [Tue Jul 27 08:34:28 2010].479369 msOracleSpatialLayerGetItems was called. [Tue Jul 27 08:34:28 2010].483218 msOracleSpatialLayerInitItemInfo was called. [Tue Jul 27 08:34:28 2010].484048 msOracleSpatialLayerInitItemInfo was called. [Tue Jul 27 08:34:28 2010].484806 msOracleSpatialLayerWhichShapes was called. [Tue Jul 27 08:34:28 2010].485589 msOracleSpatialLayerWhichShapes. Using this Sql to retrieve the data: SELECT OBJECTID, HECTARES, ZONENR, NULZONES, AREA, PERIMETER, rownum, SHAPE FROM (SELECT OBJECTID, HECTARES, ZONENR, NULZONES, AREA, PERIMETER, SHAPE FROM GRENZEN.OV_ZONERING) WHERE SDO_FILTER( SHAPE, MDSYS.SDO_GEOMETRY(2003, :srid, NULL,MDSYS.SDO_ELEM_INFO_ARRAY(1,1003,3),:ordinates ),'querytype=window')

'TRUE'

[Tue Jul 27 08:34:28 2010].490377 msOracleSpatialLayerWhichShapes. Bind values: srid:-1 minx:158432.845251 miny:458914.635053 maxx:158432.845251 maxy:458914.635053 [Tue Jul 27 08:34:28 2010].491033 msOracleSpatialLayerWhichShapes getting ordinate definition. [Tue Jul 27 08:34:28 2010].494052 msOracleSpatialLayerWhichShapes converting to OCIColl. [Tue Jul 27 08:34:28 2010].495951 msOracleSpatialLayerWhichShapes bind by name and object. [Tue Jul 27 08:34:28 2010].496910 msOracleSpatialLayerWhichShapes name and object now bound. [Tue Jul 27 08:34:28 2010].521059 msOracleSpatialLayerNextShape on layer 0x8630840, row_num: 0 [Tue Jul 27 08:34:28 2010].522846 msOracleSpatialLayerNextShape on layer 0x8630840, row_num: 1 [Tue Jul 27 08:34:28 2010].524501 msOracleSpatialLayerNextShape on layer 0x8630840, Fetched 0 more rows (1 total) Content-type: application/vnd.ogc.gml

<?xml version="1.0" encoding="ISO-8859-1"?>

<msGMLOutput

xmlns:gml="http://www.opengis.net/gml" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">

<ORA_POLY_layer>

[Tue Jul 27 08:34:28 2010].528379 msOracleSpatialLayerResultGetShape(): OracleSpatial error. msOracleSpatialLayerResultGetShape record out of range [Tue Jul 27 08:34:28 2010].529060 msOracleSpatialLayerClose was called. Layer: 0x8630840, Layer name: ORA_POLY. Layer connection: RWS_LEZEN/RWS_LEZEN@…:1521/PGPR.rws.nl [Tue Jul 27 08:34:28 2010].530614 msOracleSpatialLayerClose. Cleaning layerinfo handlers. [Tue Jul 27 08:34:28 2010].531716 msOracleSpatialLayerClose. Release the Oracle Pool. [Tue Jul 27 08:34:28 2010].532320 msConnPoolRelease(ORA_POLY,RWS_LEZEN/xxxx@…:1521/PGPR.rws.nl,0x87722b0) [Tue Jul 27 08:34:28 2010].533766 freeLayer(): freeing layer at 0x8630840. [Tue Jul 27 08:34:28 2010].534630 msConnPoolClose(RWS_LEZEN/RWS_LEZEN@…:1521/PGPR.rws.nl,0x87722b0)

Layer string is:

CONNECTION "RWS_LEZEN/xxx@…:1521/PGPR.rws.nl" CONNECTIONTYPE ORACLESPATIAL DATA "SHAPE FROM (SELECT OBJECTID, HECTARES, ZONENR, NULZONES, AREA,

PERIMETER, SHAPE FROM GRENZEN.OV_ZONERING) USING UNIQUE OBJECTID"

[assefa]

Fix committed in r10438.

Bart, can you confirm that this fixes your problem?

Alan, there was a change done in #3462 that I reverted back. I believe the order of the parameters are correct (at least it won't work for me in the current state). I think the message error in php comes from the fact the the tileindex defaults to -1? If that is the error message is not that meaningful and should be ignored. Sorry I did not reopen the same bug. Please review if my assumptions are incorrect.

Thanks

[bartvde]

Hi Assefa, thanks for looking into this. I'll check your fix first thing tomorrow morning and report back.

[bartvde]

I can confirm this fix works for us, thanks again Assefa.

[assefa]

fixed in trunk.closing