Review Request 126917: Fix check whether window is closed by session management

Wolfgang Bauer wbauer at tmo.at
Thu Jan 28 20:05:10 GMT 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126917/
-----------------------------------------------------------

(Updated Jan. 28, 2016, 8:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for Dolphin.


Changes
-------

Submitted with commit bf1dbee1d6a709983d9f5034c61e2f005fdc6aee by Wolfgang Bauer to branch Applications/15.12.


Bugs: 353548
    https://bugs.kde.org/show_bug.cgi?id=353548


Repository: dolphin


Description
-------

qApp->isSessionRestored() specifies whether the application has been *restored* by session management. This is obviously wrong here. Use qApp->isSavingSession() instead.


Diffs
-----

  src/dolphinmainwindow.cpp 81c55e8 

Diff: https://git.reviewboard.kde.org/r/126917/diff/


Testing
-------

Open multiple tabs and logout. Dolphin doesn't ask for confirmation any more.
It still asks when closing the window manually.


Thanks,

Wolfgang Bauer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20160128/d1026e07/attachment.htm>


More information about the kfm-devel mailing list