whytofly
/
qt6-bb10
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

QMimeData: fix setData on URI list without ending CRLF 

Commit 8d127c5737 refactored this code
from QByteArray::split to manually iterate over the list (and
QStringTokenizer doesn't work for QByteArray), so we missed the last
element in the iteration.

Pick-to: 6.7
Fixes: QTBUG-124580
Change-Id: I455fe22ef4ad4b2f9b01fffd17c867f6ed350af7
Reviewed-by: Ahmad Samir <a.samirh78@gmail.com>
Reviewed-by: David Faure <david.faure@kdab.com>
bb10
Thiago Macieira 2024-04-21 14:12:48 -07:00
parent f5fa811e88
commit 038c199d59
2 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/corelib/kernel/qmimedata.cpp
View File

@ -90,6 +90,11 @@ static QList<QVariant> dataToUrls(QByteArrayView text)
if (from >= text.size())
break;
}
if (from != text.size()) {
const auto bav = QByteArrayView(begin + from, text.end()).trimmed();
if (!bav.isEmpty())
list.push_back(QUrl::fromEncoded(bav));
}
return list;
}

8
tests/auto/corelib/kernel/qmimedata/tst_qmimedata.cpp
View File

@ -85,6 +85,14 @@ void tst_QMimeData::data() const
QCOMPARE(mimeData.data("text/markdown"), QByteArray("vikings"));
QCOMPARE(mimeData.data("text/html"), QByteArray("ninjas"));
QCOMPARE(mimeData.data("text/plain"), QByteArray("pirates"));
// URI list
QByteArray list = "https://example.com/\r\nhttps://example.net/\r\nhttps://example.org/\r\n";
mimeData.setData("text/uri-list", list);
QCOMPARE(mimeData.data("text/uri-list"), list);
mimeData.setData("text/uri-list", list.chopped(2)); // without the ending CRLF
QCOMPARE(mimeData.data("text/uri-list"), list);
}
void tst_QMimeData::formats() const