Ubuntu UI Toolkit

Qt.labs.settings should not save to ~/.config/qt-project.org/$APP_ID.conf

Bug #1354321 reported by Cris Dywan
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Telegram app
Fix Released
Critical
Unassigned
Ubuntu UI Toolkit
Fix Released
High
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Unassigned
Vivid
New
Undecided
Unassigned

Bug Description

Qt.labs.settings currently writes settings to ~/.config/qt-project.org/$APP_ID.conf or more specifically $XDG_CONFIG_HOME/qt-project.org/$APP_ID.conf which doesn't agree with confinement.

Related branches

lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/organizationDomain
PS Jenkins bot: Approve (continuous-integration)
Zsombor Egri: Approve
Diff: 121 lines (+77/-0)
4 files modified
debian/control (+1/-0)
modules/Ubuntu/Components/plugin/ucapplication.cpp (+2/-0)
tests/unit/tst_mainview/Settings.qml (+34/-0)
tests/unit/tst_mainview/tst_mainview.cpp (+40/-0)
lp:ubuntu/utopic-proposed/ubuntu-ui-toolkit
Revision history for this message
Cris Dywan (kalikiana) wrote :
Michał Karnicki (karni)
Changed in libqtelegram:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-ui-toolkit/staging at revision 1193, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: Confirmed → Fix Committed
Michał Karnicki (karni)
Changed in libqtelegram:
importance: High → Critical
Revision history for this message
Cris Dywan (kalikiana) wrote :

What's missing for libqtelegram here? The issue has been resolved already.

Revision history for this message
Michał Karnicki (karni) wrote :

Hi Christian,

Is there a way to set organization domain from QML? Currently we don't have a main cpp file, but if that's not the case, I'll create it only to call this, as required:

QCoreApplication::setOrganizationDomain(applicationName);

Revision history for this message
Cris Dywan (kalikiana) wrote :

You are not expected to set the domain, this is already sorted by the MainView.

Michał Karnicki (karni)
Changed in libqtelegram:
status: Confirmed → Fix Released
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit:
status: Fix Committed → Fix Released
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.