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