Bug fix, set hasPhoto state immediately to prevent race condition that the TrackDetailActivity is recreated

This commit is contained in:
Jimmy Shih
2014-01-16 14:44:15 -08:00
parent 9f45405d89
commit 0ccd70a7b9
@@ -133,11 +133,11 @@ public class TrackDetailActivity extends AbstractSendToGoogleActivity
public void run() { public void run() {
trackController.update(trackId == recordingTrackId, recordingTrackPaused); trackController.update(trackId == recordingTrackId, recordingTrackPaused);
if (hasPhoto && photoUri != null) { if (hasPhoto && photoUri != null) {
hasPhoto = false;
WaypointCreationRequest waypointCreationRequest = new WaypointCreationRequest( WaypointCreationRequest waypointCreationRequest = new WaypointCreationRequest(
WaypointType.WAYPOINT, false, null, null, null, null, photoUri.toString()); WaypointType.WAYPOINT, false, null, null, null, null, photoUri.toString());
long id = TrackRecordingServiceConnectionUtils.addMarker( long id = TrackRecordingServiceConnectionUtils.addMarker(
TrackDetailActivity.this, trackRecordingServiceConnection, waypointCreationRequest); TrackDetailActivity.this, trackRecordingServiceConnection, waypointCreationRequest);
hasPhoto = false;
if (id != -1L) { if (id != -1L) {
FileUtils.updateMediaScanner(TrackDetailActivity.this, photoUri); FileUtils.updateMediaScanner(TrackDetailActivity.this, photoUri);