ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Use clearer argument names

Browse Source
This commit is contained in:
Tom Payne
2012-07-30 09:19:44 +02:00
parent a5b0fb4908
commit b291885ffa
1 changed files with 20 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ol/base/map.js
+20 -20
View File
@@ -532,12 +532,12 @@ ol.Map.prototype.handleBackgroundColorChanged = goog.nullFunction;
/** /**
* @param {goog.events.BrowserEvent} event Event. * @param {goog.events.BrowserEvent} browserEvent Browser event.
* @param {string=} opt_type Type. * @param {string=} opt_type Type.
*/ */
ol.Map.prototype.handleBrowserEvent = function(event, opt_type) { ol.Map.prototype.handleBrowserEvent = function(browserEvent, opt_type) {
var type = goog.isDef(opt_type) ? opt_type : event.type; var type = goog.isDef(opt_type) ? opt_type : browserEvent.type;
var mapBrowserEvent = new ol.MapBrowserEvent(type, this, event); var mapBrowserEvent = new ol.MapBrowserEvent(type, this, browserEvent);
var controls = this.getControls(); var controls = this.getControls();
var controlsArray = /** @type {Array.<ol.Control>} */ controls.getArray(); var controlsArray = /** @type {Array.<ol.Control>} */ controls.getArray();
goog.array.every(controlsArray, function(control) { goog.array.every(controlsArray, function(control) {
@@ -548,11 +548,11 @@ ol.Map.prototype.handleBrowserEvent = function(event, opt_type) {
/** /**
* @param {goog.fx.DragEvent} event Event. * @param {goog.fx.DragEvent} dragEvent Drag event.
*/ */
ol.Map.prototype.handleDraggerEvent = function(event) { ol.Map.prototype.handleDraggerEvent = function(dragEvent) {
var browserEvent = event.browserEvent; var browserEvent = dragEvent.browserEvent;
this.handleBrowserEvent(browserEvent, event.type); this.handleBrowserEvent(browserEvent, dragEvent.type);
}; };
@@ -589,35 +589,35 @@ ol.Map.prototype.handleLayerRemove = function(layer) {
/** /**
* @param {ol.CollectionEvent} event Event. * @param {ol.CollectionEvent} collectionEvent Collection event.
* @protected * @protected
*/ */
ol.Map.prototype.handleLayersInsertAt = function(event) { ol.Map.prototype.handleLayersInsertAt = function(collectionEvent) {
var layers = /** @type {ol.Collection} */ event.target; var layers = /** @type {ol.Collection} */ collectionEvent.target;
var layer = /** @type {ol.Layer} */ layers.getAt(event.index); var layer = /** @type {ol.Layer} */ layers.getAt(collectionEvent.index);
this.handleLayerAdd(layer); this.handleLayerAdd(layer);
}; };
/** /**
* @param {ol.CollectionEvent} event Event. * @param {ol.CollectionEvent} collectionEvent Collection event.
* @protected * @protected
*/ */
ol.Map.prototype.handleLayersRemoveAt = function(event) { ol.Map.prototype.handleLayersRemoveAt = function(collectionEvent) {
var layer = /** @type {ol.Layer} */ event.prev; var layer = /** @type {ol.Layer} */ collectionEvent.prev;
this.handleLayerRemove(layer); this.handleLayerRemove(layer);
}; };
/** /**
* @param {ol.CollectionEvent} event Event. * @param {ol.CollectionEvent} collectionEvent Collection event.
* @protected * @protected
*/ */
ol.Map.prototype.handleLayersSetAt = function(event) { ol.Map.prototype.handleLayersSetAt = function(collectionEvent) {
var prevLayer = /** @type {ol.Layer} */ event.prev; var prevLayer = /** @type {ol.Layer} */ collectionEvent.prev;
this.handleLayerRemove(prevLayer); this.handleLayerRemove(prevLayer);
var layers = /** @type {ol.Collection} */ event.target; var layers = /** @type {ol.Collection} */ collectionEvent.target;
var layer = /** @type {ol.Layer} */ layers.getAt(event.index); var layer = /** @type {ol.Layer} */ layers.getAt(collectionEvent.index);
this.handleLayerAdd(layer); this.handleLayerAdd(layer);
}; };