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

Proposed by Chris Wayne
Status: Merged
Merged at revision: 12
Proposed branch: lp:~savilerow-team/savilerow/unity-scope-selfservicebill-newapi
Merge into: lp:~savilerow-team/savilerow/unity-scope-selfservicebill
Diff against target: 155 lines (+25/-19)
6 files modified
CMakeLists.txt (+2/-2)
debian/changelog (+6/-0)
src/SelfServiceBillQuery.cpp (+2/-2)
src/SelfServiceBillQuery.h (+4/-4)
src/SelfServiceBillScope.cpp (+7/-7)
src/SelfServiceBillScope.h (+4/-4)
To merge this branch: bzr merge lp:~savilerow-team/savilerow/unity-scope-selfservicebill-newapi
Reviewer Review Type Date Requested Status
Scott Sweeny (community) Approve
Review via email: mp+212886@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 :

tested locally, and B/I/R for me.

review: Approve
13. By Chris Wayne

releasing package unity-scope-selfservicebill version 0.3

14. 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:06 +0000
3+++ CMakeLists.txt 2014-03-27 03:42:46 +0000
4@@ -26,9 +26,9 @@
5 SET(GETTEXT_PACKAGE unity-scope-selfservicebill)
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-selfservicebill/)
12+SET(SCOPE_INSTALL_DIR ${CUSTOM_INSTALL_LIBDIR}/unity-scopes/scope-selfservicebill/)
13
14 CONFIGURE_FILE(
15 "${CMAKE_CURRENT_SOURCE_DIR}/src/config.h.in"
16
17=== modified file 'debian/changelog'
18--- debian/changelog 2014-02-03 19:38:06 +0000
19+++ debian/changelog 2014-03-27 03:42:46 +0000
20@@ -1,3 +1,9 @@
21+unity-scope-selfservicebill (0.3) trusty; urgency=medium
22+
23+ * Update to new api
24+
25+ -- Chris Wayne <chris.wayne@canonical.com> Wed, 26 Mar 2014 22:26:30 -0400
26+
27 unity-scope-selfservicebill (0.2) UNRELEASED; urgency=low
28
29 * Update renderer.json with karni's fixes
30
31=== modified file 'src/SelfServiceBillQuery.cpp'
32--- src/SelfServiceBillQuery.cpp 2014-02-22 04:36:06 +0000
33+++ src/SelfServiceBillQuery.cpp 2014-03-27 03:42:46 +0000
34@@ -33,8 +33,8 @@
35 using namespace std;
36 using namespace unity::scopes;
37
38-SelfServiceBillQuery::SelfServiceBillQuery(Query const& query, QStringList const& scopes, QString const& locale)
39- : SearchQuery(),
40+SelfServiceBillQuery::SelfServiceBillQuery(CannedQuery const& query, QStringList const& scopes, QString const& locale)
41+ : SearchQueryBase(),
42 query_(query),
43 scopes_(scopes)
44 {
45
46=== modified file 'src/SelfServiceBillQuery.h'
47--- src/SelfServiceBillQuery.h 2014-02-12 02:13:42 +0000
48+++ src/SelfServiceBillQuery.h 2014-03-27 03:42:46 +0000
49@@ -20,21 +20,21 @@
50 #define SELFSERVICEBILLQUERY_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 SelfServiceBillQuery: public unity::scopes::SearchQuery
60+class SelfServiceBillQuery: public unity::scopes::SearchQueryBase
61 {
62 public:
63- SelfServiceBillQuery(unity::scopes::Query const& query, QStringList const& scopes, QString const& locale);
64+ SelfServiceBillQuery(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/SelfServiceBillScope.cpp'
77--- src/SelfServiceBillScope.cpp 2014-02-19 03:31:56 +0000
78+++ src/SelfServiceBillScope.cpp 2014-03-27 03:42:46 +0000
79@@ -72,17 +72,17 @@
80 return VERSION;
81 }
82
83-QueryBase::UPtr SelfServiceBillScope::create_query(Query const& query, SearchMetadata const&)
84+SearchQueryBase::UPtr SelfServiceBillScope::search(const CannedQuery &query, SearchMetadata const&)
85 {
86 QStringList scopes;
87- QueryBase::UPtr q(new SelfServiceBillQuery(query, scopes, m_locale));
88+ SearchQueryBase::UPtr q(new SelfServiceBillQuery(query, scopes, m_locale));
89 return q;
90 }
91
92-QueryBase::UPtr SelfServiceBillScope::preview(Result const& result,
93+PreviewQueryBase::UPtr SelfServiceBillScope::preview(Result const& result,
94 ActionMetadata const& hints)
95 {
96- QueryBase::UPtr previewer(new SelfServiceBillPreview(*this, result));
97+ PreviewQueryBase::UPtr previewer(new SelfServiceBillPreview(*this, result));
98 return previewer;
99 }
100
101@@ -112,10 +112,10 @@
102 reply->register_layout({layout});
103
104 PreviewWidget header("header", "header");
105- header.add_attribute("title", Variant("Your next phone bill is:"));
106+ header.add_attribute_value("title", Variant("Your next phone bill is:"));
107
108 PreviewWidget description("description", "text");
109- description.add_component("text", "title");
110+ description.add_attribute_mapping("text", "title");
111
112 PreviewWidget actions("actions", "actions");
113 {
114@@ -124,7 +124,7 @@
115 {"id", Variant("open")},
116 {"label", Variant("Pay bill")}
117 });
118- actions.add_attribute("actions", builder.end());
119+ actions.add_attribute_value("actions", builder.end());
120 }
121
122 reply->push({header, description, actions});
123
124=== modified file 'src/SelfServiceBillScope.h'
125--- src/SelfServiceBillScope.h 2014-02-19 03:31:56 +0000
126+++ src/SelfServiceBillScope.h 2014-03-27 03:42:46 +0000
127@@ -24,7 +24,7 @@
128 #include <unity/scopes/PreviewReply.h>
129 #include <unity/scopes/PreviewWidget.h>
130 #include <unity/scopes/VariantBuilder.h>
131-#include <unity/scopes/Query.h>
132+#include <unity/scopes/SearchQueryBase.h>
133 #include <string>
134 #include <QString>
135 #include <QStringList>
136@@ -41,8 +41,8 @@
137 virtual int start(std::string const&, unity::scopes::RegistryProxy const&) override;
138 virtual void stop() override {}
139 virtual void run() override {}
140- virtual unity::scopes::QueryBase::UPtr create_query(unity::scopes::Query const& query, unity::scopes::SearchMetadata const&) override;
141- virtual unity::scopes::QueryBase::UPtr preview(unity::scopes::Result const&, unity::scopes::ActionMetadata const&) override;
142+ virtual unity::scopes::SearchQueryBase::UPtr search(unity::scopes::CannedQuery const& query, unity::scopes::SearchMetadata const&) override;
143+ virtual unity::scopes::PreviewQueryBase::UPtr preview(unity::scopes::Result const&, unity::scopes::ActionMetadata const&) override;
144
145 private:
146 void appendIfEnabled(QStringList &scopeToQuery, const QString &scopeId) const;
147@@ -56,7 +56,7 @@
148 //gulong m_disabledScopesSig;
149 };
150
151-class SelfServiceBillPreview : public unity::scopes::PreviewQuery
152+class SelfServiceBillPreview : public unity::scopes::PreviewQueryBase
153 {
154 public:
155 SelfServiceBillPreview(SelfServiceBillScope &scope, unity::scopes::Result const& result);

Subscribers

People subscribed via source and target branches

to all changes: