Opened 9 years ago

Closed 5 years ago

#6056 closed defect (wontfix)

geojson test 22 intvalue property is a float

Reported by: Kurt Schwehr Owned by: warmerdam
Priority: low Milestone: closed_because_of_github_migration
Component: default Version: unspecified
Severity: normal Keywords:
Cc:

Description (last modified by Kurt Schwehr)

I am pretty sure that the properties don't mean anything, but was there a particular point to having 3.5 for the 'intvalue'?

​https://trac.osgeo.org/gdal/browser/trunk/autotest/ogr/ogr_geojson.py#L928 

    if (feature.GetFieldAsString("_id") != 'aid2' or
        feature.GetFieldAsString("_rev") != 'arev2' or
        feature.GetFieldAsDouble("intvalue") != 3.5 or
        feature.GetFieldAsString("str2value") != 'bar' or
        ogrtest.check_feature_geometry(feature, ref_geom) != 0):

If there was a specific reason, we should doc it. If not, I'm considering making intvalue -> doublevalue

Change History (3)

comment:1 by Kurt Schwehr, 9 years ago

Description: modified (diff)

comment:2 by Even Rouault, 9 years ago

The test probably tries to check auto promotion from int to double when seeing 2 and then 3.5. I agree the name of the property isn't really clear as currently.

comment:3 by Even Rouault, 5 years ago

Milestone: → closed_because_of_github_migration
Resolution: → wontfix
Status: new → closed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to ​https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.