Commit graph

61 commits

Author SHA1 Message Date
Ivailo Monev
b83771b4c2 kdecore: handle local file paths with and without trailing slash in KUrl::upUrl()
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-26 05:42:51 +03:00
Ivailo Monev
181e119ecc kdecore: KUrl::equals() rework to use KUrl::url() for the comparison
same thing (all URL components are compared) except the trailing slash
option can be re-used for it essentially adding one more option (to add
trailing slash when comparing)

also fixed and tested KUrl::upUrl() for some cases

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-26 01:30:23 +03:00
Ivailo Monev
180f34d2e3 kdecore: fix null/empty KUrl comparison and test it
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-26 00:34:45 +03:00
Ivailo Monev
b1c7c3b392 kdecore: move KUrl code where it belongs
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-26 00:10:11 +03:00
Ivailo Monev
47d0b5a297 kdecore: remove unused KUrl metadata
was used to pass the referrer and while it is still possible to
reimplement it realisticly speaking it is impossible to preserve the
referrer (and other HTTP(S) header metadata) because it will be lost in
transition in some cicumstances such as when drag-n-drop is done to
application unaware of application/x-kio-metadata

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 21:13:59 +03:00
Ivailo Monev
e4a4798d60 kdecore: remove redundant KUrl::fromMimeDataByteArray() method
calls KUrl constructor with the unmodified str argument in both cases

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 19:54:51 +03:00
Ivailo Monev
03bd3652c3 kdecore: compiler warning fix
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 19:36:10 +03:00
Ivailo Monev
865711ae83 kdecore: KUrl::url() optimization
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 18:51:15 +03:00
Ivailo Monev
40da298881 kdecore: never output file:// for local files
there is a fix for that legacy thing

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 18:15:06 +03:00
Ivailo Monev
cb8a624390 kdecore: fix recursion due to kCheckLocalFile()
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 17:47:18 +03:00
Ivailo Monev
54ee8ee1cd kdecore: clarfication on the note in KUrl test
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 16:50:36 +03:00
Ivailo Monev
d63bb71869 kdecore: rename constant for consistency
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 16:36:45 +03:00
Ivailo Monev
f1df519d20 kdecore: test qHash() for KUrl
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 06:31:53 +03:00
Ivailo Monev
7e056ee4a1 kdecore: use KUrl::url() for the hash
otherwise different user, password, host or port will not result in
different hash, duh

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 05:11:39 +03:00
Ivailo Monev
0917277201 kdecore: remove unused KUrl::CompareWithoutFragment enum
doubles as optimization for KUrl comparisons

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 05:05:08 +03:00
Ivailo Monev
94a5a3f641 kdecore: remove redundant KUrl destructor
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 04:00:49 +03:00
Ivailo Monev
a7fe953ca9 kdecore: ensure KUrl::url() handles only full paths to local files specially
it's dirty, it's magic, it's a fix

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-25 03:15:21 +03:00
Ivailo Monev
728dc0838e kdecore: remove unused headers inclusions in kurl source file
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 23:20:20 +03:00
Ivailo Monev
13fdc7b7cd kdecore: do not prettify the URL in KUrl::pathOrUrl()
here is how to make browsers barf - check the tooltip and where the browser
thinks the URL points to:
https://ibb.co/cyhwppK

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 23:13:38 +03:00
Ivailo Monev
ef02376c35 kdecore: add KUrl query and fragment test
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 21:25:20 +03:00
Ivailo Monev
0a5fadde75 kdecore: do not change the scheme from KUrl::cleanPath()
the fact that many KUrl methods change it (by calling KUrl::setPath()) is
just bonkers but that's how it was implemented - to convert to file scheme
from setters

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 20:44:03 +03:00
Ivailo Monev
78c109df85 kdecore: more compatible KUrl
abusing the QUrl path for local files

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 20:10:09 +03:00
Ivailo Monev
8f348b4c5b kdecore: remove FIXME for KUrl
so far so good:
https://ibb.co/TYN4z3x

now on to the quircks..

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 03:00:21 +03:00
Ivailo Monev
ea02aa2c1a kdecore: fix regression since 7b8ff71eba
oops

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-24 00:53:35 +03:00
Ivailo Monev
341e16d4c6 kdecore: update comment in KUrl::prettyUrl()
konqueror is no more

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 23:58:08 +03:00
Ivailo Monev
5e4732f809 kdecore: KUrl::List from QList<QUrl> optimization
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 23:55:19 +03:00
Ivailo Monev
c81c3bd71d kdecore use the new QUrl::AddTrailingSlash enum
requires the following commit from Katie repo:
3369835da10412451d082713433321a72f25e0df

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 23:08:09 +03:00
Ivailo Monev
7b8ff71eba kdecore: KUrl rewrite to output encoded strings
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 22:33:18 +03:00
Ivailo Monev
b4861f86d5 kdecore: rename variable in uriListData()
bad variable naming - clashes with the function name and inconsistent

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 18:22:41 +03:00
Ivailo Monev
840f3a3b68 kdecore: assume strings passed to KUrl constructors are percentage-encoded
for compat, test case for it is KUrlMimeTest::testURLList()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 18:14:39 +03:00
Ivailo Monev
30cd526513 kdecore: set the scheme to file when it KUrl is constructor from string and it is empty
the test case for it is "local file 3" case of KUrl::cleanPath() test and
the KUrlNavigator test

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 17:53:20 +03:00
Ivailo Monev
34b75e4f7f kdecore: different take on KUrl::upUrl()
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 13:08:49 +03:00
Ivailo Monev
fab969cf3e kdecore: remove most of the KUrl compat methods
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 12:26:07 +03:00
Ivailo Monev
10eb45205c kdecore: move static KUrl functions to the top
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 11:47:07 +03:00
Ivailo Monev
1e99d802a7 kdecore: KUrl::List to QList<QUrl> conversion optimization
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 11:46:34 +03:00
Ivailo Monev
3cdc7da129 kdecore: simplify KUrl by using QDir::cleanPath() and QUrl::resolved()
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-23 11:30:35 +03:00
Ivailo Monev
639d8def3b generic: adjust to QUrl changes
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-22 15:23:29 +03:00
Ivailo Monev
a424284337 kdecore: remove unused and redundant KUrl methods
plus some disabled bits

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-22 13:42:25 +03:00
Ivailo Monev
4966846ed7 generic: drop sub-URL support
was used by the archive KIO slave for example (which is no longer
maintained)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-06 08:34:19 +03:00
Ivailo Monev
dcdc02baf2 kdecore: remove archive KIO slaves leftovers
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-01-02 07:54:18 +02:00
Ivailo Monev
028414cb07 kdecore: KDebug rewrite
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-10-22 19:17:52 +03:00
Ivailo Monev
894a2d52bb kdecore: remove KLocale getters related to text codec
see the comment in KLocalePrivate::initEncoding()

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-09-30 20:00:04 +03:00
Ivailo Monev
d3cb89b9e2 kdecore: do not deep-copy objects when iterating
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-09-21 04:09:59 +03:00
Ivailo Monev
3422ca4413 generic: get rid of KGLobal::staticQString()
global statics are static objects and the method is using one

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-02-18 19:49:54 +02:00
Ivailo Monev
da9a1745c5 generic: remove most compiler checks
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2021-01-09 16:39:44 +02:00
Ivailo Monev
6c67e5adf8 generic: make use of QHostInfo::localHostName()
Signed-off-by: Ivailo Monev <xakepa10@laimg.moc>
2019-06-21 19:56:58 +00:00
Ivailo Monev
885e632451 kdecore: compiler warnings fix
Signed-off-by: Ivailo Monev <xakepa10@laimg.moc>
2019-05-15 21:52:19 +00:00
Ivailo Monev
82376a6c02 kdecore: KUrl cleanup
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2017-08-03 11:20:08 +00:00
Ivailo Monev
c334b5b1b0 kdecore: remove overload for KGlobal::staticQString()
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2016-05-26 15:51:32 +00:00
Ivailo Monev
d1c90f17ea kdecore: fix bug where KUrl::setPath() can set "file" scheme unnecessaraly
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2016-04-24 17:50:43 +00:00