devs/foxbox
1
0
Fork 0
forked from len0rd/rockbox
Code Pull requests Activity

Check selection before trying to create talk files.

Browse source 
This prevents users from accidentially trying to create talk files for nothing.

Change-Id: I92eae0e0dc14a772c786bcdad67f404fe3786376
This commit is contained in:
Dominik Riebeling 2012-10-06 11:30:26 +02:00
parent 4d96b4c7b0
commit a2dd1625bc
1 changed files with 10 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

14
rbutil/rbutilqt/installtalkwindow.cpp
View file

@ -87,12 +87,20 @@ void InstallTalkWindow::change()
void InstallTalkWindow::accept() void InstallTalkWindow::accept()
{ {
saveSettings();
QStringList foldersToTalk
= RbSettings::value(RbSettings::TalkFolders).toStringList();
if(foldersToTalk.size() == 0) {
QMessageBox::information(this, tr("Empty selection"),
tr("No files or folders selected. Please select files or "
"folders first."));
return;
}
logger = new ProgressLoggerGui(this); logger = new ProgressLoggerGui(this);
saveSettings();
connect(logger,SIGNAL(closed()),this,SLOT(close())); connect(logger,SIGNAL(closed()),this,SLOT(close()));
logger->show(); logger->show();
saveSettings();
talkcreator->setMountPoint(RbSettings::value(RbSettings::Mountpoint).toString()); talkcreator->setMountPoint(RbSettings::value(RbSettings::Mountpoint).toString());
@ -108,8 +116,6 @@ void InstallTalkWindow::accept()
connect(talkcreator, SIGNAL(logProgress(int, int)), logger, SLOT(setProgress(int, int))); connect(talkcreator, SIGNAL(logProgress(int, int)), logger, SLOT(setProgress(int, int)));
connect(logger,SIGNAL(aborted()),talkcreator,SLOT(abort())); connect(logger,SIGNAL(aborted()),talkcreator,SLOT(abort()));
QStringList foldersToTalk
= RbSettings::value(RbSettings::TalkFolders).toStringList();
for(int i = 0; i < foldersToTalk.size(); i++) { for(int i = 0; i < foldersToTalk.size(); i++) {
qDebug() << "[InstallTalkWindow] creating talk files for folder" qDebug() << "[InstallTalkWindow] creating talk files for folder"
<< foldersToTalk.at(i); << foldersToTalk.at(i);