merge activity records

[dhiltonp]

It would be nice to have automatic facilities for merging 2 activities, for cases where either via accidental stop or crash an activity has had to be restarted.

[jonasoreland]

agree!

On Sun, Apr 26, 2015 at 9:47 PM, David Hilton [email protected]
wrote:

It would be nice to have automatic facilities for merging 2 activities,
for cases where either via accidental stop or crash an activity has had to
be restarted.

—
Reply to this email directly or view it on GitHub
#245.

[paradix]

would this be possible by simply adding a the ID of the previous or next activity and handling this at presentation layer?

[jonasoreland]

i think an simple update would work.
something like (roughly):

begin;
update location set activity_id = <new_activity_id>, lap = lap + (max lap
new activity_id) where activity_id = <old_activity_id>;
update lap set activity_id = <new_activity_id>, lap = lap + (max lap new
activity_id) where activity_id = <old_activity_id>;
update activity set total_time=total_time + (time old acitivity),
total_distance=(...) ... where activity_id = <new_activity_id>;
delete from activity where activity_id = <old_activity_id>;
commit;

I can do the SQL...but...we need a GUI to select which activities to
merge...

/Jonas

On Mon, Apr 27, 2015 at 10:56 AM, paradix [email protected] wrote:

would this be possible by simply adding a the ID of the previous or next
activity and handling this at presentation layer?

—
Reply to this email directly or view it on GitHub
#245 (comment)
.

[paradix]

Would be nice to make some plausibility checks before the user ruins his history.
i.e.

• checking start time of next against end time of previous activity
• checking sport types
• ...?

[jonasoreland]

yes,

On Mon, Apr 27, 2015 at 12:01 PM, paradix [email protected] wrote:

Would be nice to make some plausibility checks before the user ruins his
history.
i.e.

• checking start time of next against end time of previous activity
• checking sport types
• ...?

—
Reply to this email directly or view it on GitHub
#245 (comment)
.

[paradix]

I could try to do this, but at this point of the week I'm swamped with work.

[gerhardol]

#287 implements merging (needs some rework though). Closing issue

[dhiltonp]

It's weird to close this as the issue isn't completed.

[gerhardol]

The PR is open (there were 150 open issues before so the relevant issues were not found..)