[Discover] [Bug 401856] New: Apps sorting order isn't remembered after Discover restart

[Alexander Mentyu]

https://bugs.kde.org/show_bug.cgi?id=401856

Bug ID: 401856
Summary: Apps sorting order isn't remembered after Discover
restart
Product: Discover
Version: 5.14.4
Platform: Neon Packages
OS: Linux
Status: REPORTED
Severity: normal
Priority: NOR
Component: discover
Assignee: ***@kde.org
Reporter: ***@gmail.com
Target Milestone: ---

STEPS TO REPRODUCE
1. Open Discover.
2. Select top left applications/addons pages - 'Rating' is default sorting
order - selected from top right 'Sort' dropdown or select bottom left
'Installed' page - 'Name' is default sorting order.
3. Select any other sorting order.
4. Close Discover.
5. Open Discover.
6. Select top left applications/addons pages - or select bottom left
'Installed' page.

OBSERVED RESULT
Previously selected order on these pages isn't remembered and default one is
selected

EXPECTED RESULT
Previously selected order on these pages is remembered and selected

SOFTWARE/OS VERSIONS
Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.14.80
KDE Frameworks Version: 5.53.0
Qt Version: 5.11.2
Kernel Version: 4.15.0-42-generic

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=401856

Nate Graham <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@kde.org
Status|REPORTED |CONFIRMED
Ever confirmed|0 |1
Keywords| |usability

--- Comment #1 from Nate Graham <***@kde.org> ---
Confirmed.

[Patrick Silva]

https://bugs.kde.org/show_bug.cgi?id=401856

Patrick Silva <***@gmx.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gmx.com

[Aleix Pol]

https://bugs.kde.org/show_bug.cgi?id=401856

Aleix Pol <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Latest Commit| |https://commits.kde.org/dis
| |cover/4b9bdde405a70140d4942
| |66647bdf0f0001dade0
Status|CONFIRMED |RESOLVED

--- Comment #2 from Aleix Pol <***@kde.org> ---
Git commit 4b9bdde405a70140d494266647bdf0f0001dade0 by Aleix Pol.
Committed on 11/12/2018 at 00:41.
Pushed by apol into branch 'master'.

Save last used sort order

Separately for installed page and the rest

M +1 -0 discover/CMakeLists.txt
M +8 -0 discover/DiscoverObject.cpp
A +8 -0 discover/discoversettings.kcfg
A +5 -0 discover/discoversettings.kcfgc
M +11 -6 discover/qml/ApplicationsListPage.qml
M +1 -1 discover/qml/InstalledPage.qml

https://commits.kde.org/discover/4b9bdde405a70140d494266647bdf0f0001dade0