Falkon/falkonOfficial
Falkon/falkonOfficial
1
Fork 0
mirror of https://invent.kde.org/network/falkon.git synced 2024-12-24 04:36:34 +01:00
Code Releases Activity

Use KIO::OpenFileManagerWindowJob to open folder

Browse Source 
BUG: 460997
FIXED-IN: 24.12.0

Signed-off-by: Juraj Oravec <jurajoravec@mailo.com>
This commit is contained in:
Juraj Oravec 2024-10-01 23:33:15 +02:00
parent 4772efb483
commit 37441945c4
5 changed files with 42 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
CMakeLists.txt
View File

@ -107,6 +107,9 @@ if (EXISTS "${CMAKE_SOURCE_DIR}/po")
find_package(KF6I18n REQUIRED)
endif()
# Mandatory: KIO
find_package(KF6KIO ${KF_MIN_VERSION} REQUIRED)
# Optional: GnomeKeyring
find_package(PkgConfig)
if (PKG_CONFIG_FOUND)
@ -116,11 +119,9 @@ if (PKG_CONFIG_FOUND)
endif()
endif()
# Optional: KWallet, KIO, KCrash, KCoreAddons
# Optional: KWallet, KCrash, KCoreAddons
find_package(KF6Wallet ${KF_MIN_VERSION} CONFIG)
set_package_properties(KF6Wallet PROPERTIES DESCRIPTION "KDE Frameworks Integration plugin" TYPE OPTIONAL)
find_package(KF6KIO ${KF_MIN_VERSION} CONFIG)
set_package_properties(KF6KIO PROPERTIES DESCRIPTION "KDE Frameworks Integration plugin" TYPE OPTIONAL)
find_package(KF6Crash ${KF_MIN_VERSION} CONFIG)
set_package_properties(KF6Crash PROPERTIES DESCRIPTION "KDE Frameworks Integration plugin" TYPE OPTIONAL)
find_package(KF6CoreAddons ${KF_MIN_VERSION} CONFIG)
@ -129,7 +130,7 @@ find_package(KF6Purpose ${KF_MIN_VERSION} CONFIG)
set_package_properties(KF6Purpose PROPERTIES DESCRIPTION "KDE Frameworks Integration plugin" TYPE OPTIONAL)
find_package(KF6JobWidgets ${KF_MIN_VERSION} CONFIG)
set_package_properties(KF6JobWidgets PROPERTIES DESCRIPTION "KDE Frameworks Integration plugin" TYPE OPTIONAL)
if (KF6Wallet_FOUND AND KF6KIO_FOUND AND KF6Crash_FOUND AND KF6CoreAddons_FOUND AND KF6Purpose_FOUND AND KF6JobWidgets_FOUND)
if (KF6Wallet_FOUND AND KF6Crash_FOUND AND KF6CoreAddons_FOUND AND KF6Purpose_FOUND AND KF6JobWidgets_FOUND)
set(ENABLE_KDE_FRAMEWORKS_INTEGRATION_PLUGIN TRUE)
endif()

1
src/lib/CMakeLists.txt
View File

@ -603,6 +603,7 @@ target_link_libraries(FalkonPrivate
Qt6::WebChannel
Qt6::Core5Compat
KF6::Archive
KF6::KIOGui
${OPENSSL_CRYPTO_LIBRARY}
)

20
src/lib/downloads/downloaditem.cpp
View File

@ -39,11 +39,6 @@
#include <QWebEngineDownloadRequest>
#include <QtWebEngineWidgetsVersion>
#ifdef Q_OS_WIN
#include "Shlwapi.h"
#include "shellapi.h"
#endif
//#define DOWNMANAGER_DEBUG
DownloadItem::DownloadItem(QListWidgetItem *item, QWebEngineDownloadRequest* downloadItem, const QString &path, const QString &fileName, bool openFile, DownloadManager* manager)
@ -354,19 +349,12 @@ void DownloadItem::openFile()
void DownloadItem::openFolder()
{
#ifdef Q_OS_WIN
QString winFileName = QSL("%1/%2").arg(m_path, m_fileName);
QString fileName = m_fileName;
if (m_downloading) {
winFileName.append(QSL(".download"));
fileName.append(QSL(".download"));
}
winFileName.replace(QLatin1Char('/'), QSL("\\"));
QString shExArg = QSL("/e,/select,\"") + winFileName + QSL("\"");
ShellExecute(NULL, NULL, TEXT("explorer.exe"), shExArg.toStdWString().c_str(), NULL, SW_SHOW);
#else
QDesktopServices::openUrl(QUrl::fromLocalFile(m_path));
#endif
QFileInfo info(m_path, fileName);
QzTools::openFolder({QUrl::fromLocalFile(info.absoluteFilePath())});
}
QUrl DownloadItem::url() const

30
src/lib/tools/qztools.cpp
View File

@ -45,6 +45,8 @@
#include <QUrlQuery>
#include <QtGuiVersion>
#include <KIO/OpenFileManagerWindowJob>
#ifdef QZ_WS_X11
#include <xcb/xcb.h>
#endif
@ -1037,3 +1039,31 @@ void QzTools::paintDropIndicator(QWidget *widget, const QRect &r)
p.drawLine(x + i, r.top(), x + i, r.bottom());
}
}
void QzTools::openFolder(const QList<QUrl> &urls)
{
/* Filter the URL list for existing files, otherwise it will cause crash */
QList<QUrl> validUrls;
for (const QUrl &url : urls) {
if (QFileInfo::exists(url.toLocalFile())) {
validUrls.append(url);
}
else {
qWarning() << "OpenFolder Filtered:" << url;
}
}
if (validUrls.isEmpty()) {
return;
}
auto *job = new KIO::OpenFileManagerWindowJob();
job->setHighlightUrls(validUrls);
job->start();
QObject::connect(job, &KJob::result, job, [&](KJob *job) {
if (job->error() != 0) {
qWarning() << "OpenFileManagerWindowJob:" << job->errorString();
}
});
}

2
src/lib/tools/qztools.h
View File

@ -98,6 +98,8 @@ public:
}
static void paintDropIndicator(QWidget *widget, const QRect &r);
static void openFolder(const QList<QUrl> &urls);
};
#endif // QZTOOLS_H