author | llornkcor <llornkcor> | 2002-07-02 11:56:46 (UTC) |
---|---|---|
committer | llornkcor <llornkcor> | 2002-07-02 11:56:46 (UTC) |
commit | 2ffc9e4a46ec40075e6ee55e4583120e6964168d (patch) (side-by-side diff) | |
tree | 0a38d56cd249d71075a9cb784e3461b3dda1551a /library | |
parent | 75b6c4bb13fcc6d54a249b7885b212f473d654a3 (diff) | |
download | opie-2ffc9e4a46ec40075e6ee55e4583120e6964168d.zip opie-2ffc9e4a46ec40075e6ee55e4583120e6964168d.tar.gz opie-2ffc9e4a46ec40075e6ee55e4583120e6964168d.tar.bz2 |
attempt to fix bug #95, now it checks for existing file before trying to delete
-rw-r--r-- | library/lnkproperties.cpp | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/library/lnkproperties.cpp b/library/lnkproperties.cpp index 28a71e9..bd43e21 100644 --- a/library/lnkproperties.cpp +++ b/library/lnkproperties.cpp @@ -114,38 +114,39 @@ LnkProperties::LnkProperties( AppLnk* l, QWidget* parent ) d->preload->setChecked( apps.contains(l->exec()) ); if ( Global::isBuiltinCommand(lnk->exec()) ) d->preload->hide(); // builtins are always fast currentLocation = 0; // apps not movable (yet) } setCaption( l->file()); } LnkProperties::~LnkProperties() { } void LnkProperties::unlinkLnk() { if ( QPEMessageBox::confirmDelete( this, tr("Delete"), lnk->name() ) ) { - lnk->removeFiles(); - if ( QFile::exists(lnk->file()) ) { - QMessageBox::warning( this, tr("Delete"), tr("File deletion failed.") ); - } else { - reject(); - } + if ( QFile::exists(lnk->file()) ) + lnk->removeFiles(); + if ( QFile::exists(lnk->file()) ) { + QMessageBox::warning( this, tr("Delete"), tr("File deletion failed.") ); + } else { + reject(); + } } } /* * remove only the link from documents, not also the file */ void LnkProperties::unlinkIcon() { if ( QPEMessageBox::confirmDelete( this, tr("Delete Icon and leave file"), lnk->name() ) ) { lnk->removeLinkFile(); if ( QFile::exists(lnk->linkFile()) ) { QMessageBox::warning( this, tr("Delete"), tr("Icon deletion failed.") ); } else { reject(); } |