summaryrefslogtreecommitdiffstats
path: root/mappingtreewidget.cpp
diff options
context:
space:
mode:
authorArno <am@disconnect.de>2012-10-27 13:39:25 +0200
committerArno <am@disconnect.de>2012-10-27 13:39:25 +0200
commit97e8c68d7a92d6ccdb5f6ead8bba0d0af58efb3b (patch)
tree9cf27100410fe74b4794927ef1171301b7a1ceb5 /mappingtreewidget.cpp
parentdc7cc269a4fffe43bdac2ae2dc6548fbd1e3bb5e (diff)
downloadSheMov-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 'mappingtreewidget.cpp')
-rw-r--r--mappingtreewidget.cpp5
1 files changed, 2 insertions, 3 deletions
diff --git a/mappingtreewidget.cpp b/mappingtreewidget.cpp
index e9d0c93..fd3ebe6 100644
--- a/mappingtreewidget.cpp
+++ b/mappingtreewidget.cpp
@@ -94,13 +94,12 @@ MappingTreeWidget::MappingTreeWidget(QWidget *parent) : QWidget(parent){
}
MappingData MappingTreeWidget::selectedItem() const {
- MappingData retval = { -1, -1, -1, QString(), QList<QStringList>() };
QModelIndex sel = selected();
if(!sel.isValid()){
- return retval;
+ return MappingData();
}
QModelIndex real = mProxy->mapToSource(sel);
- return mModel->mappingDataFromIndex(real);
+ return mModel->mappingDataFromIndex(real);
}
void MappingTreeWidget::addChild(){