fixes to Ovi, particularly related to getBounds and setBounds. Unlike…

… almost

every other map provider it seems Ovi uses the north-west and south-east
corners for bounds. I've thus also added getNorthWest and getSouthEast methods
to mxn.BoundingBox as well.

source/mxn.core.js

@@ -1248,9 +1248,9 @@ var LatLonPoint = mxn.LatLonPoint = function(lat, lon) {
 	// TODO error if undefined?
 	//  if (lat == undefined) alert('undefined lat');
 	//  if (lon == undefined) alert('undefined lon');
-	this.lat = lat;
-	this.lon = lon;
-	this.lng = lon; // lets be lon/lng agnostic
+	this.lat = lat*1; // force to be numeric
+	this.lon = lon*1;
+	this.lng = lon*1; // lets be lon/lng agnostic
 
 	this.invoker = new mxn.Invoker(this, 'LatLonPoint');		
 };
@@ -1346,9 +1346,10 @@ LatLonPoint.prototype.lonConv = function() {
  */
 var BoundingBox = mxn.BoundingBox = function(swlat, swlon, nelat, nelon) {
 	//FIXME throw error if box bigger than world
-	//alert('new bbox ' + swlat + ',' +  swlon + ',' +  nelat + ',' + nelon);
 	this.sw = new LatLonPoint(swlat, swlon);
 	this.ne = new LatLonPoint(nelat, nelon);
+	this.se = new LatLonPoint(swlat, nelon);
+	this.nw = new LatLonPoint(nelat, swlon);
 };
 
 /**
@@ -1360,6 +1361,15 @@ BoundingBox.prototype.getSouthWest = function() {
 	return this.sw;
 };
 
+/**
+ * getSouthWest returns a LatLonPoint of the south-west point of the bounding box
+ * @returns the south-east point of the bounding box
+ * @type LatLonPoint
+ */
+BoundingBox.prototype.getSouthEast = function() {
+	return this.se;
+};
+
 /**
  * getNorthEast returns a LatLonPoint of the north-east point of the bounding box
  * @returns the north-east point of the bounding box
@@ -1369,6 +1379,16 @@ BoundingBox.prototype.getNorthEast = function() {
 	return this.ne;
 };
 
+/**
+ * getNorthWest returns a LatLonPoint of the north-west point of the bounding box
+ * @returns the north-west point of the bounding box
+ * @type LatLonPoint
+ */
+BoundingBox.prototype.getNorthWest = function() {
+	return this.nw;
+};
+
+
 /**
  * isEmpty finds if this bounding box has zero area
  * @returns whether the north-east and south-west points of the bounding box are the same point

source/mxn.ovi.core.js

@@ -66,11 +66,12 @@ Mapstraction: {
 						eventStates.mapsize = false;
 						me.load.fire();
 					}
-				}
-				if (eventStates.center) {
+				} else {
+				    if (eventStates.center) {
 					eventStates.center = false;
 					me.moveendHandler(me);
 					me.endPan.fire();
+				    }
 				}
 				if (eventStates.zoom) {
 					eventStates.zoom = false;
@@ -88,7 +89,12 @@ Mapstraction: {
 	},
 
 	applyOptions: function() {
-		// TODO
+	    var map = this.maps[this.api];
+	    if(this.options.enableScrollWheelZoom){
+		map.addComponent(new ovi.mapsapi.map.component.zoom.MouseWheel());
+	    } else{
+		map.removeComponent(map.getComponentById("zoom.MouseWheel"));
+	    }  
 	},
 
 	resizeTo: function(width, height) {
@@ -135,13 +141,11 @@ Mapstraction: {
 	// style of Zoom controls so, for now, make them functionally equivalent
 	addSmallControls: function() {
 		var map = this.maps[this.api];
-
 		map.addComponent(new ovi.mapsapi.map.component.ZoomBar());
 	},
 
 	addLargeControls: function() {
 		var map = this.maps[this.api];
-
 		map.addComponent(new ovi.mapsapi.map.component.ZoomBar());
 	},
 
@@ -265,19 +269,18 @@ Mapstraction: {
 	getBounds: function() {
 		var map = this.maps[this.api];
 		var bbox = map.getViewBounds();
-		var sw = bbox.topLeft;
-		var ne = bbox.bottomRight;
+		var nw = bbox.topLeft;
+		var se = bbox.bottomRight;
+	        return new mxn.BoundingBox(se.latitude,nw.longitude,nw.latitude,se.longitude);
 
-		return new mxn.BoundingBox(sw.latitude, sw.longitude, ne.latitude, ne.longitude);
 	},
 
 	setBounds: function(bounds) {
 		var map = this.maps[this.api];
-		var sw = bounds.getSouthWest().toProprietary(this.api);
-		var ne = bounds.getNorthEast().toProprietary(this.api);
-		var ovi_bb = new ovi.mapsapi.geo.BoundingBox(sw, ne);
+		var nw = bounds.getNorthWest().toProprietary(this.api);
+		var se = bounds.getSouthEast().toProprietary(this.api);
+	        var ovi_bb = new ovi.mapsapi.geo.BoundingBox(nw,se);
 		var keepCentre = false;
-
 		map.zoomTo(ovi_bb, keepCentre);
 	},