The login API also tries to update the user's geolocation. I missed this when I removed the rest of the GeoIP stuff, which meant that the API php file tried to import a now-deleted class. Oops!
The removed $user->save() call has always been superfluous - User::login() saves before returning, and nothing in between that point and this one mutates the $user object.
The login API also tries to update the user's geolocation. I missed this when I removed the rest of the GeoIP stuff, which meant that the API php file tried to import a now-deleted class. Oops!
The removed
$user->save()
call has always been superfluous -User::login()
saves before returning, and nothing in between that point and this one mutates the$user
object.