D29326: Bookmarks Runner: Fix extraction of firefox profile

Stefan BrĂ¼ns noreply at phabricator.kde.org
Fri May 1 20:23:51 BST 2020


bruns requested changes to this revision.
bruns added a comment.
This revision now requires changes to proceed.


  The summary needs some text which works without any images
  
  The code needs some comment which tells how the selection logic works and why it is as it is.

INLINE COMMENTS

> firefox.cpp:184
>          } else {
> -            // There are multiple profiles, find the default one
> +            const QStringList installConfig = firefoxProfile.groupList().filter(QRegularExpression("Install.*"));
> +            if (installConfig.size() == 1) {

Missing start of string anchor (^)

> firefox.cpp:185
> +            const QStringList installConfig = firefoxProfile.groupList().filter(QRegularExpression("Install.*"));
> +            if (installConfig.size() == 1) {
> +                profilePath = firefoxProfile.group(installConfig.first()).readEntry("Default");

Can this ever be > 1?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D29326

To: alex, #plasma, ngraham, bruns, broulik
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200501/ff9ba65c/attachment.html>


More information about the Plasma-devel mailing list