Merge lp:~mzanetti/unity-api/private-quicklist-entries into lp:unity-api

Proposed by Michael Zanetti
Status: Merged
Approved by: Albert Astals Cid
Approved revision: 279
Merged at revision: 276
Proposed branch: lp:~mzanetti/unity-api/private-quicklist-entries
Merge into: lp:unity-api
Diff against target: 85 lines (+17/-3)
5 files modified
debian/changelog (+7/-0)
include/unity/shell/launcher/CMakeLists.txt (+1/-1)
include/unity/shell/launcher/QuickListModelInterface.h (+5/-1)
test/qmltest/mocks/plugins/Unity/Launcher/Mocks/MockQuickListModel.cpp (+2/-0)
test/qmltest/unity/shell/launcher/tst_Launcher.qml (+2/-1)
To merge this branch: bzr merge lp:~mzanetti/unity-api/private-quicklist-entries
Reviewer Review Type Date Requested Status
Albert Astals Cid (community) Approve
Unity8 CI Bot continuous-integration Approve
Review via email: mp+318220@code.launchpad.net

Commit message

Add RoleIsPrivate to launcher's quicklist model api

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:277
https://unity8-jenkins.ubuntu.com/job/lp-unity-api-ci/145/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4205
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4233
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4067/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4067/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4067/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4067/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4067/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4067
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4067/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity-api-ci/145/rebuild

review: Approve (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote :

Q_ENUMS is "bad" use Q_ENUM

review: Needs Fixing
278. By Michael Zanetti

Q_ENUMS -> Q_ENUM

Revision history for this message
Michael Zanetti (mzanetti) wrote :

> Q_ENUMS is "bad" use Q_ENUM

oops, yeah, fixed

Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:278
https://unity8-jenkins.ubuntu.com/job/lp-unity-api-ci/146/
Executed test runs:
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build/4218
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/4246
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/4081/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=zesty/4081/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/4081/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=zesty/4081/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/4081/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4081
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=zesty/4081/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity-api-ci/146/rebuild

review: Approve (continuous-integration)
279. By Michael Zanetti

add some comment about the new role

Revision history for this message
Albert Astals Cid (aacid) wrote :

Looks good

review: Approve
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

In debian/changelog:

"""
+unity-api (8.5) zesty; urgency=medium
"""

Please next time use UNRELEASED instead of zesty. Besides being the correct state to be in, it also makes debchange work correcly. So that instead of debchange adding a new release on top of this one it will append to it instead and reformat for it to be per-author (in case they differ).

Revision history for this message
Daniel d'Andrada (dandrader) wrote :

Just run into this issue as I was rebasing a branch of mine on top of this one.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2017-02-23 11:22:38 +0000
+++ debian/changelog 2017-02-27 11:42:12 +0000
@@ -1,3 +1,10 @@
1unity-api (8.5) zesty; urgency=medium
2
3 [ Michael Zanetti ]
4 * Add RoleIsPrivate to launcher's quicklist entries
5
6 -- Michael Zanetti <michael.zanetti@canonical.com> Fri, 24 Feb 2017 13:57:12 +0000
7
1unity-api (8.4+17.04.20170223-0ubuntu1) zesty; urgency=medium8unity-api (8.4+17.04.20170223-0ubuntu1) zesty; urgency=medium
29
3 [ Pete Woods ]10 [ Pete Woods ]
411
=== modified file 'include/unity/shell/launcher/CMakeLists.txt'
--- include/unity/shell/launcher/CMakeLists.txt 2016-11-30 10:43:52 +0000
+++ include/unity/shell/launcher/CMakeLists.txt 2017-02-27 11:42:12 +0000
@@ -7,7 +7,7 @@
77
8set(UNITY_API_LIB_HDRS ${UNITY_API_LIB_HDRS} ${headers} ${internal_headers} PARENT_SCOPE)8set(UNITY_API_LIB_HDRS ${UNITY_API_LIB_HDRS} ${headers} ${internal_headers} PARENT_SCOPE)
99
10set(VERSION 11)10set(VERSION 12)
11set(PKGCONFIG_NAME "unity-shell-launcher")11set(PKGCONFIG_NAME "unity-shell-launcher")
12set(PKGCONFIG_DESCRIPTION "Unity shell Launcher APIs")12set(PKGCONFIG_DESCRIPTION "Unity shell Launcher APIs")
13set(PKGCONFIG_REQUIRES "Qt5Core")13set(PKGCONFIG_REQUIRES "Qt5Core")
1414
=== modified file 'include/unity/shell/launcher/QuickListModelInterface.h'
--- include/unity/shell/launcher/QuickListModelInterface.h 2016-12-14 15:57:28 +0000
+++ include/unity/shell/launcher/QuickListModelInterface.h 2017-02-27 11:42:12 +0000
@@ -39,6 +39,7 @@
39 * - RoleIcon (icon): The icon to be shown for this entry (QString).39 * - RoleIcon (icon): The icon to be shown for this entry (QString).
40 * - RoleClickable (clickable): Determines if the entry can be triggered or is just a static text (boolean)40 * - RoleClickable (clickable): Determines if the entry can be triggered or is just a static text (boolean)
41 * - RoleHasSeparator (hasSeparator): Determines if the entry has a separator (boolean)41 * - RoleHasSeparator (hasSeparator): Determines if the entry has a separator (boolean)
42 * - RoleIsPrivate (isPrivate): Determines whether the entry should be shown or not in locked mode (e.g. greeter is locked)
42 */43 */
43class UNITY_API QuickListModelInterface: public QAbstractListModel44class UNITY_API QuickListModelInterface: public QAbstractListModel
44{45{
@@ -51,6 +52,7 @@
51 m_roleNames.insert(RoleIcon, "icon");52 m_roleNames.insert(RoleIcon, "icon");
52 m_roleNames.insert(RoleClickable, "clickable");53 m_roleNames.insert(RoleClickable, "clickable");
53 m_roleNames.insert(RoleHasSeparator, "hasSeparator");54 m_roleNames.insert(RoleHasSeparator, "hasSeparator");
55 m_roleNames.insert(RoleIsPrivate, "isPrivate");
54 }56 }
55 /// @endcond57 /// @endcond
56public:58public:
@@ -63,8 +65,10 @@
63 RoleLabel,65 RoleLabel,
64 RoleIcon,66 RoleIcon,
65 RoleClickable,67 RoleClickable,
66 RoleHasSeparator68 RoleHasSeparator,
69 RoleIsPrivate
67 };70 };
71 Q_ENUM(Roles)
6872
69 /// @cond73 /// @cond
70 virtual ~QuickListModelInterface() {}74 virtual ~QuickListModelInterface() {}
7175
=== modified file 'test/qmltest/mocks/plugins/Unity/Launcher/Mocks/MockQuickListModel.cpp'
--- test/qmltest/mocks/plugins/Unity/Launcher/Mocks/MockQuickListModel.cpp 2016-10-28 11:14:37 +0000
+++ test/qmltest/mocks/plugins/Unity/Launcher/Mocks/MockQuickListModel.cpp 2017-02-27 11:42:12 +0000
@@ -39,6 +39,8 @@
39 return true;39 return true;
40 case RoleHasSeparator:40 case RoleHasSeparator:
41 return true;41 return true;
42 case RoleIsPrivate:
43 return false;
42 }44 }
43 return QVariant();45 return QVariant();
44}46}
4547
=== modified file 'test/qmltest/unity/shell/launcher/tst_Launcher.qml'
--- test/qmltest/unity/shell/launcher/tst_Launcher.qml 2016-11-30 10:38:23 +0000
+++ test/qmltest/unity/shell/launcher/tst_Launcher.qml 2017-02-27 11:42:12 +0000
@@ -175,7 +175,8 @@
175 { tag: "Model.roles[label]", role: "label", type: "string" },175 { tag: "Model.roles[label]", role: "label", type: "string" },
176 { tag: "Model.roles[icon]", role: "icon", type: "string" },176 { tag: "Model.roles[icon]", role: "icon", type: "string" },
177 { tag: "Model.roles[clickable]", role: "clickable", type: "boolean" },177 { tag: "Model.roles[clickable]", role: "clickable", type: "boolean" },
178 { tag: "Model.roles[hasSeparator]", role: "hasSeparator", type: "boolean" }178 { tag: "Model.roles[hasSeparator]", role: "hasSeparator", type: "boolean" },
179 { tag: "Model.roles[isPrivate]", role: "isPrivate", type: "boolean" }
179 ];180 ];
180 }181 }
181182

Subscribers

People subscribed via source and target branches

to all changes: