feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

VcsOutputWindow: Use FilePath for repository

Browse Source 
Change-Id: Ie6669c94a1fa29e48524b9338e74413bb0830229
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
This commit is contained in:
Jarek Kobus
2022-08-03 13:25:51 +02:00
parent 175867e54e
commit 4edb238073
6 changed files with 28 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/plugins/git/gitclient.cpp
View File

@@ -1144,7 +1144,7 @@ void GitClient::merge(const FilePath &workingDirectory, const QStringList &unmer
void GitClient::status(const FilePath &workingDirectory) const
{
VcsOutputWindow::setRepository(workingDirectory.toString());
VcsOutputWindow::setRepository(workingDirectory);
VcsCommand *command = vcsExec(workingDirectory, {"status", "-u"}, nullptr, true);
connect(command, &VcsCommand::finished, VcsOutputWindow::instance(),
&VcsOutputWindow::clearRepository, Qt::QueuedConnection);

2
src/plugins/subversion/subversionplugin.cpp
View File

@@ -874,7 +874,7 @@ void SubversionPluginPrivate::svnStatus(const FilePath &workingDir, const QStrin
args << SubversionClient::addAuthenticationOptions(m_settings);
if (!relativePath.isEmpty())
args.append(SubversionClient::escapeFile(relativePath));
VcsOutputWindow::setRepository(workingDir.toString());
VcsOutputWindow::setRepository(workingDir);
runSvn(workingDir, args, VcsCommand::ShowStdOut | VcsCommand::ShowSuccessMessage);
VcsOutputWindow::clearRepository();
}

2
src/plugins/vcsbase/vcsbaseclient.cpp
View File

@@ -495,7 +495,7 @@ void VcsBaseClient::status(const FilePath &workingDir,
{
QStringList args(vcsCommandString(StatusCommand));
args << extraOptions << file;
VcsOutputWindow::setRepository(workingDir.toString());
VcsOutputWindow::setRepository(workingDir);
VcsCommand *cmd = createCommand(workingDir, nullptr, VcsWindowOutputBind);
connect(cmd, &VcsCommand::finished,
VcsOutputWindow::instance(), &VcsOutputWindow::clearRepository,

2
src/plugins/vcsbase/vcscommand.cpp
View File

@@ -120,7 +120,7 @@ VcsCommand::VcsCommand(const FilePath &workingDirectory, const Environment &envi
{
connect(&d->m_watcher, &QFutureWatcher<void>::canceled, this, &VcsCommand::cancel);
VcsOutputWindow::setRepository(d->m_defaultWorkingDirectory.toString());
VcsOutputWindow::setRepository(d->m_defaultWorkingDirectory);
VcsOutputWindow *outputWindow = VcsOutputWindow::instance(); // Keep me here, just to be sure it's not instantiated in other thread
connect(this, &VcsCommand::append, outputWindow, [outputWindow](const QString &t) {
outputWindow->append(t);

53
src/plugins/vcsbase/vcsoutputwindow.cpp
View File

@@ -83,11 +83,11 @@ const char zoomSettingsKey[] = "Vcs/OutputPane/Zoom";
class RepositoryUserData : public QTextBlockUserData
{
public:
explicit RepositoryUserData(const QString &repo) : m_repository(repo) {}
const QString &repository() const { return m_repository; }
explicit RepositoryUserData(const FilePath &repository) : m_repository(repository) {}
const FilePath &repository() const { return m_repository; }
private:
const QString m_repository;
const FilePath m_repository;
};
// A plain text edit with a special context menu containing "Clear"
@@ -97,9 +97,9 @@ class OutputWindowPlainTextEdit : public Core::OutputWindow
public:
explicit OutputWindowPlainTextEdit(QWidget *parent = nullptr);
void appendLines(const QString &s, const QString &repository = QString());
void appendLines(const QString &s, const FilePath &repository = {});
void appendLinesWithStyle(const QString &s, VcsOutputWindow::MessageStyle style,
const QString &repository = QString());
const FilePath &repository = {});
VcsOutputLineParser *parser();
protected:
@@ -108,7 +108,7 @@ protected:
private:
void setFormat(VcsOutputWindow::MessageStyle style);
QString identifierUnderCursor(const QPoint &pos, QString *repository = nullptr) const;
QString identifierUnderCursor(const QPoint &pos, FilePath *repository = nullptr) const;
OutputFormat m_format;
VcsOutputLineParser *m_parser = nullptr;
@@ -135,7 +135,7 @@ static inline int firstWordCharacter(const QString &s, int startPos)
return 0;
}
QString OutputWindowPlainTextEdit::identifierUnderCursor(const QPoint &widgetPos, QString *repository) const
QString OutputWindowPlainTextEdit::identifierUnderCursor(const QPoint &widgetPos, FilePath *repository) const
{
if (repository)
repository->clear();
@@ -169,25 +169,23 @@ void OutputWindowPlainTextEdit::contextMenuEvent(QContextMenuEvent *event)
const QString href = anchorAt(event->pos());
QMenu *menu = href.isEmpty() ? createStandardContextMenu(event->pos()) : new QMenu;
// Add 'open file'
QString repository;
const QString token = identifierUnderCursor(event->pos(), &repository);
if (!repository.isEmpty()) {
FilePath repo;
const QString token = identifierUnderCursor(event->pos(), &repo);
if (!repo.isEmpty()) {
if (VcsOutputLineParser * const p = parser()) {
if (!href.isEmpty())
p->fillLinkContextMenu(menu, FilePath::fromString(repository), href);
p->fillLinkContextMenu(menu, repo, href);
}
}
QAction *openAction = nullptr;
if (!token.isEmpty()) {
// Check for a file, expand via repository if relative
QFileInfo fi(token);
if (!repository.isEmpty() && !fi.isFile() && fi.isRelative())
fi = QFileInfo(repository + '/' + token);
if (fi.isFile()) {
if (!repo.isEmpty() && !repo.isFile() && repo.isRelativePath())
repo = repo.pathAppended(token);
if (repo.isFile()) {
menu->addSeparator();
openAction = menu->addAction(VcsOutputWindow::tr("Open \"%1\"").
arg(QDir::toNativeSeparators(fi.fileName())));
openAction->setData(fi.absoluteFilePath());
openAction = menu->addAction(VcsOutputWindow::tr("Open \"%1\"").arg(repo.nativePath()));
openAction->setData(repo.absoluteFilePath().toString());
}
}
QAction *clearAction = nullptr;
@@ -217,7 +215,7 @@ void OutputWindowPlainTextEdit::handleLink(const QPoint &pos)
const QString href = anchorAt(pos);
if (href.isEmpty())
return;
QString repository;
FilePath repository;
identifierUnderCursor(pos, &repository);
if (repository.isEmpty()) {
OutputWindow::handleLink(pos);
@@ -226,10 +224,10 @@ void OutputWindowPlainTextEdit::handleLink(const QPoint &pos)
if (outputFormatter()->handleFileLink(href))
return;
if (VcsOutputLineParser * const p = parser())
p->handleVcsLink(FilePath::fromString(repository), href);
p->handleVcsLink(repository, href);
}
void OutputWindowPlainTextEdit::appendLines(const QString &s, const QString &repository)
void OutputWindowPlainTextEdit::appendLines(const QString &s, const FilePath &repository)
{
if (s.isEmpty())
return;
@@ -251,7 +249,7 @@ void OutputWindowPlainTextEdit::appendLines(const QString &s, const QString &rep
void OutputWindowPlainTextEdit::appendLinesWithStyle(const QString &s,
VcsOutputWindow::MessageStyle style,
const QString &repository)
const FilePath &repository)
{
setFormat(style);
@@ -299,7 +297,7 @@ class VcsOutputWindowPrivate
{
public:
Internal::OutputWindowPlainTextEdit widget;
QString repository;
FilePath repository;
const QRegularExpression passwordRegExp = QRegularExpression("://([^@:]+):([^@]+)@");
};
@@ -473,14 +471,9 @@ VcsOutputWindow *VcsOutputWindow::instance()
return m_instance;
}
QString VcsOutputWindow::repository() const
void VcsOutputWindow::setRepository(const FilePath &repository)
{
return d->repository;
}
void VcsOutputWindow::setRepository(const QString &r)
{
d->repository = r;
d->repository = repository;
}
void VcsOutputWindow::clearRepository()

5
src/plugins/vcsbase/vcsoutputwindow.h
View File

@@ -41,7 +41,6 @@ namespace Internal { class VcsPlugin; }
class VCSBASE_EXPORT VcsOutputWindow : public Core::IOutputPane
{
Q_OBJECT
Q_PROPERTY(QString repository READ repository WRITE setRepository)
public:
QWidget *outputWidget(QWidget *parent) override;
@@ -63,8 +62,6 @@ public:
static VcsOutputWindow *instance();
QString repository() const;
// Helper to consistently format log entries for commands as
// 'Executing <dir>: <cmd> <args>'. Hides well-known password option
// arguments.
@@ -80,7 +77,7 @@ public:
};
public slots:
static void setRepository(const QString &);
static void setRepository(const Utils::FilePath &repository);
static void clearRepository();
// Set the whole text.