Merge pull request #285 from strongloop/revert/geopoint-orientation

Revert "switch long and lat for geopoint type"
This commit is contained in:
Biniam Admikew 2017-05-31 10:58:34 -04:00 committed by GitHub
commit fc55994d8c
2 changed files with 4 additions and 29 deletions

View File

@ -361,7 +361,7 @@ MySQL.prototype.toColumnValue = function(prop, val) {
if (prop.type.name === 'GeoPoint') {
return new ParameterizedSQL({
sql: 'Point(?,?)',
params: [val.lng, val.lat],
params: [val.lat, val.lng],
});
}
if (prop.type === Buffer) {
@ -422,8 +422,8 @@ MySQL.prototype.fromColumnValue = function(prop, val) {
case 'GeoPoint':
case 'Point':
val = {
lng: val.x,
lat: val.y,
lat: val.x,
lng: val.y,
};
break;
case 'List':

View File

@ -7,7 +7,7 @@
require('./init.js');
var assert = require('assert');
var db, BlobModel, EnumModel, ANIMAL_ENUM, City;
var db, BlobModel, EnumModel, ANIMAL_ENUM;
var mysqlVersion;
describe('MySQL specific datatypes', function() {
@ -86,26 +86,6 @@ describe('MySQL specific datatypes', function() {
});
});
});
it('should create a model instance with geopoint type', function(done) {
var city1 = {
name: 'North York',
loc: {
lat: 43.761539,
lng: -79.411079,
},
};
City.create(city1, function(err, res) {
assert.ok(!err);
res.loc.should.deepEqual(city1.loc);
res.name.should.equal(city1.name);
City.find({where: {name: city1.name}}, function(err, found) {
assert.ok(!err);
found[0].name.should.equal(city1.name);
found[0].loc.should.deepEqual(city1.loc);
done();
});
});
});
it('should disconnect when done', function(done) {
db.disconnect();
done();
@ -130,11 +110,6 @@ function setup(done) {
bin: {type: Buffer, dataType: 'blob', null: false},
name: {type: String},
});
City = db.define('City', {
name: {type: String},
loc: {type: 'GeoPoint'},
});
query('SELECT VERSION()', function(err, res) {
mysqlVersion = res && res[0] && res[0]['VERSION()'];
blankDatabase(db, done);