Merge lp:~stolowski/unity-lens-video/unity-lens-video.previews into lp:unity-lens-video

[Paweł Stołowski]

Implementation of previews.

Initial implementation of previews.

[lp:~stolowski/unity-lens-video/unity-lens-video.previews]

89. By Paweł Stołowski on 2012-08-13

Preview: require Gst/GstPbutils 1.0. Combined width, height info hints into "Dimensions" hint with aspect ratio.

90. By Paweł Stołowski on 2012-08-13

Preview: added 'Size' hint. Changed order of Format & Dimensions hints.

91. By Paweł Stołowski on 2012-08-13

Preview: use GLib.format_size function instead of custom one.

[Michal Hruby (mhr3)]

51 + while iter != end_iter:
52 + if model.get_value(iter, 0) == uri:

Linear iteration of the model :/ Not too nice, but hopefully it shouldn't be too much of an issue. s/get_value/get_string/

72 + preview.add_info(Unity.InfoHint.new("dimensions", "Dimensions", None, dimensions))

Please use gettext for the user-visible strings.

GLib.format_size(os.path.getsize(uri[7:]))))

Uris escape some characters, so they don't map exactly to path, this should be fixed.

87 + return Unity.ActivationResponse(goto_uri=uri, handled=2 )

Just because this is python doesn't make it ok to use numeric values for enums :) Anyway, you don't really need to connect to the activated signal here, the activation response will by default return NOT_HANDLED, so unity will run the default handler for the file.

review: Needs Fixing

[lp:~stolowski/unity-lens-video/unity-lens-video.previews]

92. By Paweł Stołowski on 2012-08-14

Previews: properly handle uri when passing it to os.path.getsize().
Use is_last and get_string when iterating over dee model.

93. By Paweł Stołowski on 2012-08-14

Previews: added 'Show in folder' action for local videos.

94. By Paweł Stołowski on 2012-08-14

Previews: display modification time as subtitle.

95. By Paweł Stołowski on 2012-08-14

Previews: use gettext for user-visible strings.

[Michal Hruby (mhr3)]

Looking good now. +1

review: Approve

=== modified file 'src/unity-lens-video'
--- src/unity-lens-video	2012-07-04 16:13:49 +0000
+++ src/unity-lens-video	2012-08-14 12:58:18 +0000
@@ -25,6 +25,11 @@
 from zeitgeist.client import ZeitgeistClient
 from zeitgeist import client, datamodel
 import time
+import fractions
+
+import gi
+gi.require_version("Gst", "1.0")
+gi.require_version("GstPbutils", "1.0")
 
 #pylint: disable=E0611
 from gi.repository import (
@@ -33,6 +38,8 @@
     Gio,
     Unity,
     Dee,
+    Gst,
+    GstPbutils
 )
 #pylint: enable=E0611
 
@@ -70,7 +77,6 @@
     """Creation of a lens with a local scope."""
 
     def __init__(self):
-        
         #Create the lens
         self._lens = Unity.Lens.new("/net/launchpad/lens/video", "video")
         self._lens.props.search_hint = HINT
@@ -108,6 +114,7 @@
         self._scope.connect("filters-changed",self.on_filtering_changed)
         self._scope.props.sources.connect("notify::filtering",
             self.on_filtering_changed)
+        self._scope.connect('preview-uri', self.on_preview_uri)
         self._lens.add_local_scope(self._scope)
         self._lens.export()
 
@@ -124,6 +131,60 @@
         else:
             return False
     
+    def on_preview_uri(self, scope, uri):
+        """Preview request handler"""
+        preview = None
+        model = self._scope.props.results_model
+        iter = model.get_first_iter()
+        while not model.is_last(iter):
+            if model.get_string(iter, 0) == uri:
+                title = model.get_string(iter, 4);
+                subtitle = time.strftime("%x, %X", time.localtime(os.path.getmtime(GLib.filename_from_uri(uri, None))))
+                desc = model.get_string(iter, 5);
+                preview = Unity.MoviePreview.new(title, subtitle, desc, None)
+                # we may get remote uris from zeitgeist - fetch details for local files only
+                if uri.startswith("file://"):
+                    preview.props.image_source_uri = uri
+                    try:
+                        discoverer = GstPbutils.Discoverer.new(2 * 1000000000) #2 seconds
+                        discoverer_info = discoverer.discover_uri(uri)
+                        video_streams = discoverer_info.get_video_streams()
+                        # note: the container may have multiple video and audio streams, we just get properties of the first one
+                        if video_streams and len(video_streams) > 0:
+                            vstream = video_streams[0]
+                            dimensions = str(vstream.get_width()) + "*" + str(vstream.get_height())
+                            if vstream.get_width() > 0 and vstream.get_height() > 0:
+                                gcd = fractions.gcd(vstream.get_width(), vstream.get_height())
+                                dimensions += ", " + str(vstream.get_width() / gcd) + ":" + str(vstream.get_height() / gcd)
+                            preview.add_info(Unity.InfoHint.new("format", _("Format"), None, GstPbutils.pb_utils_get_codec_description(vstream.get_caps())))
+                            preview.add_info(Unity.InfoHint.new("dimensions", _("Dimensions"), None, dimensions))
+                            preview.add_info(Unity.InfoHint.new("size", _("Size"), None, GLib.format_size(os.path.getsize(GLib.filename_from_uri(uri, None)))))
+                    except Exception as e:
+                        print "Couldn't get video details", e
+                show_folder = Unity.PreviewAction.new("show-in-folder", _("Show in Folder"), None)
+                show_folder.connect('activated', self.show_in_folder)
+                preview.add_action(show_folder)
+                play_video = Unity.PreviewAction.new("play", _("Play"), None)
+                preview.add_action(play_video)
+                break
+            iter = model.next(iter)
+        if preview == None:
+            print "Couldn't find model row for requested preview uri:", uri
+        return preview
+
+    def show_in_folder(self, scope, uri):
+        """ Open folder that contains given video """
+        file = Gio.file_new_for_uri (uri)
+        parent = file.get_parent()
+        if parent == None:
+            parent = Gio.file_new_for_path("/")
+        try:
+            Gio.app_info_launch_default_for_uri(parent.get_uri(), None)
+        except Exception as e:
+            print "Couldn't launch default app for uri", parent.get_uri(), e
+            return Unity.ActivationResponse(handled=Unity.HandledType.NOT_HANDLED)
+        return Unity.ActivationResponse(handled=Unity.HandledType.HIDE_DASH)
+
     def on_lens_active(self, *_):
         """ Run a search when the lens is opened """
         if self._lens.props.active:
@@ -437,6 +498,7 @@
     except:
         raise SystemExit(1)
 
+    Gst.init(sys.argv)
     daemon = Daemon()
     GObject.MainLoop().run()