[PATCH] Re: Issue with dive merge
Robert Helling
helling at atdotde.de
Mon May 18 01:53:49 PDT 2015
> On 18.05.2015, at 10:00, Miika Turkia <miika.turkia at gmail.com> wrote:
>
> On Mon, May 18, 2015 at 10:56 AM, Robert Helling <helling at atdotde.de <mailto:helling at atdotde.de>> wrote:
>> Hi Christophe,
>>
>> On 18.05.2015, at 09:41, Christophe Démaret <christophe at demaret.eu> wrote:
>>
>>
>> Expected: Dives are merged
>> What actually happens: Nothing. Dives are not merged, no error message, no
>> crash.
>>
>>
>> Subsurface only merges dives with less than 30 minutes interval between
>> them. Change the starting time of the dive 178 by a little bit and the dives
>> will be merged.
>>
>> Maybe instead of silently failing, we should give an error message or better
>> ask the user if she still wants to merge the dives? Is there a situation
>> where we actually want to fail silently (e.g. merging many dives but amongst
>> those one the ones that make sense?!?)
>
> I vote for merging the dives, as the user has explicitly requested
> this to occur. However, a confirmation dialog when the interval is
> more than 30 minutes would make sense. Just make sure there are no
> multiple dialogs when merging multiple dives.
OK, here is a patch that does this.
Best
Robert
--
.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oO
Robert C. Helling Elite Master Course Theoretical and Mathematical Physics
Scientific Coordinator
Ludwig Maximilians Universitaet Muenchen, Dept. Physik
Phone: +49 89 2180-4523 Theresienstr. 39, rm. B339
http://www.atdotde.de
Enhance your privacy, use cryptography! My PGP keys have fingerprints
A9D1 A01D 13A5 31FA 6515 BB44 0820 367C 36BC 0C1D and
DCED 37B6 251C 7861 270D 5613 95C7 9D32 9A8D 9B8F
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150518/752b1196/attachment.html>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Ask-user-about-merging-dives-more-than-30m-apart.txt
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150518/752b1196/attachment.txt>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150518/752b1196/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150518/752b1196/attachment.sig>
More information about the subsurface
mailing list