Differential D24743 Diff 68260 kded-module/configvalueprovider.cpp

Changeset View

Standalone View

View Options

kded-module/configvalueprovider.cpp

  • This file was added.
1​/*
2​ * Copyright (C) 2019 Mikhail Zolotukhin <zomial@protonmail.com>
3​ *
4​ * This program is free software; you can redistribute it and/or
5​ * modify it under the terms of the GNU General Public License as
6​ * published by the Free Software Foundation; either version 2 of
7​ * the License or (at your option) version 3 or any later version
8​ * accepted by the membership of KDE e.V. (or its successor approved
9​ * by the membership of KDE e.V.), which shall act as a proxy
10​ * defined in Section 14 of version 3 of the license.
11​ *
12​ * This program is distributed in the hope that it will be useful,
13​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
14​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15​ * GNU General Public License for more details.
16​ *
17​ * You should have received a copy of the GNU General Public License
18​ * along with this program. If not, see <http://www.gnu.org/licenses/>.
19​ */
20​
21​#include <QFont>
22​#include <QString>
23​
24​#include <KConfig>
25​#include <KIconTheme>
26​#include <KSharedConfig>
27​#include <KConfigGroup>
28​
29​#include "configvalueprovider.h"
30​
31​QString ConfigValueProvider::getConfigFontName() const
32​{
33​ static const QFont defaultFont(QStringLiteral("Noto Sans"), 10);
34​
35​ KSharedConfigPtr config(KSharedConfig::openConfig(QStringLiteral("kdeglobals")));
ervinUnsubmitted
Done

I will contradict my previous comment about making those functions static or turning into a namespace. I think it'd be better to avoid creating and ditching those KSharedConfig instances at each call, so why not instantiate it when ConfigValueProvider is created?

You might have to load() from the config at right point in times though.

ervin: I will contradict my previous comment about making those functions static or turning into a…
36​ KConfigGroup configGroup(config->group(QStringLiteral("General")));
37​ QString fontAsString(configGroup.readEntry(QStringLiteral("font"), defaultFont.toString()));
38​ static QFont font;
39​ font.fromString(fontAsString);
broulikUnsubmitted
Done

theme() can return null

broulik: `theme()` can return null
40​ return font.family() + ' ' + font.styleName() + ' ' + QString::number(font.pointSize());
41​}
42​
43​QString ConfigValueProvider::getConfigIconThemeName() const
44​{
45​ KIconTheme *newIconTheme(KIconLoader::global()->theme());
46​ if (newIconTheme == nullptr) {
47​ return QStringLiteral("breeze");
48​ } else {
49​ return newIconTheme->internalName();
50​ }
51​}
52​
53​QString ConfigValueProvider::getConfigCursorThemeName() const
54​{
broulikUnsubmitted
Done

Pass true as second argument (which is the default), then it will return you a bool directly.

broulik: Pass `true` as second argument (which is the default), then it will return you a `bool`…
nicolasfellaUnsubmitted
Done

if (!newIconTheme)

nicolasfella: if (!newIconTheme)
55​ KSharedConfigPtr config(KSharedConfig::openConfig(QStringLiteral("kcminputrc")));
56​ KConfigGroup configGroup(config->group(QStringLiteral("Mouse")));
57​ return configGroup.readEntry(QStringLiteral("cursorTheme"));
58​}
59​
60​QString ConfigValueProvider::getIconsOnButtonsConfigValue() const
61​{
62​ KSharedConfigPtr config(KSharedConfig::openConfig(QStringLiteral("kdeglobals"), KConfig::NoGlobals));
63​ KConfigGroup configGroup(config->group(QStringLiteral("KDE")));
64​ bool kdeConfigValue(configGroup.readEntry(QStringLiteral("ShowIconsOnPushButtons"), true));
65​
66​ if (kdeConfigValue) {
67​ return QStringLiteral("1");
68​ } else {
69​ return QStringLiteral("0");
70​ }
71​}
72​
73​QString ConfigValueProvider::getIconsInMenusConfigValue() const
74​{
75​ KSharedConfigPtr config(KSharedConfig::openConfig(QStringLiteral("kdeglobals"), KConfig::NoGlobals));
76​ KConfigGroup configGroup(config->group(QStringLiteral("KDE")));
77​ bool kdeConfigValue(configGroup.readEntry(QStringLiteral("ShowIconsInMenuItems"), true));
78​
79​ if (kdeConfigValue) {
80​ return QStringLiteral("1");
broulikUnsubmitted
Done

Default value TextBesideIcon

broulik: Default value `TextBesideIcon`
81​ } else {
82​ return QStringLiteral("0");
83​ }
84​}
85​
86​QString ConfigValueProvider::getToolbarStyle(ConfigValueProvider::ToolbarStyleNotation notation) const
87​{
88​ KSharedConfigPtr config(KSharedConfig::openConfig(QStringLiteral("kdeglobals"), KConfig::NoGlobals));
89​ KConfigGroup configGroup(config->group(QStringLiteral("Toolbar style")));
90​ QString kdeConfigValue(configGroup.readEntry(QStringLiteral("ToolButtonStyle"), "TextBesideIcon"));
91​ return getToolbarStyleInDesiredNotation(kdeConfigValue, notation);
92​}
93​
94​QString ConfigValueProvider::getToolbarStyleInDesiredNotation(const QString &kdeConfigValue, ConfigValueProvider::ToolbarStyleNotation notation) const
95​{
96​ QStringList toolbarStyles;
97​ if (notation == ToolbarStyleNotation::SettingsIni) {
98​ toolbarStyles.append({"GTK_TOOLBAR_ICONS", "GTK_TOOLBAR_TEXT", "GTK_TOOLBAR_BOTH_HORIZ", "GTK_TOOLBAR_BOTH"});
99​ } else if (notation == ToolbarStyleNotation::Xsettingsd) {
100​ toolbarStyles.append({"0", "1", "3", "2"});
101​ } else {
102​ toolbarStyles.append({"icons", "text", "both-horiz", "both"});
103​ }
104​
105​ if (kdeConfigValue == QStringLiteral("NoText")) {
106​ return toolbarStyles[0];
107​ } else if (kdeConfigValue == QStringLiteral("TextOnly")) {
108​ return toolbarStyles[1];
109​ } else if (kdeConfigValue == QStringLiteral("TextBesideIcon")) {
110​ return toolbarStyles[2];
111​ } else {
112​ return toolbarStyles[3];
113​ }
114​}