Fix #476: Resume tracking btn change the state to stopped

[miltonials]

• Fix: "Resume tracking" button changes the tracking status #476

The “Resume tracking” btn should execute the setActiveTrack function only when the app has no active tracking.

[Binnette]

Hi @miltonials, may be you can improve your check. Since we are in case R.id.trackmgr_contextmenu_resume we know that the user clicked on resume item on the context menu. Also TRACK_ID_NO_TRACK is final so its value is always -1.

Consider a condition like: "if current recording track is different from the one clicked in contextMenu", then close the current one and resume the contextMenu one.

So maybe something like :

case R.id.trackmgr_contextmenu_resume:
    // let's activate the track and start the TrackLogger activity
    if (currentTrackId != contextMenuSelectedTrackid) {
        // Resume a track that is not currently recording. Stop all recording and resume track choose by user
        setActiveTrack(contextMenuSelectedTrackid);
    }
    // ...

Can you please try/debug this, to see if it make "more sens". Also feel free to add more comments in code 👍
TY for your PR

[Binnette]

See previous comment

[miltonials]

Hi @Binnette! Thank you for your suggestions. I’ve reviewed and implemented them, and the logic works smoothly

[Binnette]

TY @miltonials