OpenLayers OpenLayers

Ticket #762: encodeBBOX.2.patch

File encodeBBOX.2.patch, 9.7 kB (added by crschmidt, 1 year ago)

  • tests/Tile/test_Image.html

    old new  
    6464            SERVICE: "WMS", VERSION: "1.1.1", 
    6565            REQUEST: "GetMap", STYLES: "", 
    6666            EXCEPTIONS: "application/vnd.ogc.se_inimage", FORMAT: "image/jpeg", 
    67             SRS: "EPSG:4326", BBOX: "1,2,3,4", 
     67            SRS: "EPSG:4326", BBOX: [1,2,3,4], 
    6868            WIDTH: String(size.w), HEIGHT: String(size.h) 
    6969        }; 
    7070        t.eq( img.src, 
    …… 
    8686        var size = new OpenLayers.Size(5,6); 
    8787        var map = new OpenLayers.Map('map'); 
    8888        var layer = new OpenLayers.Layer.WMS( "OpenLayers WMS",  
    89             "http://labs.metacarta.com/wms/vmap0?", {layers: 'basic'}, {reproject:false});  
     89            "http://labs.metacarta.com/wms/vmap0?", {layers: 'basic'}, {reproject:false, encodeBBOX: true});  
    9090        map.addLayer(layer); 
    9191        tile = new OpenLayers.Tile.Image(layer, position, new OpenLayers.Bounds(-185,-90,-180,90), url, size); 
    9292        tile.draw() 

  • tests/BaseTypes/test_Bounds.html

    old new  
    6060         bounds = new OpenLayers.Bounds(1,2,3,4); 
    6161         t.eq( bounds.toString(), "left-bottom=(1,2) right-top=(3,4)", "toString() returns correct value." );  
    6262     } 
     63     function test_Bounds_toArray(t) { 
     64         t.plan( 1 ); 
     65         bounds = new OpenLayers.Bounds(1,2,3,4); 
     66         t.eq( bounds.toArray(), [1,2,3,4], "toArray() returns correct value." );  
     67     } 
    6368 
    6469     function test_04_Bounds_contains(t) { 
    6570         t.plan( 6 ); 

  • tests/Layer/test_WrapDateLine.html

    old new  
    127127        t.plan( 3 ); 
    128128 
    129129        var url = "http://octo.metacarta.com/cgi-bin/mapserv"; 
    130         layer = new OpenLayers.Layer.WMS(name, url, params, {'wrapDateLine':true}); 
     130        layer = new OpenLayers.Layer.WMS(name, url, params, {'wrapDateLine':true,encodeBBOX:true}); 
    131131        var m = new OpenLayers.Map('map'); 
    132132        m.addLayer(layer); 
    133133        m.zoomToMaxExtent(); 
    …… 
    159159            {layers: "bathymetry,land_fn,park,drain_fn,drainage," + 
    160160                     "prov_bound,fedlimit,rail,road,popplace", 
    161161             transparent: "true", format: "image/png"}, 
    162             {wrapDateLine: true, reproject: false}); 
     162            {wrapDateLine: true, reproject: false,encodeBBOX:true}); 
    163163        var m = new OpenLayers.Map('map'); 
    164164        m.addLayers([baselayer,layer]); 
    165165        m.zoomToMaxExtent(); 

  • tests/Layer/test_WFS.html

    old new  
    2828        t.eq(setSize, true, "Renderer resize called on map size change."); 
    2929 
    3030    } 
     31    function test_Layer_WFS_drawmap(t) { 
     32        t.plan(2); 
     33        map = new OpenLayers.Map('map'); 
     34        layer = new OpenLayers.Layer.WMS( "OpenLayers WMS",  
     35            "http://labs.metacarta.com/wms/vmap0", {layers: 'basic'} ); 
     36        map.addLayer(layer); 
     37 
     38        layer = new OpenLayers.Layer.WFS( "Owl Survey", 
     39            "http://www.bsc-eoc.org/cgi-bin/bsc_ows.asp?", 
     40            {typename: "OWLS", maxfeatures: 10}, 
     41            { featureClass: OpenLayers.Feature.WFS}); 
     42        map.addLayer(layer); 
     43        map.addControl(new OpenLayers.Control.LayerSwitcher()); 
     44        try {  
     45            map.setCenter(new OpenLayers.LonLat(-100, 60), 3); 
     46        } catch (Exception) { 
     47        }     
     48        t.eq(layer.tile.url, "http://www.bsc-eoc.org/cgi-bin/bsc_ows.asp?typename=OWLS&maxfeatures=10&SERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&SRS=EPSG%3A4326&BBOX=-187.890625,-36.6796875,-12.109375,156.6796875", "Tile URL is set correctly when not encoded"); 
     49        map = new OpenLayers.Map('map'); 
     50        layer = new OpenLayers.Layer.WMS( "OpenLayers WMS",  
     51            "http://labs.metacarta.com/wms/vmap0", {layers: 'basic'} 
     52            ); 
     53        map.addLayer(layer); 
     54 
     55        layer = new OpenLayers.Layer.WFS( "Owl Survey", 
     56            "http://www.bsc-eoc.org/cgi-bin/bsc_ows.asp?", 
     57            {typename: "OWLS", maxfeatures: 10}, 
     58            { featureClass: OpenLayers.Feature.WFS, 'encodeBBOX': true}); 
     59        map.addLayer(layer); 
     60        map.addControl(new OpenLayers.Control.LayerSwitcher()); 
     61        try {  
     62            map.setCenter(new OpenLayers.LonLat(-100, 60), 3); 
     63        } catch (Exception) { 
     64        }   
     65        t.eq(layer.tile.url, "http://www.bsc-eoc.org/cgi-bin/bsc_ows.asp?typename=OWLS&maxfeatures=10&SERVICE=WFS&VERSION=1.0.0&REQUEST=GetFeature&SRS=EPSG%3A4326&BBOX=-187.890625%2C-36.679687%2C-12.109375%2C156.679688", "Tile URL is set correctly when not encoded"); 
     66    } 
     67 
    3168  // --> 
    3269  </script> 
    3370</head> 

  • tests/Layer/test_WMS.html

    old new  
    7070            SERVICE: "WMS", VERSION: "1.1.1", 
    7171            REQUEST: "GetMap", STYLES: "", 
    7272            EXCEPTIONS: "application/vnd.ogc.se_inimage", 
     73            SRS: "EPSG:4326", BBOX: [1,2,3,4], 
     74            WIDTH: "256", HEIGHT: "256" 
     75        }); 
     76        t.eq( img.src, 
     77             url + "?" + OpenLayers.Util.getParameterString(tParams), 
     78             "image src is created correctly via addtile" ); 
     79        t.eq( tile.frame.style.top, "6px", "image top is set correctly via addtile" ); 
     80        t.eq( tile.frame.style.left, "5px", "image top is set correctly via addtile" ); 
     81 
     82        var firstChild = layer.div.firstChild.firstChild; 
     83        if (!isMozilla) 
     84            t.ok( true, "skipping element test outside of Mozilla"); 
     85        else 
     86            t.ok( firstChild instanceof HTMLElement, "div first child is an image object" ); 
     87        t.eq( firstChild.src, 
     88             url + "?" + OpenLayers.Util.getParameterString(tParams), 
     89             "div first child is correct image object" ); 
     90        t.eq( tile.position.toString(), "x=5,y=6", "Position of tile is set correctly." ); 
     91        map.destroy(); 
     92    } 
     93     
     94    function test_Layer_WMS_bboxEncoding (t) { 
     95        t.plan( 6 ); 
     96     
     97        var url = "http://octo.metacarta.com/cgi-bin/mapserv"; 
     98        layer = new OpenLayers.Layer.WMS(name, url, params, {encodeBBOX:true}); 
     99        var map = new OpenLayers.Map('map'); 
     100        map.addLayer(layer); 
     101        var pixel = new OpenLayers.Pixel(5,6); 
     102        var tile = layer.addTile(new OpenLayers.Bounds(1,2,3,4), pixel); 
     103        tile.draw(); 
     104 
     105        var img = tile.imgDiv; 
     106        var tParams = OpenLayers.Util.extend({}, 
     107                        OpenLayers.Util.upperCaseObject(params)); 
     108        tParams = OpenLayers.Util.extend(tParams, { 
     109            SERVICE: "WMS", VERSION: "1.1.1", 
     110            REQUEST: "GetMap", STYLES: "", 
     111            EXCEPTIONS: "application/vnd.ogc.se_inimage", 
    73112            SRS: "EPSG:4326", BBOX: "1,2,3,4", 
    74113            WIDTH: "256", HEIGHT: "256" 
    75114        }); 

  • lib/OpenLayers/BaseTypes/Bounds.js

    old new  
    9999                 + " right-top=(" + this.right + "," + this.top + ")" ); 
    100100    }, 
    101101 
     102    /** 
     103     * APIMethod: toArray 
     104     * 
     105     * Return: 
     106     * {Array} array of left, bottom, right, top 
     107     */ 
     108    toArray: function() { 
     109        return [this.left, this.bottom, this.right, this.top]; 
     110    },     
     111 
    102112    /**  
    103113     * APIMethod: toBBOX 
    104114     *  

  • lib/OpenLayers/Layer/WMS.js

    old new  
    4343     * {Boolean} Default is true for WMS layer 
    4444     */ 
    4545    isBaseLayer: true, 
     46     
     47    /** 
     48     * APIProperty: encodeBBOX 
     49     * {Boolean} Should the BBOX commas be encoded? The WMS spec says 'no',  
     50     * but some services want it that way. Default false. 
     51     */ 
     52    encodeBBOX: false, 
    4653  
    4754    /** 
    4855     * Constructor: OpenLayers.Layer.WMS 
    …… 
    146153         
    147154        var imageSize = this.getImageSize();  
    148155        return this.getFullRequestString( 
    149                      {BBOX:bounds.toBBOX(), 
     156                     {BBOX: this.encodeBBOX ?  bounds.toBBOX() : bounds.toArray(), 
    150157                      WIDTH:imageSize.w, 
    151158                      HEIGHT:imageSize.h}); 
    152159    }, 

  • lib/OpenLayers/Layer/WFS.js

    old new  
    5151     * {Boolean} Should be calculated automatically. 
    5252     */ 
    5353    vectorMode: true,  
     54     
     55    /** 
     56     * APIProperty: encodeBBOX 
     57     * {Boolean} Should the BBOX commas be encoded? The WMS spec says 'no',  
     58     * but some services want it that way. Default false. 
     59     */ 
     60    encodeBBOX: false, 
    5461 
    5562    /** 
    5663    * Constructor: OpenLayers.Layer.WFS 
    …… 
    196203            //formulate request url string 
    197204            var url = this.getFullRequestString(); 
    198205         
    199             var params = { BBOX:tileBounds.toBBOX() }; 
     206            var params = {BBOX: this.encodeBBOX ? tileBounds.toBBOX()  
     207                                                : tileBounds.toArray()}; 
    200208            url += "&" + OpenLayers.Util.getParameterString(params); 
    201209 
    202210            if (!this.tile) {