Changeset 11046


Ignore:
Timestamp:
Jan 20, 2011 5:20:03 AM (6 years ago)
Author:
bartvde
Message:

WMS GetFeatureInfo control should use integers for X and Y, r=ahocevar (closes #3019)

Location:
trunk/openlayers
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/openlayers/lib/OpenLayers/Control/WMSGetFeatureInfo.js

    r10995 r11046  
    371371            {
    372372                crs: projection,
    373                 i: clickPosition.x,
    374                 j: clickPosition.y
     373                i: parseInt(clickPosition.x),
     374                j: parseInt(clickPosition.y)
    375375            } :
    376376            {
    377377                srs: projection,
    378                 x: clickPosition.x,
    379                 y: clickPosition.y
     378                x: parseInt(clickPosition.x),
     379                y: parseInt(clickPosition.y)
    380380            }
    381381        );
  • trunk/openlayers/tests/Control/WMSGetFeatureInfo.html

    r10948 r11046  
    234234    // params in the WMS Layers involved
    235235    function test_mixedParams(t) {
    236         t.plan(3);
     236        t.plan(5);
    237237        var map = new OpenLayers.Map("map", {
    238238            getExtent: function() {return(new OpenLayers.Bounds(-180,-90,180,90));}
     
    272272        };
    273273        click.activate();
    274         click.getInfoForClick({xy: {x: 50, y: 50}});
     274        click.getInfoForClick({xy: {x: 50.2, y: 50.1}});
    275275        OpenLayers.Request.GET = _request;
     276
     277        t.eq(
     278            log.options && log.options.params.X,
     279            50,
     280            "X should be an integer"
     281        );
     282
     283        t.eq(
     284            log.options && log.options.params.Y,
     285            50,
     286            "Y should be an integer"
     287        );
    276288
    277289        t.eq(
     
    507519        };
    508520        click.activate();
    509         click.getInfoForClick({xy: {x: 50, y: 60}});
     521        click.getInfoForClick({xy: {x: 50.1, y: 60.2}});
    510522        OpenLayers.Request.GET = _request;
    511523        t.eq(
Note: See TracChangeset for help on using the changeset viewer.