OpenLayers OpenLayers

Changeset 3034

Show
Ignore:
Timestamp:
04/09/07 10:08:09 (2 years ago)
Author:
crschmidt
Message:

From Erik, "add strategic TBD 3.0 comments to remind us to remove the 'url'
var from OpenLayers.Tile's constructor." This has no affects other than to
let people know now that this behavior will change in future versions of
OpenLayers. #647.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/openlayers/lib/OpenLayers/Tile.js

    r2543 r3034  
    1313 * size - but do not add themselves to the layer div automatically, for  
    1414 * example. 
     15 *  
     16 * TBD 3.0 - remove reference to url in above paragraph 
     17 *  
    1518 */ 
    1619OpenLayers.Tile = OpenLayers.Class.create(); 
    …… 
    2326    layer: null, 
    2427     
    25     /** @type String url of the request */ 
     28    /** TBD 3.0  
     29     * @deprecated The base tile class does not need an url. This should be  
     30     *              handled in subclasses. Does not belong here. 
     31     *  
     32     * @type String url of the request */ 
    2633    url:null, 
    2734 
    …… 
    3946    drawn: false, 
    4047 
    41     /** 
     48    /** TBD 3.0 -- remove 'url' from the list of parameters to the constructor. 
     49     *             there is no need for the base tile class to have a url. 
     50     *  
    4251    * @constructor 
    4352    * 

  • trunk/openlayers/lib/OpenLayers/Tile/Image.js

    r2979 r3034  
    1212OpenLayers.Tile.Image.prototype =  
    1313  OpenLayers.Class.inherit( OpenLayers.Tile, { 
     14 
     15    /** @type String */ 
     16    url: null, 
    1417     
    1518    /** @type DOMElement img */ 
    …… 
    2326    frame: null,  
    2427 
    25     /**  
     28    /** TBD 3.0 - reorder the parameters to the init function to put URL  
     29     *             as last, so we can continue to call tile.initialize()  
     30     *             without changing the arguments.  
     31     *  
    2632    * @constructor 
    2733    * 
    …… 
    3440    initialize: function(layer, position, bounds, url, size) { 
    3541        OpenLayers.Tile.prototype.initialize.apply(this, arguments); 
     42 
     43        this.url = url; 
     44 
    3645        this.frame = document.createElement('div');  
    3746        this.frame.style.overflow = 'hidden';  

  • trunk/openlayers/lib/OpenLayers/Tile/WFS.js

    r3029 r3034  
    1919    url: null, 
    2020     
    21     /**  
     21    /** TBD 3.0 - reorder the parameters to the init function to put URL  
     22     *             as last, so we can continue to call tile.initialize()  
     23     *             without changing the arguments.  
     24     *  
    2225    * @constructor 
    2326    *