SIGABORT in dive site manager

[Lubomir I. Ivanov]

On 14 February 2015 at 17:58, Dirk Hohndel <dirk at hohndel.org> wrote:
> On Sat, Feb 14, 2015 at 04:00:49PM +0100, Henrik Brautaset Aronsen wrote:
>> On 14 Feb 2015, at 15:32, Lubomir I. Ivanov <neolit123 at gmail.com> wrote:
>> >
>> > oops, i was able to.
>> > we just need to make sure that the user can't close the logbook while
>> > editing the location info.
>> > or...the location info UI should close with the logbook.
>> >
>> > i will leave it to Tomaz and Dirk to decide what is best.
>
> I think we should prevent closing the logfile while in dive site edit
> mode. This is pretty straight forward to do - anyone care to send a patch?
>

patch attached.

i use this in "MainWindow::okToClose()":
currentApplicationState == "EditDiveSite")
...
return false;

because i think this is the new way of checking if something is being edited.

on top of it there is this similar check:
information()->isEditing()

but i assume this is the old way, so i didn't follow it.

lubomir
--
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-mainwindow.cpp-don-t-close-a-file-if-a-dive-site-is-.patch
Type: application/octet-stream
Size: 940 bytes
Desc: not available
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150214/6df31585/attachment.obj>