Merge lp:~davidc3/unity-scope-texdoc/glib-bindings-crash into lp:unity-scope-texdoc

[David Callé]

Don't crash Unity bindings when building results.

- Don't crash Unity bindings when building results.
- Add tex and latex keywords

Test plan : https://wiki.ubuntu.com/Process/Merges/TestPlan/unity-scope-texdoc

 * Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes) Yes
 * Did you build your software in a clean sbuild/pbuilder chroot or ppa? Yes
 * Has your component "TestPlan” been executed successfully on desktop? Yes
 * If you changed the packaging (debian), did you subscribe a core-dev to this MP? N/A
 * If you changed the UI, did you subscribe the design-reviewers to this MP? N/A
 * What components might get impacted by your changes? None
 * Have you requested review by the teams of these owning components? N/A

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:27
http://jenkins.qa.ubuntu.com/job/unity-scope-texdoc-ci/5/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-scope-texdoc-trusty-amd64-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-scope-texdoc-trusty-armhf-ci/2
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-scope-texdoc-trusty-i386-ci/2

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-scope-texdoc-ci/5/rebuild

review: Approve (continuous-integration)

[Paweł Stołowski (stolowski)]

Looking good. +1

review: Approve

=== modified file 'data/texdoc.scope.in'
--- data/texdoc.scope.in	2013-07-08 07:48:09 +0000
+++ data/texdoc.scope.in	2014-03-28 08:55:18 +0000
@@ -5,7 +5,7 @@
 QueryBinary=texdoc
 Module=texdoc.unity_texdoc_daemon
 ModuleType=python3
-_Keywords=texdoc;
+_Keywords=texdoc;tex;latex;
 RequiredMetadata=
 OptionalMetadata=
 Type=help
 
=== modified file 'src/unity_texdoc_daemon.py'
--- src/unity_texdoc_daemon.py	2013-12-16 01:00:56 +0000
+++ src/unity_texdoc_daemon.py	2014-03-28 08:55:18 +0000
@@ -195,19 +195,7 @@
                     i['comment'] = ''
                 if not 'dnd_uri' in i or not i['dnd_uri'] or i['dnd_uri'] == '':
                     i['dnd_uri'] = i['uri']
-                i['metadata'] = {}
-                if EXTRA_METADATA:
-                    for e in i:
-                        for m in EXTRA_METADATA:
-                            if m['id'] == e:
-                                i['metadata'][e] = i[e]
-                i['metadata']['provider_credits'] = GLib.Variant('s', PROVIDER_CREDITS)
-                result = Unity.ScopeResult.create(str(i['uri']), str(i['icon']),
-                                                  i['category'], i['result_type'],
-                                                  str(i['mimetype']), str(i['title']),
-                                                  str(i['comment']), str(i['dnd_uri']),
-                                                  i['metadata'])
-                result_set.add_result(result)
+                result_set.add_result(**i)
         except Exception as error:
             print(error)