kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-08-22 01:40:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Don't session-restore invalid paths"

Browse source 
This reverts commit c733b3aa97.

There are people who prefer the old behavior of session-restoring all
paths including the ones which are currently inaccessible. The idea is
that Dolphin is like a web browser; if a website is not currently
available, the browser tells you that instead of silently discarding the
URL and showing you the homepage. Doing this can be considered a form
of data loss.

In addition, the reason for the change was to prevent Dolphin from
showing invalid paths on launch for unmounted removable disks, but that
specific case was fixed separately by switching to ~ for any tabs/views
that were showing locations on a just-unmounted removable disk. So you
will never run into the problem if yo unmount a disk in Dolphin and then
close it.

Accordingly, let's revert the change because it causes more problems than
it resolves.

BUG: 439864
CCBUG: 427619
FIXED-IN: 21.08
This commit is contained in:
Nate Graham 2021-07-28 12:54:10 -06:00
parent 0cdcf041cf
commit ea7d4f059d
3 changed files with 0 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/dolphinmainwindow.cpp
View file

@ -239,20 +239,6 @@ QVector<DolphinViewContainer*> DolphinMainWindow::viewContainers() const
return viewContainers;
}
void DolphinMainWindow::setViewsWithInvalidPathsToHome()
{
const QVector<DolphinViewContainer*> theViewContainers = viewContainers();
for (DolphinViewContainer *viewContainer : theViewContainers) {
// Only consider local dirs, not remote locations and abstract protocols
if (viewContainer->url().isLocalFile()) {
if (!QFileInfo::exists(viewContainer->url().toLocalFile())) {
viewContainer->setUrl(QUrl::fromLocalFile(QDir::homePath()));
}
}
}
}
void DolphinMainWindow::openDirectories(const QList<QUrl>& dirs, bool splitView)
{
m_tabWidget->openDirectories(dirs, splitView);

6
src/dolphinmainwindow.h
View file

@ -105,12 +105,6 @@ public:
*/
void setViewsToHomeIfMountPathOpen(const QString& mountPath);
/**
* Sets any of the window's view containers which are currently displaying
* invalid locations to the home path
*/
void setViewsWithInvalidPathsToHome();
bool isFoldersPanelEnabled() const;
bool isInformationPanelEnabled() const;

4
src/main.cpp
View file

@ -204,10 +204,6 @@ int main(int argc, char **argv)
if (startedWithURLs) {
mainWindow->openDirectories(urls, splitView);
}
// Now handle invalid locations in the set of active views to
// avoid issues like https://bugs.kde.org/show_bug.cgi?id=427619
mainWindow->setViewsWithInvalidPathsToHome();
} else {
qCWarning(DolphinDebug) << "Unknown class " << className << " in session saved data!";
}