~ken-vandine/ubuntu-system-settings/rtm_click_framework_check

« back to all changes in this revision

Viewing changes to plugins/about/click.cpp

[ CI bot ]
* Resync trunk
[ Michael Zanetti ]
* update code to reset launcher after launcher backend implementation
  was reworked (LP: #1376707)
[ Sebastien Bacher ]
* [storage] translator comment for the string "%1 free" (LP: #1377929)
* [system-update] ellipsize labels if needed (LP: #1376286)
* [sound] Use the correct low icon (LP: #1376286)
[ Lars Uebernickel ]
* notification-manager: fix issues with GVariant API usage
[ jonas-drange ]
* [security/privacy] allow user to change HERE and location detection
  settings. (LP: #1375322)
[ Ken VanDine ]
* Updated the pot file.
[ jonas-drange ]
* [cellular] indirectly recreate ConnectionManager interface's dbus
  proxy so it reports the true value of 'Powered' after flight mode
  (LP: #1376957)
[ Iain Lane ]
* [sound] Make listSounds take a list of directories and list all of
  the files in them, instead of assuming "/custom" all the time.
[ Ubuntu daily release ]
* New rebuild forced
[ Sebastien Bacher ]
* [sound] list custom ringtones as well (LP: #1381528)
[ Ken VanDine ]
* Add an APN editor. (LP: #1225174)
[ Michael Terry ]
* Complete the unlock security screens and update a host of other
  small design issues. (LP: #1365034, LP: #1348362) (LP: #1365034,
  #1348362)
[ Diego Sarmentero ]
* Improves in network backend. Detect network connection and show
  "Connect to the internet..." message or retry on netword detected. .
* Detect account added on online accounts (LP: #1362149)
[ jonas-drange ]
* [wifi/phone] dynamically chooses what SIM to use when there's only
  one present. (LP: #1375832)
[ Iain Lane ]
* [storage/about] Use a QSharedPointer to manage freeing of the
  counter which we use to display all of the collected sizes at once.
  If the measuring process was cancelled, we were freeing it multiple
  times - once for each outstanding size measurement. This led to a
  crash. (LP: #1375988)
* [security-privacy] Link with gobject, uses symbols from this
  library. Prevents "[…]libUbuntuSecurityPrivacyPanel.so contains an
  unresolvable reference to symbol
  g_signal_handlers_disconnect_matched: it's probably a plugin"
  warnings.
[ jonas-drange ]
* [cellular] react to application.state changes and update potential
  out-of-date default SIM settings (LP: #1364103)
* [wifi] let removed networks update the previous network model so
  that we can reuse the model (and not pop the pagestack twice,
  causing this ux issue) (LP: #1370389)
[ Sebastien Bacher ]
* [location] use a flickable so the list can be scrolled (LP:
  #1374017)
* [security] get localized application names from the trust-store (LP:
  #1374018)
[ Ken VanDine ]
* Fixed handling of the Downloading/Installing label based on actual
  state. . Forward updateProgress signals for system updates. Layout
  fixes (LP: #1312587)
* Use new splash screen (LP: #1376242)
* Improved tracking download states
* Use new target_build_number from system-image-dbus to determine
  visibility of the update notification (LP: #1355803)
[ CI bot ]
* Resync trunk
[ Sebastien Bacher ]
* [storage] use libclick rather than calling the command line (LP:
  #1368967)
* [storage] tweaks to the disk informations, limit the free space to
  the user available one (excluding the system partitions), don't list
  the external mounts, that's described in the design and is going to
  require work on the categories computation to give correct values
  (LP: #1284247)
* [about] display the free space value next to the storage label (LP:
  #1335144)
[ Albert Astals ]
* Add i18n.tr
[ jonas-drange ]
* [wifi] connect to hidden network is now a dialog (LP: #1366005)
[ Ken VanDine ]
* Provide more keywords to make search useful (LP: #1370219) (LP:
  #1370219)
* [background] set sourceSize in the image preview to work around an
  issue loading large images (LP: #1373462)
[ CI bot ]
* Resync trunk
[ Mathieu Trudel-Lapierre ]
* [bluetooth] Properly handle device types for connection and
  disconnection. (LP: #1369964)
[ Michael Terry ]
* Add a translator comment for what HERE means. (LP: #1368838) (LP:
  #1368838)
* Prevent a crash in unity8 when the wizard also crashes or stops
  early. (LP: #1355892) (LP: #1355892)
[ Michael Terry ]
* Prevent the buttons of the wizard's password page from overlapping
  its content when the OSK appears. (LP: #1368346) (LP: #1368346)
[ Ken VanDine ]
* Set the version information in the SystemUpdate constructor (LP:
  #1371810) (LP: #1371810)
* [reset] Don't bail out on an isValid() check on the SystemImage
  QDBusInterface, the docs say isValid() may not be reliable for
  remote interfaces (LP: #1370815) (LP: #1370815)
* Fixed reference error for count on the various device models (LP:
  #1362517) (LP: #1362517)
* [background] Use the ContentPeerPicker to select sources for
  backgrounds (LP: #1356542) (LP: #1356542)
[ Iain Lane ]
* Don't check if the timedated interface is valid before calling
  SetTime on it, it might have timed out and we should reactivate it
  in that case.
* [about] Fix developerMode property to be QMLish - read & write
  instead of separate "read" and "toggle" properties.
* [about] Don't display a frame for icons that can't be found,
  improving the visual appearance. Also return the correct path for
  click packages which use the icon theme.
[ Sebastien Bacher ]
* [storage] handle the scope click .ini and get the correct icons (LP:
  #1368920)
[ Albert Astals ]
* make pot
[ Oliver Grawert ]
* fix LP: #1365903, add a watch to Developer Mode page so the page
  gets refreshed when going back through the page stack, also fix the
  page id and bounds (LP: #1365903)
* Some packaging fixes
  - > is not a valid relationship. It should be >>
  - Don't recommend suru-icon-theme since we also depend on it
  - Bump S-V to 3.9.6 which for example allows the images we ship in
    /usr/lib/
  - Override binary-no-manpage lintian warnings. We're not going to get man
    pages for u-s-s any time soon
[ Ken VanDine ]
* Make UpdateManager a singleton and refresh the availableUpdates
  count when the model updates (LP: #1325629) (LP: #1325629)
[ Ken VanDine ]
* Include an icon instead of using the themed icon (LP: #1365408) (LP:
  #1365408)
[ Mathieu Trudel-Lapierre ]
* [bluetooth] don't try to enable Discoverable if the device isn't
  powered yet. (LP: #1355904)
[ Ken VanDine ]
* When activated by a url request for a specific panel, clear the
  pageStack to maintain the natural navigation instead of pushing a
  new page on top of a page from a the previous panel (LP: #1362025)
  (LP: #1362025, #1359953)
[ Lars Uebernickel ]
* Use Icon instead of StatusIcon
[ CI bot ]
* Resync trunk
[ John R. Lenton ]
* Take the system settings push helper closer to implementing the spec
  at https://wiki.ubuntu.com/SoftwareUpdates#Prompting (LP: #1363972)
[ Ken VanDine ]
* Don't try to tweak the bottomMargin for the OSK, it was trying to
  calculate it based on an undefined component so failing. It
  shouldn't be needed anyway.
[ jonas-drange ]
* [wifi] "Other Network" -> "Connect to hidden network" (LP: #1365984)
* [wifi] fix bad looking buttons in "Other Networks" panel (LP:
  #1362127)
[ Jussi Pakkanen ]
* Hide entry to connect to hidden network when wifi is off. (LP:
  #1365960)
[ Ken VanDine ]
* Fixed tests for the dialpad settings failing on mako
* Don't call downloadUpdate when we see a system update ready, the
  service does that for us (LP: #1368294) (LP: #1368294)
* Translate desktop file (LP: #1318008) (LP: #1318008)
[ Martin Pitt ]
* update POT
[ Sebastien Bacher ]
* [system-update] open the ubuntuone subpanel rather than the list
  (LP: #1348580)
* [security-privacy] use ngettext strings when needed (LP: #1368159)
* Increase vertical spacing in the settings grid (LP: #1348579)
[ jonas-drange ]
* [background] always keep custom images, and remove press-and-hold
  action. (LP: #1365167)
* [wifi] do not use predictive text for network name input (LP:
  #1365992)
* [language] workaround for transition bug in keyboard layout menu
  (LP: #1350809)
* [wifi] factory was setting the icon property, which was causing the
  icon to not render (LP: #1364366)
[ Alejandro J. Cura ]
* Fixing the refreshing of the apps scope (LP: #1360369)
[ Sebastien Bacher ]
* [sound] use the correct backend to control the keyboard sound toggle
  (LP: #1366208)
[ Ken VanDine ]
* Don't call downloadUpdate for system updates that are already
  downloading (LP: #1365646) (LP: #1311219, #1365646)
[ Ken VanDine ]
* Notify the user if the phone needs to be plug to power for update.
[ Sebastien Bacher ]
* [system-update] don't draw a frame around ubuntu updates (LP:
  #1367136)
[ Mathieu Trudel-Lapierre ]
* Also allow Speakers and Carkits as valid audio devices to pair.
[ Michael Terry ]
* Allow switch from PIN code to passphrase and back using the same
  password. (LP: #1357043) (LP: #1357043)
* Fix false-negatives from AccountsService when switching from swipe
  to password (LP: #1363405) (LP: #1363405)
[ Ricardo Salveti de Araujo ]
* sound: setting default audio role for ringtone and messages
[ Michael Terry ]
* Don't crash when destructing the Wizard.Utils.System plugin. This
  fixes the crash on wizard exit and the crash when pressing back on
  the HERE license.
[ CI bot ]
* Resync trunk
[ Sebastien Bacher ]
* [developer-mode] working scrolling and correct spacing
* Use standard items for the categories and don't display dividers
  (LP: #1365563)
* [orientation-lock] don't display a frame around the icon (LP:
  #1365450)
[ jonas-drange ]
* [about/versions] create reusable component of singlevalue items that
  have long values (LP: #1363927)
[ David Barth ]
* Enable location panel, display contols for authorized applications.
[ Ken VanDine ]
* Tweak the depends for accountsservice-ubuntu-schemas to handle a "~"
  version
[ CI bot ]
* Resync trunk
[ Guillermo Gonzalez ]
* Basic unittest for the push-helper script
[ Mathieu Trudel-Lapierre ]
* Downgrade qFatal failure to register bluetooth agent to a qCritical.
[ Andrea Cimitan ]
* Various fixes for wifi wizard page (LP: #1362296)
[ Sebastien Bacher ]
* [security] don't mark "%1" string as translatable
* [security] give a translation context to the 'None' string (LP:
  #1362143)
* [licenses] display an error when the copyright can't be read (LP:
  #1206150)
[ Michael Terry ]
* Avoid another race condition when switching password types by
  completing all DBus operations synchronously as we exit the
  policykit agent. LP: 1361137 (LP: #1361137)
* Fix the OSK not working on first boot. When the wizard closes, shut
  down maliit-server and the indicators so that unity8 can start them
  itself. LP: 1362679 (LP: #1362679)
[ jonas-drange ]
* [about] print out phone numbers for dual sims as well
* [about/wifi] adding mac addresses for bt and wifi, and ip for wifi
  (LP: #1356239)
* [language] explain to the user that a reboot is necessary for the
  change to take effect (LP: #1240875)
[ Michael Terry ]
* Make the HERE license optional, clearly mark it as a HERE license
  instead of an Ubuntu license, and connect it to the HERE backend.
[ CI bot ]
* Resync trunk
[ Martin Pitt ]
* POT update
[ Ken VanDine ]
* Added setting for dialpad sounds to both the sound and phone panels.
[ CI bot ]
* Resync trunk
[ Diego Sarmentero ]
* Fix out of sync image download. Refresh scope after installation
  (LP: #1324923)
[ Sebastien Bacher ]
* [battery] refresh the graph on a minute basis
* [notifications] use normal text style and ubuntushaped icons
[ Michael Terry ]
* When changing to swipe mode, it was possible that our policykit
  agent would not unregister itself, making the next attempt to change
  modes appear to work, but in truth fail. Also, no longer ask for a
  password to switch to swipe mode if policykit has a cached
  authentication for us. (LP: #1350381)
[ Sebastien Bacher ]
* [security] tweak some pin locking details to match the design better
[ Michael Terry ]
* Implement the "When locked, allow:" portion of the Security &
  Privacy panel, so now the user can control whether the launcher
  and/or indicators are available while locked. (LP: #1358340) (LP:
  #1358340)
[ Michael Terry ]
* Update look and feel of wizard to more closely match the visual
  design.
[ Jussi Pakkanen ]
* Minor cleanups.
[ Roberto Alsina ]
* Don't crash the push helper in locales that have no .mo file (LP:
  #1357506)
[ Ken VanDine ]
* SIM PIN lock implementation
[ jonas-drange ]
* [main panel] add visual feedback on press
* [cellular] uses modemtechnologies property from ofono to populate
  technology preference options (LP: #1346790)
[ Sebastien Bacher ]
* [language] workaround inconsistent languages casing (LP: #1362123)
* [background] Display the default background selected (LP: #1349329)
* [storage] display translated names (LP: #1360207)
* [updates] don't display animations when clicking on update rows (LP:
  #1360364)
* [background] don't use an icon in the headerbar (LP: #1362122)
[ Mathieu Trudel-Lapierre ]
* Fix PIN code or passcode requests by properly matching the tag for
  the original request. (LP: #1355152)
[ Martin Pitt ]
* Update PO template
[ Sebastien Bacher ]
* Include extra files for translations (LP: #1359267)
[ jonas-drange ]
* [main panel] show rotation lock, point to correct setting
[ Ken VanDine ]
* Hide sleep locks immediately setting, which isn't implemented yet
  (LP: #1361297) (LP: #1361297)
[ Sebastien Bacher ]
* [system-update] rework the ui to use a column rather than anchors,
  the layout is quite dynamic and it should be easier to position and
  stack widgets this way (LP: #1343172)
[ Michael Terry ]
* After selecting the language, update the session environment
  immediately and restart indicators so that their notifications (like
  wifi prompt) are translated. (LP: #1354325)
[ Ken VanDine ]
* Added autopilot tests for security-privacy
[ jonas-drange ]
* [phone] adding dual sim functionality to phone panel
[ Charles Kerr ]
* In the welcome wizard, refresh the text in the 'no sim card' and
  'all done' pages to match the latest from design.
[ Michael Terry ]
* Add a location page and a terms and conditions page for the HERE
  service.
[ Ken VanDine ]
* Settings for idle/lock timeout (LP: #1230345)
* Added flickable to fix scrolling (LP: #1354161)
[ Jussi Pakkanen ]
* Fix a bunch of compiler warnings.
[ jonas-drange ]
* [cellular] displays cellular UIs based on how many SIMs are present.
  (LP: #1357393)

Show diffs side-by-side

added added

removed removed

Lines of Context:
19
19
 
20
20
#include "click.h"
21
21
 
 
22
#include <click.h>
22
23
#include <gio/gio.h>
23
24
#include <gio/gdesktopappinfo.h>
24
25
#include <glib.h>
37
38
    m_clickPackages = buildClickList();
38
39
}
39
40
 
40
 
/* Look through `hooks' for a desktop file in `directory'
 
41
/* Look through `hooks' for a desktop or ini file in `directory'
41
42
 * and set the display name and icon from this.
42
43
 *
43
 
 * Will set with information from the first desktop file found and parsed.
 
44
 * Will set with information from the first desktop or ini file found and parsed.
44
45
 */
45
 
void ClickModel::populateFromDesktopFile (Click *newClick,
46
 
                                          QVariantMap hooks,
47
 
                                          QDir directory)
 
46
void ClickModel::populateFromDesktopOrIniFile (Click *newClick,
 
47
                                               QVariantMap hooks,
 
48
                                               QDir directory)
48
49
{
49
50
    QVariantMap appHooks;
50
51
    GKeyFile *appinfo = g_key_file_new();
51
 
    gchar *desktopFileName = NULL;
 
52
    gchar *desktopOrIniFileName = nullptr;
52
53
 
53
54
    QVariantMap::ConstIterator begin(hooks.constBegin());
54
55
    QVariantMap::ConstIterator end(hooks.constEnd());
55
56
 
 
57
    const gchar *keyGroup = G_KEY_FILE_DESKTOP_GROUP;
 
58
    const gchar *keyName = G_KEY_FILE_DESKTOP_KEY_NAME;
 
59
 
56
60
    // Look through the hooks for a 'desktop' key which points to a desktop
57
 
    // file referring to this app.
 
61
    // file referring to this app, or a 'scope' key pointing to an ini
58
62
    while (begin != end) {
59
63
        appHooks = (*begin++).toMap();
60
64
        if (!appHooks.isEmpty() &&
61
 
            appHooks.contains("desktop") &&
 
65
            (appHooks.contains("desktop") || appHooks.contains("scope")) &&
62
66
            directory.exists()) {
63
 
            QFile desktopFile(directory.absoluteFilePath(
64
 
                        appHooks.value("desktop", "undefined").toString()));
65
 
 
66
 
           desktopFileName =
67
 
               g_strdup(desktopFile.fileName().toLocal8Bit().constData());
68
 
 
69
 
            g_debug ("Desktop file: %s", desktopFileName);
70
 
 
71
 
            if (!desktopFile.exists())
72
 
                goto out;
 
67
 
 
68
            if (appHooks.contains("scope"))
 
69
            {
 
70
                keyGroup = "ScopeConfig";
 
71
                keyName = "DisplayName";
 
72
 
 
73
                QDir scopeDirectory(
 
74
                            directory.absoluteFilePath(appHooks.value("scope", "").toString()));
 
75
                scopeDirectory.setNameFilters(QStringList()<<"*.ini");
 
76
 
 
77
                QStringList iniEntry(scopeDirectory.entryList());
 
78
 
 
79
                if (iniEntry.isEmpty())
 
80
                    goto out;
 
81
 
 
82
                QFile desktopOrIniFile(scopeDirectory.absoluteFilePath(iniEntry[0]));
 
83
                desktopOrIniFileName =
 
84
                        g_strdup(desktopOrIniFile.fileName().toLocal8Bit().constData());
 
85
                if (!desktopOrIniFile.exists())
 
86
                    goto out;
 
87
 
 
88
                /* replace directory so the icon is correctly loaded */
 
89
                directory = scopeDirectory;
 
90
            }
 
91
            else
 
92
            {
 
93
                keyGroup = G_KEY_FILE_DESKTOP_GROUP;
 
94
                keyName = G_KEY_FILE_DESKTOP_KEY_NAME;
 
95
 
 
96
                QFile desktopOrIniFile(directory.absoluteFilePath(
 
97
                                      appHooks.value("desktop", "undefined").toString()));
 
98
 
 
99
                desktopOrIniFileName =
 
100
                   g_strdup(desktopOrIniFile.fileName().toLocal8Bit().constData());
 
101
 
 
102
                if (!desktopOrIniFile.exists())
 
103
                    goto out;
 
104
            }
 
105
 
 
106
            g_debug ("Desktop or ini file: %s", desktopOrIniFileName);
 
107
 
 
108
 
73
109
 
74
110
            gboolean loaded = g_key_file_load_from_file(appinfo,
75
 
                                                        desktopFileName,
 
111
                                                        desktopOrIniFileName,
76
112
                                                        G_KEY_FILE_NONE,
77
 
                                                        NULL);
 
113
                                                        nullptr);
78
114
 
79
115
            if (!loaded) {
80
 
                g_warning ("Couldn't parse desktop file %s", desktopFileName);
 
116
                g_warning ("Couldn't parse desktop or ini file %s", desktopOrIniFileName);
81
117
                goto out;
82
118
            }
83
119
 
84
 
            gchar * name = g_key_file_get_string (appinfo,
85
 
                                                  G_KEY_FILE_DESKTOP_GROUP,
86
 
                                                  G_KEY_FILE_DESKTOP_KEY_NAME,
87
 
                                                  NULL);
 
120
            gchar * name = g_key_file_get_locale_string (appinfo,
 
121
                                                  keyGroup,
 
122
                                                  keyName,
 
123
                                                  nullptr,
 
124
                                                  nullptr);
88
125
 
89
126
            if (name) {
90
127
                g_debug ("Name is %s", name);
91
128
                newClick->displayName = name;
92
129
                g_free (name);
93
 
                name = NULL;
 
130
                name = nullptr;
94
131
            }
95
132
 
96
 
            // Overwrite the icon with the .desktop file's one if we have it.
 
133
            // Overwrite the icon with the .desktop or ini file's one if we have it.
97
134
            // This is the one that the app scope displays so use that if we
98
135
            // can.
99
136
            gchar * icon = g_key_file_get_string (appinfo,
100
 
                                                  G_KEY_FILE_DESKTOP_GROUP,
 
137
                                                  keyGroup,
101
138
                                                  G_KEY_FILE_DESKTOP_KEY_ICON,
102
 
                                                  NULL);
 
139
                                                  nullptr);
103
140
 
104
141
            if (icon) {
105
142
                g_debug ("Icon is %s", icon);
113
150
                    if (iconFile.exists())
114
151
                        newClick->icon = iconFile.fileName();
115
152
                    else if (QIcon::hasThemeIcon(qIcon)) // try the icon theme
116
 
                        newClick->icon = qIcon;
 
153
                        newClick->icon = QString("icon://theme/%1").arg(qIcon);
117
154
                }
 
155
                goto out;
118
156
            }
119
157
        }
120
158
out:
121
 
        g_free (desktopFileName);
 
159
        g_free (desktopOrIniFileName);
122
160
        g_key_file_free (appinfo);
123
161
        return;
124
162
    }
139
177
        // Set the icon from the click package. Might be a path or a reference to a themed icon.
140
178
        QString iconFile(manifest.value("icon", "undefined").toString());
141
179
 
142
 
        if (directory.exists()) {
 
180
        if (directory.exists() && iconFile != "undefined") {
143
181
            QFile icon(directory.absoluteFilePath(iconFile.simplified()));
144
182
            if (!icon.exists() && QIcon::hasThemeIcon(iconFile)) // try the icon theme
145
 
                newClick.icon = iconFile;
 
183
                newClick.icon = QString("image://theme/%1").arg(iconFile);
146
184
            else
147
185
                newClick.icon = icon.fileName();
148
186
        }
149
187
 
150
188
    }
151
189
 
152
 
    // "hooks" → title → "desktop" / "icon"
 
190
    // "hooks" → title → "desktop" or "ini" / "icon"
153
191
    QVariant hooks(manifest.value("hooks"));
154
192
 
155
193
    if (hooks.isValid()) {
156
194
        QVariantMap allHooks(hooks.toMap());
157
 
        // The desktop file contains an icon and the display name
158
 
        populateFromDesktopFile(&newClick, allHooks, directory);
 
195
        // The desktop or ini file contains an icon and the display name
 
196
        populateFromDesktopOrIniFile(&newClick, allHooks, directory);
159
197
   }
160
198
 
161
199
    newClick.installSize = manifest.value("installed-size",
168
206
 
169
207
QList<ClickModel::Click> ClickModel::buildClickList()
170
208
{
171
 
    QFile clickBinary("/usr/bin/click");
172
 
    if (!clickBinary.exists()) {
173
 
        return QList<ClickModel::Click>();
174
 
    }
175
 
 
176
 
    QProcess clickProcess;
177
 
    clickProcess.start("/usr/bin/click",
178
 
                       QStringList() << "list" << "--all" << "--manifest");
179
 
 
180
 
    if (!clickProcess.waitForFinished(5000)) {
181
 
        qWarning() << "Timeout retrieving the list of click packages";
182
 
        return QList<ClickModel::Click>();
183
 
    }
184
 
 
185
 
    if (clickProcess.exitStatus() == QProcess::CrashExit) {
186
 
        qWarning() << "The click utility exited abnormally" <<
187
 
                      clickProcess.readAllStandardError();
 
209
    ClickDB *clickdb;
 
210
    GError *err = nullptr;
 
211
    gchar *clickmanifest = nullptr;
 
212
 
 
213
    clickdb = click_db_new();
 
214
    click_db_read(clickdb, nullptr, &err);
 
215
    if (err != nullptr) {
 
216
        g_warning("Unable to read Click database: %s", err->message);
 
217
        g_error_free(err);
 
218
        g_object_unref(clickdb);
 
219
        return QList<ClickModel::Click>();
 
220
    }
 
221
 
 
222
    clickmanifest = click_db_get_manifests_as_string(clickdb, TRUE, &err);
 
223
    g_object_unref(clickdb);
 
224
 
 
225
    if (err != nullptr) {
 
226
        g_warning("Unable to get the manifests: %s", err->message);
 
227
        g_error_free(err);
188
228
        return QList<ClickModel::Click>();
189
229
    }
190
230
 
191
231
    QJsonParseError error;
192
232
 
193
233
    QJsonDocument jsond =
194
 
            QJsonDocument::fromJson(clickProcess.readAllStandardOutput(),
195
 
                                    &error);
 
234
            QJsonDocument::fromJson(clickmanifest, &error);
 
235
    g_free(clickmanifest);
196
236
 
197
237
    if (error.error != QJsonParseError::NoError) {
198
238
        qWarning() << error.errorString();