[OpenLayers-Trac] [OpenLayers] #2903: addTile: don't set the deprecated url param

OpenLayers trac-20090302 at openlayers.org
Wed Oct 27 06:24:32 EDT 2010


#2903: addTile: don't set the deprecated url param
-----------------------------+----------------------------------------------
 Reporter:  fredj            |       Owner:  tschaub     
     Type:  bug              |      Status:  new         
 Priority:  trivial          |   Milestone:  2.11 Release
Component:  Layer.TileCache  |     Version:  SVN         
 Keywords:                   |       State:  Review      
-----------------------------+----------------------------------------------
 In {{{OpenLayers.Layer.TileCache.addTile()}}} the url is computed and
 passed to the {{{OpenLayers.Tile.Image}}} constructor but this parameter
 is deprecated.

 The following patch removes the url computation:
 {{{
 Index: lib/OpenLayers/Layer/TileCache.js
 ===================================================================
 --- lib/OpenLayers/Layer/TileCache.js   (revision 10860)
 +++ lib/OpenLayers/Layer/TileCache.js   (working copy)
 @@ -157,9 +157,8 @@
       * {<OpenLayers.Tile.Image>} The added <OpenLayers.Tile.Image>
       */
      addTile:function(bounds, position) {
 -        var url = this.getURL(bounds);
          return new OpenLayers.Tile.Image(this, position, bounds,
 -                                             url, this.tileSize);
 +                                         null, this.tileSize);
      },

      CLASS_NAME: "OpenLayers.Layer.TileCache"
 }}}

 Tests still pass, please review.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2903>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list