diff options
author | Arno <am@disconnect.de> | 2012-10-27 13:39:25 +0200 |
---|---|---|
committer | Arno <am@disconnect.de> | 2012-10-27 13:39:25 +0200 |
commit | 97e8c68d7a92d6ccdb5f6ead8bba0d0af58efb3b (patch) | |
tree | 9cf27100410fe74b4794927ef1171301b7a1ceb5 /pictureswidget.cpp | |
parent | dc7cc269a4fffe43bdac2ae2dc6548fbd1e3bb5e (diff) | |
download | SheMov-97e8c68d7a92d6ccdb5f6ead8bba0d0af58efb3b.tar.gz SheMov-97e8c68d7a92d6ccdb5f6ead8bba0d0af58efb3b.tar.bz2 SheMov-97e8c68d7a92d6ccdb5f6ead8bba0d0af58efb3b.zip |
Bugfixes: don't select dirs and don't crash on write settings
Only select files when archiving pics from filemanager. While at it, I
noticed a crash on close when no item is selected in PicturesWidget. So
check if MappingData is valid on writing settings. Best way is
MappingData::isValid(), and for that I needed a explicit constructor.
Diffstat (limited to 'pictureswidget.cpp')
-rw-r--r-- | pictureswidget.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/pictureswidget.cpp b/pictureswidget.cpp index 92b0490..54d86e2 100644 --- a/pictureswidget.cpp +++ b/pictureswidget.cpp @@ -64,7 +64,9 @@ void PicturesWidget::showPicViewer(bool toggled){ void PicturesWidget::writeSettings(){ QSettings s; MappingData selected = mMappingTree->selectedItem(); - s.setValue("ui/selectedmapping", selected.path.first()); + if(selected.isValid()){ + s.setValue("ui/selectedmapping", selected.path.first()); + } mPictureView->writeHeaderConfig(); } |