kde/dolphin
kde/dolphin
1
0
Fork 0
mirror of https://invent.kde.org/system/dolphin synced 2024-08-21 09:20:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make the view/Terminal Panel synchronization less error-prone

Browse source 
The previous solution could cause problems if the user navigates to a
different URL in one view, and then activates another split view very
quickly: the new active view might be switched to the same URL as the
first view, which is unwanted.

To fix this problem, we record a history of "cd" commands that Dolphin
sends to the Terminal Panel in a queue. If a currentDirectoryChanged
signal is received, and the new terminal directory is "dir", this patch
does the following:

1.  If the queue is empty, change the view URL to "dir".
2.  Otherwise, take the queue's head, and check if it is equal to
    "dir". If that is the case, ignore the signal and return.
3.  Go back to step 1.

This ensures that every currentDirectoryChange signal that is caused by
a "cd" that was sent from Dolphin to the terminal is ignored.

BUG: 339009
BUG: 314038
REVIEW: 120768
This commit is contained in:
Frank Reininghaus 2014-10-24 18:13:47 +02:00
parent 592e1730e2
commit d9700c4d43
2 changed files with 20 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
src/panels/terminal/terminalpanel.cpp
View file

@ -42,7 +42,8 @@ TerminalPanel::TerminalPanel(QWidget* parent) :
m_terminal(0),
m_terminalWidget(0),
m_konsolePart(0),
m_konsolePartCurrentDirectory()
m_konsolePartCurrentDirectory(),
m_sendCdToTerminalHistory()
{
m_layout = new QVBoxLayout(this);
m_layout->setMargin(0);
@ -161,7 +162,12 @@ void TerminalPanel::sendCdToTerminal(const QString& dir)
}
m_terminal->sendInput(" cd " + KShell::quoteArg(dir) + '\n');
m_konsolePartCurrentDirectory = dir;
// We want to ignore the currentDirectoryChanged(QString) signal, which we will receive after
// the directory change, because this directory change is not caused by a "cd" command that the
// user entered in the panel. Therefore, we have to remember 'dir'. Note that it could also be
// a symbolic link -> remember the 'canonical' path.
m_sendCdToTerminalHistory.enqueue(QDir(dir).canonicalPath());
if (m_clearTerminal) {
m_terminal->sendInput(" clear\n");
@ -182,16 +188,17 @@ void TerminalPanel::slotMostLocalUrlResult(KJob* job)
void TerminalPanel::slotKonsolePartCurrentDirectoryChanged(const QString& dir)
{
m_konsolePartCurrentDirectory = dir;
m_konsolePartCurrentDirectory = QDir(dir).canonicalPath();
// Only change the view URL if 'dir' is different from the current view URL.
// Note that the current view URL could also be a symbolic link to 'dir'
// -> use QDir::canonicalPath() to check that.
const KUrl oldUrl(url());
const KUrl newUrl(dir);
if (newUrl != oldUrl && dir != QDir(oldUrl.path()).canonicalPath()) {
emit changeUrl(newUrl);
// Only emit a changeUrl signal if the directory change was caused by the user inside the
// terminal, and not by sendCdToTerminal(QString).
while (!m_sendCdToTerminalHistory.empty()) {
if (m_konsolePartCurrentDirectory == m_sendCdToTerminalHistory.dequeue()) {
return;
}
}
emit changeUrl(dir);
}
#include "terminalpanel.moc"

3
src/panels/terminal/terminalpanel.h
View file

@ -22,6 +22,8 @@
#include <panels/panel.h>
#include <QQueue>
class TerminalInterfaceV2;
class QVBoxLayout;
class QWidget;
@ -82,6 +84,7 @@ private:
QWidget* m_terminalWidget;
KParts::ReadOnlyPart* m_konsolePart;
QString m_konsolePartCurrentDirectory;
QQueue<QString> m_sendCdToTerminalHistory;
};
#endif // TERMINALPANEL_H