[Dash] Activating a preview with a scope URI for the current scope doesn't send you back to the results view

[Pete Woods]

You'll have to compile (prototype) the Youtube scope from source (https://launchpad.net/unity-scope-youtube/), as it's not in distro yet.

Basically the workflow is:
1) Search for something that has a result with a scope URI for the current scope. In the case of Youtube, search for "queenofficial" to get Queen's music channel.
2) Preview the first item (the channel itself), and click the search button.
3) This runs a new query for the current scope, but doesn't switch away from the preview view).
4) You will see a preview of the first video in the new results list.

Like when searching other scopes, the view should be returned to the results list, instead of saying in the preview "view".

[Michał Sawicz (saviq)]

Are you sending HIDE_PREVIEW activation result?

[Michał Sawicz (saviq)]

Or whatever calls showDash()? ;)

[Paweł Stołowski (stolowski)]

This is also appears to be broken for actions which have "uri" attribute set to a canned query, except for Scopes scope previews (works for favorited scopes?)

[Paweł Stołowski (stolowski)]

It seems that openScope() signal of Scope object that's emitted in such cases doesn't cause change active scope when in the preview.

[kevin gunn (kgunn72)]

@saviq - so it doesn't get lost, but this feels like it might be addressed through bug 1368670

[Robert Schroll (rschroll)]

I have a simple scope that seems to exhibit the same problem: https://code.launchpad.net/~rschroll/+junk/multiples

[Riccardo Padovani (rpadovani)]

I have the same problem here: https://github.com/rpadovani/discerning-duck

Do a search for "countries in Europe", click on the first result, click on "Tell me more" - a new search is done, but the preview isn't closed

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Paweł Stołowski (stolowski)]

I've just checked vivid and it's still the case. Note, the original steps to reproduce no longer apply because youtube scope doesn't seem to be producing previews for channels with 'Search' button. I'm not sure which of the existing scopes exhibit it offhand.

I've reproduced it again with slightly modifed mediascanner-video scope (added a 'Search' button to its preview). After some more tests I can conclude the following about preview buttons which have uris set to canned queries (should execute search):

* it works, when it points to a different scope than the one displaying the preview.
* it doesn't work when it points to the same scope. In such case the search is executed in the background, but preview is not dismissed. After exiting the preview manually one can see new search results proving the query was in fact executed.

Here is the change I applied to mediascanner scope for reproducing; change uri to "scope://mediascanner-music?q=" to see it working as expected because different scope needs to be opened).

=== modified file 'src/myvideos/video-scope.cpp'
--- src/myvideos/video-scope.cpp 2015-04-15 15:58:52 +0000
+++ src/myvideos/video-scope.cpp 2015-04-17 09:58:29 +0000
@@ -281,6 +281,12 @@
                 {"uri", Variant(uri)},
                 {"label", Variant(_("Play"))}
             });
+ builder.add_tuple({
+ {"id", Variant("sr")},
+ {"uri", Variant("scope://mediascanner-video?q=foo")},
+ {"label", Variant(_("Search"))}
+ });
+
         actions.add_attribute_value("actions", builder.end());
     }

[Michał Sawicz (saviq)]

Assuming we get notice from the Scopes object, it would be a relatively easy fix on our side.

[Albert Astals Cid (aacid)]

After a quick chat with Pawel we don't need any change from the unity8 side

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity-scopes-shell - 0.5.4+15.04.20150429-0ubuntu1

---------------
unity-scopes-shell (0.5.4+15.04.20150429-0ubuntu1) vivid; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Pawel Stolowski ]
  * Emit showDash() signal if canned query is executed for current
    scope. (LP: #1335761)
  * Reset user data set with previously executed canned query when shell
    calls setSearchQuery() (i.e.user typed a new search). (LP: #1449928)

 -- CI Train Bot <email address hidden> Wed, 29 Apr 2015 11:44:18 +0000