Accepting request 717529 from home:wolfi323:test

- Add kde408632.patch to fix kde-open5 incorrectly handling port
  numbers in URLs (boo#1138956, kde#408632)

OBS-URL: https://build.opensuse.org/request/show/717529
OBS-URL: https://build.opensuse.org/package/show/KDE:Frameworks5/kde-cli-tools5?expand=0&rev=239
This commit is contained in:
Luca Beltrame 2019-07-22 10:41:13 +00:00 committed by Git OBS Bridge
parent fb8e8b8e73
commit 0498fcd4b1
3 changed files with 48 additions and 0 deletions

View File

@ -1,3 +1,9 @@
-------------------------------------------------------------------
Sat Jul 20 19:09:02 UTC 2019 - wbauer@tmo.at
- Add kde408632.patch to fix kde-open5 incorrectly handling port
numbers in URLs (boo#1138956, kde#408632)
-------------------------------------------------------------------
Tue Jul 9 17:29:05 UTC 2019 - fabian@ritter-vogt.de

View File

@ -32,6 +32,8 @@ Source2: plasma.keyring
%endif
# PATCH-FIX-OPENSUSE kdesu-add-some-i18n-love.patch -- boo#852256
Patch0: kdesu-add-some-i18n-love.patch
# PATCH-FIX-UPSTREAM
Patch1: kde408632.patch
BuildRequires: extra-cmake-modules >= 1.3.0
BuildRequires: kf5-filesystem
BuildRequires: xz

40
kde408632.patch Normal file
View File

@ -0,0 +1,40 @@
From 42ef318a9d1c454f96b60181d8231a59233720ea Mon Sep 17 00:00:00 2001
From: Gleb Popov <6yearold@gmail.com>
Date: Thu, 18 Jul 2019 13:00:19 +0400
Subject: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs
starting with remote schemes.
BUG: 408632
Test Plan: `kioclient5 exec http://localhost:9000` now works correctly.
Subscribers: plasma-devel, #frameworks
Differential Revision: https://phabricator.kde.org/D22525
---
kioclient/urlinfo.h | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/kioclient/urlinfo.h b/kioclient/urlinfo.h
index b9635c7..ebe0419 100644
--- a/kioclient/urlinfo.h
+++ b/kioclient/urlinfo.h
@@ -52,6 +52,15 @@ public:
}
/**
+ * if the path starts with http:// or any other scheme, except file://
+ * we also don't want to do anything with URL
+ */
+ if (!QUrl(pathOrUrl).isLocalFile()) {
+ url = QUrl::fromUserInput(pathOrUrl, QDir::currentPath(), QUrl::DefaultResolution);
+ return;
+ }
+
+ /**
* ok, the path as is, is no existing file, now, cut away :xx:yy stuff as cursor
* this will make test:50 to test with line 50
*/
--
cgit v1.1