Merge lp:~jury-verrigni/slingshot/fix-1636632 into lp:~elementary-pantheon/slingshot/trunk

[Jury Verrigni]

Providing a link to the appcenter with the searched term.
Note that this depends on this merge-request https://code.launchpad.net/~jury-verrigni/appcenter/handle-search-via-uri-scheme

[Corentin Noël (tintou)]

I don't think this is the right thing to do here. You're introducing hidden dependency to AppCenter when you can just use the libappstream library to search. Each AppStream Components has an unique id which can be given to AppCenter (or any software center) via the appstream:// URI.

review: Disapprove

[David Hewitt (davidmhewitt)]

This has now been implemented in an alternative way. Closing.

review: Disapprove

Unmerged revisions

712. By Jury Verrigni on 2017-01-02

Now providing search on the AppCenter if result is empty

=== modified file 'src/Widgets/SearchView.vala'
--- src/Widgets/SearchView.vala	2016-12-28 16:13:57 +0000
+++ src/Widgets/SearchView.vala	2017-01-02 09:59:19 +0000
@@ -57,7 +57,7 @@
                         case SearchItem.ResultType.APP_ACTIONS:
                         case SearchItem.ResultType.LINK:
                         case SearchItem.ResultType.SETTINGS:
-                            search_item.app.match.execute (null);    
+                            search_item.app.match.execute (null);
                             break;
                         default:
                             search_item.app.launch ();
@@ -151,6 +151,7 @@
 
                 stack.set_visible_child_name ("results");
             } else {
+                alert_view.description = _("Try changing terms or search the <a href=\"appstream://search/%s\">AppCenter</a>".printf (search_term));
                 alert_view.title = _("No Results for “%s”".printf (search_term));
                 stack.set_visible_child_name ("alert");
             }