andrius
/
partitionmanager
1
0
Fork 0
Code Pull Requests Releases Activity

Fixed the "remove existing mount point" bug 

Summary:
The QString "contains method" is also searching for empty QStrings.
Due to this all entries in /etc/fstab were removed . After this patch it is
searching for only non-empty strings in removeMountPoint() function.

Reviewers: stikonas

Reviewed By: stikonas

Tags: #kde_partition_manager

Differential Revision: https://phabricator.kde.org/D11542
This commit is contained in:
abhijeet sharma 2018-03-22 01:28:56 +05:30
parent 98209f90bc
commit a38a6cd35d
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/gui/editmountpointdialogwidget.cpp
View File

@ -163,9 +163,9 @@ void EditMountPointDialogWidget::removeMountPoint()
{
int i=0;
for (const auto &e : fstabEntries()) {
if(e.fsSpec().contains(partition().deviceNode()) || e.fsSpec().contains(partition().fileSystem().uuid()) ||
e.fsSpec().contains(partition().fileSystem().label()) || e.fsSpec().contains(partition().label()) || e.fsSpec().contains(partition().uuid()) )
{
if((e.fsSpec().contains(partition().deviceNode()) && !partition().deviceNode().isEmpty() ) || (e.fsSpec().contains(partition().fileSystem().uuid()) && !partition().fileSystem().uuid().isEmpty()) ||
(e.fsSpec().contains(partition().fileSystem().label()) && !partition().fileSystem().label().isEmpty()) || (e.fsSpec().contains(partition().label()) && !partition().label().isEmpty() ) || (e.fsSpec().contains(partition().uuid()) && !partition().uuid().isEmpty() ) )
{
fstabEntries().removeAt(i);
partition().setMountPoint(QString());
i--;