Summary: | cervisa removes all local files | ||
---|---|---|---|
Product: | [Applications] cervisia | Reporter: | colesen |
Component: | general | Assignee: | Bernd Gehrmann <bernd> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
colesen
2003-02-01 04:16:08 UTC
I was a little too fast. First of all the above should have been "Click the Cervisa toolbar button, select some files and select "Remove from repository". Secondly, I discovered that the (my?) problem was caused by the entire directory already having been preselected (highlighted on yellow background) upon entry into Cervisa (i.e. it looks to me like Cervisa preselects the entire directory by default). I'd have preferred that it did not do that. Subject: kdesdk/cervisia CVS commit by cloose: The dot for the root directory of the repository is hard to see in the listbox of the dialog, so we convert it to the absolut path. This way it's less likely that you accidentally remove all files from the repository. CCMAIL: 53799-done@bugs.kde.org M +13 -1 addremovedlg.cpp 1.2 --- kdesdk/cervisia/addremovedlg.cpp #1.1:1.2 @@ -15,4 +15,5 @@ #include "addremovedlg.h" +#include <qfileinfo.h> #include <qlabel.h> #include <qlayout.h> @@ -54,4 +55,15 @@ AddRemoveDialog::AddRemoveDialog(ActionT void AddRemoveDialog::setFileList(const QStringList &list) { + // the dot for the root directory is hard to see, so + // we convert it to the absolut path + if( list.find(".") != list.end() ) + { + QStringList copy(list); + int idx = copy.findIndex("."); + copy[idx] = QFileInfo(".").absFilePath(); + + listbox->insertStringList(copy); + } + else listbox->insertStringList(list); } |