Merge pull request #2 from dkunin-me/has-point-fix

Fix: now hasPoint() returns false for invalid point.
This commit is contained in:
Viktor Govako 2013-08-14 06:41:50 -07:00
commit f9682c014c

View file

@ -52,16 +52,21 @@ public class MWMResponse
public static MWMResponse extractFromIntent(Context context, Intent intent)
{
final MWMResponse response = new MWMResponse();
// parse status
// parse point
final double lat = intent.getDoubleExtra(Const.EXTRA_MWM_RESPONSE_POINT_LAT, 0);
final double lon = intent.getDoubleExtra(Const.EXTRA_MWM_RESPONSE_POINT_LON, 0);
final double lat = intent.getDoubleExtra(Const.EXTRA_MWM_RESPONSE_POINT_LAT, INVALID_LL);
final double lon = intent.getDoubleExtra(Const.EXTRA_MWM_RESPONSE_POINT_LON, INVALID_LL);
final String name = intent.getStringExtra(Const.EXTRA_MWM_RESPONSE_POINT_NAME);
final String id = intent.getStringExtra(Const.EXTRA_MWM_RESPONSE_POINT_ID);
response.mPoint = new MWMPoint(lat, lon, name, id);
if (lat != INVALID_LL && lon != INVALID_LL)
response.mPoint = new MWMPoint(lat, lon, name, id);
else
response.mPoint = null;
return response;
}
private final static double INVALID_LL = Double.MIN_VALUE;
private MWMResponse() {}
}