The Savilerow project

Merge lp:~savilerow-team/savilerow/unity-scope-selfservicebundles-newapi into lp:~savilerow-team/savilerow/unity-scope-selfservicebundles

  1. unity-scope-selfservicebundles-newapi
  2. Merge into unity-scope-selfservicebun...
Proposed by Chris Wayne
Status: Merged
Merged at revision: 9
Proposed branch: lp:~savilerow-team/savilerow/unity-scope-selfservicebundles-newapi
Merge into: lp:~savilerow-team/savilerow/unity-scope-selfservicebundles
Diff against target: 151 lines (+24/-18)
6 files modified
CMakeLists.txt (+2/-2)
debian/changelog (+6/-0)
src/SelfServiceBundlesQuery.cpp (+2/-2)
src/SelfServiceBundlesQuery.h (+4/-4)
src/SelfServiceBundlesScope.cpp (+6/-6)
src/SelfServiceBundlesScope.h (+4/-4)
To merge this branch: bzr merge lp:~savilerow-team/savilerow/unity-scope-selfservicebundles-newapi
Reviewer Review Type Date Requested Status
Scott Sweeny (community) Approve
Review via email: mp+212893@code.launchpad.net

Description of the change

New api

To post a comment you must log in.
Revision history for this message
Scott Sweeny (ssweeny) wrote :

B/I/R for me. Please add a changelog entry and merge.

review: Approve
lp:~savilerow-team/savilerow/unity-scope-selfservicebundles-newapi updated
10. By Chris Wayne

releasing package unity-scope-selfservicebundles version 0.2

11. By Chris Wayne

Fix install location

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2014-02-22 04:36:47 +0000
3+++ CMakeLists.txt 2014-03-27 03:43:23 +0000
4@@ -26,9 +26,9 @@
5 SET(GETTEXT_PACKAGE unity-scope-selfservicebundles)
6 SET(LOCALE_DIR "${CMAKE_INSTALL_DATADIR}/locale")
7
8-SET(CUSTOM_INSTALL_LIBDIR /custom/usr/lib/arm-linux-gnueabihf)
9+SET(CUSTOM_INSTALL_LIBDIR /custom/lib/arm-linux-gnueabihf)
10 SET(CUSTOM_INSTALL_DATADIR /custom/xdg/data/)
11-SET(SCOPE_INSTALL_DIR ${CUSTOM_INSTALL_LIBDIR}/unity/scopes/scope-selfservicebundles/)
12+SET(SCOPE_INSTALL_DIR ${CUSTOM_INSTALL_LIBDIR}/unity-scopes/scope-selfservicebundles/)
13
14 CONFIGURE_FILE(
15 "${CMAKE_CURRENT_SOURCE_DIR}/src/config.h.in"
16
17=== modified file 'debian/changelog'
18--- debian/changelog 2014-01-31 19:55:50 +0000
19+++ debian/changelog 2014-03-27 03:43:23 +0000
20@@ -1,3 +1,9 @@
21+unity-scope-selfservicebundles (0.2) trusty; urgency=medium
22+
23+ * Update to new api
24+
25+ -- Chris Wayne <chris.wayne@canonical.com> Wed, 26 Mar 2014 22:27:00 -0400
26+
27 unity-scope-selfservicebundles (0.1) UNRELEASED; urgency=low
28
29 * Initial release.
30
31=== modified file 'src/SelfServiceBundlesQuery.cpp'
32--- src/SelfServiceBundlesQuery.cpp 2014-02-22 04:36:47 +0000
33+++ src/SelfServiceBundlesQuery.cpp 2014-03-27 03:43:23 +0000
34@@ -33,8 +33,8 @@
35 using namespace std;
36 using namespace unity::scopes;
37
38-SelfServiceBundlesQuery::SelfServiceBundlesQuery(Query const& query, QStringList const& scopes, QString const& locale)
39- : SearchQuery(),
40+SelfServiceBundlesQuery::SelfServiceBundlesQuery(CannedQuery const& query, QStringList const& scopes, QString const& locale)
41+ : SearchQueryBase(),
42 query_(query),
43 scopes_(scopes)
44 {
45
46=== modified file 'src/SelfServiceBundlesQuery.h'
47--- src/SelfServiceBundlesQuery.h 2014-02-12 02:13:59 +0000
48+++ src/SelfServiceBundlesQuery.h 2014-03-27 03:43:23 +0000
49@@ -20,21 +20,21 @@
50 #define SELFSERVICEBUNDLESQUERY_H
51
52 #include <unity/scopes/ScopeBase.h>
53-#include <unity/scopes/Query.h>
54+#include <unity/scopes/SearchQueryBase.h>
55 #include <QStringList>
56 #include <string>
57 #include <memory>
58
59-class SelfServiceBundlesQuery: public unity::scopes::SearchQuery
60+class SelfServiceBundlesQuery: public unity::scopes::SearchQueryBase
61 {
62 public:
63- SelfServiceBundlesQuery(unity::scopes::Query const& query, QStringList const& scopes, QString const& locale);
64+ SelfServiceBundlesQuery(unity::scopes::CannedQuery const& query, QStringList const& scopes, QString const& locale);
65
66 virtual void run(unity::scopes::SearchReplyProxy const& reply) override;
67 virtual void cancelled() override;
68
69 private:
70- unity::scopes::Query query_;
71+ unity::scopes::CannedQuery query_;
72 QStringList scopes_;
73 };
74
75
76=== modified file 'src/SelfServiceBundlesScope.cpp'
77--- src/SelfServiceBundlesScope.cpp 2014-02-19 03:51:14 +0000
78+++ src/SelfServiceBundlesScope.cpp 2014-03-27 03:43:23 +0000
79@@ -72,17 +72,17 @@
80 return VERSION;
81 }
82
83-QueryBase::UPtr SelfServiceBundlesScope::create_query(Query const& query, SearchMetadata const&)
84+SearchQueryBase::UPtr SelfServiceBundlesScope::search(const CannedQuery &query, SearchMetadata const&)
85 {
86 QStringList scopes;
87- QueryBase::UPtr q(new SelfServiceBundlesQuery(query, scopes, m_locale));
88+ SearchQueryBase::UPtr q(new SelfServiceBundlesQuery(query, scopes, m_locale));
89 return q;
90 }
91
92-QueryBase::UPtr SelfServiceBundlesScope::preview(Result const& result,
93+PreviewQueryBase::UPtr SelfServiceBundlesScope::preview(Result const& result,
94 ActionMetadata const& hints)
95 {
96- QueryBase::UPtr previewer(new SelfServiceBundlesPreview(*this, result));
97+ PreviewQueryBase::UPtr previewer(new SelfServiceBundlesPreview(*this, result));
98 return previewer;
99 }
100
101@@ -112,7 +112,7 @@
102 reply->register_layout({layout});
103
104 PreviewWidget art("art", "image");
105- art.add_component("source", "art");
106+ art.add_attribute_mapping("source", "art");
107
108 PreviewWidget actions("actions", "actions");
109 {
110@@ -121,7 +121,7 @@
111 {"id", Variant("open")},
112 {"label", Variant("Buy now")}
113 });
114- actions.add_attribute("actions", builder.end());
115+ actions.add_attribute_value("actions", builder.end());
116 }
117
118 reply->push({art, actions});
119
120=== modified file 'src/SelfServiceBundlesScope.h'
121--- src/SelfServiceBundlesScope.h 2014-02-19 03:51:14 +0000
122+++ src/SelfServiceBundlesScope.h 2014-03-27 03:43:23 +0000
123@@ -21,7 +21,7 @@
124 #define SELFSERVICEBUNDLESSCOPE_H
125
126 #include <unity/scopes/ScopeBase.h>
127-#include <unity/scopes/Query.h>
128+#include <unity/scopes/SearchQueryBase.h>
129 #include <unity/scopes/PreviewReply.h>
130 #include <unity/scopes/PreviewWidget.h>
131 #include <unity/scopes/VariantBuilder.h>
132@@ -41,8 +41,8 @@
133 virtual int start(std::string const&, unity::scopes::RegistryProxy const&) override;
134 virtual void stop() override {}
135 virtual void run() override {}
136- virtual unity::scopes::QueryBase::UPtr create_query(unity::scopes::Query const& query, unity::scopes::SearchMetadata const&) override;
137- virtual unity::scopes::QueryBase::UPtr preview(unity::scopes::Result const&, unity::scopes::ActionMetadata const&) override;
138+ virtual unity::scopes::SearchQueryBase::UPtr search(unity::scopes::CannedQuery const& query, unity::scopes::SearchMetadata const&) override;
139+ virtual unity::scopes::PreviewQueryBase::UPtr preview(unity::scopes::Result const&, unity::scopes::ActionMetadata const&) override;
140
141 private:
142 void appendIfEnabled(QStringList &scopeToQuery, const QString &scopeId) const;
143@@ -56,7 +56,7 @@
144 //gulong m_disabledScopesSig;
145 };
146
147-class SelfServiceBundlesPreview : public unity::scopes::PreviewQuery
148+class SelfServiceBundlesPreview : public unity::scopes::PreviewQueryBase
149 {
150 public:
151 SelfServiceBundlesPreview(SelfServiceBundlesScope &scope, unity::scopes::Result const& result);

Subscribers

People subscribed via source and target branches

to all changes: