Akregator: fix URL/path confusion which led to KIO warnings and possibly missing icons

Authored by dfaure on Apr 5 2020, 8:15 AM.



The faviconUrl stored in ~/.local/share/akregator/data was a path rather
than a URL.

It's also pretty pointless to then do a download job on that local file...

Test Plan

I've been having some trouble finding RSS sites with icons...
Please double-check.

Diff Detail

R201 Akregator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
dfaure requested review of this revision.Apr 5 2020, 8:15 AM
dfaure created this revision.
mlaurent accepted this revision.Apr 5 2020, 10:12 AM

it seems ok for me.

This revision is now accepted and ready to land.Apr 5 2020, 10:12 AM
This revision was automatically updated to reflect the committed changes.
Restricted Application added a project: KDE PIM. · View Herald TranscriptApr 5 2020, 11:08 AM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript