Skip to content
Commit 93f0f5ee authored by Luca Weiss's avatar Luca Weiss Committed by David Edmundson
Browse files

Fix favicons in firefox bookmarks runner

Summary:
The favicons are located in the file favicons.sqlite file since FF 37 and the old table was removed in FF 41 resulting in crashes because the table wasn't found.

* Update SQL to match the tables in FF 58
* query.bindValue() needs a double colon in the string, see https://doc.qt.io/qt-5/qsqlquery.html#prepare
* Fix intendation (tab->spaces) in one line

BUG: 363136

Test Plan:
Firefox bookmark favicons now work and don't crash Krunner and Kickoff
{F5715653}

{F5715654}

Reviewers: #plasma, broulik, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: ngraham, elvisangelaccio, davidedmundson, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D10610
parent 4149b474
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