No subject


Tue Mar 27 15:02:52 EDT 2007


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.


Modified: trunk/openlayers/lib/OpenLayers/Tile/Image.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Tile/Image.js	2007-04-09 13:22:32 UTC (rev 3033)
+++ trunk/openlayers/lib/OpenLayers/Tile/Image.js	2007-04-09 14:08:09 UTC (rev 3034)
@@ -11,6 +11,9 @@
 OpenLayers.Tile.Image = OpenLayers.Class.create();
 OpenLayers.Tile.Image.prototype = 
   OpenLayers.Class.inherit( OpenLayers.Tile, {
+
+    /** @type String */
+    url: null,
     
     /** @type DOMElement img */
     imgDiv: null,
@@ -22,7 +25,10 @@
      * @type DOMElement div */ 
     frame: null, 
 
-    /** 
+    /** TBD 3.0 - reorder the parameters to the init function to put URL 
+     *             as last, so we can continue to call tile.initialize() 
+     *             without changing the arguments. 
+     * 
     * @constructor
     *
     * @param {OpenLayers.Grid} layer
@@ -33,6 +39,9 @@
     */
     initialize: function(layer, position, bounds, url, size) {
         OpenLayers.Tile.prototype.initialize.apply(this, arguments);
+
+        this.url = url;
+
         this.frame = document.createElement('div'); 
         this.frame.style.overflow = 'hidden'; 
         this.frame.style.position = 'absolute'; 

Modified: trunk/openlayers/lib/OpenLayers/Tile/WFS.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Tile/WFS.js	2007-04-09 13:22:32 UTC (rev 3033)
+++ trunk/openlayers/lib/OpenLayers/Tile/WFS.js	2007-04-09 14:08:09 UTC (rev 3034)
@@ -18,7 +18,10 @@
     /** @type Array(String) */
     url: null,
     
-    /** 
+    /** TBD 3.0 - reorder the parameters to the init function to put URL 
+     *             as last, so we can continue to call tile.initialize() 
+     *             without changing the arguments. 
+     * 
     * @constructor
     *
     * @param {OpenLayers.Layer} layer

Modified: trunk/openlayers/lib/OpenLayers/Tile.js
===================================================================
--- trunk/openlayers/lib/OpenLayers/Tile.js	2007-04-09 13:22:32 UTC (rev 3033)
+++ trunk/openlayers/lib/OpenLayers/Tile.js	2007-04-09 14:08:09 UTC (rev 3034)
@@ -12,6 +12,9 @@
  * about themselves -- such as the URL that they are related to, and their 
  * size - but do not add themselves to the layer div automatically, for 
  * example.
+ * 
+ * TBD 3.0 - remove reference to url in above paragraph
+ * 
  */
 OpenLayers.Tile = OpenLayers.Class.create();
 OpenLayers.Tile.prototype = {
@@ -22,7 +25,11 @@
     /** @type OpenLayers.Layer */
     layer: null,
     
-    /** @type String url of the request */
+    /** TBD 3.0 
+     * @deprecated The base tile class does not need an url. This should be 
+     *              handled in subclasses. Does not belong here.
+     * 
+     * @type String url of the request */
     url:null,
 
     /** @type OpenLayers.Bounds */
@@ -38,7 +45,9 @@
     /** @type Boolean */
     drawn: false,
 
-    /**
+    /** TBD 3.0 -- remove 'url' from the list of parameters to the constructor.
+     *             there is no need for the base tile class to have a url.
+     * 
     * @constructor
     *
     * @param {OpenLayers.Layer} layer



More information about the Commits mailing list