Commit 68c1aaab authored by Alex Nemeth's avatar Alex Nemeth Committed by David Edmundson

Removed duplicate preview screenshots

Summary:
Now it checks if the preview screenshot URL is already in the list.
Sometimes it happens that preview1 and preview2 are the same URL.

Test Plan:
Before:
{F5669395}

After:
{F5669397}

Reviewers: apol, #discover_software_store, davidedmundson, abetts

Reviewed By: apol, #discover_software_store, davidedmundson, abetts

Subscribers: ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D10027
parent ecad96b0
......@@ -166,7 +166,7 @@ QString KNSResource::section()
static void appendIfValid(QList<QUrl>& list, const QUrl &value, const QUrl &fallback = {})
{
if (value.isValid() && !value.isEmpty())
if (value.isValid() && !value.isEmpty() && !list.contains(value))
list << value;
else if (!fallback.isEmpty())
appendIfValid(list, fallback);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment