the logic in PR is already found in Tracker\Request overrideLocation($visitorInfo);
Calling this method will also bring country, region, city, lat, long overrides. Cheers
overrideLocation() only allows to pass lat/long in request in case when current request is first action of visit.
Any following action will not change visit's lat/long.
This PR purpose is to allow update lat/long in any following action of visit (for example for mobile users).
overrideLocation will override the following:
static $locationOverrideParams = array( 'country' => array('string', 'location_country'), 'region' => array('string', 'location_region'), 'city' => array('string', 'location_city'), 'lat' => array('float', 'location_latitude'), 'long' => array('float', 'location_longitude'), );
So it does also country / region / city. My feedback is to use this function rather than rewrite the code only for lat/long.