Merge lp:~didrocks/unity-lens-applications/conditionnally-show-more-suggestions into lp:unity-lens-applications

Proposed by Didier Roche-Tolomelli
Status: Rejected
Rejected by: Didier Roche-Tolomelli
Proposed branch: lp:~didrocks/unity-lens-applications/conditionnally-show-more-suggestions
Merge into: lp:unity-lens-applications
Diff against target: 64 lines (+21/-3)
3 files modified
configure.ac (+1/-1)
src/daemon.vala (+6/-2)
tests/software-center-suggestions.txt (+14/-0)
To merge this branch: bzr merge lp:~didrocks/unity-lens-applications/conditionnally-show-more-suggestions
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+126407@code.launchpad.net

Description of the change

Add conditionnal more suggestions show/hide based on the global online/commercial
key (LP: #1054746)

Manual test included.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Rejecting myself the merge request. Design descoped the app lens

Unmerged revisions

308. By Didier Roche-Tolomelli

add manual test

307. By Didier Roche-Tolomelli

dep on latest libunity

306. By Didier Roche-Tolomelli

add conditionally showing the suggestions (LP: #1054746).

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'configure.ac'
--- configure.ac 2012-09-19 11:33:38 +0000
+++ configure.ac 2012-09-26 08:24:20 +0000
@@ -65,7 +65,7 @@
65 gee-1.065 gee-1.0
66 dee-1.0 >= 0.5.1666 dee-1.0 >= 0.5.16
67 zeitgeist-1.0 >= 0.3.867 zeitgeist-1.0 >= 0.3.8
68 unity >= 6.5.168 unity >= 6.7
69 libgnome-menu)69 libgnome-menu)
7070
71AC_SUBST(LENS_DAEMON_CFLAGS)71AC_SUBST(LENS_DAEMON_CFLAGS)
7272
=== modified file 'src/daemon.vala'
--- src/daemon.vala 2012-09-25 10:24:12 +0000
+++ src/daemon.vala 2012-09-26 08:24:20 +0000
@@ -122,6 +122,8 @@
122122
123 private PurchaseInfoHelper purchase_info = null;123 private PurchaseInfoHelper purchase_info = null;
124124
125 private PreferencesManager preferences = PreferencesManager.get_default ();
126
125 construct127 construct
126 {128 {
127 populate_type_queries ();129 populate_type_queries ();
@@ -183,7 +185,8 @@
183 // TRANSLATORS: Please make sure this string is short enough to fit185 // TRANSLATORS: Please make sure this string is short enough to fit
184 // into the filter button186 // into the filter button
185 local_apps_option = scope.sources.add_option ("local", _("Local Apps"));187 local_apps_option = scope.sources.add_option ("local", _("Local Apps"));
186 if (display_available_apps)188 if (display_available_apps &&
189 (preferences.remote_content_search == Unity.PreferencesManager.RemoteContent.ALL))
187 {190 {
188 // TRANSLATORS: Please make sure this string is short enough to fit191 // TRANSLATORS: Please make sure this string is short enough to fit
189 // into the filter button192 // into the filter button
@@ -569,7 +572,8 @@
569 purchase_info = new PurchaseInfoHelper ();572 purchase_info = new PurchaseInfoHelper ();
570573
571 /* If we don't have a search we display 6 random apps */574 /* If we don't have a search we display 6 random apps */
572 if (usc_apps_active () && display_available_apps && pkgsearcher != null)575 if (usc_apps_active () && display_available_apps && pkgsearcher != null &&
576 (preferences.remote_content_search == Unity.PreferencesManager.RemoteContent.ALL))
573 {577 {
574 if (has_search)578 if (has_search)
575 {579 {
576580
=== added file 'tests/software-center-suggestions.txt'
--- tests/software-center-suggestions.txt 1970-01-01 00:00:00 +0000
+++ tests/software-center-suggestions.txt 2012-09-26 08:24:20 +0000
@@ -0,0 +1,14 @@
1'Disable' Software Center suggestions
2----------------------------------------------------
3Disabling the commercial/online suggestions should not show available apps.
4
5Setup:
6
7Actions:
81. Disable the online/commercial search in gnome-control-center or directly in gsettings
9(com.canonical.Unity.Lenses "remote-content-search" to none)
102. Perfom a search in the application lens
11
12Expected Results:
13No "More suggestions" category should appear.
14

Subscribers

People subscribed via source and target branches